Skip to content
This repository has been archived by the owner on Jun 17, 2022. It is now read-only.

Import oligo as a dependency #16

Merged
merged 1 commit into from
Jan 25, 2022
Merged

Import oligo as a dependency #16

merged 1 commit into from
Jan 25, 2022

Conversation

hectorespert
Copy link
Collaborator

Import oligo as a dependency instead of maintain a copy of that,

@ad-ha
Copy link
Owner

ad-ha commented Jan 20, 2022

Buenas,

Uno de los cambios que propones al final lo que da es el consumo instantáneo, no?

Quizás esté equivocado, pero de la forma como lo tenía montado sacaba el acumulado de lectura del contador e iba acumulando la lectura, sobre todo para poder incorporar los datos al panel de energía de HA.

Yo de esto no sé mucho, que al final fui mirando código y copiando lo que creía. Que hacía falta para que funcionara.

Ya me dices y los vamos ajustando.

Muchas gracias por la ayuda

@hectorespert
Copy link
Collaborator Author

Uno de los cambios que propones al final lo que da es el consumo instantáneo, no?

Quizás esté equivocado, pero de la forma como lo tenía montado sacaba el acumulado de lectura del contador e iba acumulando la lectura, sobretodo para poder incorporar los datos al panel de energía de HA.

He actualizado la librería con los cambios que creaste para sacar el acumulado de el contador: hectorespert/python-oligo#29

@ad-ha
Copy link
Owner

ad-ha commented Jan 21, 2022

He actualizado la librería con los cambios que creaste para sacar el acumulado de el contador: hectorespert/python-oligo#29

De acuerdo, gracias por la nota y el cambio.

Esta noche, cuando pueda me pongo con ello y lo vamos ajustando.

Y muchas gracias por la ayuda y colaboración en esto.

@ad-ha ad-ha added the enhancement New feature or request label Jan 21, 2022
@hectorespert
Copy link
Collaborator Author

@ad-ha Como se prueban los cambios? Conoces algún tutorial o algo?

@ad-ha
Copy link
Owner

ad-ha commented Jan 21, 2022

@ad-ha Como se prueban los cambios? Conoces algún tutorial o algo?

Yo lo tengo montado en un container separado para pruebas, y corriendo directamente con Visual Studio.

Seguí lo que pone aquí: https://developers.home-assistant.io/docs/development_environment

Y luego el código de la integración lo puse debajo de custom-components y las definiciones en el configuration.yaml

@hectorespert hectorespert marked this pull request as ready for review January 21, 2022 19:23
@ad-ha
Copy link
Owner

ad-ha commented Jan 24, 2022

Buenas @hectorespert

Has probado el cambio? Funciona bien la importación de oligo con HA?

@hectorespert
Copy link
Collaborator Author

Has probado el cambio? Funciona bien la importación de oligo con HA?
En local funciona con una imagen de docker, no he llegado a probarlo con Home Assistant OS, pero debería de funcionar.

Desde la rama master creo que se puede probar antes de generar una nueva release.

@ad-ha
Copy link
Owner

ad-ha commented Jan 24, 2022

Desde la rama master creo que se puede probar antes de generar una nueva release.

A ver si lo puedo probar luego y si funciona todo bien. Gracias

@ad-ha
Copy link
Owner

ad-ha commented Jan 25, 2022

Tested and working on my developer system. Must check the installation of python-oligo, since it took me a few tries to get it to work.

@ad-ha ad-ha merged commit 8ac1fbc into ad-ha:main Jan 25, 2022
@hectorespert hectorespert deleted the import_oligo branch January 25, 2022 11:26
@hectorespert
Copy link
Collaborator Author

Se puede crear una release beta para probar?

@ad-ha
Copy link
Owner

ad-ha commented Jan 25, 2022

Se puede crear una release beta para probar?

La acabo de hacer: https://github.com/ad-ha/ide_api/releases/tag/v0.2.0-alpha

Estoy probando ahora en mi HA principal... a ver cómo va

@ad-ha
Copy link
Owner

ad-ha commented Jan 25, 2022

Al menos de momento no me ha salido ningún error y parece que ha ido todo bien. A ver si actualiza en las próximas lecturas.

@ad-ha
Copy link
Owner

ad-ha commented Jan 25, 2022

Al menos de momento no me ha salido ningún error y parece que ha ido todo bien. A ver si actualiza en las próximas lecturas.

@hectorespert Confirmado. Me funciona correctamente y sigo obteniendo lecturas 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants