-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
MAINT: asv with py3 on windows #16139
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
Conversation
Codecov Report
@@ Coverage Diff @@
## master #16139 +/- ##
=======================================
Coverage 90.84% 90.84%
=======================================
Files 159 159
Lines 50779 50779
=======================================
Hits 46128 46128
Misses 4651 4651
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #16139 +/- ##
=======================================
Coverage 90.84% 90.84%
=======================================
Files 159 159
Lines 50779 50779
=======================================
Hits 46128 46128
Misses 4651 4651
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #16139 +/- ##
=======================================
Coverage 90.84% 90.84%
=======================================
Files 159 159
Lines 50779 50779
=======================================
Hits 46128 46128
Misses 4651 4651
Continue to review full report at Codecov.
|
Can you keep those entries but specify it is only for python 3? |
@jorisvandenbossche - asv actually still works on python 2 with this change, as long as the right version of MSVC is installed (https://www.microsoft.com/en-us/download/details.aspx?id=44266). This would be the same compiler our release binaries / tests are built with, and easy enough to install now (wasn't always), so I think we should prefer it over the |
OK, I currently don't use python 2 on windows anymore, so won't push too hard. I just know that being able to just conda install the correct compilers was very handy at that time :-) But if that has improved, then certainly OK! Our contributing docs still mention |
Yeah, that's fair, I think MSVC would be the 'recommended' way now, but |
thanks! |
Since benchmarking on python 3, no longer want to install
libpython