Skip to content

Commit

Permalink
edits and new script
Browse files Browse the repository at this point in the history
  • Loading branch information
megbedell committed Jul 26, 2019
1 parent 87d9078 commit a2fb927
Show file tree
Hide file tree
Showing 2 changed files with 28 additions and 4 deletions.
9 changes: 5 additions & 4 deletions paper/paper.tex
Original file line number Diff line number Diff line change
Expand Up @@ -211,7 +211,7 @@ \subsection{Model Specification}
\end{equation}
Because $P$ is quite sparse, in practice we do not instantiate the full matrix when performing calculations.
At this point, it is also useful to note that $P$ could, in principle, encode an instrumental line spread function (\LSF).
We return to this point in Section \label{s:future}.
We return to this point in Section \ref{s:future}.

The apparent stellar \RV, $v_{obs,n}$, is a \replaced{sum}{combination} of the star's actual velocity \replaced{about its center of mass}{in the barycentric reference frame} $v_{\star,n}$ and the projected motion of the Earth about the Solar System barycenter (Barycentric Earth Radial Velocity or \BERV), the latter of which is known.
\added{For the purposes of this work, we use the \BERV furnished by the \HARPS pipeline.
Expand Down Expand Up @@ -653,8 +653,9 @@ \subsection{Changes to the Data}

Another way a less well-calibrated instrument may violate \wobble's assumptions is in the non-negligible variability of all spectral lines due to changes in the instrumental line spread function (\LSF).
In principle, if the changes to the \LSF were well-understood they could be hard-coded into linear operators like the $P$ operator used in Equation \ref{eqn:star}, leaving the essential equations of \wobble largely unchanged.
\added{In this case the \LSF operator should be applied to both the stellar and telluric component models after combining them.}
Realistically speaking, if \LSF variations are present they are probably not sufficiently well-understood to take this approach; instead, we might want to fit these variations as a data-driven model component.
\added{In this case the \LSF operator should be applied to both the stellar and telluric component models after combining them. Indeed, technically the current \wobble model does not deal correctly with line broadening: the templates we infer for star and tellurics are post-broadening spectra, but this goes against the correct order of operations if line broadening is dominated by instrumental effects that are applied to the \textit{combined} spectra. We have assumed that this effect is negligible for the purposes of this work, but it should certainly be considered more carefully in the future.}

Realistically speaking, if \LSF variations are present they are probably not sufficiently well-understood to \replaced{take this approach}{take a prescriptive approach to their behavior and hard-code them into the form of the $P$ operator}; instead, we might want to fit these variations \replaced{as a data-driven model component}{using a data-driven approach, which would make the \wobble model significantly non-linear}.
This leads us into the territory of making more fundamental changes to the \wobble model.

\subsection{Changes to the Model}
Expand Down Expand Up @@ -774,7 +775,7 @@ \section{Conclusion}
}

\acknowledgements{\added{We gratefully acknowledge the referee, Ian Czekala, for a thoughtful and constructive review.}
It is a pleasure to acknowledge Jacob L. Bean, John M. Brewer, Heather Knutson, Timothy Morton, Melissa Ness, Andreas Seifahrt, and Julian St\"{u}rmer for helpful discussions.
It is a pleasure to acknowledge Jacob L. Bean, John M. Brewer, Heather Knutson, Timothy Morton, Melissa Ness, Andreas Seifahrt, Julian St\"{u}rmer, \added{and Sharon Xuesong Wang} for helpful discussions.
We are grateful to Hans-Walter Rix and the Max-Planck-Institut f\"ur Astronomie for their hospitality while developing key parts of this project. Special thanks go to Geert Barentsen, Michael Gully-Santiago, Christina Hedges, Susan Mullaly, and Z\'{e} Vin\'{i}cius for their vital contribution to the most important part of \wobble's documentation (the pronunciation guide).

This research has made use of the services of the ESO Science Archive Facility.
Expand Down
23 changes: 23 additions & 0 deletions scripts/missing_wavelength_files.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,23 @@
import numpy as np
from astropy.io import fits
import os
import glob

def missing_wavelength_files(filelist):
missing_files = []
for f in filelist:
path = f[0:str.rfind(f,'/')+1]
sp = fits.open(f)
header = sp[0].header
wave_file = header['HIERARCH ESO DRS CAL TH FILE']
if os.path.isfile(path+wave_file):
continue
else:
missing_files = np.append(missing_files, wave_file)

return np.unique(missing_files)


if __name__ == '__main__':
filelist = glob.glob('/Users/mbedell/python/wobble/data/51peg/HARPS*ccf_G2_A.fits')
np.savetxt('missing_files.txt', missing_files, fmt='%s')

0 comments on commit a2fb927

Please sign in to comment.