Ticket #1364 (closed: fixed)

Opened 10 years ago

Last modified 5 years ago

Small bugs in the SANS reduction software

Reported by: Steve Williams Owned by: Steve Williams
Priority: major Milestone: Iteration 25
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Martyn Gigg

Description

I some times recieve bugs that Richard Heenan and Stephen King sent to Mantid help.

Change History

comment:1 Changed 10 years ago by Steve Williams

  • Status changed from new to accepted

Problem with the labeling of monitors in the SANS GUI

Subject: Re: [Mantid-help] SANS GUI: fatal crash bug
Ahhh… I think I understand why now!
 
Under the analysis tab in bank selection there are two boxes to specify spectrum numbers.  By chance I hit reduce without changing the spectrum for transmission monitor (as I had been doing yesterday) and everything worked fine and looked right.  When I realized what I’d done I changed the spectrum number for the transmission monitor from 2 to 3 as I’d been doing and Mantid bombed.
 
The way I’d interpreted these boxes – and I think anyone without a priori knowledge would given the wording alongside - was that these were the spectrum numbers containing the incident beam monitor spectrum and the transmission monitor spectrum, respectively.
 
Whereas, what I now suspect is that these boxes are actually BOTH supposed to specify incident beam monitor spectra but one to be used normalizing the SANS data and one to be used normalizing the TRANS.  Can you confirm if this is the case, please?!
 
If you concur, can I suggest we change the wording such that:
 
Monitor spectrum -> Incident monitor (SANS)
Transmission monitor -> Incident monitor (TRANS)
 
Thanks,
Steve
 
From: Draper, Nick (-,RAL,ISIS) 
Sent: 02 July 2010 15:24
To: Mantid Help; Williams, Steve (STFC,RAL,ISIS)
Cc: King, Stephen (STFC,RAL,ISIS)
Subject: RE: [Mantid-help] SANS GUI: fatal crash bug
 
Steve, Can you have a look at  this please.
 
Regards,
 
Nick Draper
Mantid Project Manager
R78, 1.02
 
Phone +44 1296 445228
Skype: nicholas_draper
 
________________________________________
From: mantid-help-bounces@mantidproject.org on behalf of stephen.king@stfc.ac.uk
Sent: Thu 7/1/2010 5:33 PM
To: Mantid Help
Cc: King, Stephen (STFC,RAL,ISIS)
Subject: [Mantid-help] SANS GUI: fatal crash bug
I’ve discovered a bug that crashes Mantid.
SANS2D /cycle_09_2 data.  Mask attached.
Sample SANS: 1017, Sample TRANS: 990, Sample DIRECT: 1002
Trans monitor is Sp 3
Mantid falls over during the transmission calculation. I think it is because the transmission runs have slightly different chopper phasing (our fault) so it hits a divide by zero.  But the condition is not trapped.
 
Steve

comment:2 Changed 10 years ago by Steve Williams

(In [5079]) When CalculateTransmission is called from a SANS reduction the same monitor can't be specified as both the incident and transmitted re #1364

comment:3 Changed 10 years ago by Steve Williams

  • Status changed from accepted to verify
  • Resolution set to fixed

(In [6059]) The SANS interface now checks if the Python MaskFile() function completed successfully, if not an error is shown to the user. This improves the situation when the LoadEmptyInstrument() takes a long time to load blocking Python closes #1364

comment:4 Changed 10 years ago by Mathieu Doucet

(In [6064]) Copied Steve's edits to the new ISISReducer, and corrected minor bugs. Re #1364.

comment:5 Changed 10 years ago by Mathieu Doucet

(In [6070]) Added TransmissionSample and TransmissionCan. Re #1364.

comment:6 Changed 10 years ago by Mathieu Doucet

Punched in the wrong ticket ID for [6070]. Meant #1447

comment:7 Changed 10 years ago by Mathieu Doucet

(In [6091]) Added AssignSample. Re #1364.

comment:8 Changed 10 years ago by Mathieu Doucet

(In [6092]) Added AssignSample tests. Re #1364.

comment:9 Changed 10 years ago by Nick Draper

  • Component set to Mantid
  • Milestone changed from Ongoing to Iteration 25

comment:10 Changed 10 years ago by Martyn Gigg

  • Status changed from verify to verifying
  • Tester set to Martyn Gigg

Confirmed fixed.

comment:11 Changed 10 years ago by Martyn Gigg

  • Status changed from verifying to closed

comment:12 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 2211

Note: See TracTickets for help on using tickets.