Allow setting --debug-model flag to false

Author: alexdewar

src/cli.rs

@@ -36,8 +36,8 @@ pub struct RunOpts {
     #[arg(long)]
     pub overwrite: bool,
     /// Whether to write additional information to CSV files
-    #[arg(long)]
-    pub debug_model: bool,
+    #[arg(long, value_name = "BOOL", num_args = 0..=1, default_missing_value = "true")]
+    pub debug_model: Option<bool>,
 }
 
 /// The available commands.
@@ -116,8 +116,8 @@ pub fn handle_run_command(
     };
 
     // These settings can be overridden by command-line arguments
-    if opts.debug_model {
-        settings.debug_model = true;
+    if let Some(opt) = opts.debug_model {
+        settings.debug_model = opt;
     }
     if opts.overwrite {
         settings.overwrite = true;

tests/regression.rs

@@ -43,7 +43,7 @@ fn run_regression_test_debug_opt(example_name: &str, debug_model: bool) {
     let opts = RunOpts {
         output_dir: Some(output_dir.clone()),
         overwrite: false,
-        debug_model: true, // NB: Always enable this as it helps to have the files for debugging
+        debug_model: Some(true), // NB: Always enable this as it helps to have the files for debugging
     };
     handle_example_run_command(example_name, &opts, Some(Settings::default())).unwrap();
 

tests/run.rs

@@ -25,7 +25,7 @@ fn test_handle_run_command() {
     let opts = RunOpts {
         output_dir: Some(output_dir),
         overwrite: false,
-        debug_model: false,
+        debug_model: None,
     };
     handle_run_command(&get_model_dir(), &opts, Some(Settings::default())).unwrap();