Sarah Foxley

@warunawickramasingha this should be ready to review again - the last test failures were not associated with this PR and have been fixed.

Can you add which versions of Mantid these errors are occuring on? We're finding it hard to replicate without that information

We have managed to replicate the issue as far back as mantidplot v3.9.2. That is a long time for it to go unnoticed.

It looks like this is working as it should then @smk78 ? If thats the case perhaps this issue could be closed? What do you think @SimonHeybrock ?

I'm closing this as it's been fixed by #36581

I've been looking into this and MergeRuns could be a good solution. It looks like it uses the Instrument Parameter File for the default merging behaviour for sample logs. Otherwise...

@perenon I have cancelled the tests that were rerunning on this PR as there are some merge conflicts

This is still in draft so I assume that there are more commits to come. Can you keep an eye on the Doxygen build? It is not failing if there...

Actually it looks like it is affecting several PRs so is not being caused by this one. I've raised #37286 so you no longer need to deal with it as...

It is possibly worth noting that when using the Load() algorithm on Windows it seems to cope with spaces in the path if the folder is listed in Manage Directories....