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

Broken link in pvsystem.retrieve_sam #2288

Open
cwhanse opened this issue Nov 4, 2024 · 3 comments · May be fixed by #2312
Open

Broken link in pvsystem.retrieve_sam #2288

cwhanse opened this issue Nov 4, 2024 · 3 comments · May be fixed by #2312

Comments

@cwhanse
Copy link
Member

cwhanse commented Nov 4, 2024

The link to documentation of CEC parameter sets is broken.

Expected behavior
Link to a current webpage? A bit of searching hasn't turned up a target for this link.

I don't think pvlib should be responsible to maintain a description of those databases. I favor just removing the dead link.

@cwhanse
Copy link
Member Author

cwhanse commented Nov 4, 2024

@janinefreeman

@kandersolar
Copy link
Member

archive.org says it's just the PV publications page: https://web.archive.org/web/20200618135912/https://sam.nrel.gov/photovoltaic/pv-sub-page-2.html

I agree that the link is not really helpful for these databases, but for completeness, the new url is: https://sam.nrel.gov/photovoltaic/pv-publications.html

@janinefreeman
Copy link

Let me know if you need any other links!

@RDaxini RDaxini linked a pull request Nov 27, 2024 that will close this issue
4 tasks
@RDaxini RDaxini added this to the v0.11.2 milestone Nov 27, 2024
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.

4 participants