-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
BUG: GH34677 Roll forward asv benchmark environment to py3.8 #34699
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
I think we could also just remove this entry altogether |
Just tested and that works for me as well. You think that's the better route to take? |
Yea I think so |
We need to verify that changing this won't cause a break in the timeseries at https://pandas.pydata.org/speed/pandas/. |
How do we do that? I have another PR pending that needs benchmarking |
I'm not sure what conditions cause asv to consider it a different environment.
Hopefully that isn't blocked by this issue. You can make the changes locally and run the ASVs. |
Oh duh yeah thanks |
I think this will cause a break in the timeseries (in a similar way as the cython version pin caused a new timeseries, see the two colors at eg https://pandas.pydata.org/speed/pandas/#algorithms.Hashing.time_frame), but I don't think there is a way around that? We need to do this update at some point, so the question is maybe if we are ok with doing it now. |
Even if this breaks the time series I would be OK with doing it. Maybe after the next release so we get a clean delineation? |
Yes, I think waiting until after the release sounds as a good idea |
Partially closes #34677