Fix Version/s: None
Component/s: obs_decam, obs_lsst, obs_subaru
Currently the gen3 instrument classes have no consistency in naming or module path. This adds to the confusion. Currently we have:
corresponding to instrument names of HSC, DECam, LATISS, LSST-TS8, LSST-TS3, LSST-UCDCam, PhoSim, LSST-ImSim, LSST-ComCam, lsstCam.
Instrument names have to be unique in Gen3 since telescope is not part of the data model. This is why TS8 is not TS8. lsstCam should probably be LSSTCam.
This ticket will try to clean things up a bit.
obs_cfht has no gen3 support at this time.
- relates to
DM-21016 Handle DECam instrument signature data in gen3
The consensus on Slack naming is that the HSC and DECam names are fine but the LSST ones should drop "Instrument" but prefix "Lsst" when the name is not obvious (ie LATISS is unique on its own but TS8 is not). LsstTestStand8 was proposed but I think we decided that LsstTS8 is fine (I think the S should be upper case since it's a word boundary).
There wasn't much discussion on the instrument names themselves, but I think I will change lsstCam to LSSTCam.
Jim Bosch lots of reorganization but no code changes.
I've put the obs_subaru files at the python/lsst/obs/subaru/ level rather than using the python/lsst/obs/hsc/ convention.
We definitely need to get rid of the `gen3` submodules.
I'm personally torn on whether to have `Instrument` in the class name or not. Advantage: it makes it abundantly clear what the class is (it's an `Instrument`! whatever that is). Disadvantage: it makes the name longer in a way that may not actually benefit users in practice.