Skip to content

Conversation

@EtienneLt
Copy link
Contributor

@EtienneLt EtienneLt commented Jan 12, 2026

PR Summary

migrate parameters ComponentMode according to powsybl release : powsybl/powsybl-core#3583

Signed-off-by: Etienne LESOT <etienne.lesot@rte-france.com>
Signed-off-by: Etienne LESOT <etienne.lesot@rte-france.com>
Signed-off-by: Etienne LESOT <etienne.lesot@rte-france.com>
@EtienneLt EtienneLt self-assigned this Jan 12, 2026
@Mathieu-Deharbe Mathieu-Deharbe self-requested a review January 13, 2026 13:39
Mathieu-Deharbe

This comment was marked as resolved.

@Column(name = "connectedComponentMode")
@Enumerated(EnumType.STRING)
private LoadFlowParameters.ConnectedComponentMode connectedComponentMode;
private LoadFlowParameters.ComponentMode connectedComponentMode;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If the type name changes you should probably change the variable name too :

Suggested change
private LoadFlowParameters.ComponentMode connectedComponentMode;
private LoadFlowParameters.ComponentMode componentMode;

(edit : ok I can see that this gets complicated given that this is an entity, maybe it is better to keep it like this, do as you wish)

@sonarqubecloud
Copy link

@EtienneLt EtienneLt merged commit e9e9d28 into main Jan 20, 2026
4 checks passed
@EtienneLt EtienneLt deleted the powsybl-dependencies-v2025.3.0 branch January 20, 2026 07:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants