NatLabRockies / NatLabRockies/EnergyPlus

HeatPump:PlantLoop:EIR:Heating adding heat to source loop instead of removing

Open
#8,147 8 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
C++
Stars
1.6k
Forks
490
Avg merge
6d 21h
Merged PRs (30d)
22

Description

The HeatPump:PlantLoo:EIR:Heating adds heat to the source loop instead of removing it. The output variable "Heat Pump Source Side Outlet Temperature" is reporting the correct temperature, being lower than the inlet temperature. But the "System Node Temperature" for the same node is source heat pump inlet temperature plus the temp difference, not minus the temp difference. And the source loop is behaving as it is seeing the higher temperature, with heat added, not the lower temperature, with heat removed.

Also, not shown in this file, but having multiple, parallel heat pump objects, all branches request flow when unnecessary
Next 42 Results GSHP3b bug report.txt

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the HeatPump:PlantLoop:EIR:Heating source-side temperature and flow handling, using the attached Next.42 Results GSHP3b bug report to reproduce the behavior. Trace the source loop's System Node Temperature against the reported outlet temperature, and inspect parallel heat-pump flow requests. Done means the source loop reflects heat removal and unnecessary parallel branch flow is not requested.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
backend
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.