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

Overload management system Serialization/Deserialization #2844

Merged
merged 7 commits into from
Jan 15, 2024

Conversation

olperr1
Copy link
Member

@olperr1 olperr1 commented Jan 4, 2024

Please check if the PR fulfills these requirements

  • The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

Does this PR already have an issue describing the problem?

No

What kind of change does this PR introduce?

Feature

What is the current behavior?

Overload management systems are not serialized / deserialized.

What is the new behavior (if this is a feature change)?
Overload management systems can now be serialized or deserialized.

2 options were introduced:

  • iidm.export.xml.with-automation-systems export option:
    • When set to false, the overload management systems are not exported when serializing a network (default value is true);
  • iidm.import.xml.with-automation-systems import option:
    • When set to false, the overload management systems are not imported when deserializing a network (default value is true);

Does this PR introduce a breaking change or deprecate an API?

  • Yes
  • No

Other information:

Base automatically changed from evolution_xiidm/1.12 to main January 10, 2024 12:07
@olperr1 olperr1 changed the base branch from main to automation_systems January 10, 2024 12:27
@olperr1 olperr1 force-pushed the overload_management_system_serde branch from 0ad09a0 to f987230 Compare January 10, 2024 12:54
@olperr1 olperr1 changed the title [WIP] Overload management system Serialization/Deserialization Overload management system Serialization/Deserialization Jan 10, 2024
Base automatically changed from automation_systems to main January 11, 2024 14:41
…tation

Signed-off-by: Olivier Perrin <olivier.perrin@rte-france.com>
…rrors

Signed-off-by: Olivier Perrin <olivier.perrin@rte-france.com>
Signed-off-by: Olivier Perrin <olivier.perrin@rte-france.com>
Signed-off-by: Olivier Perrin <olivier.perrin@rte-france.com>
Signed-off-by: Olivier Perrin <olivier.perrin@rte-france.com>
@olperr1 olperr1 force-pushed the overload_management_system_serde branch from f987230 to 691d509 Compare January 11, 2024 15:02
@olperr1 olperr1 marked this pull request as ready for review January 11, 2024 15:03
@olperr1 olperr1 requested review from geofjamg and flo-dup January 11, 2024 15:17
Signed-off-by: Olivier Perrin <olivier.perrin@rte-france.com>
Copy link

Quality Gate Passed Quality Gate passed

The SonarCloud Quality Gate passed, but some issues were introduced.

1 New issue
0 Security Hotspots
95.4% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@olperr1 olperr1 merged commit a9c4678 into main Jan 15, 2024
6 checks passed
@olperr1 olperr1 deleted the overload_management_system_serde branch January 15, 2024 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants