IPP Software Navigation Tools IPP Links Communication Pan-STARRS Links

Opened 19 years ago

Closed 19 years ago

Last modified 19 years ago

#925 closed enhancement (fixed)

pantasks.pro needs detrend.off to turn off dettool.* tasks

Reported by: jester@… Owned by: Paul Price
Priority: high Milestone:
Component: ippTasks Version: 2.2
Severity: minor Keywords:
Cc:

Description

Created an attachment (id=105)
Patch provides requested enhancement

In the pantasks.pro tasks, register chip cam warp stack diff can be turned off by saying step.off, but to turn off the tasks listed as dettool.* in status, I need to say detrend.off, which is a bit counterintuitive. Suggested patch attached.

Attachments (1)

pantasks.pro.patch (552 bytes ) - added by jester@… 19 years ago.
Patch provides requested enhancement

Download all attachments as: .zip

Change History (5)

by jester@…, 19 years ago

Attachment: pantasks.pro.patch added

Patch provides requested enhancement

comment:1 by Paul Price, 19 years ago

Status: newassigned

If anything, the intuitive short version would be 'det', rather than 'dettool' (e.g., we don't say 'warptool.off', but 'warp.off'). I prefer 'detrend' to 'det', but am willing to change to that. Which would you prefer?

comment:2 by jester@…, 19 years ago

My logic was that I looked at the 'status' output, and all the tasks with names like warp.something.orother can be turned off with warp.off, hence if I want to turn off the things that are listed as dettool.sub.task, I need to say dettool.off.

In other words, as long as the status display and the shortcut for turning on and off start with the same "rootname", it doesn't matter what it is; but the display and the toggle command should have the same rootname to make it intuitive.

comment:3 by Paul Price, 19 years ago

Resolution: fixed
Status: assignedclosed

Ah, I see --- the detrend tasks are listed as "dettool". That shouldn't be. Fixed in CVS head:

Changing names of tasks from dettoool.whatever to detrend.whatever, to be consistent with the other task names, which are not named after the tool but the action (bug 925).

comment:4 by Paul Price, 19 years ago

Component: ippScriptsippTasks
Note: See TracTickets for help on using tickets.