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

Indicador de ajuste de mercado y cálculo de Indexada #69

Merged
merged 6 commits into from
Mar 10, 2024

Conversation

MiguelAngelLV
Copy link
Contributor

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í.

@azogue
Copy link
Owner

azogue commented Mar 10, 2024

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?

@MiguelAngelLV
Copy link
Contributor Author

MiguelAngelLV commented Mar 10, 2024

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.

Copy link
Owner

@azogue azogue left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Muchas gracias @MiguelAngelLV 💪

aiopvpc/const.py Show resolved Hide resolved
@azogue azogue merged commit 0e251a8 into azogue:master Mar 10, 2024
2 checks passed
@azogue
Copy link
Owner

azogue commented Mar 10, 2024

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 👍

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

Successfully merging this pull request may close these issues.

2 participants