Error handling fixes

nusamai/src/main.rs

@@ -165,6 +165,8 @@ fn main() -> ExitCode {
         sink_provider.create(&sink_params)
     };
 
+    let mut had_error = false;
+
     let updated_transformer_registry = TransformerRegistry {
         configs: transformer_registry
             .configs
@@ -177,28 +179,50 @@ fn main() -> ExitCode {
                     match &mut config.parameter {
                         // If the parameter is of type Selection, update the selected value
                         ParameterType::Selection(selection) => {
-                            if let Err(err) = selection.set_selected_value(value) {
-                                eprintln!("{} for option '{}'", err, config.key);
+                            if let Err(_err) = selection.set_selected_value(value) {
+                                let available_options: Vec<String> = selection.get_options()
+                                    .iter()
+                                    .map(|option| format!("'{}'", option.get_value()))
+                                    .collect();
+                                log::error!(
+                                    "Non-existent value '{}' specified for option '{}'. Available options are: {}",
+                                    value,
+                                    config.key,
+                                    available_options.join(", ")
+                                );
+                                had_error = true;
                             }
                         }
                         // If the parameter is of type Boolean, update the boolean value
                         ParameterType::Boolean(bool_param) => match value.as_str() {
                             "true" => *bool_param = true,
                             "false" => *bool_param = false,
-                            _ => eprintln!(
-                                "Invalid boolean value '{}' for option '{}'",
-                                value, config.key
-                            ),
+                            _ => {
+                                log::error!(
+                                    "Invalid boolean value '{}' for option '{}'. Only 'true' or 'false' are allowed.",
+                                    value,
+                                    config.key
+                                );
+                                had_error = true;
+                            }
                         },
                         // Handle other parameter types if needed
-                        _ => eprintln!("Unsupported parameter type for key '{}'", config.key),
+                        _ => {
+                            log::error!("Unsupported parameter type for key '{}'", config.key);
+
+                            had_error = true;
+                        }
                     }
                 }
-                config // Return the (potentially updated) config
+                config
             })
-            .collect(), // Collect all configs into a new Vec
+            .collect(),
     };
 
+    if had_error {
+        return ExitCode::FAILURE;
+    }
+
     let mut requirements = sink.make_requirements(updated_transformer_registry);
     requirements.set_output_epsg(match args.sink.0.as_ref() {
         "kml" => 6697, // temporary hack for KML output

nusamai/src/sink/cesiumtiles/mod.rs

@@ -97,7 +97,7 @@ impl DataSinkProvider for CesiumTilesSinkProvider {
             key: "use_lod".to_string(),
             label: "出力LODの選択".to_string(),
             parameter: transformer::ParameterType::Selection(Selection::new_lod_selections(
-                "max_lod",
+                "maxlod",
             )),
             requirements: vec![transformer::Requirement::UseLod(LodSelection::MaxLod)],
         });

nusamai/src/sink/czml/mod.rs

@@ -63,7 +63,7 @@ impl DataSinkProvider for CzmlSinkProvider {
             key: "use_lod".to_string(),
             label: "出力LODの選択".to_string(),
             parameter: transformer::ParameterType::Selection(Selection::new_lod_selections(
-                "max_lod",
+                "maxlod",
             )),
             requirements: vec![transformer::Requirement::UseLod(LodSelection::MaxLod)],
         });

nusamai/src/sink/geojson/mod.rs

@@ -63,7 +63,7 @@ impl DataSinkProvider for GeoJsonSinkProvider {
             key: "use_lod".to_string(),
             label: "出力LODの選択".to_string(),
             parameter: transformer::ParameterType::Selection(Selection::new_lod_selections(
-                "max_lod",
+                "maxlod",
             )),
             requirements: vec![transformer::Requirement::UseLod(LodSelection::MaxLod)],
         });

nusamai/src/sink/gltf/mod.rs

@@ -85,7 +85,7 @@ impl DataSinkProvider for GltfSinkProvider {
             key: "use_lod".to_string(),
             label: "出力LODの選択".to_string(),
             parameter: transformer::ParameterType::Selection(Selection::new_lod_selections(
-                "max_lod",
+                "maxlod",
             )),
             requirements: vec![transformer::Requirement::UseLod(LodSelection::MaxLod)],
         });

nusamai/src/sink/gpkg/mod.rs

@@ -64,7 +64,7 @@ impl DataSinkProvider for GpkgSinkProvider {
             key: "use_lod".to_string(),
             label: "出力LODの選択".to_string(),
             parameter: transformer::ParameterType::Selection(Selection::new_lod_selections(
-                "max_lod",
+                "maxlod",
             )),
             requirements: vec![transformer::Requirement::UseLod(LodSelection::MaxLod)],
         });

