SpectrumView should have a refresh/update method to change the workspace data

[Federico M Pouzols]

This method would make it possible to update the Spectrum View window with a different workspace/underlying data.

The idea is that if one is using a single spectrum viewer instance, it should not be necessary to close it and open a new one to visualize a different workspace.

This seems to be mostly (if not fully) implemented in SpectrumView::renderWorkspace(), but it needs to be tested (potentially amended) and documented.

This is related to/motivated by ticket #8091.

Destroying a spectrum viewer window and creating a new one does not seem to take a significant amount of time, at least in the few tests that I did. So I'd say that ticket has a low benefit/effort ratio and consequently low priority, as it may require testing of proper initialization, etc.

This ticket is a brother of #10445 and #10446.

[Dan Nixon]

Gave this a quick test and seems to work fine, will do some more testing and documentation to what the function actually does.

[Dan Nixon]

Better documentation of SpectrumView functions

Start supporting workspace refresh events.

Refs #10447

Changeset: 0a5f881c3d72d5a07242728e6400e27b495304aa

[Dan Nixon]

Remove old plots on new data

Refs #10447

Changeset: 1c364b9e994741fd2ff566f9f6f7152cf8cb4f22

[Dan Nixon]

To test:

• Load a file (e.g. IRS26173.raw)
• Open the spectrum viewer for the workspace
• Select a region on the colour plot so that data is shown on the slice plots also
• Load a different file (e.g. IRS26176.raw) with the same workspace name such that the original workspace is replaced
• Notice the spectrum viewer is updated

• Review function documentation

[Raquel Alvarez Banos]

Got this error when doing git test start:

$ git test start 10447 git fetch -p remote: Counting objects: 185, done. remote: Compressing objects: 100% (153/153), done. remote: Total 185 (delta 113), reused 66 (delta 32)R Receiving objects: 100% (185/185), 1017.90 KiB | 605.00 KiB/s, done. Resolving deltas: 100% (113/113), done. From github.com:mantidproject/mantid

a67a0a8..e928e4a develop -> origin/develop 8a98cc6..87d4e78 feature/10672_save_button_updates -> origin/feature/10672_save_button_updates

• [new branch] feature/9915_add_usage_example_sinq_focus -> origin/feature/9915_add_usage_example_sinq_focus
• [new branch] feature/9952_spectrogram_crash -> origin/feature/9952_spectrogram_crash

ERROR: Branch 'feature/10447_spec_view_refresh_function' has not been fully merged to develop, meaning it is not possible to know whether the code is valid on all environments.

Contact Dan Nixon on dan@… (or via skype) and ask them to merge the code to develop using 'git checkbuild' from their branch feature/10447_spec_view_refresh_function and let you know when it passes testing on the buildservers so you can start testing.

[Dan Nixon]

Should be fine now.

[Raquel Alvarez Banos]

Merge remote-tracking branch 'origin/feature/10447_spec_view_refresh_function'

Full changeset: ​5d1221493a6af3a276b8ecfb5252fd196a172f58

[Stuart Campbell]

This ticket has been transferred to github issue ​11289