Skip to content

Add hot-gas reheat unitary system (Coil:Heating:Desuperheater)#891

Open
Ski90Moo wants to merge 6 commits into
openstudiocoalition:developfrom
Ski90Moo:feature/desuperheater-hotgas-reheat
Open

Add hot-gas reheat unitary system (Coil:Heating:Desuperheater)#891
Ski90Moo wants to merge 6 commits into
openstudiocoalition:developfrom
Ski90Moo:feature/desuperheater-hotgas-reheat

Conversation

@Ski90Moo

Copy link
Copy Markdown

Summary

  • Adds "Unitary - Single Speed DX cooling - Elec heat - CAV - Desuperheater reheat" to the default HVAC library: the existing single-speed DX cooling coil paired with a new Coil:Heating:Desuperheater as the supplemental/reheat coil under CoolReheat dehumidification control, so reheat is partially sourced from the cooling coil's waste heat instead of just resistance heat.
  • Fixes AirLoopHVACUnitarySystem/loop cloning (both "drag from library" and "Copy System"): ModelObject::clone() only remaps true parent-child relationships. Lateral (sibling-to-sibling) object-list references — like CoilHeatingDesuperheater::heatingSource(), or a SetpointManager:MixedAir's node fields — are left broken on the clone, sometimes stale, sometimes cleared outright. Added a generic post-clone reference fixup (buildCloneHandleMap + remapLateralReferences in HVACSystemsController.cpp, using IddObject::objectLists()/WorkspaceObject::getTarget()/setPointer()) rather than a Desuperheater-specific special case.
  • The traversal walks children(), a loop's supplyComponents()/demandComponents(), and an outdoor air system's oaComponents()/reliefComponents() as independently size-matched groups, since branch equipment (units on a loop, or equipment like an evaporative cooler on an OA system's branches) isn't reachable via children() alone, and demand-side counts can legitimately differ between original and clone since connected thermal zones are never duplicated.

Context

Modeling dehumidification via hot-gas/desuperheater reheat is a recurring real-world need — see this UnmetHours thread, where CoilCoolingDXTwoStageWithHumidityControlMode proved awkward to control directly and commenters pointed toward Coil:Heating:Desuperheater as a workaround. Worth noting the thread's caveat: a desuperheater coil reclaims waste heat from the refrigerant's superheat and doesn't affect compressor performance, so it's a lower-fidelity approximation of "true" hot-gas reheat, not an exact substitute — but it's a standard, supported EnergyPlus object and a reasonable default-library addition for this use case.

Proof this was broken before this fix

You don't need the new desuperheater system to see the underlying bug — it reproduces with a completely stock model:

  1. In the app, File > Examples > Example Model to generate a fresh example model.
  2. Go to the HVAC Systems tab, select the one airloop, click "Copy System."
  3. Select the copy and open its outdoor air system. It has an evaporative cooler with a SetpointManager:MixedAir attached — but on the copy, that setpoint manager's Reference Setpoint / Fan Inlet / Fan Outlet Node fields are all empty (shown as warning icons in the inspector), even though the original's are filled in correctly.
  4. Run a simulation on the copy: EnergyPlus fails immediately with a fatal error (missing required properties on that object) before any calculation happens.

That's ModelObject::clone()'s lateral-reference bug on a totally unmodified example model — this PR's fixup resolves it for both this stock case and the new desuperheater system.

Test plan

  • cmake --build . --target openstudio_lib --config Release succeeds
  • Dragged the new system onto a zone in the app and ran the simulation successfully
  • Used "Copy System" on that airloop, reassigned zones on the copy, and ran the simulation successfully
  • Verified against the SDK directly (Ruby) for same-model/cross-model clone scenarios, and against OpenStudio::Model.exampleModel's stock outdoor-air-system evaporative cooler + SetpointManager:MixedAir
  • Confirmed live in the rebuilt app that "Copy System" correctly re-wires node references on that stock example airloop too

Ski90Moo and others added 6 commits July 15, 2026 14:01
This repo is built in-place at the repo root rather than in an
out-of-source build/ dir, so CMake, Conan, MSBuild, and Qt's moc/rcc all
drop generated files directly alongside tracked source. None of that was
covered by the existing (out-of-source-build-oriented) ignore rules,
so every local build left ~150+ untracked files/dirs in git status.

Also ignores the handful of configure_file()-generated files produced
from tracked .in templates -- notably AnalyticsHelperSecrets.hxx, which
is populated with real secret values at configure time and should never
be committed.

Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
Adds 'Unitary - Single Speed DX cooling - Elec heat - CAV - Hotgas reheat' to
the default HVAC library, using CoolReheat dehumidification control with a
Coil:Heating:Desuperheater as the reheat coil. The desuperheater reclaims
waste heat from the unitary system's own DX cooling coil rather than using
resistance reheat, avoiding the extra energy cost of electric reheat.

Also fixes AirLoopHVACUnitarySystem cloning (drag-from-library and "copy
system"): ModelObject::clone() only remaps parent-child ownership, but
CoilHeatingDesuperheater::heatingSource() is a lateral reference to a sibling
coil, so it was left unset on the clone. fixupClonedDesuperheaterHeatingSource()
re-links the cloned desuperheater coil to the cloned cooling coil.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
ModelObject::clone() clones each child individually and reattaches it via
setParent(), so parent-child edges are correctly remapped, but lateral
(sibling-to-sibling) object-list references within the cloned subtree are
left pointing at the original objects. The previous fix special-cased just
CoilHeatingDesuperheater::heatingSource(), but the same bug applies to any
lateral reference (e.g. SetpointManager:MixedAir's node fields).

Replace it with a generic fixup: build an old-handle -> clone map by
walking the original and cloned subtrees in parallel, then use
IddObject::objectLists()/WorkspaceObject::getTarget()/setPointer() to
rewrite any object-list field in the clone that still points into the
original subtree. This fixes the desuperheater case as before, and any
future lateral-reference case, without needing a new special case per
object type.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Rename the library template to "Desuperheater reheat" for clarity.

Testing the generalized fixup against the real SDK (loading
hvac_library.osm and cloning the new UnitarySystem both within the same
model and across models) showed remapLateralReferences was a no-op for
CoilHeatingDesuperheater::heatingSource(): clone() doesn't leave that
field pointing at the stale original object, it clears it outright. The
previous implementation only rewrote fields that already had some target
set on the clone, so it silently did nothing for this case -- reproduced
live in the app as a fatal EnergyPlus error (missing heating_source_name
/ heating_source_object_type) when simulating the new system.

Fixed by reading the field to remap from `original` instead of from the
clone, and unconditionally forcing the clone's field to match whenever
the original's target was itself part of the cloned subtree. Verified
against the SDK directly (Ruby) for both same-model ("copy system") and
cross-model (drag-from-library) clone scenarios, and confirmed the
simulation now runs cleanly in the rebuilt app.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
ParentObject::children() is true ownership (e.g. a UnitarySystem's own
fan/coils) but does not include HVACComponents placed on a Loop's
supply/demand branches. A UnitarySystem sitting on an AirLoopHVAC's
supply branch is only reachable via supplyComponents(), not children().
Cloning a whole loop ("Copy System") therefore never visited the branch
equipment at all -- including the desuperheater's heating source inside
it -- because buildCloneHandleMap/remapLateralReferences only recursed
via children().

Reproduced live: copying an existing (working) desuperheater airloop and
reassigning zones still hit the same fatal EnergyPlus error, since the
copy's desuperheater was never touched by the fixup.

Fixed by walking children(), supplyComponents(), and demandComponents()
as independently size-matched groups rather than one combined list --
demand-side counts legitimately differ between original and clone
(connected thermal zones are never duplicated by clone(), by design), so
that mismatch must not block fixing up the supply side, where the
equipment lines up 1:1 with the original.

Verified against the SDK directly (Ruby) by reproducing the actual
AirLoopHVAC topology (OA system, unitary system, zone, terminal) and
confirmed live in the app: both "drag from library" and "Copy System" on
a whole airloop now correctly re-wire the desuperheater's heating source
and a manually-added SetpointManager:MixedAir's node references.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Same traversal gap as loop supply/demand branches, one level deeper:
equipment placed on an AirLoopHVACOutdoorAirSystem's outdoor-air or
relief branch (e.g. an evaporative cooler, or a SetpointManager:MixedAir
sitting on one of those nodes) is reachable only via
oaComponents()/reliefComponents(), not children() and not the loop's own
supplyComponents()/demandComponents() (which only see the OA system as a
single opaque object on the main branch).

Reproduced with the stock File > Examples > Example Model: its airloop's
outdoor air system has an evaporative cooler with a SetpointManager:
MixedAir on it. Copying that airloop left the copy's setpoint manager's
node fields empty, since the fixup never visited that branch at all.

Fixed by adding oaComponents()/reliefComponents() as two more
independently size-matched groups in childSubtreeObjectGroups(). Verified
against the SDK directly (Ruby, using OpenStudio::Model.exampleModel) --
all 4 of the example model's SetpointManagerMixedAir objects correctly
resolve their node references on the clone -- and confirmed live in the
rebuilt app.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
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

Successfully merging this pull request may close these issues.

1 participant