Stuck on an issue?
Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.
And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.
Improve docstring or behavior for irradiance.get_total_irradiance and irradiance.get_sky_diffuse
See original GitHub issuepvlib.irradiance.get_total_irradiance accepts kwargs dni_extra and airmass, both default to None. However, values for these kwargs are required for several of the irradiance transposition models.
See discussion here
Docstring should specify when dni_extra and airmass are required, and which airmass is appropriate for each model.
Could also test for kwarg values if e.g. model=='perez'
Issue Analytics
- State:
- Created 3 years ago
- Comments:5 (5 by maintainers)
Top Results From Across the Web
No results found
Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free
Top Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
PVSystem.get_irradiancehas some relevant shim code:https://github.com/pvlib/pvlib-python/blob/04a523fafbd61bc2e49420963b84ed8e2bd1b3cf/pvlib/pvsystem.py#L289-L294
I’ll volunteer to take this up for v0.8.1, since I was the complainer.