Ticket #5815 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

SANS: Bug with trans fit when one data point

Reported by: Anders Markvardsen Owned by: Anders Markvardsen
Priority: critical Milestone: Release 2.3
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Michael Reuter

Description


Change History

comment:1 Changed 8 years ago by Anders Markvardsen

Bug with trans fit - does this fix it. re #5815

Changeset: 564ec1a653abaa83ab061d0a857928063b8339b1

comment:2 Changed 8 years ago by Anders Markvardsen

Removed commented out lines of code. Re #5815

Changeset: 9c631640a99d61afab005edb24468cedb33327ed

comment:3 Changed 8 years ago by Anders Markvardsen

  • Status changed from new to accepted

To test:

  1. Open ISIS SANS GUI
  1. pick some mask file
  1. Load any SANS2D data
  1. Go to analysis details tab. For 'Wavelength' select a vary narrow Min and Max range. E.g. from 3.0 to 3.02 (it has to be small enough only to contain one data point)
  1. Make sure the tick box to the left of Trans Fit is ticked
  1. Hit 1D reduce button after hitting the Load Data button
  1. This will create various workspace including one that contain the reduced that for the selected range. Plot this workspace
  1. Go back to Analysis Details tab and untick tick box to the left of Trans Fit
  1. Go to Run Numbers tab and do a 1D reduce

what you should see is that the plotted reduced data should look basically the same, i.e. not scaled heavily

comment:4 Changed 8 years ago by Anders Markvardsen

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

comment:5 Changed 8 years ago by Anders Markvardsen

Bug with trans fit - does this fix it. re #5815

Changeset: 564ec1a653abaa83ab061d0a857928063b8339b1

comment:6 Changed 8 years ago by Anders Markvardsen

Removed commented out lines of code. Re #5815

Changeset: 9c631640a99d61afab005edb24468cedb33327ed

comment:7 Changed 8 years ago by Michael Reuter

  • Status changed from verify to verifying
  • Tester set to Michael Reuter

comment:8 Changed 8 years ago by Michael Reuter

  • Status changed from verifying to closed

This looks to be working correctly.

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 6661

Note: See TracTickets for help on using tickets.