Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

move rec su vars to system control page #1961

Merged
merged 1 commit into from
Dec 7, 2024
Merged

Conversation

tyneises
Copy link
Collaborator

@tyneises tyneises commented Dec 7, 2024

Both the Trough and Fresnel models included inputs on the Solar Field page for “Receiver startup delay time” and “receiver startup delay energy fraction”. These variables originated from the power tower model, where the component performance model uses these values to determine when the receiver can generate useful thermal energy. However, the trough and Fresnel performance model tracks the solar field temperature when the field is “off” (not sending heat to TES or cycle), so these two inputs are not used in the performance model. However, when trough and Fresnel is used in the dispatch model, we use these two inputs as a way for the dispatch model to guess the time and energy startup penalty for the field.

This pull request moves these two inputs from the Solar Field page to the System Control page for both trough and Fresnel.

@tyneises tyneises requested a review from cpaulgilman December 7, 2024 00:27
@tyneises tyneises merged commit ba373aa into develop Dec 7, 2024
2 of 8 checks passed
@tyneises tyneises deleted the move-linear-rec_su branch December 7, 2024 02:51
@cpaulgilman cpaulgilman added this to the SAM Fall 2024 Release milestone Dec 9, 2024
@cpaulgilman cpaulgilman added requires help revision Requires a Help revision before releasing public version added to release notes PR and/or issue has been added to release notes for a public release and removed requires help revision Requires a Help revision before releasing public version labels Dec 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
added to release notes PR and/or issue has been added to release notes for a public release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants