Add maxlogicalwindow param to celldetect call

sot / astromon

Astrometric accuracy (celestial location) monitor

0 stars 0 forks source link

Add maxlogicalwindow param to celldetect call #10

Closed jeanconn closed 5 years ago

jeanconn commented 6 years ago

See #9

jeanconn commented 6 years ago

I think I'm actually going to install this without testing or merging and use the cron job to test it. Looks like all the files are owned by the cron job owner.

jeanconn commented 6 years ago

I think the job runs on Tuesday mornings, so we can either see if the change is enough from the logs tomorrow, or @taldcroft, you can delete your celldetect.par file too. At some point we should probably also configure this job so it uses a sandbox'd PFILES.

jeanconn commented 6 years ago

Live test was apparently a dud

celldetect source_img.fits source_src2.fits thresh=3.0 maxlogicalwindow=2048 clobber=yes 
parammatch : can't find parameter maxlogicalwindow in reference list 
# celldetect (CIAO 4.10): ERROR: Could not open parameter file 'celldetect.par'.

Maybe run punlearn celldetect from a new ciao?

jeanconn commented 6 years ago

It looks like the 'punlearn celldetect' seems to have gotten this script going again. If this doesn't break anything for @taldcroft in regular use of CIAO we can merge this and keep it this way. Otherwise we can move ahead with a plan to use an isolated PFILES for this trending task.