Skip to content

Final test data cache - inside CI docker images#40689

Merged
ydshieh merged 4 commits intomainfrom
cache_test_files_in_docker
Sep 4, 2025
Merged

Final test data cache - inside CI docker images#40689
ydshieh merged 4 commits intomainfrom
cache_test_files_in_docker

Conversation

@ydshieh
Copy link
Copy Markdown
Collaborator

@ydshieh ydshieh commented Sep 4, 2025

What does this PR do?

Cache the test data during the CI docker image build time.

Hope you would be happy with this :-) @manueldeprada

I tried with the dev images which works. I will push the real images then merge if you are happy.

@HuggingFaceDocBuilderDev
Copy link
Copy Markdown

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
Copy link
Copy Markdown
Collaborator Author

ydshieh commented Sep 4, 2025

cc @zucchini-nlp too

@ydshieh ydshieh requested a review from zucchini-nlp September 4, 2025 11:27
Copy link
Copy Markdown
Contributor

@manueldeprada manueldeprada left a comment

Choose a reason for hiding this comment

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

Oh this js a much better solution than what I proposed. Thanks a lot!!♥️

Copy link
Copy Markdown
Member

@zucchini-nlp zucchini-nlp left a comment

Choose a reason for hiding this comment

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

One step closer to ✅ CI

Comment on lines -1212 to +1216
"https://www.ilankelman.org/stopsigns/australia.jpg",
"https://www.ilankelman.org/stopsigns/australia.jpg",
],
url_to_local_path(
"https://huggingface.co/datasets/huggingface/documentation-images/resolve/main/transformers/tasks/australia.jpg"
)
]
* 2,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

cool, can you approve #40684 (comment) pls and we'll be done with processor tests 🤞🏻

@ydshieh ydshieh enabled auto-merge (squash) September 4, 2025 13:11
@ydshieh ydshieh merged commit 5b0c01b into main Sep 4, 2025
24 checks passed
@ydshieh ydshieh deleted the cache_test_files_in_docker branch September 4, 2025 13:12
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