Skip to content

Improve docstring or behavior for irradiance.get_total_irradiance and irradiance.get_sky_diffuse #949

Closed
@cwhanse

Description

@cwhanse

pvlib.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'

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions