Header fixes and LSSTCamSim support #473
Conversation
jchiang87
force-pushed
the
u/jchiang/header_fixes
branch
from
71401e9
to
303954a
Compare
| telcode = 'MC' | ||
| dayobs = eimage.header['DAYOBS'] | ||
| seqnum = eimage.header['SEQNUM'] | ||
| contrllr = eimage.header['CONTRLLR'] | ||
| phdu.header['TELESCOP'] = SIMONYI_TELESCOPE |
There was a problem hiding this comment.
Maybe factor out some of the other common keywords too? E.g., 'RAFTBAY', 'CCDSLOT', 'RA', 'DEC', 'ROTCOORD', 'ROTPA', ...?
There was a problem hiding this comment.
In order to do that more cleanly, I'd like to get rid of support for LsstCamImSim, otherwise I suspect the number of if statements will actually increase. In fact, in addition to getting rid of LsstCamImSim support, I'd like to support just *Sim cameras explicitly, e.g., LsstCamSim and LsstComCamSim, so also removing LsstCam as an option. I'll post an issue with this proposal.
| if eimage.header['IMGTYPE'] == 'SKYEXP': | ||
| phdu.header['RADESYS'] = 'ICRS' | ||
| phdu.header['TRACKSYS'] = 'RADEC' | ||
| else: |
There was a problem hiding this comment.
When does this get used? Simulated flats?
There was a problem hiding this comment.
(by this, I mean TRACKSYS = LOCAL).
There was a problem hiding this comment.
Yes, for anything not on-sky, specifically any calibration frames.
jchiang87
force-pushed
the
u/jchiang/header_fixes
branch
from
303954a
to
af7bcf7
Compare
…out contents of test eimage file and compress
jchiang87
force-pushed
the
u/jchiang/header_fixes
branch
from
af7bcf7
to
4c167f2
Compare
No description provided.