nusamai/src/sink/kml/mod.rs

@@ -63,7 +63,7 @@ impl DataSinkProvider for KmlSinkProvider {
             key: "use_lod".to_string(),
             label: "出力LODの選択".to_string(),
             parameter: transformer::ParameterType::Selection(Selection::new_lod_selections(
-                "max_lod",
+                "maxlod",
             )),
             requirements: vec![transformer::Requirement::UseLod(LodSelection::MaxLod)],
         });

nusamai/src/sink/minecraft/mod.rs

@@ -68,7 +68,7 @@ impl DataSinkProvider for MinecraftSinkProvider {
             key: "use_lod".to_string(),
             label: "出力LODの選択".to_string(),
             parameter: transformer::ParameterType::Selection(Selection::new_lod_selections(
-                "max_lod",
+                "maxlod",
             )),
             requirements: vec![transformer::Requirement::UseLod(LodSelection::MaxLod)],
         });

nusamai/src/sink/mvt/mod.rs

@@ -94,7 +94,7 @@ impl DataSinkProvider for MvtSinkProvider {
             key: "use_lod".to_string(),
             label: "出力LODの選択".to_string(),
             parameter: transformer::ParameterType::Selection(Selection::new_lod_selections(
-                "max_lod",
+                "maxlod",
             )),
             requirements: vec![transformer::Requirement::UseLod(LodSelection::MaxLod)],
         });

nusamai/src/sink/obj/mod.rs

@@ -102,7 +102,7 @@ impl DataSinkProvider for ObjSinkProvider {
             key: "use_lod".to_string(),
             label: "出力LODの選択".to_string(),
             parameter: transformer::ParameterType::Selection(Selection::new_lod_selections(
-                "max_lod",
+                "maxlod",
             )),
             requirements: vec![transformer::Requirement::UseLod(LodSelection::MaxLod)],
         });

nusamai/src/sink/ply/mod.rs

@@ -72,7 +72,7 @@ impl DataSinkProvider for StanfordPlySinkProvider {
             key: "use_lod".to_string(),
             label: "出力LODの選択".to_string(),
             parameter: transformer::ParameterType::Selection(Selection::new_lod_selections(
-                "max_lod",
+                "maxlod",
             )),
             requirements: vec![transformer::Requirement::UseLod(LodSelection::MaxLod)],
         });

nusamai/src/sink/shapefile/mod.rs

@@ -67,7 +67,7 @@ impl DataSinkProvider for ShapefileSinkProvider {
             key: "use_lod".to_string(),
             label: "出力LODの選択".to_string(),
             parameter: transformer::ParameterType::Selection(Selection::new_lod_selections(
-                "max_lod",
+                "maxlod",
             )),
             requirements: vec![transformer::Requirement::UseLod(LodSelection::MaxLod)],
         });

nusamai/src/transformer/setting.rs

@@ -17,6 +17,14 @@ impl SelectionOptions {
             value: value.to_string(),
         }
     }
+
+    pub fn get_label(&self) -> String {
+        self.label.clone()
+    }
+
+    pub fn get_value(&self) -> String {
+        self.value.clone()
+    }
 }
 
 #[derive(Debug, Serialize, Deserialize, Clone)]
@@ -46,8 +54,8 @@ impl Selection {
     pub fn new_lod_selections(selected_value: &str) -> Self {
         Self::new(
             vec![
-                ("最大LOD", "max_lod"),
-                ("最小LOD", "min_lod"),
+                ("最大LOD", "maxlod"),
+                ("最小LOD", "minlod"),
                 ("LOD0", "lod0"),
                 ("LOD1", "lod1"),
                 ("LOD2", "lod2"),
@@ -66,6 +74,10 @@ impl Selection {
             Err("Invalid value".to_string())
         }
     }
+
+    pub fn get_options(&self) -> Vec<SelectionOptions> {
+        self.options.clone()
+    }
 }
 
 #[derive(Debug, Serialize, Deserialize, Clone)]
@@ -160,13 +172,13 @@ impl TransformerRegistry {
                 ParameterType::Selection(value) => {
                     if config.key == "use_lod" {
                         match value.selected_value.as_str() {
-                            "max_lod" => {
+                            "maxlod" => {
                                 data_requirements.set_lod_filter(transformer::LodFilterSpec {
                                     mode: transformer::LodFilterMode::Highest,
                                     ..Default::default()
                                 })
                             }
-                            "min_lod" => {
+                            "minlod" => {
                                 data_requirements.set_lod_filter(transformer::LodFilterSpec {
                                     mode: transformer::LodFilterMode::Lowest,
                                     ..Default::default()