Ticket #7749 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

EQSANS: make sure sensitivity calculation works w/beam center

Reported by: Mathieu Doucet Owned by: Mathieu Doucet
Priority: major Milestone: Release 3.0
Component: SANS Keywords:
Cc: Blocked By:
Blocking: Tester: Jay Rainey

Description

Clicking the "Compute Sensitivity" button on the EQSANS UI doesn't work if the option to compute the beam center is turned on at the same time.

Change History

comment:1 Changed 7 years ago by Mathieu Doucet

  • Status changed from new to inprogress

Re #7749 Fix issue with setup

Changeset: 60c66708f5b38e5dadf3d7c153bb26e6a57dd060

comment:2 Changed 7 years ago by Mathieu Doucet

Re #7749 Fix issue with output sensitivity

Changeset: e3bc6e1175af1aa355573e40e6e8d1a0de768cb4

comment:3 Changed 7 years ago by Mathieu Doucet

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

To test:

  • Start the EQSANS reduction UI.
  • In the Tools menu, turn debug mode ON.
  • On the Detector tab, check the "Use beam finder" box, then enter a run number in the data file box under it (run 1466 from the systemtests would work).
  • Check the "Perform sensitivity correction" box, then enter a run number in the sensitivity data file box (run 4061 from the systemtests data would work well here).
  • Check the "Patch sensitivity" box.
  • Click the "Create sensitivity file" button.
  • A new workspace named 'sensitivity' should appear in the ADS and there should be no error.

comment:4 Changed 7 years ago by Jay Rainey

  • Status changed from verify to verifying
  • Tester set to Jay Rainey

comment:5 Changed 7 years ago by Jay Rainey

Testing

  • Computing sensitivity now works if the beam center is turned on at the same time.
  • Tested on Ubuntu 12.04

Code review

  • Clean code with no visible issues.

comment:6 Changed 7 years ago by Jay Rainey

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/feature/7749_fix_eqsans_sensitivity'

comment:7 Changed 7 years ago by Nick Draper

  • Component changed from Framework to SANS

comment:8 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 8594

Note: See TracTickets for help on using tickets.