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

Manual Testing Indirect interfaces #37985

Closed
MohamedAlmaki opened this issue Sep 17, 2024 · 12 comments
Closed

Manual Testing Indirect interfaces #37985

MohamedAlmaki opened this issue Sep 17, 2024 · 12 comments
Assignees
Labels
Manual Tests Only for issues that are unscripted testing tasks during the release period
Milestone

Comments

@MohamedAlmaki
Copy link
Contributor

You have been assigned manual testing. The hope is to catch as many problems with the code before release, so it would be great if you can take some time to give a serious test to your assigned area. Thank you!!

The general guide to manual testing:

  • The tests must be performed on the installer versions of the final release candidate. Not on local compiled code.
  • Serious errors involving loss of functionality, crashes etc. should be raised
    as issues with the current release as a milestone and an email sent to the project manager immediately.
  • Minor and cosmetic issues should be raised as issues against the forthcoming
    releases.
  • First try things that should work, then try to break Mantid, e.g. entering invalid values, unexpected characters etc.
  • Don't spend more than a few hours on the testing as fatigue will kick in.
  • If you find errors in the documentation, please correct them.
  • Comment against this ticket the OS environment you are testing against.
  • Disable Usage reporting before you start testing
  • Close the this issue once you are done.
  • Time how long this manual test takes for you to do and leave a comment about it in this issue.

Specific Notes:

Follow the directions at:

@MohamedAlmaki MohamedAlmaki added the Manual Tests Only for issues that are unscripted testing tasks during the release period label Sep 17, 2024
@MohamedAlmaki MohamedAlmaki added this to the Release 6.11 milestone Sep 17, 2024
@robertapplin
Copy link
Contributor

robertapplin commented Sep 18, 2024

On Windows there is a freeze and then crash with these instructions:

  1. Open Interfaces > Indirect > Data reduction
  2. Make sure Instrument is set to IRIS
  3. Go to the ISIS calibration
  4. Enter Input Runs 26173
  5. Click Run
  6. This should generate a workspace with _calib at the end, delete this workspace
  7. Check the Scale by factor checkbox and change the value of the scale to 0.5.
  8. Click Run
  9. This should generate a workspace with _calib
  10. Check the Create RES box
  11. Click Run
  12. This should generate a workspace with _res at the end
  13. Freeze and crash!

@robertapplin
Copy link
Contributor

robertapplin commented Sep 18, 2024

On Windows only:

  1. On ISIS Diagnostics
  2. Load 26176 for sample.
  3. Use 26173 for calibration from previous test.
  4. Tick Use Two ranges.
  5. Widen the start and end of the Peak region.
  6. Move the background region to the end of the spectrum.
  7. Click Run
  8. Freeze and crash!

@robertapplin
Copy link
Contributor

@rbauststfc I've re-tried these tests on Windows and linuxthis morning but unfortunately I can't replicate the crashes. Are you able to replicate them still on your Windows machine?

@rbauststfc
Copy link
Contributor

Yes, I'm afraid it does. In case it's useful, on the crash from the ISIS Diagnostics tab, it doesn't happen if I exclude step 5 from the instructions to reproduce.

I think my Windows laptop is quite low spec, so not sure if that could have anything to do with it. You could get a few Windows users to give this a try, if you think it's worth doing a bit more digging.

@rbauststfc
Copy link
Contributor

I've just had a go in my installation of 6.10 and I don't get crashes in either case. The first works fine and the second flashes up an error message box and logs the following in the messages pane:

Error in execution of algorithm TimeSlice:
Problem setting "StartWorkspaceIndex" in CropWorkspace-v1: When converting parameter "StartWorkspaceIndex": When setting value of property "StartWorkspaceIndex": Selected value -1 is < the lower bound (0)
  at line 137 in 'C:/MantidInstall/plugins/python\algorithms/WorkflowAlgorithms/TimeSlice.py'
  caused by line 215 in 'C:/MantidInstall/plugins/python\algorithms/WorkflowAlgorithms/TimeSlice.py'
  caused by line 1081 in 'C:\MantidInstall\bin\lib\site-packages\mantid\simpleapi.py'
  caused by line 970 in 'C:\MantidInstall\bin\lib\site-packages\mantid\simpleapi.py'
  caused by line 953 in 'C:\MantidInstall\bin\lib\site-packages\mantid\simpleapi.py'
Error starting batch algorithm
Got error from algorithm "TimeSlice"
Stopping batch algorithm because of execution error

@robertapplin
Copy link
Contributor

Thank you for checking! @MohamedAlmaki and @adriazalvarez can either of you replicate these crashes with your Windows machines?

@adriazalvarez
Copy link
Contributor

adriazalvarez commented Sep 19, 2024

@robertapplin I'm not able to replicate either of the crashes (Conda version on Windows) .
ISIS Diagnostic gives an error if you don't set Spectra Min to at least 3, but it doesn't freeze and crash.

@rbauststfc
Copy link
Contributor

Thanks @adriazalvarez, that's interesting because I've been using the standalone installer, not a Conda install. @robertapplin, how had you installed Mantid when you tested on Windows?

@robertapplin
Copy link
Contributor

Thanks for testing @adriazalvarez . I was using the standalone installer too. @MohamedAlmaki do you have any luck replicating this?

@MohamedAlmaki
Copy link
Contributor Author

@robertapplin I could reproduce both crashes on my Windows machine, but on Linux works fine. The crash happens in the reading file step. I'm going to investigate more with a development build

@robertapplin
Copy link
Contributor

@robertapplin I could reproduce both crashes on my Windows machine, but on Linux works fine. The crash happens in the reading file step. I'm going to investigate more with a development build

Thats great, thank you!

@MohamedAlmaki
Copy link
Contributor Author

I've raised an issue for this here #38078 so I guess you can close this issue if you don't have any other unraised issues

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Manual Tests Only for issues that are unscripted testing tasks during the release period
Projects
None yet
Development

No branches or pull requests

5 participants