search

virtualdj / pun_sensor

Prezzi PUN del mese - Home Assistant Integration
MIT License
78 stars 13 forks source link

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

Closed nanomad closed 2 months ago

nanomad commented 2 months ago

Fixes #40

nanomad commented 2 months ago

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 commented 2 months ago

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 commented 2 months ago

Ciao,

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

virtualdj commented 2 months ago

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 commented 2 months ago

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 commented 2 months ago

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

virtualdj commented 2 months ago

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.