Skip to content
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

Compatibility with pandas 2.0.0 #1672

Merged
merged 3 commits into from
Apr 24, 2023
Merged

Compatibility with pandas 2.0.0 #1672

merged 3 commits into from
Apr 24, 2023

Conversation

kandersolar
Copy link
Member

@kandersolar kandersolar commented Feb 21, 2023

- [ ] Closes #xxxx

  • I am familiar with the contributing guidelines
    - [ ] Tests added
    - [ ] Updates entries in docs/sphinx/source/reference for API changes.
    - [ ] Adds description and name entries in the appropriate "what's new" file in docs/sphinx/source/whatsnew for all changes. Includes link to the GitHub Issue with :issue:`num` or this Pull Request with :pull:`num`. Includes contributor name and/or GitHub username (link with :ghuser:`user`).
    - [ ] New code is fully documented. Includes numpydoc compliant docstrings, examples, and comments where necessary.
  • Pull request is nearly complete and ready for detailed review.
  • Maintainer: Appropriate GitHub Labels (including remote-data) and Milestone are assigned to the Pull Request and linked Issue.

The first release candidate for pandas 2.0.0 was released yesterday: https://github.com/pandas-dev/pandas/releases/tag/v2.0.0rc0. It's available on PyPI with pip install --pre pandas==2.0.0rc0.

The list of changes is pretty extensive, but it seems like we escape nearly unscathed: it doesn't seem to break anything in pvlib itself and only causes one small issue in our tests. This PR fixes that test issue.

Since pvlib itself seems to be compatible, I suggest we wait to merge this PR until pandas 2.0.0 is officially released (anticipated in "a couple weeks") in case there are any other breaking changes made between now and then.

@kandersolar kandersolar added this to the 0.9.5 milestone Feb 21, 2023
@adriesse
Copy link
Member

The first release candidate for pandas 2.0.0 was released yesterday

This PR fixes that test issue.

I'm impressed!

@kandersolar kandersolar mentioned this pull request Mar 2, 2023
12 tasks
@kandersolar
Copy link
Member Author

This branch has no failures on the just-released 2.0.0rc1 as well. Their ETA for the actual release of pandas 2.0.0 is still "about two weeks".

@kandersolar kandersolar modified the milestones: 0.9.5, 0.9.6 Mar 18, 2023
@kandersolar kandersolar closed this Apr 3, 2023
@kandersolar kandersolar reopened this Apr 3, 2023
@kandersolar kandersolar added the remote-data triggers --remote-data pytests label Apr 3, 2023
@kandersolar kandersolar added remote-data triggers --remote-data pytests and removed remote-data triggers --remote-data pytests labels Apr 4, 2023
@kandersolar kandersolar marked this pull request as ready for review April 4, 2023 13:43
@kandersolar
Copy link
Member Author

This is ready for review now that pandas 2.0.0 has been released.

Note that wheels are available on PyPI, but the new version isn't yet available on conda, so the CI isn't currently using it for the remote-data tests. In lieu of that I ran pytest pvlib/tests/iotools/ --remote-data locally with pandas 2.0.0 installed and all looks good.

Copy link
Member

@AdamRJensen AdamRJensen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@AdamRJensen AdamRJensen merged commit b0ac666 into pvlib:main Apr 24, 2023
@kandersolar kandersolar deleted the pd200 branch April 24, 2023 12:23
@kandersolar kandersolar modified the milestones: 0.9.6, 0.10.0 May 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
remote-data triggers --remote-data pytests testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants