Ticket #5916 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

Calib background range is not being used

Reported by: Nick Draper Owned by: Dereck Kachere
Priority: critical Milestone: Release 2.3
Component: Mantid Keywords: PatchCandidate,Released
Cc: Blocked By:
Blocking: Tester: Alex Buts

Description

A severe bug has appeared in the last release within “calib” of the C2E tab for indirect spec.

When calculating the res file mantid wants now that the background is taken over the whole energy range Before one could use e.g. -0.6 - -0.5 meV just a range to define the background as it is done in tof for the calib file. This large range seems to have consequences for the res file, because it is now negative away from the elastic range!!! It seems to subtract too much now. And negative values in the resolution file makes conv fit and Fourier trafo more than doubtful or unuseable.

This was introduced within the last release and needs fixing with high priority.

Thanks,

franz Demmel

Derek,

Please work on this with Martyn / Anders

Change History

comment:1 Changed 8 years ago by Dereck Kachere

  • Status changed from new to accepted

comment:2 Changed 8 years ago by Dereck Kachere

refs #5916: One can now choose arbitrary ranges for indirect bkg

This is related to the calibration tab.

Changeset: f7469ea8b38b8554dc80756cfcc8b99de22c7dfb

comment:3 Changed 8 years ago by Dereck Kachere

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

comment:4 Changed 8 years ago by Dereck Kachere

  • Keywords PatchCandidate added

comment:5 Changed 8 years ago by Alex Buts

  • Status changed from verify to verifying
  • Tester set to Alex Buts

comment:6 Changed 8 years ago by Alex Buts

  • Status changed from verifying to closed

one can choose arbitrary ranges indeed. The ranges affect resolution file base.

comment:7 Changed 8 years ago by Dereck Kachere

refs #5916: One can now choose arbitrary ranges for indirect bkg

This is related to the calibration tab.

Changeset: f7469ea8b38b8554dc80756cfcc8b99de22c7dfb

comment:8 Changed 7 years ago by Nick Draper

  • Keywords PatchCandidate,Released added; PatchCandidate removed

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 6762

Note: See TracTickets for help on using tickets.