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

Default Cooling Coil should be CounterFlow #581

Closed
Ski90Moo opened this issue Dec 12, 2022 · 1 comment
Closed

Default Cooling Coil should be CounterFlow #581

Ski90Moo opened this issue Dec 12, 2022 · 1 comment
Labels
OpenStudio SDK Issue An issue that is not related to OpenStudioApplication but to NREL/OpenStudio SDK

Comments

@Ski90Moo
Copy link

Issue overview

The default cooling coil OS:Coil:Cooling:Water from the HVAC library has ! Heat Exchanger Configuration as nil. This is correct and should be translated to EnergyPlus as the default (CounterFlow for cooling coils).

However, when dropping the object into a loop using the OpenStudio App, the App appears to assign CrossFlow to the object (shown in the drop down menu). Inspecting the osm file, it does not actually assign CrossFlow (it is still nil). But, when the simulation is run, it gets translated to EnergyPlus as CrossFlow.

Current Behavior

Expected Behavior

Steps to Reproduce

  1. Create air loop and drop in a chilled water coil. App shows CrossFlow is selected:
    2022-12-12

  2. Inspecting the osm file, it shows nil:
    2022-12-12 (1)

  3. Inspecting the idf input file, it shows CrossFlow:
    2022-12-12 (2)

Possible Solution

Details

Environment

Some additional details about your environment for this issue (if relevant):

  • Platform (Operating system, version):
  • Version of OpenStudioApplication (if using an intermediate build, include SHA):
    2022-12-12 (4)

Context

@Ski90Moo Ski90Moo added the Triage Issue needs to be assessed and labeled, further information on reported might be needed label Dec 12, 2022
@macumber
Copy link
Collaborator

Thanks @Ski90Moo, this is an OS SDK issue so I posted this over at NREL/OpenStudio#4772

As a work around in the meantime, you can select CrossFlow in the app and then select CounterFlow, this will hard assign the values in that field

@macumber macumber added OpenStudio SDK Issue An issue that is not related to OpenStudioApplication but to NREL/OpenStudio SDK and removed Triage Issue needs to be assessed and labeled, further information on reported might be needed labels Dec 20, 2022
@jmarrec jmarrec closed this as completed Dec 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OpenStudio SDK Issue An issue that is not related to OpenStudioApplication but to NREL/OpenStudio SDK
Projects
None yet
Development

No branches or pull requests

3 participants