-
Notifications
You must be signed in to change notification settings - Fork 169
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
Pull Imagen feature #285
base: main
Are you sure you want to change the base?
Pull Imagen feature #285
Conversation
@@ -448,6 +448,13 @@ export interface GenerateContentStreamResult { | |||
stream: AsyncGenerator<EnhancedGenerateContentResponse>; | |||
} | |||
|
|||
// @public | |||
export interface GeneratedImage { | |||
// Warning: (ae-forgotten-export) The symbol "PredictServiceValueType" needs to be exported by the entry point index.d.ts |
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.
Check this file for comments telling you what's still missing in the exported types.
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.
Thanks for pointing out. I removed the field from GeneratedImage.
btw while this works for me :
While using this branch and following the sample:
I am getting :
And with:
I just get 404. any idea @junyanxu ? |
@jony89 Are you in the tester list? Your project need to be in trusted project list. Otherwise the you cannot use the API. Target GA date should be in mid Nov. After that this branch will be merged. |
Yes @junyanxu I do have access. I also was able to use OAuth to access that API and generate images. but with the code here I got that error. |
I messed up the old branch after doing a merge. Create this new branch for us to keep reviewing.
I have resolved all comments except one from @hsubox76
The ...request will cause testing error like this, because original request has more parameters than
parameters
could accept. And in new predict api theparameters
are not typed.Related to #276