Skip to content

Swap nlr-pysam for nrel-pysam and bound python above at 3.15 exclusive - #2945

Closed
markcampanelli wants to merge 1 commit into
pvlib:mainfrom
markcampanelli:update_pysam
Closed

markcampanelli wants to merge 1 commit into
pvlib:mainfrom
markcampanelli:update_pysam

Conversation

@markcampanelli

@markcampanelli markcampanelli commented Nov 14, 2026 •

Copy link
Copy Markdown
Contributor
  • Closes Support PySAM on Python 3.14 via the nlr-pysam package name #2871 and #2994
  • 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
  • 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.

@markcampanelli

Copy link
Copy Markdown
Contributor Author

Here, I try to install the pvlib, including the nlr-pysam extra, using Python 3.15, which isn't quite fully compatible due to nlr-pysam and h5py being tardy with the latest wheels.

% uv sync --python 3.15 --no-install-package h5py --extra all
Using CPython 3.15.2
error: The requested interpreter resolved to Python 3.15.2, which is incompatible with the project's Python requirement:
       `>=3.11, <3.15` (from `project.requires-python`)

@markcampanelli

Copy link
Copy Markdown
Contributor Author

I'm happy to chase down conda-only test failures, so long as maintainers are ok with the upper bound on Python version.

@echedey-ls

Copy link
Copy Markdown
Member

This is a duplicate of #2872 - without pinning versions. I understand your motivation to pin versions, but I think at this point it is clear that it's just a debate with its own set of technical limitations and maintenance burden. I suggest closing this PR.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support PySAM on Python 3.14 via the nlr-pysam package name

2 participants