-
Notifications
You must be signed in to change notification settings - Fork 9
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
Indicador de ajuste de mercado y cálculo de Indexada #69
Conversation
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
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. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Muchas gracias @MiguelAngelLV 💪
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í.