keflavich / brick-jwst-2221

Data reduction & analysis for the Brick JWST 2221 data
4 stars 3 forks source link

added vgroup to tblfns #23

Closed SpacialTree closed 2 months ago

SpacialTree commented 2 months ago

Added vgroup to glob list of table filenames so merge catalogs merges the correct catalogs.

SpacialTree commented 2 months ago

The previous version will not find the new catalog names as there is no wildcard in the old filename.

..visit{visitid:03d}_exp{exposure:05d}... versus ...visit{visitid:03d}_vgroup*_exp{exposure:05d}...

The script without this update will only merge the old catalog files, and it will fail if only the current version has been run.

keflavich commented 2 months ago

ah, maybe it's better to just put a wildcard (*) instead of _vgroup*?

SpacialTree commented 2 months ago

...visit{visitid:03d}*_exp{exposure:05d}...

Like this?