Ticket #3402 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Should deprecated algorithms be hidden in the MantidPlot algorithm list?

Reported by: Russell Taylor Owned by: Nick Draper
Priority: minor Milestone: Release 2.5
Component: Mantid Keywords:
Cc: Blocked By: #5209
Blocking: Tester: Peter Peterson

Description

....either automatically or via an option in the preferences panel

Change History

comment:1 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:2 Changed 9 years ago by Nick Draper

  • Milestone changed from Iteration 32 to Iteration 33

Moved to iteration 33 at iteration 32 code freeze

comment:3 Changed 8 years ago by Nick Draper

  • Milestone changed from Release 2.1 to Release 2.2

Moved at end of release 2.1

comment:4 Changed 8 years ago by Nick Draper

  • Milestone changed from Release 2.2 to Release 2.3

http://www.mantidproject.org/Release_Notes_2_2_Full_List

comment:5 Changed 8 years ago by Nick Draper

  • Milestone changed from Release 2.3 to Release 2.4

Moved to milestone 2.4

comment:6 Changed 8 years ago by Nick Draper

  • Milestone changed from Release 2.4 to Release 2.5

comment:7 Changed 8 years ago by Nick Draper

  • Owner changed from Nick Draper to Anyone
  • Status changed from new to assigned

Hide deprecated algorithms. This should be done by changing the category entry to Deprecated and making that category hidden by default.

comment:8 Changed 8 years ago by Nick Draper

  • Status changed from assigned to accepted
  • Owner changed from Anyone to Nick Draper

comment:9 Changed 8 years ago by Nick Draper

  • Blocked By 5209 added

Ticket 5209 must be tested first

comment:10 Changed 8 years ago by Nick Draper

re #3402 moved several deprecated algorithms to the deprecated catorgy

Changeset: b2474cc0e39ce1f411dff63afaed399b861d971c

comment:11 Changed 8 years ago by Nick Draper

re #3402 remove test on algorithm category

Changeset: d0dc55904f466653652d31f02953489aa599d70a

comment:12 Changed 8 years ago by Nick Draper

re #3402 remove test on algorithm category

Changeset: d0dc55904f466653652d31f02953489aa599d70a

comment:13 Changed 8 years ago by Nick Draper

Moved many of the deprecated algorithms to the Deprecated category. This should be hidden by default (done in #5209).

I have not moved some algorithms where they are just deprecated because they have newer versions of the same named algorithm.

Tester: just check that algorithms such as ExtractMasking are no longer visible in the algorithm tree.

comment:14 Changed 8 years ago by Nick Draper

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

branch feature/3402_deprecated_category

comment:15 Changed 8 years ago by Peter Peterson

  • Status changed from verify to verifying
  • Tester set to Peter Peterson

comment:16 Changed 8 years ago by Nick Draper

re #3402 moved several deprecated algorithms to the deprecated catorgy

Changeset: 29a8fa941f1348f0f0465b4819fc44aebeaeec5f

comment:17 Changed 8 years ago by Nick Draper

re #3402 remove test on algorithm category

Changeset: 12710617ff1657f4d0d50d3e071b0c18acbfabc0

comment:18 Changed 8 years ago by Peter Peterson

  • Status changed from verifying to closed

This was done correctly. It may have been better to implement DeprecatedAlgorithm::category() so people could just delete their own category method. Another way is to add the deprecated category rather than replace with it.

comment:19 Changed 8 years ago by Nick Draper

re #3402 moved several deprecated algorithms to the deprecated catorgy

Changeset: 29a8fa941f1348f0f0465b4819fc44aebeaeec5f

comment:20 Changed 8 years ago by Nick Draper

re #3402 remove test on algorithm category

Changeset: 12710617ff1657f4d0d50d3e071b0c18acbfabc0

comment:21 Changed 8 years ago by Nick Draper

re #3402 moved several deprecated algorithms to the deprecated catorgy

Changeset: 29a8fa941f1348f0f0465b4819fc44aebeaeec5f

comment:22 Changed 8 years ago by Nick Draper

re #3402 remove test on algorithm category

Changeset: 12710617ff1657f4d0d50d3e071b0c18acbfabc0

comment:23 Changed 8 years ago by Nick Draper

re #3402 moved several deprecated algorithms to the deprecated catorgy

Changeset: 29a8fa941f1348f0f0465b4819fc44aebeaeec5f

comment:24 Changed 8 years ago by Nick Draper

re #3402 remove test on algorithm category

Changeset: 12710617ff1657f4d0d50d3e071b0c18acbfabc0

comment:25 Changed 7 years ago by Nick Draper

re #3402 moved several deprecated algorithms to the deprecated catorgy

Changeset: 29a8fa941f1348f0f0465b4819fc44aebeaeec5f

comment:26 Changed 7 years ago by Nick Draper

re #3402 remove test on algorithm category

Changeset: 12710617ff1657f4d0d50d3e071b0c18acbfabc0

comment:27 Changed 7 years ago by Nick Draper

re #3402 moved several deprecated algorithms to the deprecated catorgy

Changeset: 29a8fa941f1348f0f0465b4819fc44aebeaeec5f

comment:28 Changed 7 years ago by Nick Draper

re #3402 remove test on algorithm category

Changeset: 12710617ff1657f4d0d50d3e071b0c18acbfabc0

comment:29 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​4249

Note: See TracTickets for help on using tickets.