MiguelAngelLV / balance_neto

Componente para Home Assistant que calcula el Balance Neto Horario para instalaciones fotovoltáicas.
34 stars 5 forks source link

Valor erróneo #3

Closed jjvelar closed 1 year ago

jjvelar commented 1 year ago

Hola! Gracias por la integración. En ocasiones sale un valor muy loco en los sensores de exportación e importación neta (-500000 o +500000). Adjunto captura de pantalla. Gracias!

image

MiguelAngelLV commented 1 year ago

Jums... ¿Podrías mirar que valores tienes en el sensor original?

Quizás el sensor original se resetea por algún motivo o se queda como desconocido y es lo que provoca todo el problema.

jjvelar commented 1 year ago

Efectivamente el sensor original da un salto muy grande provocado por un error puntual en los datos originales. ¿Puedo sugerir que tus sensores incluyan un filtro de manera que si el valor de entrada es undefined, unavailable o, en valor absoluto es mayor que, por ejemplo, 100000, devuelva 0, por ejemplo? Saludos,

José

MiguelAngelLV commented 1 year ago

Pues si se podría poner que el caso de cambio anormal decida omitir el cambio.

Mañana a la tarde te subo una beta para que la pruebes.

jjvelar commented 1 year ago

Perfecto!

El El dom, 28 may 2023 a las 7:38, Miguel Ángel López Vicente < @.***> escribió:

Pues si se podría poner que el caso de cambio anormal decida omitir el cambio.

Mañana a la tarde te subo una beta para que la pruebes.

El sáb, 27 may 2023 22:53, jjvelar @.***> escribió:

Efectivamente el sensor original da un salto muy grande provocado por un error puntual en los datos originales. ¿Puedo sugerir que tus sensores incluyan un filtro de manera que si el valor de entrada es undefined, unavailable o, en valor absoluto es mayor que, por ejemplo, 100000, devuelva 0, por ejemplo? Saludos,

José

— Reply to this email directly, view it on GitHub < https://github.com/MiguelAngelLV/balance_neto/issues/3#issuecomment-1565677877 , or unsubscribe < https://github.com/notifications/unsubscribe-auth/AAE3ZHS5RWS6S4O5GWRUW33XIJSVHANCNFSM6AAAAAAYRICMPU

. You are receiving this because you commented.Message ID: @.***>

— Reply to this email directly, view it on GitHub https://github.com/MiguelAngelLV/balance_neto/issues/3#issuecomment-1565906226, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALKMG4UGTLPSLX7RSJOQZ6LXILQD5ANCNFSM6AAAAAAYRICMPU . You are receiving this because you authored the thread.Message ID: @.***>

jjvelar commented 1 year ago

Pues si se podría poner que el caso de cambio anormal decida omitir el cambio. Mañana a la tarde te subo una beta para que la pruebes.

Hola! Tengo que hacer algo para descargar la beta y probarla? Gracia y un saludo,

José

MiguelAngelLV commented 1 year ago

Perdona, lo modifiqué y lo dejé probando y olvidé subirlo y avisarte.

Ya está subido. Simplemente en el HACS ve al repo y darle a descargar, en el selector de versión, activa «versiones prueba» o algo así y deberia salirte la beta.

jjvelar commented 1 year ago

OK. ya he instalado la versión "1.1 beta". te cuento si veo algún fallo? qué tipo de control de error has introducido? sólo si el sensor original está "unavailable" o también filtras para descartar si el valor absoluto del sensor en es mayor que "x", siendo "x" un número muy algo, como por ejemplo 9999999? gracias!

El mié, 31 may 2023 a las 8:25, Miguel Ángel López Vicente (< @.***>) escribió:

Perdona, lo modifiqué y lo dejé probando y olvidé subirlo y avisarte.

Ya está subido. Simplemente en el HACS ve al repo y darle a descargar, en el selector de versión, activa «versiones prueba» o algo así y deberia salirte la beta.

— Reply to this email directly, view it on GitHub https://github.com/MiguelAngelLV/balance_neto/issues/3#issuecomment-1569567887, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALKMG4SOM2VZVJNABI5AE43XI3P47ANCNFSM6AAAAAAYRICMPU . You are receiving this because you authored the thread.Message ID: @.***>

MiguelAngelLV commented 1 year ago

Básicamente descarto por «diferencia».

De esta forma mato dos pájaros de un tiro.

Si está como no disponible, no llegan valores, y en cuanto llegue el primero, habrá mucha diferencia (desde 0...) y por tanto descarta y empieza a calcular de nuevo.

Básicamente, siempre que la diferencia sea de más de 100kWh (no creo que nadie palme 100kWh use HA para llevar el cálculo) descarta el cálculo que lleva y empieza a contar.

jjvelar commented 1 year ago

Totalmente de acuerdo.

El El mié, 31 may 2023 a las 9:33, Miguel Ángel López Vicente < @.***> escribió:

Básicamente descarto por «diferencia».

De esta forma mato dos pájaros de un tiro.

Si está como no disponible, no llegan valores, y en cuanto llegue el primero, habrá mucha diferencia (desde 0...) y por tanto descarta y empieza a calcular de nuevo.

Básicamente, siempre que la diferencia sea de más de 100kWh (no creo que nadie palme 100kWh use HA para llevar el cálculo) descarta el cálculo que lleva y empieza a contar.

— Reply to this email directly, view it on GitHub https://github.com/MiguelAngelLV/balance_neto/issues/3#issuecomment-1569651173, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALKMG4TE5W5BEYPTKOJKRVTXI3X6LANCNFSM6AAAAAAYRICMPU . You are receiving this because you authored the thread.Message ID: @.***>