Closed bsutherland333 closed 3 months ago
Looks good!
In case you are looking at the commit history, I thought, based on some testing, that the set_parameters method wasn't needed, so I deleted it, but then realized that my tests weren't complete. So I added it back in, since we do need the function calls to set_parameters.
This commit add the ability to change previously declared parameters in code. It also renames some of the functions to explicitly include the datatype related to that parameter, as I found declare_param("string", "value") would get mapped to the bool overload instead of the string one. Only declare_param("string", std::string("value")) resulted in correct behavior.