Skip to content

Conversation

@kandersolar
Copy link
Member

  • [ ] 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.

Latest remote-data tests are failing with (See https://github.com/pvlib/pvlib-python/actions/runs/21646769714/job/62400674396):

FAILED tests/iotools/test_meteonorm.py::test_get_meteonorm_training
requests.exceptions.HTTPError: Meteonorm API returned an error:
invalid time range: make sure 'start' and 'end' are in the range 2024-01-01T00:00:00Z ... 2026-01-27T20:37:28Z`

Meteonorm docs say that "Available time period: the current year excluding the last 7 days, plus the latest two full calendar years."

This PR updates the requested dates and associated expected values to follow this requirement.

@kandersolar kandersolar added this to the v0.15.0 milestone Feb 3, 2026
@kandersolar kandersolar added the remote-data triggers --remote-data pytests label Feb 3, 2026
@kandersolar
Copy link
Member Author

I won't bother anyone for a review for such a trivial fix. Merging...

@kandersolar kandersolar merged commit 247b31f into pvlib:main Feb 3, 2026
36 of 37 checks passed
@kandersolar kandersolar deleted the test_meteonorm branch February 3, 2026 21:02
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant