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

Replace ucte-x-node-code with pairing_key #702

Merged
merged 6 commits into from
Feb 15, 2024
Merged

Conversation

colinepiloquet
Copy link
Member

@colinepiloquet colinepiloquet commented Feb 8, 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?

Update

What is the current behavior?

Dangling lines have a field ucte-x-node-code

What is the new behavior (if this is a feature change)?
This field is renamed pairing_key and ucte-x-node-code is deprecated

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

  • Yes
  • No

If yes, please check if the following requirements are fulfilled

  • The Breaking Change or Deprecated label has been added
  • The migration steps are described in the following section

What changes might users need to make in their application due to this PR? (migration steps)

When creating a dangling line, the column or keyword argument ucte-x-node-code should be replaced by pairing_key.
When accessing the field from the dangling dataframe, the name of the column is also updated to pairing_key.

Signed-off-by: Coline PILOQUET <coline.piloquet@rte-france.com>
@EtienneLt EtienneLt force-pushed the deprecate_ucte_x_node branch from 54c5623 to ae99006 Compare February 13, 2024 14:55
Signed-off-by: Etienne LESOT <etienne.lesot@rte-france.com>
@EtienneLt EtienneLt force-pushed the deprecate_ucte_x_node branch from ae99006 to 3375e9a Compare February 13, 2024 16:02
Signed-off-by: Etienne LESOT <etienne.lesot@rte-france.com>
Signed-off-by: Etienne LESOT <etienne.lesot@rte-france.com>
EtienneLt and others added 2 commits February 15, 2024 09:06
Signed-off-by: Etienne LESOT <etienne.lesot@rte-france.com>
@EtienneLt EtienneLt merged commit fdb5420 into main Feb 15, 2024
6 checks passed
@EtienneLt EtienneLt deleted the deprecate_ucte_x_node branch February 15, 2024 09:51
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