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
build: pandas min version #16308
base: main
Are you sure you want to change the base?
build: pandas min version #16308
Conversation
Thank you for your contribution to Astropy! 🌌 This checklist is meant to remind the package maintainers who will review this pull request of some common things to look for.
|
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.
With v6.1rc out, we don't backport to v6.0.x anymore, so I adjusted the milestone. FYI.
Also, need to update this:
Line 80 in 7542bc9
oldestdeps: pandas==1.4.* |
Actually the |
It would be nice if we could tie oldest-deps in tox to our pyproject. |
@WilliamJamieson might have wrote something up for such things at one point but I don't remember where. I vaguely remember @jdavies-st had ideas too. 🤞 |
From a quick grep, we also don't need this anymore: astropy/astropy/table/table.py Line 95 in 7542bc9
Also the blurb about "Version 0.14" in Maybe can also get rid of this but @taldcroft should confirm: astropy/astropy/table/table.py Lines 4147 to 4154 in 7542bc9
|
https://github.com/spacetelescope/minimum_dependencies is the package you want. It was originally a script within |
Thanks @jdavies-st! Perfect for a follow up PR simplifying our tox.ini. |
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.
Fine for table
But @taldcroft , can we remove the snippet mentioned here? |
Yes, that can go since pandas >= 0.24 is always true. The one thing is that this removes the warning if the user specifies
This is a minor API change but a good cleanup for 7.0. In a perfect world this change will break a test (for the warning) but I'm not sure off-hand. |
pre-commit autofix |
pre-commit.ci autofix |
for more information, see https://pre-commit.ci
Hm. Appears that pandas v1.5.3 doesn't raise the same error as later versions, instead emitting a warning. |
See #15849 (comment)