-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
[Enhancement] Add win-ci #1296
[Enhancement] Add win-ci #1296
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1296 +/- ##
==========================================
+ Coverage 90.22% 90.24% +0.02%
==========================================
Files 130 131 +1
Lines 7560 7599 +39
Branches 1258 1264 +6
==========================================
+ Hits 6821 6858 +37
Misses 531 531
- Partials 208 210 +2
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
requirements/runtime.txt
Outdated
@@ -1,4 +1,5 @@ | |||
matplotlib | |||
numpy | |||
opencv-python>=3 |
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.
From mmcv experiments, they don't recommend adding opencv in requirements as opencv have 4 versions and there is a conflict between them
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.
Delete opencv-python>=3
would raise error. Windows system needs cv2. So I added it back.
* [Enhancement] Add win-ci * add timm in win unittest * remove mmflow with mmseg in win unittest * remove opencv-python in requirements * add opencv2 back * move opencv installation into build.yml
Motivation
Add win-ci in .github/workflows/build.yml
ref pr: open-mmlab/mmagic#732, open-mmlab/mmflow#92.