Improved handling of corsika.Library

Closes #718 (closed)

This update handles "unknown" directories by simply skipping them instead of throwing an error. For example,

In [2]: lib = corsika.Library("test_dir")
[WARNING] - corsika - Could not find a configuration file in test_dir/interaction_hist. Skipping sub-directory
[WARNING] - corsika - An error occured loading a RadioProcess: single positional indexer is out-of-bounds
[WARNING] - corsika - Unable to instantiate an instance of 'RadioProcess' for a process called 'CoREAS'. Skipping 'CoREAS'
[WARNING] - corsika - 'RadioProcess' object has no attribute '_RadioProcess__data'
[WARNING] - corsika - An error occured loading a RadioProcess: single positional indexer is out-of-bounds
[WARNING] - corsika - Unable to instantiate an instance of 'RadioProcess' for a process called 'ZHS'. Skipping 'ZHS'
[WARNING] - corsika - 'RadioProcess' object has no attribute '_RadioProcess__data'

python/corsika/io/library.py

@@ -133,7 +133,7 @@ class Library(object):
 
         """
 
-        return self.__load_yaml(path, "config.yaml")
+        return Library.__load_yaml(path, "config.yaml")
 
     @staticmethod
     def load_summary(path: str) -> Optional[Dict[str, Any]]:
@@ -153,7 +153,7 @@ class Library(object):
 
         """
 
-        return self.__load_yaml(path, "summary.yaml")
+        return Library.__load_yaml(path, "summary.yaml")
 
     @staticmethod
     def __valid_library(path: str) -> bool:
@@ -185,7 +185,9 @@ class Library(object):
         return config["creator"] == "CORSIKA8"
 
     @staticmethod
-    def __build_outputs(path: str, dirs: Union[list, None]) -> Dict[str, outputs.Output]:
+    def __build_outputs(
+        path: str, dirs: Union[list, None]
+    ) -> Dict[str, outputs.Output]:
         """
         Build the outputs contained in this library.
 
@@ -233,7 +235,10 @@ class Library(object):
 
             # if `out_type` was None, this is an invalid output
             if out_type is None or name is None:
-                msg = f"'{subdir}' does not contain a valid config." "Missing 'type' or 'name' keyword."
+                msg = (
+                    f"'{subdir}' does not contain a valid config."
+                    "Missing 'type' or 'name' keyword."
+                )
                 logging.getLogger("corsika").warning(msg)
                 continue  # skip to the next output, don't error
 
@@ -245,13 +250,19 @@ class Library(object):
 
                 # check if the read failed
                 if not component.is_good():
-                    msg = f"'{name}' encountered an error while reading. " "This process will be not be loaded."
+                    msg = (
+                        f"'{name}' encountered an error while reading. "
+                        "This process will be not be loaded."
+                    )
                     logging.getLogger("corsika").warning(msg)
                 else:
                     components[name] = component
 
             except AttributeError as e:
-                msg = f"Unable to instantiate an instance of '{out_type}' " f"for a process called '{name}'. Skipping '{subdir}'"
+                msg = (
+                    f"Unable to instantiate an instance of '{out_type}' "
+                    f"for a process called '{name}'. Skipping '{subdir}'"
+                )
                 logging.getLogger("corsika").warning(msg)
                 logging.getLogger("corsika").warning(e)
                 continue  # skip to the next output, don't error