Skip to content

Fix calcolo PUN a fine mese; closes #73#74

Merged
virtualdj merged 1 commit into
masterfrom
fix-73
Jan 4, 2025
Merged

Fix calcolo PUN a fine mese; closes #73#74
virtualdj merged 1 commit into
masterfrom
fix-73

Conversation

@virtualdj

Copy link
Copy Markdown
Owner

Come ben descritto nell'issue #73, questa PR risolve il problema verificatosi a fine anno.
Il problema si manifesta l'ultimo giorno del mese (per dicembre, il 31) in questa porzione di codice:

# Calcola l'intervallo di date per il mese corrente
date_end = dt_util.now().date() + timedelta(
days=1
) # Necessario per prezzo zonale (domani)
date_start = date(date_end.year, date_end.month, 1)
# All'inizio del mese, aggiunge i valori del mese precedente
# a meno che CONF_ACTUAL_DATA_ONLY non sia impostato
if (not self.actual_data_only) and (date_end.day < 5):
date_start = date_start - timedelta(days=3)
start_date_param = date_start.strftime("%Y%m%d")
end_date_param = date_end.strftime("%Y%m%d")

Avendolo modificato nella commit bf13936 (rispetto alla precedente 1630326) per scaricare i dati del giorno successivo (PUN orario e prezzo zonale), il giorno successivo del 31/12 è il 01/01 che quindi modifica il mese di date_start perdendo tutto lo storico dei prezzi del mese di dicembre. Infatti nel log si notano pochissimi file XML:

2024-12-31 02:02:12.108 DEBUG (MainThread) [custom_components.pun_sensor.coordinator] 3 file trovati nell'archivio (20241229MGPPrezzi.xml, 20241230MGPPrezzi.xml, 20241231MGPPrezzi.xml)

Questo è chiaramente errato e viene sistemato da questa PR.

@virtualdj virtualdj added the patch New patch/hotfix/minor changes label Jan 4, 2025
@virtualdj virtualdj merged commit e7d9f84 into master Jan 4, 2025
@virtualdj virtualdj deleted the fix-73 branch January 4, 2025 14:51
@virtualdj virtualdj added the bug Something isn't working label Jan 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working patch New patch/hotfix/minor changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant