Ticket #7246 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

ConvertAxisByFormula Algorithm

Reported by: Nick Draper Owned by: Nick Draper
Priority: major Milestone: Release 2.6
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Roman Tolchenov

Description

To allow people to do custom axis adjustments by defining a math formula.

Using muparser.

Change History

comment:1 Changed 7 years ago by Nick Draper

  • Summary changed from CovertAxisByFormula Algorithm to ConvertAxisByFormula Algorithm

comment:2 Changed 7 years ago by Nick Draper

re #7246 first draft with bugs, commits to transfer computer

Changeset: e8bc78d7e67ff3f04ca8ba77eea5f5db9302b2eb

comment:3 Changed 7 years ago by Nick Draper

re #7246 better now, but needs more tests and testing

Changeset: 9ced42a6e7111d1d7ea14842e3b399e7141aa581

comment:4 Changed 7 years ago by Nick Draper

re #7246 better now, but needs more tests and testing

Changeset: 7c3ea8005fb7ebefe20c933b9f8ff67df30b8c3b

comment:5 Changed 7 years ago by Nick Draper

re #7246 Tests good, progress added

Changeset: 126e99d91a196c9b3e520921c3d6f860674f309a

comment:6 Changed 7 years ago by Nick Draper

  • Status changed from new to accepted

To test:

TEST 1

  • load up a workspace
  • Plot a 1D graph and note the axes labels and values.
  • Run the algorithm, change the X axis using a formula of something like x*x, or X2 or log(x), do not set the title or unit
  • replot the graph, check you see the changes, and the unit has not changed
  • Rerun the test this time change the unit and title as well, check that you see the changes

TEST 2

  • load up a workspace
  • Run it throughConvertSpectrumAxis and change the axis to theta, phi or something like that
  • Plot a 1D graph and note the axes labels and values.
  • Run the algorithm, change the Y axis using a formula of something like y/1000, do not set the title or unit
  • replot the graph, check you see the changes, and the unit has not changed
  • Rerun the test this time change the unit and title as well, check that you see the changes
  • You could also check that you can change the X axis on this dataset as well.

comment:7 Changed 7 years ago by Nick Draper

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

comment:8 Changed 7 years ago by Roman Tolchenov

  • Status changed from verify to verifying
  • Tester set to Roman Tolchenov

comment:9 Changed 7 years ago by Roman Tolchenov

Renamed property name to start with a capital letter.

Fixed compiler warning. Re #7246

Changeset: ffecbda169a8e6a3701665e318668d83ea2aed2e

comment:10 Changed 7 years ago by Roman Tolchenov

  • Status changed from verifying to closed

comment:11 Changed 7 years ago by Nick Draper

  • Component changed from Mantid to Framework

comment:12 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 8092

Note: See TracTickets for help on using tickets.