nikshriv / hass_navien_water_heater

Home Assistant custom component for Navien water heaters using NaviLink
40 stars 14 forks source link

Cumulative Gas Use is too high #18

Open gardiner4 opened 1 year ago

gardiner4 commented 1 year ago

My NaviLink shows Total Gas Usage as 230 therms, but Cumulative Gas Use in this integration reports 22,407.2 ft³. Yesterday the app says I used .3 therms, but the Cumulative Gas Use increased 183.6 ft³. I think a unit conversion is off. This is a NPE-240 in the US.

nikshriv commented 1 year ago

I have been using the old conversion factor from the prior version, assuming that it wouldn't change. I'll take a closer look at it this weekend and update. Thanks

wisem2540 commented 7 months ago

FYI this appears to be broken. Can you share your conversion so we can at least adjust the sensor?

gardiner4 commented 6 months ago

Here is the conversion from /navien_water_heater/sensor.py. My unit is offline right now, but if the unit is reporting cubic feet you might just have to divide by 35.3147.

"accumulatedGasUsage": GenericSensorDescription( state_class = SensorStateClass.TOTAL_INCREASING, native_unit_of_measurement=VOLUME_CUBIC_METERS if hass_units == "metric" else VOLUME_CUBIC_FEET, name="Cumulative Gas Use", conversion_factor = 1 if hass_units == navien_units else 35.3147 if hass_units == "us_customary" else 0.0283168732, device_class=SensorDeviceClass.GAS

gardiner4 commented 6 months ago

So the Navien App has a lifetime count of therms. This integration reports in cubic feet. My natural gas provider National Grid calculates the bill using ccf which is a hundred cubic feet. I am now using a template to do the approximate conversion to ccf.

sensor: