so-artn / ARTN

Arizona Robotic Telescope Network
MIT License
8 stars 4 forks source link

CRVAL1 and CRVAL2 are now being set to 0 #35

Closed tepickering closed 4 years ago

tepickering commented 4 years ago

The correct RA and Dec are being populated in the RA and DEC headers, but are not being populated into the WCS headers CRVAL1 and CRVAL2.

pkubanek commented 4 years ago

This is most likely easy to fix. Just add CRVAL[12] to FITS template with the correct variable.

On Wed, Sep 18, 2019, 6:30 PM T. E. Pickering notifications@github.com wrote:

The correct RA and Dec are being populated in the RA and DEC headers, but are not being populated into the WCS headers CRVAL1 and CRVAL2.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/bjweiner/ARTN/issues/35?email_source=notifications&email_token=ABJYKBQW6OWBJB4FKWXYXYLQKLI3DA5CNFSM4IYFKTXKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HMJEPJA, or mute the thread https://github.com/notifications/unsubscribe-auth/ABJYKBTPFI7QS5DNPKWW753QKLI3DANCNFSM4IYFKTXA .

srswinde commented 4 years ago

I believe this is working now for at least the primary header. We will have to restart RTS2 to see if it is working for the 2 extensions.

Thanks Petr!

dsand commented 4 years ago

Hey Petr -- We are still having problems with this. We can populate the primary header, but we cannot seem to populate the image extension headers. Even when we put [CHANNEL1], [CHANNEL2] in the init file. Any chance you can take a look and see what we are doing wrong?

pkubanek commented 4 years ago

Sorry, got to it only now. You need to provide EXTNAME in [], CHANNEL[12] is just example (from LSST CCD testing days). I edited fitstemplate to:

[IM1]
CRVAL1 = @BIG61.TEL:ra
CRVAL2 = @BIG61.TEL:dec
SCTEST = TEST / this is a test

[IM2]
CRVAL1 = @BIG61.TEL:ra
CRVAL2 = @BIG61.TEL:dec

SCTEST = @W0.filter
SCTEST2 = FOOBAR / do you see this on chanel

give it a try (kill -1 $(pidof rts2-executor) shall be enough, if not, restart rts2-executor.

srswinde commented 4 years ago

I will implement the IM1 and IM2 stuff soon.

dsand commented 4 years ago

I believe this is working now--Closing