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

Delete CGMES update to only use CGMES full export #1949

Merged
merged 7 commits into from
Feb 10, 2022
Merged

Conversation

miovd
Copy link
Contributor

@miovd miovd commented Jan 24, 2022

Please check if the PR fulfills these requirements (please use '[x]' to check the checkboxes, or submit the PR and then click the checkboxes)

  • 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 ? If so, link to this issue using '#XXX' and skip the rest
No

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Delete unsupported CGMES update and replace it by CGMES full export + add parameter to select profiles to export + Correct substation geographicalTag import

Does this PR introduce a breaking change or deprecate an API? If yes, check the following:
Yes

  • The Breaking Change or Deprecated label has been added
  • The migration guide has been updated in the github wiki (What changes might users need to make in their application due to this PR?)

@miovd miovd changed the title [WIP] Update cgmes export Delete CGMES update to only use CGMES full export Jan 24, 2022
@zamarrenolm zamarrenolm requested a review from marcosmc January 26, 2022 16:45
@miovd miovd removed the request for review from marcosmc January 27, 2022 08:49
@miovd miovd changed the title Delete CGMES update to only use CGMES full export [WIP] Delete CGMES update to only use CGMES full export Jan 28, 2022
@miovd miovd changed the title [WIP] Delete CGMES update to only use CGMES full export Delete CGMES update to only use CGMES full export Jan 28, 2022
@miovd miovd requested a review from marcosmc January 28, 2022 13:36
miovd added 6 commits February 9, 2022 10:37
Signed-off-by: VEDELAGO MIORA <miora.ralambotiana@rte-france.com>
…sion

Signed-off-by: VEDELAGO MIORA <miora.ralambotiana@rte-france.com>
Signed-off-by: VEDELAGO MIORA <miora.ralambotiana@rte-france.com>
Signed-off-by: VEDELAGO MIORA <miora.ralambotiana@rte-france.com>
…in NODE-BREAKER

Signed-off-by: VEDELAGO MIORA <miora.ralambotiana@rte-france.com>
Signed-off-by: VEDELAGO MIORA <miora.ralambotiana@rte-france.com>
@miovd miovd force-pushed the update_cgmes_export branch from 7d2a341 to 5bca881 Compare February 9, 2022 09:53
@miovd miovd requested a review from zamarrenolm February 9, 2022 09:54
@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 9, 2022

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

85.3% 85.3% Coverage
0.0% 0.0% Duplication

@miovd miovd merged commit 09506b3 into main Feb 10, 2022
@miovd miovd deleted the update_cgmes_export branch February 10, 2022 08:42
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.

3 participants