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

CGMES functional logs (first steps) #3161

Merged
merged 3 commits into from
Sep 27, 2024
Merged

CGMES functional logs (first steps) #3161

merged 3 commits into from
Sep 27, 2024

Conversation

rcourtier
Copy link
Member

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?

Implements #3140

What kind of change does this PR introduce?

Feature

What is the current behavior?

There is some reporting done during CGMES import conversion but the logs are for the moment scattered and not hierarchically organized.

What is the new behavior (if this is a feature change)?

The purpose of that PR is to set a canvas of the main conversion steps in order to add the detailed functional logs.

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

  • Yes
  • No

Other information:

…on, and attach already existing CgmesReports to these subnodes

Signed-off-by: Romain Courtier <[email protected]>
@rcourtier rcourtier requested a review from olperr1 September 27, 2024 10:36
@rcourtier rcourtier changed the title CGMES functional logs CGMES functional logs (first steps) Sep 27, 2024
@rcourtier rcourtier marked this pull request as ready for review September 27, 2024 12:38
Copy link

sonarcloud bot commented Sep 27, 2024

@olperr1 olperr1 merged commit bfee0f0 into main Sep 27, 2024
7 checks passed
@olperr1 olperr1 deleted the cgmes_functional_logs branch September 27, 2024 12:58
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