Merge pull request #648 from FAIRmat-NFDI/no-log-error-for-missing-concepts

lukaspie · web-flow · commit b317069318d3 · 2025-05-28T14:23:14.000+02:00
do not log error for missing concepts
diff --git a/src/pynxtools/dataconverter/helpers.py b/src/pynxtools/dataconverter/helpers.py
@@ -98,9 +98,9 @@ def _log(self, path: str, log_type: ValidationProblem, value: Optional[Any], *ar
                 f"The value at {path} does not match with the enumerated items from the open enumeration: {value}."
             )
         elif log_type == ValidationProblem.MissingRequiredGroup:
-            logger.error(f"The required group, {path}, hasn't been supplied.")
+            logger.warning(f"The required group, {path}, hasn't been supplied.")
         elif log_type == ValidationProblem.MissingRequiredField:
-            logger.error(
+            logger.warning(
                 f"The data entry corresponding to {path} is required "
                 "and hasn't been supplied by the reader.",
             )

Original file line number	Diff line number	Diff line change
`@@ -98,9 +98,9 @@ def _log(self, path: str, log_type: ValidationProblem, value: Optional[Any], *ar`
`98`	`98`	`f"The value at {path} does not match with the enumerated items from the open enumeration: {value}."`
`99`	`99`	`)`
`100`	`100`	`elif log_type == ValidationProblem.MissingRequiredGroup:`
`101`		`- logger.error(f"The required group, {path}, hasn't been supplied.")`
	`101`	`+ logger.warning(f"The required group, {path}, hasn't been supplied.")`
`102`	`102`	`elif log_type == ValidationProblem.MissingRequiredField:`
`103`		`- logger.error(`
	`103`	`+ logger.warning(`
`104`	`104`	`f"The data entry corresponding to {path} is required "`
`105`	`105`	`"and hasn't been supplied by the reader.",`
`106`	`106`	`)`