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

Project itz fixes v2 (Klaas) to master #1379

Merged
merged 337 commits into from
Jan 23, 2025
Merged

Conversation

jelgerjansen
Copy link
Contributor

@kldjonge I merged your changes into the project_itz_fixes_v2 branch of the open-ideas repo (via PR #1376). I'll use this branch to fix typos and update the reference results. If you want to change something, please pull the latest changes of the project_itz_fixes_v2 to your local branch and later create a PR to merge your changes into this branch.

If everything appears to be fine, we can merge in the stack_effect_project_itz_master branch, after which we can merge to the master branch.

@Mathadon FYI

Mathadon and others added 30 commits October 15, 2022 15:26
WallUnitTest update for interzonal airflow v2
 - refactored partialsurface such that it includes CrackOrOperableDoor
 - CrackOrOperableDoor implements both a regular orifice equation and a door
 - depending on the settings of interzonalairflow the former or the latter is used
 - both in a window and an internal wall the operable door can be used to increase the air flow rate
 - control inputs are added for this
 - added an optional control signal for a trickle vent in a window
 - added unit tests for the above
 - the amount of diffuse/direct light entering the zone is reduced/increased depending on the window opening control signal
 - an input for adding custom water vapor to the zone model ahs been added
 - an input for adding custom co2 to the zone model ahs been added
 - an output for the relative humidity has been added to the zone model
 - the reflection coefficient implementation in the window has been improved
 - the bridge type implementation has been refactored such that the user no longer must provide the bridge type length to get a reasonable estimate of the bridge
@lucasverleyen lucasverleyen added this to the Release 3.x milestone Jan 10, 2025
@jelgerjansen jelgerjansen changed the base branch from stack_effect_project_itz_master to master January 20, 2025 13:46
@jelgerjansen
Copy link
Contributor Author

I changed the base branch from "stack_effect_project_itz_master" to "master". Merging the latest master branch (and solving the conflicts) changes the following reference results:

IDEAS.Buildings.Components.Examples
IDEAS_Buildings_Components_Examples_CavityInternalWall

IDEAS.Buildings.Validation.Tests
IDEAS_Buildings_Validation_Tests_EnergyConservationValidation

IDEAS_Buildings_Validation_Tests_n50Test2_Statistics

IDEAS_Buildings_Validation_Tests_n50Test3_Statistics

IDEAS.Examples.PPD12
IDEAS_Examples_PPD12_VentilationRBC
IDEAS_Examples_PPD12_VentilationRBC_Statistics

IDEAS.Examples.Tutorial
IDEAS_Examples_Tutorial_Example6

IDEAS_Examples_Tutorial_Example7

IDEAS_Examples_Tutorial_Example9

IDEAS_Examples_Tutorial_Example10

IDEAS.Examples.TwinHouses
IDEAS_Examples_TwinHouses_BuildingN2_Exp2_1
IDEAS_Examples_TwinHouses_BuildingN2_Exp2_2
IDEAS_Examples_TwinHouses_BuildingN2_Exp2_3
IDEAS_Examples_TwinHouses_BuildingN2_Exp2_4

IDEAS_Examples_TwinHouses_BuildingN2_Exp2_Tset_1
IDEAS_Examples_TwinHouses_BuildingN2_Exp2_Tset_2

@jelgerjansen
Copy link
Contributor Author

The changes in IDEAS.Examples.Tutorial were strange at first sight as they do not use interzonal airflow. However, it appears that these changes can be attributed to the addition of a RealExpression AExp=A in IDEAS.Buildings.Components.Interfaces.PartialSurface,which is connected to the building component area in the PropsBus instead of the area within the layMul component. This has consequences for the window, as now the total window area (including frame) is used within the zone for airflow and radiation calculations instead of the window glass area A_glass.

We decided that the new implementation is better, as the radiation of/to the window frame is not considered in the current implementation, so the use of A instead of A_glass=A*(1-frac) partially compensates for this. This is clarified in the documentation of the window model (see commit f2e1fa8)

@jelgerjansen jelgerjansen merged commit 8b3818d into master Jan 23, 2025
2 checks passed
@jelgerjansen
Copy link
Contributor Author

This closes #1244 and PR #1327

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

Successfully merging this pull request may close these issues.

5 participants