Ticket #1987 (closed: fixed)

Opened 10 years ago

Last modified 5 years ago

Allow Fit Functions to be categorized

Reported by: Anders Markvardsen Owned by: Anders Markvardsen
Priority: critical Milestone: Release 2.0
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: #3985 Tester: Karl Palmen

Description

Categorize Fit Functions.

This categorization is e.g. meant to target users who use the Fit interface in MantidPlot. For example, a Muon user wants to easily select among all the Fit Functions those relevant for fitting Muon data.

This is likely to become increasingly relevant as more specialised custom Fit functions are added to Mantid. For example the Muon group at ISIS is threatening with adding 15 or more of these.

A suggestion for a user gui is: When a user wants to select a Fit function he/she is presented with a tree of nodes, where each node is a sub-category, for example, a Muon Fitting category, a general purpose category (where functions such as Gaussian and Lorentzian lives) etc. Note a given function may possibly be a member of multiple categories and a sub-category should be allowed to be split up into further sub-categories.

Change History

comment:1 Changed 10 years ago by Nick Draper

  • Owner set to Anders Markvardsen
  • Status changed from new to assigned
  • Milestone changed from Iteration 26 to Iteration 27

comment:2 Changed 10 years ago by Nick Draper

see ticket #2125 for a suggested impl

comment:3 Changed 10 years ago by Nick Draper

  • Milestone changed from Iteration 27 to Iteration 28

Bulk move of tickets at the end of iteration 27

comment:4 Changed 9 years ago by Nick Draper

  • Milestone changed from Iteration 28 to Iteration 29

Bulk move of tickets at the end of iteration 28

comment:5 Changed 9 years ago by Anders Markvardsen

(In [12031]) Extend IFitFunction API to allow for categorisation. Test for this and demonstrate how to do this from UserAlgorithm. re #1987

comment:6 Changed 9 years ago by Nick Draper

  • Milestone changed from Iteration 29 to Iteration 30

Accepted and assigned tickets moved at iteration 29 code freeze

comment:7 Changed 9 years ago by Nick Draper

  • Milestone changed from Iteration 30 to Iteration 31

Bulk move of tickets to iteration 31 at the iteration 30 code freeze

comment:8 Changed 9 years ago by Anders Markvardsen

  • Owner changed from Anders Markvardsen to Robert Whitley

comment:9 Changed 9 years ago by Nick Draper

  • Blocking 3985 added

follow the same approach as for algorithms.

comment:10 Changed 9 years ago by Anders Markvardsen

  • Priority changed from major to critical

comment:11 Changed 9 years ago by Anders Markvardsen

I should say the reason for putting this up to critical - is because the Muon is now starting to add muon fitting functions where the categorisation becomes important

comment:12 Changed 9 years ago by Anders Markvardsen

Changed default categorization seperator for fit functions. re #1987

1) add categories() method (as from Algorithm) 2) categorization fit function "ExpDecay" and added unit test

Changeset: 1ebb0d92ec54041afd1e9e5591cd44b6ae1dae99

comment:13 Changed 9 years ago by Anders Markvardsen

  • Owner changed from Robert Whitley to Anders Markvardsen
  • Summary changed from Categorize Fit Functions to Allow Fit Functions to be categorized

This ticket changed to dealing with add the code for allowing fit function categorization. Other tickets #3985 and #4383 created which will deal with exposing this to users.

comment:14 Changed 9 years ago by Anders Markvardsen

  • Status changed from assigned to accepted

comment:15 Changed 9 years ago by Anders Markvardsen

Get more than the windows build to work hopefully. Re #1987

Changeset: ac40e7c8468474103c2bafe572efc31cf655d1c8

comment:16 Changed 9 years ago by Anders Markvardsen

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

comment:17 Changed 9 years ago by Karl Palmen

  • Status changed from verify to verifying
  • Tester set to Karl Palmen

comment:18 Changed 9 years ago by Karl Palmen

  • Status changed from verifying to closed

The categories of General and Muon have been added to the functions ExpDecay and ExpDecayOsc and this addition has been unit tested successfully in each case.

comment:19 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 2834

Note: See TracTickets for help on using tickets.