search

NREL / SolarPILOT

SolarPILOT power tower layout and optimization tool
https://www.nrel.gov/csp/solarpilot.html
Other
47 stars 17 forks source link

Visualization of land boundaries #19

Closed mschmitz2 closed 6 years ago

mschmitz2 commented 6 years ago

See images below. The first image correctly shows the default land boundaries (-180° to °180°) with the correct radial limits. For the second image, the land boundaries have been changed to (-175°,+180°). However, the plot shows them as if they were (-180°,-175°).

-180_ 180

-175_ 180

Conversely, the land boundaries for (-180°,-175°) are plotted as (-175°,+180°) (not shown).

This error only appears in the 'Land Boundaries' plot; heliostats are positioned correctly according to the desired land boundaries.

mjwagner2 commented 6 years ago

@mschmitz2 I've also confirmed this issue. The relevant code is in app/FieldPlot.cpp, lines 370-387. It appears the arc points are drawn in reverse order. I'm adding a fix for this and will put it in the next patch release.

mschmitz2 commented 6 years ago

@mjwagner2 cool! Thanks!