-
Notifications
You must be signed in to change notification settings - Fork 12
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
Add Proof of Concept work #116
Conversation
Co-authored-by: Douglas dougmoy@bu.edu Co-authored-by: Dima dimak@bu.edu
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
@@ -0,0 +1,99 @@ | |||
{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It was good to try at least. So now we now the performance.
Btw, it would be helpful to state explicitly which model and what date you used, since OpenAI is constantly updating the models.
Reply via ReviewNB
@@ -0,0 +1,99 @@ | |||
{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -0,0 +1,549 @@ | |||
{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove hardcoded keys before merging. Read from environment variable instead. Should be example code on how to do this in both OpenAI and Azure documentation.
Co-authored-by: Dima dimak@bu.edu
Removed API keys and updated code for resized images |
Co-authored-by: Douglas dougmoy@bu.edu
Co-authored-by: Dima dimak@bu.edu