Skip to content

Remove data_version parameter from get_meteonorm_tmy#2781

Open
AdamRJensen wants to merge 3 commits into
pvlib:mainfrom
AdamRJensen:meteonorm-test-data_version
Open

Remove data_version parameter from get_meteonorm_tmy#2781
AdamRJensen wants to merge 3 commits into
pvlib:mainfrom
AdamRJensen:meteonorm-test-data_version

Conversation

@AdamRJensen

Copy link
Copy Markdown
Member
  • I am familiar with the contributing guidelines
  • I attest that all AI-generated material has been vetted for accuracy and is in compliance with the pvlib license
  • Tests added
  • 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 remote data tests are currently failing due to the get_meteonorm_tmy function no longer supporting the data_version parameter.

@AdamRJensen AdamRJensen added this to the v0.15.2 milestone Jun 10, 2026
@AdamRJensen AdamRJensen added bug io remote-data triggers --remote-data pytests labels Jun 10, 2026
@AdamRJensen AdamRJensen changed the title Meteonorm test data version Remove data_version parameter from get_meteonorm_tmy Jun 10, 2026
@AdamRJensen AdamRJensen added remote-data triggers --remote-data pytests and removed remote-data triggers --remote-data pytests labels Jun 10, 2026
@AdamRJensen

Copy link
Copy Markdown
Member Author

The change is documented in the meteonorm changelog: https://docs.meteonorm.com/overview/changelog-api#2025-12-05

@kandersolar

Copy link
Copy Markdown
Member

We can deprecate this with a message "this parameter no longer has any effect (see XX meteonorm URL) and will be removed soon", right? Can we do that instead of jumping straight to removal?

@AdamRJensen

Copy link
Copy Markdown
Member Author

We can deprecate this with a message "this parameter no longer has any effect (see XX meteonorm URL) and will be removed soon", right? Can we do that instead of jumping straight to removal?

The issue is that the API requests are currently failing when the parameter is included.

@kandersolar

Copy link
Copy Markdown
Member

Right. Can't we keep it as a parameter to the pvlib function (to avoid a breaking change), and just stop including it in the API request?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug io remote-data triggers --remote-data pytests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants