Skip to content

Conversation

@cwhanse
Copy link
Member

@cwhanse cwhanse commented Feb 9, 2022

  • Closes Advance numpy to 1.16.0 #1399
  • 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 and Milestone are assigned to the Pull Request and linked Issue.

Advance numpy to 1.16.0 or later. Repair some poorly formed tests for inverter.py.

@cwhanse cwhanse added this to the 0.9.1 milestone Feb 9, 2022
@kandersolar
Copy link
Member

setup.py, the other ci/requirements files, and benchmarks/asv.conf.json need the new version as well

@cwhanse
Copy link
Member Author

cwhanse commented Feb 9, 2022

Advancing numpy exposed two tests that should have failed, not sure why they were passing before.

Copy link
Member

@kandersolar kandersolar left a comment

Choose a reason for hiding this comment

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

Strange about the tests. I don't know why they were passing either.

@cwhanse cwhanse changed the title Numpy Advance numpy from 1.12.0 to 1.16.0 Feb 9, 2022
@cwhanse
Copy link
Member Author

cwhanse commented Feb 9, 2022

Based on comments on #1399 it appears we're in agreement. I'll merge tomorrow unless objections are raised.

@cwhanse cwhanse merged commit 95e25fd into pvlib:master Feb 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Advance numpy to 1.16.0

2 participants