Ticket #2039 (closed: fixed)

Opened 10 years ago

Last modified 5 years ago

Log binning shouldn't be allowed with negative Low in ConvertToEnergy

Reported by: Michael Reuter Owned by: Russell Taylor
Priority: major Milestone: Iteration 26
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Michael Reuter

Description

The ConvertToEnergy GUI currently allows the user to perform logarithmic binning when a negative Low value is given for the rebinning parameters. This causes the program to eat up all available memory and slows the machine. The rate at which this happens depends on the rebinning parameters. Regardless, log binning with negative axis values just shouldn't be allowed since it is incorrect.

Change History

comment:1 Changed 10 years ago by Nick Draper

  • Status changed from new to assigned
  • Owner changed from Martyn Gigg to Russell Taylor

comment:2 Changed 10 years ago by Russell Taylor

  • Status changed from assigned to accepted

comment:3 Changed 10 years ago by Russell Taylor

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

(In [7988]) Easy fix. Fixes #2039.

comment:4 Changed 10 years ago by Michael Reuter

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

comment:5 Changed 10 years ago by Michael Reuter

  • Status changed from verifying to closed

This was tested via the ConvertToEnergy UI and now works to protect the user from entering a negative low value while doing log binning.

comment:6 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​2886

Note: See TracTickets for help on using tickets.