- Fixed invalidateAllProcessed for TwoLevelTreeManager.
- Added forEachGroup and forEachRow for iterating over a
QTwoLevelTreeModel.
- Removed unnecessary variable settingsHaveChanged.
- Removed extra //s.

Caused by not getting the initial value of the settings, now done in
the accept(view) method.

Had to move/adjust expectGetOptions + similar to account for the fact
settings are fetched in the accept method rather than later in the
process method.

Previously generating the notebook was crashing for the SANS v2 GUI because it does not have post-processing. Now it will simply skip the post-processing in the output notebook.

Re #21457

- Replace functions that get options as strings with ones that return a dict for pre-processing and processing options (note that, for now, post-processing options are still always used as a string)
- Use a QMap in the C++ code so that SIP conversion works
- Update tests to use the QMap return type
- Convert from the QMap to a std::map where the results are used in the data processor so that subsequent code does not have to change

Re #21218

Re #21457

Re #21457

Re #21457

Re #21457

Re #21457

This reverts commit 2326bd7d.

Re #21457

Refactor usages of maps of maps into using OptionsMap or ColumnOptionsMap, to make it clear which map is being used. Remove usages of maps of QString to QString where the second QString is a comma separated list of key=value pairs.

Re #21457

Re #21218

Separators inside workspaces names are now supported as long as the value is wrapped in quotes

Re #21218

This reverts commit 9a799ee9.

This work will be done as part of a separate PR

Re #21218

- A full list of options applicable to CreateTransmissionWorkspaceAuto is now returned from the Settings presenter
- If preprocessing property names are the same as processing algorithm property names, then they may be overridden by values on the Runs tab
- Minor refactoring has been done so the same functions can be used to update (and optionally insert) option maps for preprocessing and processing algorithms.

Re #21218

- Refactoring to re-use the same code to parse and re-join preprocessing input values
- Refactoring to re-use the same function for working out the reduced workspace name
- Preprocessing input values are now always split on comma or plus character and always re-joined using the plus character

Re #21218

Avoids repeated definitions and fixes ambiguous symbol error on Windows

Re #21218

The workspaces to stitch were not being found due to a name change.

Re #21218

Re #21218

Parsing map or string values now uses same set of functions and same quote/escape characters

Re #21218

Re #21818

This commit fixes a coding error where the output value was not being updated.

Re #21218

Re #21218

Preprocessing options are now taken from the options map rather than parsing a string

Re #21218

Previously the settings were being put into a comma-separated string of key=value which then had to be parsed out by the data processor. This commit makes it pass a map of key=value instead.

Re #2118

We can't set non-preprocessed default options on the algorithm because they may need preprocessing before they are valid algorithm properties. This commit gathers them into a map first and sets them on the algorithm as the last step.

This needs further work to pass the correct values to preprocessing from the global options.

Re #21218

Previously, the processing algorithm defaults were being set after preprocessing was done. If the same property was being used for both preprocessing and processing, we had to remember which columns had been preprocessed and set as inputs to the processing algorithm so as not to overwrite them with the default algorithm properties. This commit changes this so that defaults are set first, and are overwritten by preprocessed values afterwards

Re #21218

Re #21218

Re #21218

Re #21218

Replace 'child' with 'children'

Re #21215