fix bug in mapping nested options in set_command_option

src/+gui_auxiliary/set_command_option.m

@@ -115,7 +115,12 @@ try
             end
             
         otherwise
-            options_.(gui_auxiliary.command_option_mapping(name)) = value;
+            if isnumeric(value)
+                eval(['options_.' gui_auxiliary.command_option_mapping(name) '=' num2str(value) ';']);
+            else
+                eval_expr = sprintf("options_.%s='%s';", gui_auxiliary.command_option_mapping(name), value);
+                eval(eval_expr);
+            end
     end
 catch
     status = 0;

[Houtan Bastani]

In this situation, why would you prefer the eval construct ? Isn't this better: options_.(gui_auxiliary.command_option_mapping(name)) = value;

[Dóra Kocsis]

There was a problem with nested options (eg: estimation.moments_posterior_density.indicator) in this case this syntax options_.(gui_auxiliary.command_option_mapping(name)) failed to populate the options_ field. There was no error because the switch is inside a try-catch statement.

[Houtan Bastani]

Sorry, I misspoke. Please mention this in the commit message

[Houtan Bastani]

And here, is the problem that sometimes the value passed is not a string? Otherwise, what was there previously should work. What variable type for value ends up here that was not handled by the previous statement, options_.(gui_auxiliary.command_option_mapping(name)) = value;