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

Speedup tf tests #10601

Merged
merged 3 commits into from Mar 9, 2021
Merged

Speedup tf tests #10601

merged 3 commits into from Mar 9, 2021

Conversation

LysandreJik
Copy link
Member

Fyi @sgugger @patrickvonplaten @stas00, I'm temporarily marking these tests as slow as they take more than 1h30 minutes and prevent the CI from completing, therefore preventing any relevant information from the TF tests.

I'm working on improving the CI times so this is temporary (will revert by Friday).

@LysandreJik LysandreJik merged commit 546cbe7 into master Mar 9, 2021
@LysandreJik LysandreJik deleted the speedup-tf-tests branch March 9, 2021 02:44
@LysandreJik
Copy link
Member Author

LysandreJik commented Mar 9, 2021

This single change made the tests on the TF CI pass from 6+ hours (non slow) to 29/31 minutes: https://github.com/huggingface/transformers/actions/runs/635711849

Will look for a way to reduce their time.

Iwontbecreative pushed a commit to Iwontbecreative/transformers that referenced this pull request Jul 15, 2021
* Pipeline tests should be slow

* Temporarily mark some tests as slow

* Temporarily mark Barthez tests as slow
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.

None yet

1 participant