Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Timeouts cannot be added #115

Open
Neel-132 opened this issue Jun 12, 2024 · 3 comments
Open

Timeouts cannot be added #115

Neel-132 opened this issue Jun 12, 2024 · 3 comments
Labels
bug Something isn't working

Comments

@Neel-132
Copy link

I am calling the Unstructured paid API via Unstructured client SDK in python. But there is no way for me to give timeouts. I already added the Retry Config, but adding timeouts is also necessary. How do I add timeouts?

Please assist

@Neel-132 Neel-132 added the bug Something isn't working label Jun 12, 2024
@MthwRobinson
Copy link

Moving this over to the client SDK repo. The Unstructured SDK gives you access to the underlying requests session and you should be able to add timeouts using a workflow similar to this.

@MthwRobinson MthwRobinson transferred this issue from Unstructured-IO/unstructured Jun 12, 2024
@awalker4
Copy link
Collaborator

Here's an example of modifying the underlying requests session to add a custom timeout. I realize this isn't an intuitive way to do this - I've added a feature request to do this with a simple parameter.

@Neel-132
Copy link
Author

Hey, I have tried this as well before posting this issue. The point is overwriting the HTTP Adapter does not work all the time. So, adding timeout as a parameter in the general function(in the unstructured client library) would be more appropriate.

@awalker4 awalker4 reopened this Jun 14, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants