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

Some HVAC components that exist in OpenStudio are missing from the HVAC library #599

Closed
DavidGoldwasser opened this issue Oct 30, 2013 · 11 comments

Comments

@DavidGoldwasser
Copy link
Collaborator

cc: @kbenne

One such example is Fan:OnOff, but there may be others.

@ghost ghost assigned asparke2 Oct 30, 2013
@mbadams5
Copy link
Contributor

mbadams5 commented Nov 1, 2013

Some comments regarding Fan:OnOff specifically since I ran into some issues with it and had to investigate. Fan:OnOff can not be added to an AirLoopHVAC but can only be uses in certain ZoneHVAC and Unitary systems. If you try to run E+ with a Fan:OnOff on an AirLoopHVAC (part of a Branch within E+) then you get the following error message...

This component may only be referenced by a parent component such as AirLoopHVAC:Unitary:Furnace:HeatCool or similar.

Thus any system that can use a Fan:OnOff object must define it as a variation within the HVAC library since there is no method to attach different fans from the GUI.

Also there are a quite a few bugs surrounding Fan:OnOff (addToNode, constructor, components hard coded to Fan:ConstantVolume, and more) but that is a separate issue.

@DavidGoldwasser
Copy link
Collaborator Author

In addition to some missing components in 1.1.1 I noticed that the category for "AirTerminal Single Duct VAV Reheat" shows up twice with "Air Terminal Single Duct Constant Volume Reheat" sandwiched in the middle.

@kbenne
Copy link
Contributor

kbenne commented Nov 21, 2013

Also parallel fan powered box.

@MatthewSteen
Copy link
Member

The following are missing in 1.6.0:

AirTerminal:SingleDuct:VAV:HeatAndCool:NoReheat

CoilCoolingWaterToAirHP

CoilHeatingWaterToAirHP

@macumber
Copy link
Contributor

This issue is too general, @kbenne let's make sure we have captured all the objects in this issue then open new issues for any future objects

@asparke2 asparke2 added this to the Version 1.7.0 milestone Feb 13, 2015
@macumber
Copy link
Contributor

macumber commented Mar 3, 2015

@kbenne @asparke2 I don't see this in pivotal. As I said before, this is too general. If there are any specific objects missing let's add those and close this issue.

@macumber
Copy link
Contributor

@kbenne @asparke2 I don't see this in pivotal. As I said before, this is too general. If there are any specific objects missing let's add those and close this issue.

@DavidGoldwasser
Copy link
Collaborator Author

I think at least 5 specific objects are listed in the issue and comments. If all of those are in, then we can close, it but not a bad idea to proactively check that everything is in there by going through them. In one case an object showed up twice in the list.

@kbenne
Copy link
Contributor

kbenne commented Jun 18, 2015

@MatthewSteen you may correct me if I'm wrong but I believe the only way to get CoilCoolingWaterToAirHP or CoilHeatingWaterToAirHP on air loop is to put it in a AirLoopHVACUnitarySystem. Even the coil system wrapper won't do it, if I recall correctly. I think the correct course of action on those two is to simply remove the category.

@kbenne
Copy link
Contributor

kbenne commented Jun 18, 2015

Parallel fan powered boxes are now covered.

screen shot 2015-06-17 at 8 42 34 pm

@kbenne
Copy link
Contributor

kbenne commented Jun 18, 2015

It appears that Air Terminal Single Duct VAV Reheat is correct in current code. See above screenshot.

kbenne added a commit that referenced this issue Jun 18, 2015
CoilCoolingWaterToAirHP and CoilHeatingWaterToAirHP are not allowed
directly on air loop, thus no drag and drop, no need to have a library
entry.  #599
@kbenne kbenne closed this as completed in d61d36c Jun 18, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants