Ticket #2652 (assigned)
Check for Algorithm and ParameterName conformance in alg registration
Reported by: | Nick Draper | Owned by: | Peter Peterson |
---|---|---|---|
Priority: | critical | Milestone: | Backlog |
Component: | Framework | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Tester: |
Description (last modified by Peter Peterson) (diff)
Algorithm and Parameter Names should be PascalCase. Neither should contain spaces or underscores.
We should check that on algorithm registration, both for C++ and Python algorithms. Non-Conformant algorithms should emit a warning log message, but still work as best they can.
This is probably best implemented in the API\AlgorithmManager class (as it creates an instance of each algorithm and is involved in registration)
Change History
comment:3 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:4 Changed 9 years ago by Nick Draper
- Milestone changed from Iteration 29 to Iteration 30
"New" tickets moved at the code freeze of iteration 29
comment:5 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:6 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:7 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:8 Changed 8 years ago by Nick Draper
- Milestone changed from Release 2.2 to Release 2.3
Moved at the end of release 2.2
comment:9 Changed 8 years ago by Nick Draper
- Milestone changed from Release 2.3 to Release 2.4
Moved to milestone 2.4
comment:11 Changed 7 years ago by Nick Draper
- Milestone changed from Release 2.5 to Release 2.6
comment:13 Changed 7 years ago by Peter Peterson
- Description modified (diff)
- Milestone changed from Release 2.6 to Backlog
comment:14 Changed 7 years ago by Nick Draper
- Status changed from new to assigned
Bulk move to assigned at the introduction of the triage step
comment:15 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 3499
Reminder of the naming rules has been sent to Mantid developers