Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for radial velocity data #2

Closed
sefffal opened this issue Feb 24, 2022 · 2 comments
Closed

Add support for radial velocity data #2

sefffal opened this issue Feb 24, 2022 · 2 comments
Assignees

Comments

@sefffal
Copy link
Owner

sefffal commented Feb 24, 2022

We would like to be able to support radial velocity data with the following requirements:

  • Multiple instruments
  • Instrument specific jitter
  • Plotting support in the analysis file (timeplot and timeplotgrid)

We can implement this by creating a new subtype of AbstractObs (types.jl) and likelihood functions (models.jl)
Users can add an rv parameter to the system to model the unperturbed radial velocity.

We would like to be able to add rv data to both the system as a whole and to planets, so we will have to write two separate likelihood functions (they can share code though).

A nice bonus feature could be a helper function to read in the GAIA RV from the HGCA catalog that is already downloaded.

@sefffal
Copy link
Owner Author

sefffal commented Mar 3, 2022

Support for single instrument RV data now implemented.
Still todo: multiple instruments, docs, and recommendations for parameterization e.g. i=0, \Omega=0.

@sefffal
Copy link
Owner Author

sefffal commented Apr 13, 2022

I think we can say this is done now, and we can punt on multiple instruments for the time being.

@sefffal sefffal closed this as completed Apr 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants