Skip to content
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

Add limit setter #25

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from
Draft

Add limit setter #25

wants to merge 7 commits into from

Conversation

mlee94
Copy link
Contributor

@mlee94 mlee94 commented Oct 25, 2023

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_setter and 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.

@mlee94
Copy link
Contributor Author

mlee94 commented Dec 7, 2023

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).

@mlee94
Copy link
Contributor Author

mlee94 commented Dec 7, 2023

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant