Extend resource-specific functionality for geography features#50
Merged
Extend resource-specific functionality for geography features#50
Conversation
* removed constraint_resource in create_node GeoAvailability
…ansmission test - document new resource-segment extension points for flow variables and coupling constraints - expand how-to guidance with user-facing migration notes for resource-specific dispatch - improve model docstrings to clarify fallback hook behavior for extension packages - add a dedicated resource-flow test module with: - custom PotentialPower resource - custom PotentialLossMode transmission mode - two-area transmission corridor case validating potential loss behavior - include the new resource-flow test in the test runner - align project and test environments with EnergyModelsBase 0.10 and EnergyModelsInvestments 0.9
…eography.jl into enhance/complex_resource_flow
…-flow example * add an unversioned release note for EMB v0.10 resource dispatch changes * add a how-to guide for extending geography models with resource-specific flow variables * align docs terminology from hooks to extension functions and fix cross-references * update public docs pages with module metadata so Documenter resolves bindings correctly * relax EMB compat and adjust EMI investment constraints to use the full time structure
espenbodal
changed the title
JulStraus
requested changes
Apr 14, 2026
Member
JulStraus
left a comment
JulStraus
left a comment
There was a problem hiding this comment.
I commented a few things which I also adjusted myself in commits. If you agree with my comments and the implementation, you can resolve them. I will merge the PR at that point and register the version.
JulStraus
approved these changes
Apr 16, 2026
Member
JulStraus
left a comment
JulStraus
left a comment
There was a problem hiding this comment.
Looks good. Due to simplicity, my additional commits were approved by @espenbodal who could not as PR creator approve them directly.
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Description
This pull request implements resource-specific extension functionality for EnergyModelsGeography, following the pattern established in EnergyModelsBase#61.
What's included
Feature Implementation
variables_flow_resourceandconstraints_couple_resourcefor transmission modes and areasPotentialPowerresource tracking potential and enforcing transmission lossesDocumentation
docs/src/how-to/extend-resource-functionality.mdTests
test/test_resource_flow.jlwith PotentialPower exampleApplications
The extension pattern enables modeling of:
Related Issues
Relates to: #61 (resource-extension functionality in EnergyModelsBase)
Branch
Development on:
enhance/complex_resource_flow