search

megbedell / wobble

precise data-driven RV fitting, now with tellurics!
https://wobble.readthedocs.io
MIT License
39 stars 16 forks source link

Missing Keyword #80

Open sushithor opened 2 months ago

sushithor commented 2 months ago

Hi,

I'm trying to run wobble with some reduced ESPRESSO data and when I try to convert the .fits files to .hdf5 it gives me the error "Keyword 'ESO QC CCF DRIFT DET0 MEAN' not found". Any idea on how to fix this error?

megbedell commented 2 months ago

This is probably happening here when the reader tries to scrape the RV drift from the FITS header and it might be coming up because the function was written for an older version of the ESPRESSO data products than what you've got. The RV drift isn't strictly required for wobble, it's just kept and applied to the wobble RVs for convenience, so you could work around this by commenting out that line (or by reading the spectra in yourself without calling the from_espresso function... there's some example of how to go about this in the CARMENES data tutorial notebook). Just remember that the instrument drift correction won't have been applied to the resulting wobble RVs.

For a long-term fix we'd need to update the from_ESPRESSO function to work with the latest version of the DRS output-- I'm not sure when I'll have the opportunity to do that, but feel free to submit a pull request if you figure it out!