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

Single Zone VAV Conventional Controller Not Cooling in Setup #2265

Closed
dhblum opened this issue Nov 20, 2020 · 2 comments
Closed

Single Zone VAV Conventional Controller Not Cooling in Setup #2265

dhblum opened this issue Nov 20, 2020 · 2 comments

Comments

@dhblum
Copy link
Contributor

dhblum commented Nov 20, 2020

I believe there is a bug in Buildings.Air.Systems.SingleZone.VAV.ChillerDXHeatingEconomizerController where the controller does not turn on the fan if cooling is needed in setup (that is, during unoccupied hours but the room temperature still goes above the higher zone air temperature cooling setpoint).

In addition, I believe:

  • the controller component model should be located in Buildings.Air.Systems.SingleZone.VAV.BaseClasses with the other controllers
  • it's name changed to ControllerChillerDXHeatingEconomizer to be more consistent with the other controllers
  • validation example added to Buildings.Air.Systems.SingleZone.VAV.BaseClasses.Validation to check for the above condition and make sure it is fixed

A validation model has been made to reproduce the bug. It can be seen in the screenshot below at the time indicated. Here, the fan should turn on to cool the space, but does not (as seen in the second plot).

Screen Shot 2020-11-20 at 5 04 11 PM

The proposed fix is to add a connection between orFan and chOnTRoo as shown in the figure below. Note that the existing description of orFan is "Switch fan on if heating, cooling, or occupied" so I believe this issue address the already intended operation.

Screen Shot 2020-11-20 at 5 06 11 PM

Then, the validation model shows the desired behavior in the Figure below.

Screen Shot 2020-11-20 at 5 07 51 PM

@dhblum
Copy link
Contributor Author

dhblum commented Nov 20, 2020

My above comments are addressed in #2266.

mwetter added a commit that referenced this issue Dec 22, 2020
@mwetter
Copy link
Member

mwetter commented Dec 22, 2020

Merged on the master.

@mwetter mwetter closed this as completed Dec 22, 2020
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