-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
[Fix] Update outdated dependencies of mmcv for downloading fine-gym dataset #2495
Conversation
Thanks for your contribution. Related functions have been moved into |
Hi there, I have made these changes, please check. Thank you for providing such detailed guidance! |
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #2495 +/- ##
==========================================
- Coverage 76.97% 76.93% -0.05%
==========================================
Files 159 159
Lines 12546 12554 +8
Branches 2099 2102 +3
==========================================
+ Hits 9657 9658 +1
- Misses 2386 2390 +4
- Partials 503 506 +3
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
Hi @yhZhai, We'd like to express our appreciation for your valuable contributions to the mmaction2. Your efforts have significantly aided in enhancing the project's quality. If you're on WeChat, we'd also love for you to join our community there. Just add our assistant using the WeChat ID: openmmlabwx. When sending the friend request, remember to include the remark "mmsig + Github ID". Thanks again for your awesome contribution, and we're excited to have you as part of our community! |
Motivation
The MMCV package now has version 2.0.0, which removes the function
mmcv.load()
, causing the following error when running tools/data/gym/download_videos.sh:Traceback (most recent call last):
File "download.py", line 100, in
main(**vars(p.parse_args()))
File "download.py", line 75, in main
youtube_ids = mmcv.load(input).keys()
AttributeError: module 'mmcv' has no attribute 'load'
Modification
Changing
pip install mmcv
topip install "mmcv<2.0.0"
addresses this problem.