Upgrade to powsybl optimizer 0.14.0 (#113)

FranckLecuyer · web-flow · commit 522200f45acc · 2025-07-22T14:42:53.000+02:00
* Upgrade to open-reac version to 0.14.0
* Change report for each missing voltage level limit from TRACE to DETAIL severity

Signed-off-by: Franck LECUYER &lt;franck.lecuyer@rte-france.com&gt;
diff --git a/pom.xml b/pom.xml
@@ -52,7 +52,7 @@
         <!-- FIXME to remove at next upgrade of powsybl-ws-dependencies -->
         <powsybl-network-store.version>1.28.0</powsybl-network-store.version>
         <gridsuite-dependencies.version>41.0.0</gridsuite-dependencies.version>
-        <powsybl-open-reac.version>0.13.0</powsybl-open-reac.version>
+        <powsybl-open-reac.version>0.14.0</powsybl-open-reac.version>
         <powsybl-optimizer-commons.version>0.13.0</powsybl-optimizer-commons.version>
         <mockwebserver3.version>5.0.0-alpha.14</mockwebserver3.version>
         <liquibase-hibernate-package>org.gridsuite.voltageinit.server</liquibase-hibernate-package>
diff --git a/src/main/java/org/gridsuite/voltageinit/server/service/parameters/VoltageInitParametersService.java b/src/main/java/org/gridsuite/voltageinit/server/service/parameters/VoltageInitParametersService.java
@@ -346,7 +346,7 @@ private static void logVoltageLimitsModifications(final ReportNode reporter,
                         .withTypedValue("voltageLevelId", voltageLevel.getId(), TypedValue.VOLTAGE_LEVEL)
                         .withTypedValue("lowVoltageLimit", computeRelativeVoltageLevel(initialLowVoltageLimit, voltageLimits.get(VoltageLimitType.LOW_VOLTAGE_LIMIT)), TypedValue.VOLTAGE)
                         .withTypedValue("highVoltageLimit", computeRelativeVoltageLevel(initialHighVoltage, voltageLimits.get(VoltageLimitType.HIGH_VOLTAGE_LIMIT)), TypedValue.VOLTAGE)
-                        .withSeverity(TypedValue.TRACE_SEVERITY)
+                        .withSeverity(TypedValue.DETAIL_SEVERITY)
                         .add();
             });
     }
diff --git a/src/test/resources/reporter_buildOpenReacParameters.json b/src/test/resources/reporter_buildOpenReacParameters.json
@@ -30,7 +30,7 @@
                 "value": "NaN → 10.0 kV",
                 "type": "VOLTAGE"
               },
-              "reportSeverity": { "value": "TRACE", "type": "SEVERITY" }
+              "reportSeverity": { "value": "DETAIL", "type": "SEVERITY" }
             }
           },
           {
@@ -39,7 +39,7 @@
               "voltageLevelId": { "value": "VLHV1", "type": "VOLTAGE_LEVEL" },
               "lowVoltageLimit": { "value": "NaN → 5.0 kV", "type": "VOLTAGE" },
               "highVoltageLimit": { "value": "20.0 kV", "type": "VOLTAGE" },
-              "reportSeverity": { "value": "TRACE", "type": "SEVERITY" }
+              "reportSeverity": { "value": "DETAIL", "type": "SEVERITY" }
             }
           },
           {
@@ -54,7 +54,7 @@
                 "value": "NaN → 88.0 kV",
                 "type": "VOLTAGE"
               },
