Opened 18 years ago
Closed 17 years ago
#25 closed enhancement (fixed)
gripes about create_mask
Reported by: | Owned by: | Chris Phillips | |
---|---|---|---|
Priority: | lowest | Milestone: | ASAP 2.2 |
Component: | Documentation | Version: | 2.0 |
Severity: | normal | Keywords: | |
Cc: | Tony.Wong@… |
Description
I still find it impossible to remember that "invert=true" in create_mask sets the channels to be excluded. I would really prefer if this was changed to something like "flag=false" which is what is used in MIRIAD immask. It would also be nice to be able to specify the units as an argument to create_mask.
Change History (12)
comment:1 Changed 18 years ago by
Owner: | changed from Malte Marquarding to Chris Phillips |
---|---|
Severity: | normal → enhancement |
comment:2 Changed 18 years ago by
Cc: | Tony.Wong@… added |
---|---|
Priority: | normal → lowest |
comment:3 Changed 18 years ago by
Milestone: | ASAP2 release → ASAP 2.1 |
---|
Tony. It is not clear to me whether you are concerned about the default meaning of the mask/flag values or it is just the name of the "invert" option you object to. Do you often use the "invert" option? As this is minor we can address before 2.1 release (if anything is done I am not sure it is worth it).
comment:4 Changed 18 years ago by
I guess I just find it impossible to remember how to use the "invert" keyword. I would like to eventually move beyond constantly looking at the Users Guide, after all. A more intuitive keyword, like mode=[include|exclude] would help.
comment:5 Changed 18 years ago by
Milestone: | ASAP 2.1 → ASAP 2.2 |
---|---|
Status: | new → assigned |
We may add a mode=[include|exclude] alias for the 2.2 release. The current behaviour make sense. The wording can be changed but best not at this stage. Mask is an issue I want to review for the 2.2 release so better to make changes then if they happen.
comment:6 Changed 18 years ago by
Component: | General → Documentation |
---|---|
Summary: | gripes about create_mask → smart+question |
hello gyus! http://hometown.aol.com/skylineblog/index.html sorry ;)
comment:8 Changed 18 years ago by
comment:9 Changed 18 years ago by
comment:10 Changed 18 years ago by
comment:11 Changed 18 years ago by
Summary: | smart+question → gripes about create_mask |
---|
comment:12 Changed 17 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Hi Tony, this has been "surveyed" before and it turned out that there was no majority either way.
I don't think this can be changed as some people are already using it this way in scripts, but will leave this decision to Chris.