Skip to content

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

Merged
merged 2 commits into from
Apr 26, 2017

Conversation

chris-b1
Copy link
Contributor

Since benchmarking on python 3, no longer want to install libpython

@chris-b1 chris-b1 added Compat pandas objects compatability with Numpy or Python functions Performance Memory or execution speed performance labels Apr 26, 2017
@codecov
Copy link

codecov bot commented Apr 26, 2017

Codecov Report

Merging #16139 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master   #16139   +/-   ##
=======================================
  Coverage   90.84%   90.84%           
=======================================
  Files         159      159           
  Lines       50779    50779           
=======================================
  Hits        46128    46128           
  Misses       4651     4651
Flag Coverage Δ
#multiple 88.62% <ø> (ø) ⬆️
#single 40.33% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d50b162...c8b3853. Read the comment docs.

@codecov
Copy link

codecov bot commented Apr 26, 2017

Codecov Report

Merging #16139 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master   #16139   +/-   ##
=======================================
  Coverage   90.84%   90.84%           
=======================================
  Files         159      159           
  Lines       50779    50779           
=======================================
  Hits        46128    46128           
  Misses       4651     4651
Flag Coverage Δ
#multiple 88.62% <ø> (ø) ⬆️
#single 40.33% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d50b162...c8b3853. Read the comment docs.

@codecov
Copy link

codecov bot commented Apr 26, 2017

Codecov Report

Merging #16139 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master   #16139   +/-   ##
=======================================
  Coverage   90.84%   90.84%           
=======================================
  Files         159      159           
  Lines       50779    50779           
=======================================
  Hits        46128    46128           
  Misses       4651     4651
Flag Coverage Δ
#multiple 88.62% <ø> (ø) ⬆️
#single 40.33% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d50b162...42f47bc. Read the comment docs.

@jorisvandenbossche
Copy link
Member

Can you keep those entries but specify it is only for python 3?
Then for people benchmarking on python 2, it keeps working

@chris-b1
Copy link
Contributor Author

@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 mingw toolchain.

@jorisvandenbossche
Copy link
Member

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 libpython: http://pandas-docs.github.io/pandas-docs-travis/contributing.html#creating-a-windows-development-environment

@chris-b1
Copy link
Contributor Author

Yeah, that's fair, I think MSVC would be the 'recommended' way now, but libpython is still valid for 2.x. I added a comment that hopefully is a decent middle ground?

@jorisvandenbossche jorisvandenbossche merged commit b8d9861 into pandas-dev:master Apr 26, 2017
@jorisvandenbossche
Copy link
Member

thanks!

@jorisvandenbossche jorisvandenbossche added this to the 0.20.0 milestone Apr 26, 2017
pcluo pushed a commit to pcluo/pandas that referenced this pull request May 22, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Compat pandas objects compatability with Numpy or Python functions Performance Memory or execution speed performance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants