You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Thought we should add AEMO's interconnector limit setting logic in the interest of modelling AEMO's procedure (#24). Would be cool if we could run market.get_interconnector_limit_setter and retrieve the interconnector limits and their associated constraints.
Before we move forward with unit tests, docstrings and examples, I would like to check that you find the namespaces (for new funcs) and structure acceptable, since you have pioneered this project. Still would like to add the FCAS limits for completeness.
New public functions added, but they are a bit of a mouthful. Happy to update...
market.get_interconnector_energy_constraint_limits:
Full list of constraints and their associated limit on that interconnector direction (import or export).
market.get_interconnector_energy_constraint_setter:
Most restrictive constraint outlined in AEMO's process for an interconnector direction (energy only).
market.get_interconnector_fcas_constraint_setter
Most restrictive constraint outlined in AEMO's process for an interconnector direction (energy + fcas).
Note, a number of unit tests will need to be rewritten if we include fcas_support_unavailable in the interconnector definitions (so that market.get_interconnector_fcas_constraint_setter can retrieve the data). But we could instead pass the fcas support data directly into market.get_interconnector_fcas_constraint_setter.
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
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.
Hi @nick-gorman,
I don't mean to overwhelm the PR log.
Thought we should add AEMO's interconnector limit setting logic in the interest of modelling AEMO's procedure (#24). Would be cool if we could run
market.get_interconnector_limit_setterand retrieve the interconnector limits and their associated constraints.The procedure is described in an old NEMMCO doc: https://aemo.com.au/-/media/files/electricity/nem/security_and_reliability/dispatch/policy_and_process/interconnector-limit-setter-reporting-changes.pdf
Before we move forward with unit tests, docstrings and examples, I would like to check that you find the namespaces (for new funcs) and structure acceptable, since you have pioneered this project. Still would like to add the FCAS limits for completeness.