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

Ventilation and infiltration definitions in building.py #253

Open
gianlucamaracchini opened this issue Jul 26, 2021 · 1 comment
Open

Ventilation and infiltration definitions in building.py #253

gianlucamaracchini opened this issue Jul 26, 2021 · 1 comment

Comments

@gianlucamaracchini
Copy link

gianlucamaracchini commented Jul 26, 2021

Hi,
looking at building.py, I found these args definitions in Building():
infil: Infiltration rate (ACH).
vent: Ventilation rate (ACH).
However, in Building() properties:
* vent -- Ventilation (m^3/s/m^2)
The same unit of measure is used for vent in SchDef. Then, it could be a mistake regarding the unit of measure of "vent".

Looking at the code in building.py, it would seems that this second definition is the correct one, indeed:
# total vent volumetric flow [m3 s-1]
volVent = self.vent * self.nFloor
volInfil = self.infil * UCM.bldHeight /
3600. # Change of units AC/H -> [m3 s-1]

However, why using a different formulation for "infil" and "vent"?

Many thanks,
G.

@saeranv
Copy link
Member

saeranv commented Jul 26, 2021

@jmrk87

Yes that's a typo, the vent metric should be [m3 s-1 m-2]. Also the subsequent volVent and volInfil metric should actually be [m3 s-1 m-2] as well, since the building energy balance (and I think canyon as well) is normalized by floor area, or else those volumetric derivations don't make sense.

The different metrics are just due to different conventions used by the original DOE reference buildings.

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

No branches or pull requests

2 participants