Skip to content

docstring updated to indicate limits of axis_tilt #1489

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

Merged
merged 9 commits into from
Jul 19, 2022
Merged

docstring updated to indicate limits of axis_tilt #1489

merged 9 commits into from
Jul 19, 2022

Conversation

kurt-rhee
Copy link
Contributor

@kurt-rhee kurt-rhee commented Jul 6, 2022

@kandersolar
Copy link
Member

Cool, thanks @kurt-rhee. I think it probably makes sense to make the same comment everywhere the axis_tilt parameter exists, which is several functions in pvlib.tracking.

Also please add yourself to the list of contributors in the 0.9.2 what's new file, if you like!

@kandersolar kandersolar added this to the 0.9.2 milestone Jul 6, 2022
@kurt-rhee
Copy link
Contributor Author

@kanderso-nrel Added those additional references when axis_tilt is used as an input and also added myself to the list of contributors. Thanks for the help!

Copy link
Member

@cwhanse cwhanse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some formatting. Thanks @kurt-rhee

kurt-rhee and others added 4 commits July 7, 2022 07:41
Co-authored-by: Cliff Hansen <[email protected]>
Co-authored-by: Cliff Hansen <[email protected]>
Co-authored-by: Cliff Hansen <[email protected]>
Co-authored-by: Cliff Hansen <[email protected]>
Copy link
Member

@kandersolar kandersolar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @kurt-rhee, both for this PR and the discussion over in #1471!

@kandersolar kandersolar merged commit 3f397ed into pvlib:master Jul 19, 2022
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.

3 participants