powsybl / pypowsybl

A PowSyBl and Python integration based on GraalVM native image
Mozilla Public License 2.0
54 stars 11 forks source link

Fix multi substation sld svg #797

Closed CBiasuzzi closed 2 months ago

CBiasuzzi commented 3 months ago

Please check if the PR fulfills these requirements

What kind of change does this PR introduce?

Bug fix

What is the current behavior?

write_matrix_multi_substation_single_line_diagram_svgand get_matrix_multi_substation_single_line_diagram: errors are raised depending on the matrix_ids parameter (the 2D list of substation ids);
Tests were added to demonstrate the issue in a couple of cases. On the java side the matrix's size, decoded from the python parameter, does not seem to be computed correctly.

What is the new behavior (if this is a feature change)? The matrix's size on the Java side is computed correctly.

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