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

Feat: aggiunto sensore per prossima fascia e prossimo cambio fascia #41

Merged
merged 1 commit into from
May 7, 2024

Conversation

nanomad
Copy link
Contributor

@nanomad nanomad commented May 5, 2024

Fixes #40

@nanomad
Copy link
Contributor Author

nanomad commented May 5, 2024

Screenshot:

image

Razionale:

La programmazione di ricarica della mia auto elettrica è molto limitata, quindi mi sono fatto una automazione che quando vede che la prossima fascia sarà F3 aggiusta inizio e termine ricarica di conseguenza

@virtualdj
Copy link
Owner

Ho visto, grazie.

Però io sono un po' contrario (passami il termine) ad avere queste 3 informazioni (fascia successiva, data/ora inizio prossima fascia e data/ora fine prossima fascia) come sensori: non sarebbe meglio, piuttosto, averle come attributi del sensorepun_fascia_corrente (ovvero nella classe FasciaPUNSensorEntity)?

Il motivo è che poi tutto questo viene loggato (uso InfluxDB) e mentre gli altri dati sono sostanzialmente necessari, in quanto non possono essere derivati gli uni dagli altri, queste 3 informazioni invece sì.

Inoltre mi pare che anche gli attributi possano scatenare trigger di cambi di stato nelle automazioni, quindi risolveresti comunque il tuo caso d'uso.

Che ne pensi?

@nanomad
Copy link
Contributor Author

nanomad commented May 6, 2024

Ciao,

faccio un esperimento e ti dico, onestamente del tracciare nello storico il cambio fascia successiva non mi importa nulla.....

@virtualdj
Copy link
Owner

Lo so, però comunque causa memorizzazione di dati che a lungo andare si accumulano (se provi InfluxDB te ne renderai conto 😄) quindi averlo negli attributi è comodo e allo stesso tempo utile.

@nanomad
Copy link
Contributor Author

nanomad commented May 6, 2024

Lo so, però comunque causa memorizzazione di dati che a lungo andare si accumulano (se provi InfluxDB te ne renderai conto 😄) quindi averlo negli attributi è comodo e allo stesso tempo utile.

Ah sisi concordo, intendevo proprio quello: la storicizzazione di sta roba non serve a nulla.
L'unica cosa brutta che vedo è che non puoi far automazioni basate sull'orario del prossimo cambio fascia...ma anche li...tempo che arriva quell'orario il valore del sensore temo sia già cambiato ed ho i miei dubbi che funzioni qualcosa

@nanomad
Copy link
Contributor Author

nanomad commented May 6, 2024

@virtualdj Sistemato, ho anche fatto un bello squash cosi da avere un commit pulito

@virtualdj
Copy link
Owner

L'unica cosa brutta che vedo è che non puoi far automazioni basate sull'orario del prossimo cambio fascia...ma anche li...tempo che arriva quell'orario il valore del sensore temo sia già cambiato ed ho i miei dubbi che funzioni qualcosa

Sì però almeno puoi estrarre la durata della fascia, credo, in modo da farti i tuoi calcoli.

Grazie intanto, domani provo con calma.

@virtualdj virtualdj merged commit 2662a1f into virtualdj:master May 7, 2024
2 checks passed
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.

Mostrare fascia sucessiva e cambio fascia
2 participants