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

Return Fan:VariableVolume Fan Power Minimum Flow Fraction imposes air loop flow #6050

Closed
ecoeficiente opened this issue Mar 8, 2017 · 5 comments · Fixed by #8401
Closed
Assignees
Labels
Defect Includes code to repair a defect in EnergyPlus PriorityLow This defect or feature has been declared low priority

Comments

@ecoeficiente
Copy link
Contributor

Issue overview

Return Fan:VariableVolume imposes air loop flow to Fan Power Minimum Flow Fraction despite E+I/O states the opposite:

This field does not enforce the system air flow rate during simulation

Details

It seems to happen with return fan but not with supply fan.

@Nigusse
Copy link
Contributor

Nigusse commented May 22, 2017

Looking at the source code confirms that the input field Fan Power Minimum Flow Fraction value is used for fan power calculation only. @ecoeficiente Do you have idf file that shows the defect?

@Nigusse Nigusse added the UnconfirmedDefect Defect has not yet confirmed to be an actual issue label May 23, 2017
@Nigusse
Copy link
Contributor

Nigusse commented May 23, 2017

@mjwitte There is no defect file to confirm the defect and looking at the source code the value of Fan Power Minimum Flow Fraction is used for fan power calculation only. I labeled the issue to unconfirmed defect.

@mjwitte mjwitte added the PriorityLow This defect or feature has been declared low priority label May 24, 2017
@ecoeficiente
Copy link
Contributor Author

So sorry I did not notice your request for example files. I attatched 2 idf files, everything identical but Fan Power Minimum Flow Fraction in AHU return fan.
You will see that the one with 0.3 fraction imposes the flow (the one with 0 fraction gets no flow).

FlowFractionBug.zip

@jmarrec
Copy link
Contributor

jmarrec commented Nov 27, 2020

I took the files from @ecoeficiente , updated them to E+ 9.4 and I can reproduce the issue : Here I am plotting the ('CIRCUITO DE AIRE SUPPLY SIDE OUTLET 1', 'System Node Mass Flow Rate', 'kg/s') variable

image

Interestingly I created a MCVE test case using OpenStudio with a simple AirLoopHVAC that has two Fan:VariableVolume, and changing the Min Flow Fraction on the return fan from 0 to 0.3 has no effect there...

image

@jmarrec jmarrec added Defect Includes code to repair a defect in EnergyPlus and removed UnconfirmedDefect Defect has not yet confirmed to be an actual issue labels Nov 27, 2020
@jmarrec jmarrec self-assigned this Nov 27, 2020
@jmarrec
Copy link
Contributor

jmarrec commented Nov 27, 2020

Ok, I found it, it's becasue my VAV:NoReheat terminal had a Constant Minimum Air Flow Fraction = 0.3... Switching it to 0 like in the defect files exhibits the defect.

image

jmarrec added a commit that referenced this issue Dec 1, 2020
jmarrec added a commit that referenced this issue Dec 1, 2020
jmarrec added a commit that referenced this issue Dec 2, 2020
… Flow Rate) to the outlet node's MassFlowRateMin
mitchute added a commit that referenced this issue Dec 11, 2020
Fix #6050 - Return Fan:VariableVolume Fan Power Minimum Flow Fraction should not impose air loop flow
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Defect Includes code to repair a defect in EnergyPlus PriorityLow This defect or feature has been declared low priority
Projects
None yet
4 participants