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

New top-down parameters for a partial retrofitted BES #78

Closed
HvanderStok opened this issue Apr 24, 2024 · 2 comments · Fixed by #79
Closed

New top-down parameters for a partial retrofitted BES #78

HvanderStok opened this issue Apr 24, 2024 · 2 comments · Fixed by #79
Assignees
Labels
enhancement New feature or request

Comments

@HvanderStok
Copy link
Contributor

To simulate a partially retrofitted BES, introducing new top-down parameters such as QBuiNoRetrofit_flow_nominal and THydSupNoRetrofit_nominal would be beneficial. These parameters could be utilized for further parameterization so that these subsystems are automatically designed for the non-retrofitted state of the building. All retrofitted subsystems would still use the parameters QBui_flow_nominal and THydSup_nominal of the actual building.

@HvanderStok HvanderStok added the enhancement New feature or request label Apr 24, 2024
@HvanderStok HvanderStok self-assigned this Apr 24, 2024
Copy link

@FWuellhorst
Copy link
Contributor

Regarding naming discussion, another option would be to use _nominal and _actual, with _nominal referring to the old values and _actual to those which are "actually" applied in the simulation. However, I think some users may think _actual is referring to the dynamic variable and not the parameter. Moreover, if _nominal is not used for all cases as the nominal value of the simulation, this further confuse. Thus, let's stay with _design and _nominal. Either way, the documentation has to specify it clearly :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants