37594 sliceviewer sample data order

[Despiix]

Description of work

Summary of work

Slice viewed displays error when sample data (Y axis) is unordered. Concluded to just displaying a warning when the unusual data is imported telling the user to order the relevant data themselves in necessary.

Fixes #37594.

@robertapplin

Further detail of work

To test:

1. Import the attached data:
   File: osiris100318-100329_graphite002_red_elwin_eq.zip

2. Open the Slice Viewer:
   Right click the data and click "Slice Viewer"

3. Check for a Warning:
   The window should display the data, but also show a warning message on the side.
   Expected behavior: The data should appear "broken" (Thin line strip), and the warning should provide further details.

4. Test with Other Data:
   Try importing and viewing other data files

---

Reviewer

Please comment on the points listed below (full description).
Your comments will be used as part of the gatekeeper process, so please comment clearly on what you have checked during your review. If changes are made to the PR during the review process then your final comment will be the most important for gatekeepers. In this comment you should make it clear why any earlier review is still valid, or confirm that all requested changes have been addressed.

Code Review

• Is the code of an acceptable quality?
• Does the code conform to the coding standards?
• Are the unit tests small and test the class in isolation?
• If there is GUI work does it follow the GUI standards?
• If there are changes in the release notes then do they describe the changes appropriately?
• Do the release notes conform to the release notes guide?

Functional Tests

• Do changes function as described? Add comments below that describe the tests performed?
• Do the changes handle unexpected situations, e.g. bad input?
• Has the relevant (user and developer) documentation been added/updated?

Does everything look good? Mark the review as Approve. A member of @mantidproject/gatekeepers will take care of it.

Gatekeeper

If you need to request changes to a PR then please add a comment and set the review status to "Request changes". This will stop the PR from showing up in the list for other gatekeepers.

[RichardWaiteSTFC]

Warning shown when following instructions, as discussed offline this is a temporary measure, the bug is somewhere in the sliceviewer.
Just some comments about confusing index and spectrum number and a suggested change. I think this also requires a release note.

[sf1919]

@Despiix - This is a victim of changes to pre-commit. See #37960 (review) for details of the changes that went in and see if you need to modify any of your files as a result and push a new commit.

[RichardWaiteSTFC]

Looks good thanks for changes - maybe we can hunt this bug down during release

The base branch was changed.

[RichardWaiteSTFC]

Approving again after target branch changed