Skip to content

Trough solar azimuth unit bug #1391

Description

@taylorbrown75

Describe the bug
Possible bug in the trough collector class (csp_solver_trough_collector_receiver.cpp) during the end gain calculations. In loop_optical_eta(), SolarAz is compared against 90 (degrees) but SolarAz has already been converted to radians. Code is likely always assuming sun is in southern sky, regardless of actual position.

Expected behavior
EndGain calculation should change depending on location of site relative to equator.
Change 90 to pi/2

Screenshots
Image

Activity

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

Metadata

Metadata

Labels

cspconcentrating solar power

Type

Projects

No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions