Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Output Plot Incorrect in MSlice Manual Testing Instructions #38672

Open
cailafinn opened this issue Jan 22, 2025 · 1 comment · May be fixed by #38703
Open

Output Plot Incorrect in MSlice Manual Testing Instructions #38672

cailafinn opened this issue Jan 22, 2025 · 1 comment · May be fixed by #38703
Assignees
@warunawickramasingha
Labels
Bug Issues and pull requests that are regressions or would be considered a bug by users (e.g. crashing) Direct Inelastic Issues and pull requests related to direct inelastic Documentation Issues and pull requests related to documentation ISIS Team: Excitations/Vesuvio Issue and pull requests managed by the Excitiations/Vesuvio subteam at ISIS
Milestone
Release 6.13

Comments

@cailafinn
Copy link
Contributor

cailafinn commented Jan 22, 2025 •
edited by warunawickramasingha
Loading

Describe the bug
The output plot looks slightly different for this test https://developer.mantidproject.org/Testing/Direct/MSliceTestGuide.html#changing-the-intensity-of-a-cut

The data has no margin at the low y and high y:

Image

In addition, the plot shown here is also needs to be updated.

4. Interactive Cuts step 9, the instructions need to be updated, since it is the x axis label that is changing when Click on Flip Integration Axis.

To Reproduce

  1. Follow the instructions in the linked documentation.

Expected behavior
The docs should match the result of the output.

Platform/Version (please complete the following information):

  • OS: All
  • Mantid Version: Nightly. Since at least 6.11.
@cailafinn cailafinn added Bug Issues and pull requests that are regressions or would be considered a bug by users (e.g. crashing) Direct Inelastic Issues and pull requests related to direct inelastic Documentation Issues and pull requests related to documentation ISIS Team: Excitations/Vesuvio Issue and pull requests managed by the Excitiations/Vesuvio subteam at ISIS labels Jan 22, 2025
@cailafinn cailafinn mentioned this issue Jan 22, 2025
Closed
@sf1919 sf1919 moved this from Icebox to Backlog in Mantid Maintenance Jan 23, 2025
@sf1919 sf1919 added this to the Release 6.13 milestone Jan 23, 2025
@warunawickramasingha warunawickramasingha self-assigned this Jan 24, 2025
@warunawickramasingha warunawickramasingha moved this from Backlog to Assigned in Mantid Maintenance Jan 24, 2025
@warunawickramasingha
Copy link
Contributor

In addition, the plot shown here is also needs to be updated.

@warunawickramasingha warunawickramasingha linked a pull request Jan 27, 2025 that will close this issue
Open
@warunawickramasingha warunawickramasingha moved this from Assigned to Ready for Review in Mantid Maintenance Jan 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@warunawickramasingha warunawickramasingha

Labels
Bug Issues and pull requests that are regressions or would be considered a bug by users (e.g. crashing) Direct Inelastic Issues and pull requests related to direct inelastic Documentation Issues and pull requests related to documentation ISIS Team: Excitations/Vesuvio Issue and pull requests managed by the Excitiations/Vesuvio subteam at ISIS
Projects
Mantid Maintenance
Status: Ready for Review
Milestone
Release 6.13
Development

Successfully merging a pull request may close this issue.

38672 mslice test instructions mantidproject/mantid
3 participants
@cailafinn @sf1919 @warunawickramasingha