-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add CO2 features to large office complex air #4
Add CO2 features to large office complex air #4
Conversation
Issue406 spawn support
1. Replace Medium Air with Buildings library air media package with CO2 2. CO2 inputs are calculated based on the number of occupant in each thermal zone 3. Add CO2 sensors for each thermal zone
However, error: could not solve simplified initialization for homotopy method; Integrator failed to start model.
Although 'TestSuccess' run successfully, we need to debug why 'TestCase' failed to initialize since two system models are almost the same.
…eOffice_complexAir
LoadWrapper needs to be put behind of HVAC to avoid model initialization failure (but reason is not clear yet)
...complex_air/models/MultizoneOfficeComplexAir/BaseClasses/HVACSide/BaseClasses/BoilerPlant.mo
Outdated
Show resolved
Hide resolved
...e_complex_air/models/MultizoneOfficeComplexAir/BaseClasses/HVACSide/BaseClasses/Component.mo
Outdated
Show resolved
Hide resolved
...s/multizone_office_complex_air/models/MultizoneOfficeComplexAir/BaseClasses/HVACSide/HVAC.mo
Show resolved
Hide resolved
...s/multizone_office_complex_air/models/MultizoneOfficeComplexAir/TestCases/TestSuccess.bak-mo
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@GuowenLi-PhD , thank you. Please see my in-line comments, make corresponding changes, and submit a new PR.
Modelica middle floor extends from E+ middle floor to ten floors
All comments resolved, I have requested a new review. |
03a9186
into
terrancelu92:issue1_addCO2_largeOffice_complexAir
The testcase model after adding CO2 sensors cannot be solved by Dymola 2021 but Dymola 2024. The following errors are reported:
|
|
@GuowenLi-PhD , please output the |
Reply to @terrancelu92
|
Use command
The reason of the nonlinear and initialization issue need to be further investigated. (Hold on) |
Main Updates:
Remain Question
MultizoneOfficeComplexAir.TestCases.TestCase
, the code ofMultizoneOfficeComplexAir.BaseClasses.LoadSide.LoadWrapper
needs to be placed afterMultizoneOfficeComplexAir.BaseClasses.HVACSide.HVAC
to simulate successfully, otherwise the model initialization fails. The specific reason for this setting needs to be clarified.Potential solution: please refer to pull request Adjusted system equipment sizing and Resolved nonlinear system warnings. #7 : To avoid initialization failure, set initial economizer damper positions as open (refer to the commit e370209)