-
Notifications
You must be signed in to change notification settings - Fork 21
Delta formulation and free starts #1253
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
base: master
Are you sure you want to change the base?
Conversation
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## master #1253 +/- ##
==========================================
- Coverage 86.48% 86.33% -0.15%
==========================================
Files 148 148
Lines 4660 4735 +75
==========================================
+ Hits 4030 4088 +58
- Misses 630 647 +17 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
It looks promising; it's the 12RP case from @aporrasc that I have, which goes from around 10m to 4.5m to create the model. So, kudos for that. Some open questions from my side are:
Thanks! |
So we take into account the node in computing the before time-slice.
So histories of different blocks aren't mixed.
This is controversial but we need to discuss it.
...to not include the middle histories.
...whenever blocks with free start overlap with blocks without free start, such as RPs.
Now we only disallow the history of blocks with free start to be related to other blocks with free start. The "non-history" part is allowed to be related so that blocks with free start can overlap.
…cks without has_free_start: - Collecting the generator into an array first - Checking if the array is empty before calling last() - Only applying initial values when last_history_t is not nothing
This PR introduces the delta formulation for the representative periods model. The main changes are as follows:
resolutionparameter and we instead use the resolution of therepresentative_periods_mappingparameter. This is because we only need time-slices at mapping resolution to write the deltas.min_node_stateandnode_state_capacityconstraints for nodes withis_long_term_storage=true. No other constraint is generated outside of representative periods.node_stateis obtained from a JuMP expression corresponding to the delta equation.Also we introduce temporal blocks with free start using middle histories, superseding #1254 (now closed).
Fixes #1233
Checklist before merging