Fix tower height cost issue#475
Conversation
cpaulgilman
commented
Dec 8, 2020
- Add h_tower to UI equation for ui_tower_height input variable.
- Adjust group box widths to match O&M costs form
- Fix alignment issues
* Add h_tower to UI equation for ui_tower_height input variable. * Adjust group box widths to match O&M costs form * Fix alignment issues
|
Can you explain this issue to me? I see that changing the tower height on the Tower and Receiver page does not update the same-named calculated variable on the System Costs page, but do we know why this happened? The input on the Tower and Receiver page is named 'THT' and 'h_tower' is just calculated by returning 'THT'. Now we're simplifying this equation by just returning 'h_tower' instead of either 'h_tower' or 'THT', but it doesn't appear it would affect the outcome (although it does). What am I missing? |
|
The change that fixes this issue is on the Tower SolarPilot Capital Costs form not on the Tower Capital Costs form. The The Tower Capital Costs form is used by the ISCC and DSPT models, which are both disabled in SAM 2020.11.29. I made the change there in case we bring those models back to life in the future. The |
|
Thanks, that should have been obvious. I added a commit to align one of the adjacent input boxes. |