njcuk9999 / apero-utils

APERO affiliated utilities and tools
MIT License
3 stars 3 forks source link

[APERO] --filetype in apero_mk_template does not accept TELLU_OBJ #264

Closed njcuk9999 closed 3 months ago

njcuk9999 commented 3 months ago

From @vandalt

while pasting the runstring from ARI did not work yesterday when I tried? (see below)

>> apero_mk_template_nirps_he.py NLTT5556 --filetype=TELLU_OBJ --fiber=A --database=True --plot=0 --program=FTTEMP1[00874] --recipe_kind=tellu-science --parallel=True --shortname=FTTEMP1 --crunfile=online_run.ini

02:05:15.262-  |MKTEMP| ***************************************************************************
02:05:15.270-  |MKTEMP| *
02:05:15.270-  |MKTEMP| *  NIRPS_HE @PID-00017235147138708780-Z860 (V0.7.290)
02:05:15.270-  |MKTEMP| *           py3.9.19  git:v0.7.290-live
02:05:15.270-  |MKTEMP| *
02:05:15.277-  |MKTEMP| ***************************************************************************
...
02:05:15.327-  |MKTEMP| ***************************************************************************
Loading Arguments. Please wait...02:05:15.338-!!|MKTEMP| E[09-001-00001]: Argument Error (Error from ArgParse):
02:05:15.338-!!|MKTEMP| argument --filetype: invalid choice: 'TELLU_OBJ' (choose from 'EXT_E2DS',
02:05:15.338-!!|MKTEMP|         'EXT_E2DS_FF')
02:05:15.339-!!

Seems to be a problem with the definition of KW_OUTPUT (DRSOUTID) types for apero_mk_template

njcuk9999 commented 3 months ago

This should be fixed now for all instruments. Wasn't a problem unless using a runstring as apero processing does not give the --filetype keyword and thus default is used avoiding this check on options.