Python and dependency bounds illustration
At this point this PR is for illustration only. Things to add/update would be:
- Adding Python 3.13 to the test matrix.
- A dependency management policy in the documentation. (See https://github.com/pvlib/pvlib-python/issues/2201#issuecomment-2558592339)
- Look into any changes/simplifications in CI that might be possible by calling out the Python upper limits in the
optionaldependencies. - ~timetimezones.rst would be revised to use the standard library's
timezonepackage instead ofpytz, which is still used bypandas.~ Moved to https://github.com/pvlib/pvlib-python/pull/2343
- [ ] Closes #xxxx (Inspired by #2201)
- [ ] I am familiar with the contributing guidelines
- [ ] Tests added
- [ ] Updates entries in
docs/sphinx/source/referencefor API changes. - [ ] Adds description and name entries in the appropriate "what's new" file in
docs/sphinx/source/whatsnewfor 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.
Could you put the changes relative to zoneinfo into another PR? I have the gut feeling one of the proposals will be more controversial than the other, and it's easier to follow a conversation on a proposal than on many at the same time.
Could you put the changes relative to zoneinfo into another PR? I have the gut feeling one of the proposals will be more controversial than the other, and it's easier to follow a conversation on a proposal than on many at the same time.
@echedey-ls Thanks for your patience and feedback as I learn how to best present my ideas to others in digestible chunks. I will split these two ideas up even though they originally came to me together :).
Question: Should I label/communicate my “demonstration/experimental” PRs with something other than [DRAFT]?
Question: Should I label/communicate my “demonstration/experimental” PRs with something other than [DRAFT]?
You can set them as drafts, there is a button on top right of the PR page, in the reviewers box, that says "Convert to draft". When you first open them, in the Open PR button there is dropdown to select Open as draft. Said that, it's not that important. Here in pvlib there is no convention, neither for the titles. I like to use the brackets so I can browse my PRs easier. Feel free to find what works best for you and be comfortable with your development procedures.
@echedey-ls Thanks for your patience
Thanks to you for proactively raising awareness of the user's needs.
@echedey-ls Please forgive the bigger diff than perhaps necessary in the pyproject.toml. I'm a stickler for alphabetizing dependencies, imports, etc.
Closing this PR as the maintainers have decided not to pursue this strategy.
Thanks for the discussion @markcampanelli, also interesting to learning about different perspectives.