Permanent limit is now mandatory in IIDM

cgmes/cgmes-conversion/src/main/java/com/powsybl/cgmes/conversion/LoadingLimitsMapping.java

@@ -6,8 +6,6 @@
  */
 package com.powsybl.cgmes.conversion;
 
-import com.google.common.collect.Iterables;
-import com.powsybl.iidm.network.LoadingLimits;
 import com.powsybl.iidm.network.LoadingLimitsAdder;
 
 import java.util.HashMap;
@@ -33,16 +31,10 @@ public class LoadingLimitsMapping {
 
     void addAll() {
         for (Map.Entry<String, LoadingLimitsAdder<?, ?>> entry : adders.entrySet()) {
-            if (!Double.isNaN(entry.getValue().getPermanentLimit()) || entry.getValue().hasTemporaryLimits()) {
-                LoadingLimits limits = entry.getValue().add();
-                if (Double.isNaN(limits.getPermanentLimit())) {
-                    double fixedPermanentLimit = Iterables.get(limits.getTemporaryLimits(), 0).getValue();
-                    context.fixed("Operational Limit Set of " + entry.getKey(),
-                            "An operational limit set without permanent limit is considered with permanent limit" +
-                                    "equal to lowest TATL value",
-                            Double.NaN, fixedPermanentLimit);
-                    limits.setPermanentLimit(fixedPermanentLimit);
-                }
+            if (!Double.isNaN(entry.getValue().getPermanentLimit())) {
+                entry.getValue().add();
+            } else {
+                context.ignored("Missing permanent limit for LoadingLimit set {}. Any temporary limits in the set will be ignored", entry.getKey());
             }
         }
         adders.clear();

cgmes/cgmes-conversion/src/test/java/com/powsybl/cgmes/conversion/test/conformity/modified/CgmesConformity1ModifiedConversionTest.java

@@ -364,9 +364,7 @@ void microBEUndefinedPatl() {
                 NetworkFactory.findDefault(), null);
         Line line = network.getLine("ffbabc27-1ccd-4fdc-b037-e341706c8d29");
         CurrentLimits limits = line.getCurrentLimits1().orElse(null);
-        assertNotNull(limits);
-        assertEquals(2, limits.getTemporaryLimits().size());
-        assertEquals(1312.0, limits.getPermanentLimit(), 0.0);
+        assertNull(limits);
     }
 
     @Test

iidm/iidm-api/src/main/java/com/powsybl/iidm/network/ValidationUtil.java

@@ -585,8 +585,7 @@ public static void checkConnected(Validable validable, Boolean connected) {
     }
 
     public static void checkPermanentLimit(Validable validable, double permanentLimit) {
-        // TODO: if (Double.isNaN(permanentLimit) || permanentLimit <= 0) {
-        if (permanentLimit <= 0) {
+        if (Double.isNaN(permanentLimit) || permanentLimit <= 0) {
             throw new ValidationException(validable, "permanent limit must be defined and be > 0");
         }
     }