Skip to content
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

test: Increase test timeout for test stability #100

Merged
merged 2 commits into from
Jan 16, 2025

Conversation

rmccorm4
Copy link
Collaborator

@rmccorm4 rmccorm4 commented Jan 16, 2025

WAR to avoid intermittent test timeouts from slow network speed for now. Should be improved in the future to have models pre-downloaded, cached, etc. when possible.

krishung5
krishung5 previously approved these changes Jan 16, 2025
Copy link
Contributor

@krishung5 krishung5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: update copyright, otherwise lgtm!

@rmccorm4 rmccorm4 changed the title test: Increase test timeout for test stability for now test: Increase test timeout for test stability Jan 16, 2025
@rmccorm4 rmccorm4 requested a review from krishung5 January 16, 2025 22:49
@rmccorm4 rmccorm4 merged commit f3abcc3 into main Jan 16, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants