Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Verify phi-2 #299

Merged
merged 2 commits into from
Mar 12, 2024
Merged

Verify phi-2 #299

merged 2 commits into from
Mar 12, 2024

Conversation

qxprakash
Copy link
Contributor

@qxprakash qxprakash commented Mar 12, 2024

This should close #274 , I have updated the model list , and added tests taking reference from issue description at #259 , @p-wysocki @ilya-lavrenov @pavel-esir in the readme I have added phi separately , since there are other phi models as well , like phi-1.5 which can be added in the future , please let me know if anything else can be done.

@ilya-lavrenov ilya-lavrenov merged commit 900a60a into openvinotoolkit:master Mar 12, 2024
5 checks passed
@qxprakash
Copy link
Contributor Author

@p-wysocki should I also verify Phi-1.5 ?

@p-wysocki
Copy link
Collaborator

Thank you for your contribution!

Sure, @Wovchena mentioned that contributors may propose their own models. :) Do you want me to create a Good First Issue for you?

@qxprakash
Copy link
Contributor Author

Thanks for the guidance @p-wysocki , I will create an issue by myself , based on the template for verifying models.

@p-wysocki
Copy link
Collaborator

Awesome, thanks! You can use the Good First Issue template and fill it just like the issue you completed. :)

@qxprakash
Copy link
Contributor Author

Ok sure @p-wysocki

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Good First Issue]: Verify phi-2 with GenAI text_generation
3 participants