Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GasMeter Power has been removed #11

Closed
B-Hartley opened this issue Jun 30, 2022 · 3 comments
Closed

GasMeter Power has been removed #11

B-Hartley opened this issue Jun 30, 2022 · 3 comments

Comments

@B-Hartley
Copy link

I think the GasMeter Power section has been removed from the MQTT feed.
(As it didn't make sense due to lack of live data).

2022-06-30 21:39:17 WARNING (MainThread) [homeassistant.components.rest.sensor] JSON result was not a dictionary or list with 0th element a dictionary
2022-06-30 21:39:18 ERROR (MainThread) [homeassistant.util.logging] Exception in mqtt_message_received when handling msg on 'glow/XXXXXXXXXXXX/SENSOR/gasmeter': '{"gasmeter":{"timestamp":"2022-06-30T20:38:50Z","energy":{"import":{"cumulative":8570.929,"day":0.000,"week":24.531,"month":282.630,"units":"kWh","cumulativevol":775.636,"cumulativevolunits":"m3","dayvol":0.000,"weekvol":24.531,"monthvol":282.630,"dayweekmonthvolunits":"kWh","mprn":"XXXXXXXXXXXX","supplier":"---","price":{"unitrate":0.06000,"standingcharge":0.15090}}}}}'
Traceback (most recent call last):
  File "/config/custom_components/hildebrand_glow_ihd/sensor.py", line 246, in mqtt_message_received
    updateGroup.process_update(message)
  File "/config/custom_components/hildebrand_glow_ihd/sensor.py", line 290, in process_update
    sensor.process_update(parsed_data)
  File "/config/custom_components/hildebrand_glow_ihd/sensor.py", line 324, in process_update
    new_value = self._func(mqtt_data)
  File "/config/custom_components/hildebrand_glow_ihd/sensor.py", line 213, in <lambda>
    "func": lambda js : js['gasmeter']['power']['value']
KeyError: 'power'
@robhawkes
Copy link

I believe this is related to the recent changes they made to the firmware (see #10)

@B-Hartley
Copy link
Author

Agreed.
I've just removed it from the sensor.py for now to stop the errors.

@megakid
Copy link
Owner

megakid commented Jul 1, 2022

Thanks for this - I also got a firmware update today as describe in #10 - I am about to push a new version that fixes this.

@megakid megakid closed this as completed Jul 1, 2022
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

No branches or pull requests

3 participants