Ticket #1677 (closed: wontfix)

Opened 10 years ago

Last modified 5 years ago

Have properties containing paths added semi-automatically to the list of possible relative paths

Reported by: Russell Taylor Owned by: Russell Taylor
Priority: major Milestone: Release 2.2
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Jean Bilheux

Description

Perhaps if the property name includes 'directory/ies' or 'path'.

Change History

comment:1 Changed 10 years ago by Russell Taylor

(In [6373]) Add parameterDefinition.directory to the list of possible relative paths. Re #1677.

comment:2 Changed 10 years ago by Russell Taylor

  • Status changed from new to accepted

comment:3 Changed 10 years ago by Nick Draper

  • Milestone changed from Iteration 26 to Iteration 27

Bulk move of tickets to iteration 27, if your ticket is essential for Iteration 26 then move it back.

comment:4 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:5 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: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 Russell Taylor

In [13793]:

Squash start-up warning messages because cmake doesn't install empty directories. Re #1677.

comment:8 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:9 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:10 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:11 Changed 8 years ago by Russell Taylor

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

Had a look at this, but have decided it's not really worth it. This kind of thing should be picked up in testing in any case.

Issues:

  1. No way of telling whether the path has to exist or not if doing it like this.
  2. Can't retrieve the whole key in one go, only the part up to the next dot. So lots of recursion and copying vectors would be required.
  3. Would have to make a decision on what words to search for and add - more useful to have more, but also more work.

comment:12 Changed 8 years ago by Jean Bilheux

  • Status changed from verify to verifying
  • Tester set to Jean Bilheux

comment:13 Changed 8 years ago by Jean Bilheux

  • Status changed from verifying to closed

Agree. Not worth it.

comment:14 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 2524

Note: See TracTickets for help on using tickets.