-
Notifications
You must be signed in to change notification settings - Fork 180
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
AbortSignal support #144
AbortSignal support #144
Conversation
docs/reference/files/generative-ai.googleaifilemanager.deletefile.md
Outdated
Show resolved
Hide resolved
import { GoogleAIFileManager } from "../../dist/files"; | ||
|
||
use(chaiAsPromised); | ||
|
||
/** | ||
* Integration tests against live backend. | ||
*/ | ||
describe("abortSignal", function () { | ||
describe("signal", function () { |
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.
Can you add a test to see if this works together with timeout (e.g., whichever comes first works)? Or if it's too hard to make an automated test, can you test it manually?
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.
Done.
any news? |
Adds
SingleRequestOptions
withAbortSignal
support to most of the asynchronous methods of:GenerativeModel
GoogleAIFileManager
ChatSession
NOTE:
AbortSignal
is a client-only operation. Using it to cancel an operation will not cancel the request in the service. You will still be charged usage for any applicable operations.Some methods are not supported as they would leave the server in an ambiguous state, namely
GoogleAIFileManager.uploadFile()
andGoogleAIFileManager.deleteFile()
.