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

insert prices in statistics, addition of zscore, removal of backup and some other changes #3

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

janmolemans
Copy link
Contributor

@janmolemans janmolemans commented Sep 18, 2022

Bedankt voor het creeren van deze repo!
Ik heb een aantal veranderingen voor mezelf gemaakt en wou deze delen in geval die u ook aanspraken:

  • de toekomstige prijzen in de statistics injecteren ipv attributes van de sensor. Dit maakt het maken van grafieken ook iets cleaner.
  • een extra sensor (zscore) die aanduidt of we op dit moment (current hour) al dan niet te maken hebben met een hoge of lage prijs tov de andere uren
  • herwerken van de api call functie en de data die wordt teruggegeven
  • verwijderen van de backup optie

Oorspronkelijk wou ik de toekomstige prijzen ook in de state injecteren, en dat lukte maar had wat nare neveneffecten dus die aanpak heb ik laten varen en voor de insert_statistics optie gegaan.
Voel u vrij om de code eruit te nemen die u nuttig acht.

@infradom
Copy link
Owner

Bedankt om dit te delen. Ik hoop deze aanpassingen te kunnen overnemen, maar het zal me wat tijd kosten om dit te onderzoeken. Vooral rond het weglaten van de (relatief ongeteste backup optie) heb ik nog wat twijfels. Ik moet ook nakijken of mijn automatisaties die verder bouwen op deze code nog goed functioneren na uw aanpassingen.
Het kan een paar dagen (misschien zelfs week) duren voor ik tijd vind om dit in detail te bekijken.

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