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
Initial change from dicts to objects #9
Conversation
cohere/__init__.py
Outdated
@@ -96,3 +96,24 @@ def __repr__(self): | |||
self.message, | |||
self.http_status, | |||
) | |||
|
|||
class CohereGenerate: |
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.
how do these show up in debugging / errors? Sholud they just be Generation
or CohereGeneration
?
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.
took a look at twillo sdk and i think we should go with Generate
. I'm going with the docs naming right now, so it won't be Generation
.
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.
yeah the docs name the endpoint as Generate
, however the product is a Generation
. Its the same reason an endpoint called Create🍌
would return a 🍌
and not a Create🍌
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'll be hard to come up with a name for choosebest 🤔
update the sanity test file as well? make sure everything runs |
Don't know about how we should name objects yet.
Originally, I wanted to have one "catch all" CohereResponse object, but the names were confusing because we had "likelihood" and "likelihoods", thus designing it this way.
This should also be refactored into bzz and docs
closes #8