Skip to content

Conversation

@jfsimoneau
Copy link
Contributor

J'ai dû ajouter beaucoup de docstrings pour faire passer les vérifications de pre-commit, mais le changement principal, c'est dans HiloRewardSensor._async_update(self).

@jfsimoneau jfsimoneau force-pushed the silence-flexd-reward branch from 4c8f479 to 5d7e554 Compare January 6, 2026 19:13
@ic-dev21
Copy link
Collaborator

ic-dev21 commented Jan 6, 2026

@jfsimoneau peut-être faire du logging avec le devcontainer un moment, j'ai pu avoir quelques jsons FlexD et je crois que vous avez probablement un champ "cost" de vôtre côté?

https://github.com/dvd-dev/python-hilo/blob/main/doc/Challenge(Event)%20JSON/EventFlexConsumptionUpdatedValuesReceived.json

@jfsimoneau
Copy link
Contributor Author

@jfsimoneau peut-être faire du logging avec le devcontainer un moment, j'ai pu avoir quelques jsons FlexD et je crois que vous avez probablement un champ "cost" de vôtre côté?

https://github.com/dvd-dev/python-hilo/blob/main/doc/Challenge(Event)%20JSON/EventFlexConsumptionUpdatedValuesReceived.json

D'accord, je vais regarder ça un peu plus et voir si on peut faire comme l'appli Hilo, i.e. donner le coût estimé et le coût réel. On pourrait considérer que la somme des récompenses est la différence entre les deux. C'est ce que l'appli affiche, en tout cas, mais elle appelle ça plutôt l'économie de la saison et montre une courbe de l'évolution de l'économie pour la saison.

@ic-dev21
Copy link
Collaborator

ic-dev21 commented Jan 7, 2026

@jfsimoneau peut-être faire du logging avec le devcontainer un moment, j'ai pu avoir quelques jsons FlexD et je crois que vous avez probablement un champ "cost" de vôtre côté?
https://github.com/dvd-dev/python-hilo/blob/main/doc/Challenge(Event)%20JSON/EventFlexConsumptionUpdatedValuesReceived.json

D'accord, je vais regarder ça un peu plus et voir si on peut faire comme l'appli Hilo, i.e. donner le coût estimé et le coût réel. On pourrait considérer que la somme des récompenses est la différence entre les deux. C'est ce que l'appli affiche, en tout cas, mais elle appelle ça plutôt l'économie de la saison et montre une courbe de l'évolution de l'économie pour la saison.

Es-tu à l’aise avec Postman? J’ai un environnement tout fait si tu veux poker le websocket hors HA voir les retours

Copy link
Collaborator

@ic-dev21 ic-dev21 left a comment

Choose a reason for hiding this comment

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

Testé en repassant au travers du reward sensor, LGTM

@ic-dev21 ic-dev21 merged commit 55dab57 into dvd-dev:main Jan 8, 2026
3 checks passed
@jfsimoneau jfsimoneau deleted the silence-flexd-reward branch January 8, 2026 15:17
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