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

Contingency lists improvements #2495

Closed
etiennehomer opened this issue Mar 10, 2023 · 0 comments
Closed

Contingency lists improvements #2495

etiennehomer opened this issue Mar 10, 2023 · 0 comments
Assignees

Comments

@etiennehomer
Copy link
Contributor

etiennehomer commented Mar 10, 2023

  • Do you want to request a feature or report a bug?
    Bug/improvements required for GridSuite's needs on identifiers based contingency lists.

  • What is the current behavior?

1/ we have a useless field identifiableType in IdentifierContingencyList

Expected behavior :
done in #2491

2/ there's no 'name' field on NetworkElementIdentifierList class.

Expected behavior :
We'd like to associate a name to each NetworkElementIdentifierList.
#2515

3/ we can create N-K defaults but the k defaults are not applied, only the first one is used in filterIdentifiable(). only N-1 defaults are supported
done in #2515

Expected behavior :
We'd like to create and filter with N-K default lists.

4/ ContingencyList type is a simple String

Expected behavior : we could use an enum but it's not a priority
not required

5/ GroovyContingencyList is not serializable because of fied GroovyCodeSource codeSource. It makes impossible the mapping with this class in GridSuite. Do we want to change the design of GroovyContingencyList ? We keep it like this and GS will not be homogeneous on this class ?
Expected behavior : not required actually

6/ There is only one nominal voltage for LineCriterionContingencyList.

Expected behavior :
We'd like to use a DoubleNominalVoltageCriterion

7/ When the list of countries of SingleCountryCriterion is empty, no contingency is returned.
#2512 (closed PR)
done in #2515

8/ 2 nominal voltage criterions for HvdcLines. Should be one ?

  • What is the motivation / use case for changing the behavior?
    These changes are needed for GridSuite integration of contingency lists.
@EtienneLt EtienneLt self-assigned this Mar 10, 2023
@etiennehomer etiennehomer changed the title Identifier contingency lists improvements Contingency lists improvements Mar 21, 2023
@annetill annetill closed this as completed Jul 1, 2024
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

No branches or pull requests

3 participants