-
Notifications
You must be signed in to change notification settings - Fork 3
Open
Description
It's bad DX having to specify this and confusing we already have a timeout parameter (in seconds) and the SDK also needs a timeout parameter (in ms)
const runResult = await client.taskRun.result(
taskRun.run_id,
{
timeout: 3600,
},
{ timeout: 3600 * 1000 }
);
It's best if the default timeout becomes 3600 so people or llms won't make the mistake by default.
Seems we can configure this: https://www.stainless.com/docs/reference/config#client-settings
Edit: it seems we are still running into timeouts even with above code!
Metadata
Metadata
Assignees
Labels
No labels