-              "reportSeverity": { "value": "TRACE", "type": "SEVERITY" }
+              "reportSeverity": { "value": "DETAIL", "type": "SEVERITY" }
             }
           },
           {
diff --git a/src/test/resources/reporter_buildOpenReacParameters_caseRelativeFalse.json b/src/test/resources/reporter_buildOpenReacParameters_caseRelativeFalse.json
@@ -42,7 +42,7 @@
                 "type": "VOLTAGE_LEVEL"
               },
               "reportSeverity": {
-                "value": "TRACE",
+                "value": "DETAIL",
                 "type": "SEVERITY"
               },
               "highVoltageLimit": {
@@ -63,7 +63,7 @@
                 "type": "VOLTAGE_LEVEL"
               },
               "reportSeverity": {
-                "value": "TRACE",
+                "value": "DETAIL",
                 "type": "SEVERITY"
               },
               "highVoltageLimit": {
@@ -84,7 +84,7 @@
                 "type": "VOLTAGE_LEVEL"
               },
               "reportSeverity": {
-                "value": "TRACE",
+                "value": "DETAIL",
                 "type": "SEVERITY"
               },
               "highVoltageLimit": {
@@ -105,7 +105,7 @@
                 "type": "VOLTAGE_LEVEL"
               },
               "reportSeverity": {
-                "value": "TRACE",
+                "value": "DETAIL",
                 "type": "SEVERITY"
               },
               "highVoltageLimit": {
diff --git a/src/test/resources/reporter_buildOpenReacParameters_caseRelativeTrue.json b/src/test/resources/reporter_buildOpenReacParameters_caseRelativeTrue.json
@@ -42,7 +42,7 @@
                 "type": "VOLTAGE_LEVEL"
               },
               "reportSeverity": {
-                "value": "TRACE",
+                "value": "DETAIL",
                 "type": "SEVERITY"
               },
               "highVoltageLimit": {
@@ -63,7 +63,7 @@
                 "type": "VOLTAGE_LEVEL"
               },
               "reportSeverity": {
-                "value": "TRACE",
+                "value": "DETAIL",
                 "type": "SEVERITY"
               },
               "highVoltageLimit": {
@@ -84,7 +84,7 @@
                 "type": "VOLTAGE_LEVEL"
               },
               "reportSeverity": {
-                "value": "TRACE",
+                "value": "DETAIL",
                 "type": "SEVERITY"
               },
               "highVoltageLimit": {
diff --git a/src/test/resources/reporter_buildOpenReacParameters_withLimitModifications.json b/src/test/resources/reporter_buildOpenReacParameters_withLimitModifications.json
@@ -29,7 +29,7 @@
                 "type": "VOLTAGE_LEVEL"
               },
               "reportSeverity": {
-                "value": "TRACE",
+                "value": "DETAIL",
                 "type": "SEVERITY"
               },
               "highVoltageLimit": {
@@ -50,7 +50,7 @@
                 "type": "VOLTAGE_LEVEL"
               },
               "reportSeverity": {
-                "value": "TRACE",
+                "value": "DETAIL",
                 "type": "SEVERITY"
               },
               "highVoltageLimit": {
@@ -71,7 +71,7 @@
                 "type": "VOLTAGE_LEVEL"
               },
               "reportSeverity": {
-                "value": "TRACE",
+                "value": "DETAIL",
                 "type": "SEVERITY"
               },
               "highVoltageLimit": {
@@ -92,7 +92,7 @@
                 "type": "VOLTAGE_LEVEL"
               },
               "reportSeverity": {
-                "value": "TRACE",
+                "value": "DETAIL",
                 "type": "SEVERITY"
               },
               "highVoltageLimit": {
diff --git a/src/test/resources/reporter_fourSubstations_noVoltageLimits.json b/src/test/resources/reporter_fourSubstations_noVoltageLimits.json
@@ -29,7 +29,7 @@
                 "type": "VOLTAGE_LEVEL"
               },
               "reportSeverity": {
-                "value": "TRACE",
+                "value": "DETAIL",
                 "type": "SEVERITY"
               },
               "highVoltageLimit": {
@@ -50,7 +50,7 @@
                 "type": "VOLTAGE_LEVEL"
               },
               "reportSeverity": {
-                "value": "TRACE",
+                "value": "DETAIL",
                 "type": "SEVERITY"
               },
               "highVoltageLimit": {
@@ -71,7 +71,7 @@
                 "type": "VOLTAGE_LEVEL"
               },
               "reportSeverity": {
-                "value": "TRACE",
+                "value": "DETAIL",
                 "type": "SEVERITY"
               },
               "highVoltageLimit": {

Original file line number	Diff line number	Diff line change
`@@ -346,7 +346,7 @@ private static void logVoltageLimitsModifications(final ReportNode reporter,`
`346`	`346`	`.withTypedValue("voltageLevelId", voltageLevel.getId(), TypedValue.VOLTAGE_LEVEL)`
`347`	`347`	`.withTypedValue("lowVoltageLimit", computeRelativeVoltageLevel(initialLowVoltageLimit, voltageLimits.get(VoltageLimitType.LOW_VOLTAGE_LIMIT)), TypedValue.VOLTAGE)`
`348`	`348`	`.withTypedValue("highVoltageLimit", computeRelativeVoltageLevel(initialHighVoltage, voltageLimits.get(VoltageLimitType.HIGH_VOLTAGE_LIMIT)), TypedValue.VOLTAGE)`
`349`		`- .withSeverity(TypedValue.TRACE_SEVERITY)`
	`349`	`+ .withSeverity(TypedValue.DETAIL_SEVERITY)`
`350`	`350`	`.add();`
`351`	`351`	`});`
`352`	`352`	`}`
Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,7 @@`
`30`	`30`	`"value": "NaN → 10.0 kV",`
`31`	`31`	`"type": "VOLTAGE"`
`32`	`32`	`},`
`33`		`- "reportSeverity": { "value": "TRACE", "type": "SEVERITY" }`
	`33`	`+ "reportSeverity": { "value": "DETAIL", "type": "SEVERITY" }`
`34`	`34`	`}`
`35`	`35`	`},`
`36`	`36`	`{`
`@@ -39,7 +39,7 @@`
`39`	`39`	`"voltageLevelId": { "value": "VLHV1", "type": "VOLTAGE_LEVEL" },`
`40`	`40`	`"lowVoltageLimit": { "value": "NaN → 5.0 kV", "type": "VOLTAGE" },`
`41`	`41`	`"highVoltageLimit": { "value": "20.0 kV", "type": "VOLTAGE" },`
`42`		`- "reportSeverity": { "value": "TRACE", "type": "SEVERITY" }`
	`42`	`+ "reportSeverity": { "value": "DETAIL", "type": "SEVERITY" }`
`43`	`43`	`}`
`44`	`44`	`},`
`45`	`45`	`{`
`@@ -54,7 +54,7 @@`
`54`	`54`	`"value": "NaN → 88.0 kV",`
`55`	`55`	`"type": "VOLTAGE"`
`56`	`56`	`},`
`57`		`- "reportSeverity": { "value": "TRACE", "type": "SEVERITY" }`
	`57`	`+ "reportSeverity": { "value": "DETAIL", "type": "SEVERITY" }`
`58`	`58`	`}`
`59`	`59`	`},`
`60`	`60`	`{`