Make HDU number available to ingest. #90
Conversation
| infoList.append(self.getInfoFromMetadata(md, info=phuInfo.copy())) | ||
| hduInfo = self.getInfoFromMetadata(md, info=phuInfo.copy()) | ||
| # We need the HDU number when registering MEF files. | ||
| hduInfo["hdu"] = extnum - 1 |
There was a problem hiding this comment.
This makes hdu a very special dataId key name. Have you checked if this could affect other obs packages?
I wonder if we want this to be added to some higher-level documentations.
There was a problem hiding this comment.
HDU is very FITS-centric for a generic butler...
There was a problem hiding this comment.
It does makes hdu special in that it could potentially collide with a header keyword named HDU. I haven't found any obs packages that use anything called HDU though. CFHT does a similar thing as this loop but calls it extension. There is a bit of a snag in that the normal translations can't affect this field, but I think I'd like to put that off in order to get decam not-broken in the 13.0 release.
@timj This isn't in the generic butler, it's in a task that parses FITS files.
This is used for ingesting MEF files (such as from Decam).
ctslater
force-pushed
the
tickets/DM-8690
branch
from
00c0a61
to
3d61c9d
Compare
This is used for ingesting MEF files (such as from Decam).