Skip to content

fix documentation references to inverter/module data #786

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

Closed
wholmgren opened this issue Oct 4, 2019 · 3 comments · Fixed by #791
Closed

fix documentation references to inverter/module data #786

wholmgren opened this issue Oct 4, 2019 · 3 comments · Fixed by #791

Comments

@wholmgren
Copy link
Member

The CEC data update in #767 broke the documentation. At least the ModelChain page is broken, maybe more.

@wholmgren wholmgren added this to the 0.7.0 milestone Oct 4, 2019
@cwhanse
Copy link
Member

cwhanse commented Oct 4, 2019

Scanning the pages, these are also broken:
PVSystem attributes
PV Power Forecast

@birgits
Copy link
Contributor

birgits commented Oct 8, 2019

In the course of fixing this it would also be nice to fix the link to where the original files are downloaded from. I would have fixed it but could not find where the data sets are hosted.
Further, it would be great to provide a link to some documentation of the data sets in order for a user to figure out the meaning and units of the provided parameters. I think this link would be helpful:
https://sam.nrel.gov/photovoltaic/pv-sub-page-2.html

@cwhanse
Copy link
Member

cwhanse commented Oct 10, 2019

In the course of fixing this it would also be nice to fix the link to where the original files are downloaded from. I would have fixed it but could not find where the data sets are hosted.

Thanks for pointing out this old link. The current link.

Further, it would be great to provide a link to some documentation of the data sets in order for a user to figure out the meaning and units of the provided parameters. I think this link would be helpful:
https://sam.nrel.gov/photovoltaic/pv-sub-page-2.html

Agree, and again, thanks!

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

Successfully merging a pull request may close this issue.

3 participants