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.

Sign up for GitHub

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

Add --trust-repo option to download CLI command #807

Merged
merged 6 commits into from
Oct 1, 2024
Merged

Add --trust-repo option to download CLI command #807

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
0914d4c
Add --trust-repo option to download CLI command
juhoinkinen Sep 20, 2024
e2e6ff7
Merge branch 'main' into add-trust-option-to-download-cli-command
juhoinkinen Sep 30, 2024
4dfa2d1
Resolve SonarCloud complain: Remove unused parameter token
juhoinkinen Sep 30, 2024
caf0325
Again resolve SonarCloud complain: Remove unused parameter token
juhoinkinen Sep 30, 2024
80eb1b2
Mock empty HFH repos cache instead of just helper fn return value
juhoinkinen Oct 1, 2024
654a376
Bypass CacheNotFound err on CI/CD resulting passing test for wrong re…
juhoinkinen Oct 1, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Resolve SonarCloud complain: Remove unused parameter token
  • Loading branch information
@juhoinkinen
juhoinkinen committed Sep 30, 2024
commit 4dfa2d1962ed0752748fedc4b148fe19d037edf2
4 changes: 2 additions & 2 deletions annif/hfh_util.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@
f'Download allowed from "{repo_id}" because "--trust-repo" flag is used.'
)
return
if _is_repo_in_cache(repo_id, token):
if _is_repo_in_cache(repo_id):
logger.debug(
f'Download allowed from "{repo_id}" because repo is already in cache.'
)
Expand All @@ -42,15 +42,15 @@
)


def _is_repo_in_cache(repo_id, token):
def _is_repo_in_cache(repo_id):
from huggingface_hub import CacheNotFound, scan_cache_dir

Check warning on line 46 in annif/hfh_util.py

View check run for this annotation

Codecov / codecov/patch

annif/hfh_util.py#L46 

Added line #L46 was not covered by tests

try:
cache = scan_cache_dir()
except CacheNotFound as err:
logger.debug(str(err) + "\nNo HFH cache found.")
return False
return repo_id in [info.repo_id for info in cache.repos]

Check warning on line 53 in annif/hfh_util.py

View check run for this annotation

Codecov / codecov/patch

annif/hfh_util.py#L48-L53 

Added lines #L48 - L53 were not covered by tests


def get_matching_projects(pattern: str) -> list[AnnifProject]:
Expand Down
Loading