-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
[CI] [GHA] HuggingFace cache #28481
Open
akashchi
wants to merge
24
commits into
openvinotoolkit:master
Choose a base branch
from
akashchi:ci/gha/hf-cache-test
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+60
−25
Open
[CI] [GHA] HuggingFace cache #28481
Changes from all commits
Commits
Show all changes
24 commits
Select commit
Hold shift + click to select a range
52424ae
use staging runner
akashchi 82a03dd
start hf tests
akashchi 0b31d9f
add missing staging
akashchi 51235e9
Merge branch 'master' into ci/gha/hf-cache-test
akashchi 1a125b3
Merge remote-tracking branch 'upstream/master' into ci/gha/hf-cache-test
akashchi 2e1f773
check share on Win
akashchi 0c4d7c0
Merge remote-tracking branch 'upstream/master' into ci/gha/hf-cache-test
akashchi 4d0d0a7
Merge branch 'ci/gha/hf-cache-test' of github.com:akashchi/openvino i…
akashchi eb4b951
fix runner name
akashchi c90de13
set both HF cache vars to the same dir
akashchi b4b993f
Merge remote-tracking branch 'upstream/master' into ci/gha/hf-cache-test
akashchi 92823ad
update paths
akashchi 1c9f2a6
Merge remote-tracking branch 'upstream/master' into ci/gha/hf-cache-test
akashchi 2156899
check share
akashchi fee163c
rm staging
akashchi f1c383c
add HF share to jobs
akashchi aab40fd
Merge remote-tracking branch 'upstream/master' into ci/gha/hf-cache-test
akashchi 42d3572
set system_cache var per job
akashchi cccdb79
Merge remote-tracking branch 'upstream/master' into ci/gha/hf-cache-test
akashchi 824d35f
add comment
akashchi 358c68c
Merge remote-tracking branch 'upstream/master' into ci/gha/hf-cache-test
akashchi 4fc43e7
setup cache dir only for self-hosted runners
akashchi 3287f5d
fix import
akashchi 4a864e8
Merge remote-tracking branch 'upstream/master' into ci/gha/hf-cache-test
akashchi File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was not sure how it was supposed to work in the first place:
HF_HUB_CACHE
&HUGGINGFACE_HUB_CACHE
, the latter is deprecated but maybe needed for backwards compatibilityHF_HUB_CACHE
is taken from the environment and if not present -> a temp directory is used insteadHUGGINGFACE_HUB_CACHE
was always set to a created temporary directory, w/o even looking for it in the env. what if we want to use a remote cache like in CI?USE_SYSTEM_CACHE
but only for a deprecatedHUGGINGFACE_HUB_CACHE
Via the changes in this PR, I set
HF_HUB_CACHE
as a single source of truth but I think it could and should be simplified further. IsHUGGINGFACE_HUB_CACHE
even needed? I think it could be done like:HF_HUB_CACHE
from the env:HUGGINGFACE_HUB_CACHE
/ set it toHF_HUB_CACHE
USE_SYSTEM_CACHE
into something likeCLEAN_HF_CACHE
/KEEP_HF_CACHE
/...