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

Allow defining GLSK on DanglingLine #99

Merged
merged 12 commits into from
Sep 25, 2023
Merged

Allow defining GLSK on DanglingLine #99

merged 12 commits into from
Sep 25, 2023

Conversation

pet-mit
Copy link
Contributor

@pet-mit pet-mit commented Apr 14, 2023

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)

What kind of change does this PR introduce?
Feature

What is the current behavior?
GLSKs are currently only possible upon generators & loads

What is the new behavior (if this is a feature change)?
It would be possible to define a GLSK on a dangling line:

  • For UCTE & CSE, the dangling line is defined by its XNode
  • For CIM, the dangling line should be identified by its ID in the network

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

Signed-off-by: Peter Mitri <peter.mitri@rte-france.com>
@pet-mit pet-mit added the enhancement New feature or request label Apr 14, 2023
pet-mit added 3 commits April 18, 2023 16:25
Signed-off-by: Peter Mitri <peter.mitri@rte-france.com>
Signed-off-by: Peter Mitri <peter.mitri@rte-france.com>
Signed-off-by: Peter Mitri <peter.mitri@rte-france.com>
@pet-mit pet-mit changed the title [WIP] Allow defining GLSK on DanglingLine Allow defining GLSK on DanglingLine Apr 18, 2023
pet-mit added 2 commits April 18, 2023 17:23
Signed-off-by: Peter Mitri <peter.mitri@rte-france.com>
Signed-off-by: Peter Mitri <peter.mitri@rte-france.com>
@pet-mit pet-mit requested review from phiedw and murgeyseb April 18, 2023 15:29
@sonarcloud
Copy link

sonarcloud bot commented Apr 18, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

96.8% 96.8% Coverage
0.0% 0.0% Duplication

glsk/glsk-document-cim/pom.xml Outdated Show resolved Hide resolved
glsk/glsk-document-cse/pom.xml Outdated Show resolved Hide resolved
glsk/glsk-document-ucte/pom.xml Outdated Show resolved Hide resolved
Signed-off-by: Peter Mitri <peter.mitri@rte-france.com>
Signed-off-by: Peter Mitri <peter.mitri@rte-france.com>
Signed-off-by: Peter Mitri <peter.mitri@rte-france.com>
Signed-off-by: Peter Mitri <peter.mitri@rte-france.com>
@sonarcloud
Copy link

sonarcloud bot commented Sep 22, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

97.4% 97.4% Coverage
0.0% 0.0% Duplication

@phiedw phiedw merged commit de70297 into main Sep 25, 2023
7 checks passed
@phiedw phiedw deleted the glsk_on_dangling_line branch September 25, 2023 08:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants