#925 closed enhancement (fixed)
pantasks.pro needs detrend.off to turn off dettool.* tasks
| Reported by: | 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)
Change History (5)
by , 19 years ago
| Attachment: | pantasks.pro.patch added |
|---|
comment:1 by , 19 years ago
| Status: | new → assigned |
|---|
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 , 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 , 19 years ago
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
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 , 19 years ago
| Component: | ippScripts → ippTasks |
|---|

Patch provides requested enhancement