-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Download----fix new bug #8088
Download----fix new bug #8088
Conversation
Thanks for your contribution! |
paddlenlp/utils/download/__init__.py
Outdated
if cache_file_name is not None and not isinstance(cache_file_name, object): | ||
return cache_file_name | ||
if cache_file_name is not None: | ||
if ((from_aistudio or from_hf_hub) and not isinstance(cache_file_name, object)) or from_bos: |
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.
没这么复杂吧
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.
if from_hf_hub and cache_file_name is _CACHED_NO_EXIST:
cache_file_name = None
if cache_file_name is not None:
xxxxxxxxxxx
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.
LGTM
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.
LGTM
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #8088 +/- ##
===========================================
- Coverage 56.46% 56.46% -0.01%
===========================================
Files 596 596
Lines 91564 91576 +12
===========================================
+ Hits 51704 51708 +4
- Misses 39860 39868 +8 ☔ View full report in Codecov by Sentry. |
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.
LGTM
PR types
Bug fixes
PR changes
修改模型默认下载优先级、修复cache加载文件时的bug、增加对下载文件是否为None的检查
Description
修复新的Download逻辑中测试所发现的新bug