There was a slight change made in 70c0b626 regarding how DisplayFormatter checks whether a plugin is in an iterative loop or not. This in turn made it unnecessary to save the 'iterative' plugin dict key in the process list NeXuS file, so since some of these commits depend on saving that info to the process list file, we might keep this pull request unmerged for now to see if the alternative approach was reasonable.
There was a slight change made in 70c0b626 regarding how
DisplayFormatterchecks whether a plugin is in an iterative loop or not. This in turn made it unnecessary to save the'iterative'plugin dict key in the process list NeXuS file, so since some of these commits depend on saving that info to the process list file, we might keep this pull request unmerged for now to see if the alternative approach was reasonable.The changes in https://github.com/jessicavers/Savu/commit/33fac045cb7cc1f466e865cfc6a2b03a2b8c0623 (which is one of the commits in the pull request) is very useful to have though, since it fixed some configurator tests, so I've included that specific commit in the branch by cherry-picking it https://github.com/DiamondLightSource/Savu/commit/f8483463f61973f7b61b7e63194badd059017b84. I hope that is a reasonable solution for now :)