Add management of specific specific parameter `PowerElectronicsCluster` #180

sBouzols · 2026-01-21T17:22:01Z

PR Summary

…PowerElectronicsCluster` Signed-off-by: sBouzols <sylvain.bouzols@gmail.com>

Signed-off-by: sBouzols <sylvain.bouzols@gmail.com>

…_PowerElectronicsClusters Signed-off-by: sBouzols <sylvain.bouzols@gmail.com>

src/main/java/org/gridsuite/shortcircuit/server/dto/FilterElements.java

...main/java/org/gridsuite/shortcircuit/server/dto/powsybl_private/PowerElectronicsCluster.java

src/main/java/org/gridsuite/shortcircuit/server/service/ShortCircuitService.java

...main/java/org/gridsuite/shortcircuit/server/dto/powsybl_private/PowerElectronicsCluster.java

Signed-off-by: sBouzols <sylvain.bouzols@gmail.com>

src/main/java/org/gridsuite/shortcircuit/server/service/ShortCircuitService.java

EtienneLt · 2026-02-06T14:01:44Z

src/main/java/org/gridsuite/shortcircuit/server/service/ShortCircuitService.java

+            }
+        } catch (Exception ex) {
+            // avoid breaking the run flow for unexpected shapes; log if you have a logger available
+            LOGGER.info("Could not normalize specific parameters for powerElectronicsClusters", ex);


Logs should be in a reportNode shouldn't it ?

EtienneLt · 2026-02-06T14:02:48Z

src/main/java/org/gridsuite/shortcircuit/server/service/ShortCircuitService.java

        // update status to running status
        setStatus(List.of(resultUuid), ShortCircuitAnalysisStatus.RUNNING);
+
+        LOGGER.debug("specificParameters types: {}",


are this log only for debug or should it stay in the code ?

src/test/java/org/gridsuite/shortcircuit/server/TestJsonNormalizer.java

Signed-off-by: sBouzols <sylvain.bouzols@gmail.com>

sonarqubecloud · 2026-02-10T17:31:08Z

Quality Gate passed

Issues
6 New issues
0 Accepted issues

Measures
0 Security Hotspots
89.4% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

sBouzols added 5 commits January 21, 2026 17:40

feat(PowerElectronicsCluster): Add mapagement of specific parameter `…

1bf7d29

…PowerElectronicsCluster` Signed-off-by: sBouzols <sylvain.bouzols@gmail.com>

clean

5a71d57

Signed-off-by: sBouzols <sylvain.bouzols@gmail.com>

checkstyle fix

241f76c

Signed-off-by: sBouzols <sylvain.bouzols@gmail.com>

fix(): filter by active cluster only

087b6bb

Signed-off-by: sBouzols <sylvain.bouzols@gmail.com>

Merge remote-tracking branch 'origin/main' into add_ENR_module_params…

3aebbf8

…_PowerElectronicsClusters Signed-off-by: sBouzols <sylvain.bouzols@gmail.com>

EtienneLt assigned sBouzols Feb 4, 2026

EtienneLt reviewed Feb 4, 2026

View reviewed changes

src/main/java/org/gridsuite/shortcircuit/server/dto/FilterElements.java Outdated Show resolved Hide resolved

EtienneLt reviewed Feb 4, 2026

View reviewed changes

...main/java/org/gridsuite/shortcircuit/server/dto/powsybl_private/PowerElectronicsCluster.java Outdated Show resolved Hide resolved

EtienneLt reviewed Feb 4, 2026

View reviewed changes

src/main/java/org/gridsuite/shortcircuit/server/service/ShortCircuitService.java Outdated Show resolved Hide resolved

EtienneLt reviewed Feb 4, 2026

View reviewed changes

...main/java/org/gridsuite/shortcircuit/server/dto/powsybl_private/PowerElectronicsCluster.java Outdated Show resolved Hide resolved

test(): Add json normalization utility class to adapt unit tests

9a040fa

Signed-off-by: sBouzols <sylvain.bouzols@gmail.com>