NatLabRockies / NatLabRockies/OpenStudio
Default Cooling Coil should be CounterFlow
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 646
- Forks
- 237
- Avg merge
- 3d 11h
- Merged PRs (30d)
- 10
Description
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).
Current Behavior
When translated to EnergyPlus, the default field is translated to EnergyPlus as CrossFlow instead of the IDD default CounterFlow .
Expected Behavior
When translated to EnergyPlus, the default field is translated to EnergyPlus as the IDD default CounterFlow .
Steps to Reproduce
- Create air loop with OS:Coil:Cooling:Water, leave
Heat Exchanger Configurationblank - Translate to EnergyPlus
Possible Solution
Respect IDD default on translation or remove IDD default
Details
Originally reported at https://github.com/openstudiocoalition/OpenStudioApplication/issues/581 by @Ski90Moo
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start at the OS:Coil:Cooling:Water translation path into EnergyPlus and inspect how a blank Heat Exchanger Configuration is resolved against the IDD default. Reproduce with an air loop and blank field; done means EnergyPlus receives CounterFlow rather than CrossFlow, with regression coverage for this path.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- tooling
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100