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.

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

6.12 Manual Testing - Testing Instruction Maintenance #38661

Open
3 tasks done
MialLewis opened this issue Jan 22, 2025 · 2 comments · May be fixed by #38702
Open
3 tasks done

6.12 Manual Testing - Testing Instruction Maintenance #38661

MialLewis opened this issue Jan 22, 2025 · 2 comments · May be fixed by #38702
Assignees
Labels
Maintenance Unassigned issues to be addressed in the next maintenance period.
Milestone

Comments

@MialLewis
Copy link
Contributor

MialLewis commented Jan 22, 2025

An umbrella issue for small documentation maintenance tasks identified during manual testing. Feel free to add to this list.

  • QENS - Convolution tab, step 17. Should state 5 plots per input spectra not in total.
  • Corrections test, Monte Carlo. Sample Workspace changed to Input Workspace.
  • Corrections test, Monte Carlo. Instructions refer to container thickness, this has been changed to container front and back.
@MialLewis MialLewis added the Maintenance Unassigned issues to be addressed in the next maintenance period. label Jan 22, 2025
@sf1919 sf1919 moved this from Icebox to Backlog in Mantid Maintenance Jan 22, 2025
@sf1919 sf1919 added this to the Release 6.13 milestone Jan 22, 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

It seems these test instructions are written to run independently such that one can start testing from Calculate Monte Carlo Absorption Tab tab if required and the Inelastic Corrections GUI can be closed/reopened in between tests.

@MialLewis Are you suggesting to select the Input Workspace from ADS by selecting Workspace rather than freshly loading from a file?

Image

Thanks!

@warunawickramasingha warunawickramasingha moved this from Assigned to In progress in Mantid Maintenance Jan 24, 2025
@MialLewis
Copy link
Contributor Author

It seems these test instructions are written to run independently such that one can start testing from Calculate Monte Carlo Absorption Tab tab if required and the Inelastic Corrections GUI can be closed/reopened in between tests.

@MialLewis Are you suggesting to select the Input Workspace from ADS by selecting Workspace rather than freshly loading from a file?

Image

Thanks!

In the testing instructions it is stated With the Sample combo box set to File click browse and select the file irs26176_graphite002_red from the Sample Data folder. This should read With the _Input Workspace_ combo box set to File click browse and select the file irs26176_graphite002_red from the Sample Data folder.

@warunawickramasingha warunawickramasingha linked a pull request Jan 27, 2025 that will close this issue
@warunawickramasingha warunawickramasingha moved this from In progress 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
Labels
Maintenance Unassigned issues to be addressed in the next maintenance period.
Projects
Status: Ready for Review
Development

Successfully merging a pull request may close this issue.

3 participants