Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated pd.date_range-API in plot_sunpath_diagrams.py #1997

Closed
wants to merge 1 commit into from

Conversation

PhilBrk8
Copy link
Contributor

freq='H' is depricated.
the parameter 'closed' was changed to 'inclusive'

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

freq='H' is depricated.
the parameter 'closed' was changed to 'inclusive'
@kandersolar
Copy link
Member

Right now our docs build uses python 3.7 (see https://github.com/pvlib/pvlib-python/blob/main/readthedocs.yml), which I'm guessing means it's also using an older pandas that doesn't support inclusive. I'm in favor of upgrading our docs build configuration to a more modern environment. @PhilBrk8 would you be up for trying that out in this PR?

@kandersolar kandersolar added this to the v0.10.5 milestone Mar 23, 2024
@PhilBrk8
Copy link
Contributor Author

Right now our docs build uses python 3.7 (see https://github.com/pvlib/pvlib-python/blob/main/readthedocs.yml), which I'm guessing means it's also using an older pandas that doesn't support inclusive. I'm in favor of upgrading our docs build configuration to a more modern environment. @PhilBrk8 would you be up for trying that out in this PR?

Hi there,
sorry - life got in the way a bit...

I'm currently doing the Solar Energy Engineering Program of the TU Delft on edx.org and i'm in the last of 4 courses.
I set my self the goal to transfer the content and calculations into code in order to review and solidify my learnings.

To go even further and incorporate useful parts into the library while bringing it up to a more modern environment might be a great opportunity to advance my python skills to the next level and polish up my github profile in the meantime as well.

@kandersolar Would you also be in favor of this?

@kandersolar
Copy link
Member

@kandersolar Would you also be in favor of this?

Hard to say without hearing some specifics, but in general we enthusiastically welcome contributions! Please open an issue describing any improvements you'd like to discuss :)

@kandersolar kandersolar modified the milestones: v0.10.5, 0.11.0 May 3, 2024
@kandersolar
Copy link
Member

Closing this PR as it is now redundant after #2029. Thanks @PhilBrk8, and hopefully we see more PRs from you in the future :)

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.

2 participants