Closed MiguelAngelLV closed 6 months ago
Hola @MiguelAngelLV 👋
El PR tiene muy buena pinta 👍, hay un par de detallitos a nivel de organización de código y un bug potencial en el cálculo de esa indexada, pero, ya que no has subido versión del paquete en este PR, prefiero hacer yo otro PR por encima, y así aprovecho para hacer algo de limpieza de ejemplos antiguos, y alguna cosilla más.
Además, creo q cambiaron el sistema de publicación en pypi, así q tengo q echarle una miradilla para poder publicar nuevas versiones (lo q hay ahora no iba a funcionar 🥲)
Respecto al otro PR en el custom-component, ¿no te animas a hacer el PR en el HA-Core?? 😜, el custom-component es un rollo, y creo q ya está a la par con el oficial, no?
Es posible que haya algún bug sí, python solo lo he tocado para HA, así que aún no tengo claro las explosiones que puede haber con los Nones y demás.
En cuanto a mandar al oficial... Todavía no tengo claro todos los protocolos pertinentes.
En cuanto a mandar al oficial... Todavía no tengo claro todos los protocolos pertinentes.
Tienen muy buena doc para montarlo todo y hacer tu 1º PR. Cuanto + pequeño mejor 😅; yo puedo revisártelo 👍
En esta PR añado el nuevo indicador para obtener el ajuste de mercado que se realiza en PVPC respecto a la indexada.
Adicionalmente, una vez se tienen todos los sensores, se calcula el valor de indexada durante la fase de procesamiento, restando al valor de PVPC el valor de mercado futuros.
Dado que Indexada no es un indicador, le he puesto 0, aunque no estoy seguro de que deba ser así.