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

Bugfix class_strompreis #198 #199

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Bugfix class_strompreis #198 #199

wants to merge 1 commit into from

Conversation

NormannK
Copy link
Collaborator

@NormannK NormannK commented Nov 14, 2024

/strompreis was not working since beginning of Oktober

  • added checking of day -1 is in the input data. if not the first value is used
  • logging added, prints removed
  • some type hints added
  • output rounded to 10 digits
  • added basic test if endpoint is available
  • removed the doubling of self.charges in get_price_for_date.

future todo:

  • add endpoint and class name
  • add proper test of class
  • use of config for TZ

@NormannK NormannK marked this pull request as draft November 28, 2024 00:15
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.

1 participant