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

Error en la carga del sensor Contador #6

Open
rexp2 opened this issue Aug 8, 2021 · 6 comments
Open

Error en la carga del sensor Contador #6

rexp2 opened this issue Aug 8, 2021 · 6 comments

Comments

@rexp2
Copy link

rexp2 commented Aug 8, 2021

Este error se originó a partir de una integración personalizada.

Logger: custom_components.edistribucion.sensor
Source: custom_components/edistribucion/sensor.py:91
Integration: EDistribucion Sensor (documentation, issues)
First occurred: 1:52:20 (3 occurrences)
Last logged: 2:32:52

Fail to setup Contador e-distribución
Traceback (most recent call last):
File "/config/custom_components/edistribucion/sensor.py", line 91, in getAttrData
attributes['Estado ICP'] = meter['data']['estadoICP']
TypeError: 'NoneType' object is not subscriptable
#########################
Me da este error al cargar home assistant. Si entro por web al contador funciona bien, incluso diría que mejor que antes, más rápido y falla menos. Pienso que han hecho algun cambio o algún tipo de baneo a la integración.
Utilizo esta versión
https://github.com/jjprz/edistribucion
tuneada un poco. El sensor Prevision de facturación es un desastre ahora mismo porque en Edistribucion no están los últimos periodos de facturación, si estuvieran sería otra cosa seguramente.
El sensor Potencia Máxima es menos interesante para mi.
Lo que funciona muy bien es el sensor Consumo no facturado, es perfecto en cuanto facturan se reinicia y se actualiza dos veces al dia, una pasadas las 00 y la otra varía pero antes de la tarde ya está actualizado hasta las 23:59 del dia anterior.
Como decía he tuneado un poco el sensor Consumono facturado:
Screenshot_20210808-024037_Home Assistant
Pero algunos datos tengo que cambiar editando cada mes porque son variables y yo no tengo conocimientos para poder importar desde un input_number o algo así para poder cambiar la potencia contratada, precio medio de cada periodo y lo que marca el contador en cada periodo. En este sensor si sería interesante importar el esiosApi.
Screenshot_20210808-030639_Home Assistant

Es todo, ojalá os sirva este comentario para mejorar y corregir.
Perdon por el tocho y gracias por vuestro trabajo.

@marc-pujol
Copy link

Buenas tardes, mismo error que @rexp2

@bvis
Copy link

bvis commented Aug 13, 2021

Hola, yo detecté un error también, no sé si exactamente el mismo y @trocotronic lo ha arreglado en una versión nueva 0.4.1.

Si se actualiza la versión de la librería volverá todo a funcionar, yo lo he hecho manualmente.

@bepece1
Copy link

bepece1 commented Aug 13, 2021

Hola, yo detecté un error también, no sé si exactamente el mismo y @trocotronic lo ha arreglado en una versión nueva 0.4.1.

Si se actualiza la versión de la librería volverá todo a funcionar, yo lo he hecho manualmente.

@bvis ¿de qué librería se trata? ¿podrías explicar cómo se actualizaría manualmente?
Gracias :)

@bvis
Copy link

bvis commented Aug 13, 2021

He creado una pull request con los cambios necesarios: https://github.com/jagalindo/edistribucion/pull/8/commits

La versión funcional está en una rama de mi fork, por si lo quieres testear antes de que se acepte la pull request, si es que se acepta, claro.

https://github.com/bvis/edistribucion/tree/feature/update-EdistribucionAPI.py-to-v0.4.1

@rexp2
Copy link
Author

rexp2 commented Aug 13, 2021

Hola, yo detecté un error también, no sé si exactamente el mismo y @trocotronic lo ha arreglado en una versión nueva 0.4.1.

Si se actualiza la versión de la librería volverá todo a funcionar, yo lo he hecho manualmente.

Gracias, ahora funciona bien el contador pero me han dejado de funcionar los otros sensores, será por la versión de jjprz. Voy a mirar si puedo arreglarlo

@bepece1
Copy link

bepece1 commented Aug 13, 2021

Hola, yo detecté un error también, no sé si exactamente el mismo y @trocotronic lo ha arreglado en una versión nueva 0.4.1.
Si se actualiza la versión de la librería volverá todo a funcionar, yo lo he hecho manualmente.

@bvis ¿de qué librería se trata? ¿podrías explicar cómo se actualizaría manualmente?
Gracias :)

Me respondo a mi mismo. Realicé los siguientes pasos y funcionó:

cd ~/docker/homeassistant/config/custom_components/edistribucion/backend
sudo mv EdistribucionAPI.py EdistribucionAPI.py_
sudo wget https://raw.githubusercontent.com/trocotronic/edistribucion/master/EdistribucionAPI.py
sudo rm pycache/ -r
docker restart homeassistant

Por si a alguien le interesa....

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

4 participants