-
Notifications
You must be signed in to change notification settings - Fork 54
Feature/ldes framework #468
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
Feature/ldes framework #468
Conversation
updates to LDES initialization
* Update README.md to not specify `coin-or-cbc` version. --------- Co-authored-by: John Jasa <[email protected]>
I am a little surprised that the LDES system has a less-negative AEP than the battery. This leads to a lower overall LCOE and other differences. It could be an efficiency thing, but I thought I checked that in the LDES tests. The maximum and minimum SOC are the same between battery and LDES. I'm wondering if perhaps @genevievestarke or @bayc you have any ideas. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good from my end!
Add meaningful title here
PR Checklist
RELEASE.md
has been updated to describe the changes made in this PRdocs/
files are up-to-date, or added when necessaryRelated issues
Impacted areas of the software
path/to/file.extension
method1
: What and why something was changed in one sentence or less.Additional supporting information
Test results, if applicable