Skip to content

make all seq commands consistent

cec m requested to merge cmd_select into master

Review all the commands applicable to sequences:

command argument comment Done
getref seqname change to sequencename [x]
light_curve sequencename [x]
merge sequence1 no change required [x]
preprocess sequencename [x]
register sequence change to sequencename [x]
select add sequencename argument + make scriptable [x]
seqapplyreg sequencename [x]
seqclean sequencename [x]
seqcosme sequencename [x]
seqcosme_cfa sequencename [x]
seqcrop sequencename [x]
seqextract_Green sequencename [x]
seqextract_Ha sequencename [x]
seqextract_HaOIII sequencename [x]
seqheader sequencename [x]
seqfind_cosme sequencename [x]
seqfind_cosme_cfa sequencename [x]
seqfindstar sequencename [x]
seqfixbanding sequencename [x]
seqmerge_cfa sequencename [x]
seqmtf sequencename [x]
seqpsf [sequencename ~~add required sequencename argument ~~ [ ]
seqplatesolve sequencename [x]
seqrl sequence change to sequencename [x]
seqsb sequence change to sequencename [x]
seqsplit_cfa sequencename [x]
seqstarnet sequence change to sequencename [x]
seqstat sequencename [x]
seqsubsky sequencename [x]
seqtilt [sequencename] add required sequencename argument [x]
seqwiener sequence [x]
setmagseq magnitude add sequencename argument + rename to seqsetmag [x]
unselect add sequencename argument + make scriptable [x]
unsetmagseq add sequencename argument + rename to sequnsetmag [x]

Make sure they all call load_sequence so that . is a valid argument (replaces loaded sequence with its name).
Make sure they all have the same consistent sequencename argument.
Make sure sequencename argument is never optional

Note: seqpsf has been left out of this round of review. sequencename is still optional. Will reconsider in the future.

Edited by cec m

Merge request reports