[ ] Implemented the source code change that satisfies the tests
[ ] Documented the feature by providing worked example
[ ] Updated the README or other documentation
[ ] Completed the pre-Request checklist below
Change Description
Use a new function, ConfigurationHelp::toString() instead of std::to_string when building HelpMaps. The new function is just a wrapper around output << std::setprecision(n) << input, returning outpt.str().
Test Description
Run nextsim --help-config MEVPDynamics and note that the default value for MEVPDynamics.DeltaMin is 2e-09 and not 0.000000.
Documentation Impact
Better self-documentation!
Other Details
N/A
Pre-Request Checklist
[ ] The requirements of this pull request are fully captured in an issue or design specification and are linked and summarised in the description of this PR
[ ] No new warnings are generated
[ ] The documentation has been updated (or an issue has been created to track the corresponding change)
[ ] Methods and Tests are commented such that they can be understood without having to obtain additional context
[ ] This PR/Issue is labelled as a bug/feature/enhancement/breaking change
[ ] File dates have been updated to reflect modification date
[ ] This change conforms to the conventions described in the README
Fix poor formatting of default values when using --help-config
Fixes #739
Task List
Change Description
Use a new function,
ConfigurationHelp::toString()instead ofstd::to_stringwhen buildingHelpMaps. The new function is just a wrapper aroundoutput << std::setprecision(n) << input, returningoutpt.str().Test Description
Run
nextsim --help-config MEVPDynamicsand note that the default value forMEVPDynamics.DeltaMinis 2e-09 and not 0.000000.Documentation Impact
Better self-documentation!
Other Details
N/A
Pre-Request Checklist