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

Refactoring of class_akku to Battery #130

Draft
wants to merge 195 commits into
base: main
Choose a base branch
from
Draft

Conversation

fortrieb
Copy link
Collaborator

@fortrieb fortrieb commented Oct 7, 2024

Note: Still a draft pull request. Will add type hints for all methods in Battery.

drbacke and others added 30 commits February 16, 2024 12:19
2 stufige Optimierung, erst binär, dann mit float
PV Prognose aufsummieren der Strings eingebaut
michaelosthege and others added 21 commits October 4, 2024 11:51
datetime.datetime.now() is incorrectly accessing the datetime class
- Optimized Imports: Removed unused imports and organized them.
- Refactored Code: Introduced split_individual function for clarity.
- Improved Efficiency: Enhanced penalty calculation and streamlined loops.
- Updated Evaluation Logic: Better handling of penalties in evaluate.
- Type Hints added
- fixed seed option added for automated tests
- verbose comment added, default False

Notes:
- isfloat is only used in flask_server.py
- start_hour is not used in this class
deleted comments
flask_server path with os lib
removed comments
fixed Bug in visualize.py (extra data empty)
removed dead cp
@fortrieb fortrieb self-assigned this Oct 9, 2024
@NormannK
Copy link
Collaborator

@fortrieb could you take a look into this PR? If you got no time id make a new one. Thanks

@fortrieb
Copy link
Collaborator Author

I will resolve the conflicts this week. Last few weeks i haven't had much time to work on it.

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.

7 participants