-
Notifications
You must be signed in to change notification settings - Fork 18
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
Updated df.applymap to df.map to silence FutureWarning #42
Conversation
Right on, thanks for the PR! Is this change backwards compatible or does it break with slightly older pandas versions? If the latter can you update the pandas requirement to >=the minimum version? Once this is done I'll merge and push out a release, thanks again!! |
Looks like the pandas version required is not released on 3.8 - I'm happy to drop 3.8 support here too and will remove it from the test matrix |
Thanks for taking a look at it! I made the change you suggested to setup.cfg and verified it with a test install. The pandas docs suggest that
|
@ray-berkeley tests are passing for me locally and in the test matrix on >3.8 - could you paste the contents of your file so I can double check it myself? Hopefully it's a local issue for you... if I don't hear back in a couple of days I'll merge here and if it breaks I'm sure I'll hear about it :-) Thanks again for the PR! |
Sure! Here's a minimal starfile that should reproduce the error. It only seems to be a problem with Pandas 2.1.0:
|
thanks! this is working with the version of pandas that is currently installed in |
Sweet, thank you! |
Small change to silence a FutureWarning that is thrown by newer versions of Pandas