Remove redundant assemble_fibermap script

desihub / desispec

DESI spectral pipeline

BSD 3-Clause "New" or "Revised" License

35 stars 24 forks source link

Remove redundant assemble_fibermap script #2181

Closed weaverba137 closed 4 months ago

weaverba137 commented 6 months ago

bin/assemble_fibermap and bin/desi_assemble_fibermap are functionally identical. We propose removing bin/assemble_fibermap. This should at least wait until desihub/nightwatch#391 is merged.

moustakas commented 4 months ago

@weaverba137 can we act on this ticket now?

weaverba137 commented 4 months ago

@sybenzvi, has the merged version of desihub/nightwatch#391 been deployed?

weaverba137 commented 4 months ago

While we're waiting on that, I'll prepare a PR.