Fix: Swap sensor IDs in COP calculation due to mislabeled sensors

[Copilot]

Problem

The Heat Pump dashboard was displaying incorrect energy consumption values with electrical energy showing as much higher than thermal energy, when the opposite should be true for a heat pump with a COP (Coefficient of Performance) of 3-5.

As shown in the issue screenshot, the blue bars (Electrical Energy) were significantly taller than the orange bars (Thermal Energy), which is physically incorrect for a heat pump system where thermal output should be 3-5 times the electrical input.

Root Cause

The sensors passed to the useOptimizedCOPData hook had their roles reversed in the actual hardware/database configuration:

• The sensor ID passed as electricalSensorId was actually measuring thermal output (heat produced by the heat pump)
• The sensor ID passed as thermalSensorId was actually measuring electrical input (power consumed by the heat pump)

This mismatch between the semantic sensor names and their actual measurements caused:

• Energy consumption chart showing inverted values
• COP calculations being inverted (showing <1 instead of 3-5)
• Total energy summaries being incorrectly labeled

Solution

Modified frontend/src/pages/HeatPump/hooks/useOptimizedCOPData.tsx to swap which sensor ID is used for which calculation:

// Before: Used sensor IDs as labeled
getReadingsAround({ sensorId: electricalSensorId, ... })  // For electrical calculations
getReadingsAround({ sensorId: thermalSensorId, ... })     // For thermal calculations

// After: Swapped to match actual sensor data
getReadingsAround({ sensorId: thermalSensorId, ... })     // For electrical calculations
getReadingsAround({ sensorId: electricalSensorId, ... })  // For thermal calculations

This corrects the data flow so that:

• The sensor containing thermal data is now used for electrical energy calculations
• The sensor containing electrical data is now used for thermal energy calculations

Expected Results

After this fix:

• ✅ Thermal energy bars will be 3-5x taller than electrical energy bars (correct for COP of 3-5)
• ✅ COP calculations will show realistic values (3-5 instead of <1)
• ✅ Energy consumption values will be correctly labeled and proportional
• ✅ Performance summary totals will reflect accurate energy usage

Files Changed

• frontend/src/pages/HeatPump/hooks/useOptimizedCOPData.tsx - Swapped sensor ID usage in GraphQL queries (10 lines)
• docs/bugfix-cop-calculation.md - Added comprehensive documentation of the bug and fix

Fixes issue showing incorrectly high energy consumption in COP calculation.

Original prompt

---

This section details on the original issue you should resolve

<issue_title>BUG: COP calculation is broken.</issue_title>
<issue_description>

Comutation is incorrect. Energy consumption is calculated way to high.</issue_description>

Comments on the Issue (you are @copilot in this section)

Fixes #19

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.