Skip to content

GSYE-891: Add HeatPumpWithSorTesTankStrategy to is_heatpump_node_type… - #614

Merged
hannesdiedrich merged 1 commit into
masterfrom
feature/GSYE-891
May 29, 2026
Merged

GSYE-891: Add HeatPumpWithSorTesTankStrategy to is_heatpump_node_type…#614
hannesdiedrich merged 1 commit into
masterfrom
feature/GSYE-891

Conversation

@hannesdiedrich

Copy link
Copy Markdown
Member

… for correctly calculating the KPIs of the building containing such a heatpump

Reason for the proposed changes

Please describe what we want to achieve and why.

Proposed changes

GSY_WEB_BRANCH=master

… for correctly calculating the KPIs of the building containing such a heatpump
@hannesdiedrich
hannesdiedrich requested a review from spyrostz May 20, 2026 13:28
@codecov

codecov Bot commented May 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 67.61%. Comparing base (545ef3e) to head (f542f9e).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #614   +/-   ##
=======================================
  Coverage   67.61%   67.61%           
=======================================
  Files          88       88           
  Lines        5571     5571           
  Branches     1262     1262           
=======================================
  Hits         3767     3767           
  Misses       1622     1622           
  Partials      182      182           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@spyrostz spyrostz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@hannesdiedrich
hannesdiedrich merged commit ec6a9f9 into master May 29, 2026
4 checks passed
@hannesdiedrich
hannesdiedrich deleted the feature/GSYE-891 branch May 29, 2026 13:44
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.

2 participants