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

Save (circleci) cache at the end of a job #29141

Merged
merged 1 commit into from
Feb 20, 2024
Merged

Conversation

ydshieh
Copy link
Collaborator

@ydshieh ydshieh commented Feb 20, 2024

What does this PR do?

This way, pytest will run before cache saving and we have access to the results earlier in the case of partial or no cache loaded.

@ydshieh ydshieh requested review from ArthurZucker and amyeroberts and removed request for ArthurZucker February 20, 2024 12:10
@@ -264,6 +248,25 @@ def to_dict(self):

steps.append({"store_artifacts": {"path": "~/transformers/tests_output.txt"}})
steps.append({"store_artifacts": {"path": "~/transformers/reports"}})

# save cache at the end: so pytest step runs before cache saving and we can see results earlier
Copy link
Collaborator Author

@ydshieh ydshieh Feb 20, 2024

Choose a reason for hiding this comment

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

see them earlier on the console. The final results will be shown on a tab only when the whole job is finished.

Copy link
Collaborator

@amyeroberts amyeroberts left a comment

Choose a reason for hiding this comment

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

Thanks for updating this!

@HuggingFaceDocBuilderDev

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@ydshieh ydshieh force-pushed the put_cache_saving_after branch from 4238e46 to 7e5d5fe Compare February 20, 2024 13:02
@ydshieh ydshieh merged commit 7688d8d into main Feb 20, 2024
22 checks passed
@ydshieh ydshieh deleted the put_cache_saving_after branch February 20, 2024 13:31
itazap pushed a commit that referenced this pull request May 14, 2024
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.

4 participants