-
Notifications
You must be signed in to change notification settings - Fork 6
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
Storage/StoragePolicy improvements #198
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
kilianp14
commented
Apr 5, 2024
•
edited
Loading
edited
- Completely refactored StoragePolicy to MicrogridPolicy: Microgrid can now switch between grid-connected and islanded mode in the DefaultMicrogridPolicy; apply function now returns an energy_delta instead of a power_delta; Microgrid now always has such a policy; grid_power has been changed to charge_power (meaning the power that the battery is (dis)charged with); better documentation (still not all too great Improve documentation of storage module #117); policy is now its own module; better distinction between Storage and Policy (closes Simplification of Storage/ StoragePolicy #169)
- Improved handling of storage: Update function now returns the total energy that was (dis)charged instead of a power_delta that is difficult to understand; Storage state is now monitored correctly (closes Incorrect monitoring of storage state #187) and monitor can now be initialized with policy and storage component
- Improved cosim: Grid is now stepped at the end, avoiding storage (dis)charging at the first time step; seperate aggregator now computes the p_delta at every step of the microgrid and sends it to controllers and grid; energy delta that is computed by the policy is given to the controllers in the following timestep
- updated tests and examples
marvin-steinke
approved these changes
Apr 9, 2024
birnbaum
approved these changes
Apr 18, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.