Opened 19 years ago
Last modified 16 years ago
#1022 closed enhancement
caution with instrument passed to detrend — at Initial Version
| Reported by: | eugene | Owned by: | Paul Price |
|---|---|---|---|
| Priority: | high | Milestone: | |
| Component: | psModules | Version: | unspecified |
| Severity: | minor | Keywords: | |
| Cc: |
Description
the call in pmFPAfileDefine to pmDetrendSelect requires the instrument name. This had been coming from the cameraName element of the config structure. However, this name includes the version information (eg, chip mosaic alternate name, etc), and is not the correct name for the detrend system. This is not a problem normally in ppImage because you are starting with a raw image and applying detrend data to that. however, it is a problem when using the detrend db to work on derived data versions. for example, to look up the astrometry model when processing psphot output.
i change the code for now to use FPA.INSTRUMENT, but this is perhaps not correct: it should be whatever given the value that is used to generate cameraName.
beyond that, I wonder if we are not overloading the name, and should not be identifying an explicit configuration word, rather than mangling the name.
