search

mrtommyb / GP_model_Kepler_data

modeling a Kepler light curve using a Gaussian Process
MIT License
0 stars 0 forks source link

Light curve jitter term #5

Closed dfm closed 10 years ago

dfm commented 10 years ago

In your final code, did you include a jitter term for the light curve? If so, I'll add some discussion to the noise model section.

mrtommyb commented 10 years ago

Yes, there were two jitter terms. One for the light curve and one for the RV model. There is a brief discussion in Section 3.1. However, I think section 3 needs to be reordered. I don’t like the subsections much. I put them in to aid the writing of the draft. The values the jitter terms take are given in the results table. However, the summary statistics we provide aren’t really representative given the non-Gaussian distribution.

I’ve not checked in my chains because the file size is very large. Do you think the paper would benefit from a triangle plot?

On May 2, 2014, at 9:12 AM, Dan Foreman-Mackey notifications@github.com wrote:

In your final code, did you include a jitter term for the light curve? If so, I'll add some discussion to the noise model section.

— Reply to this email directly or view it on GitHub.

dfm commented 10 years ago

Cool. I'll add that in and then let you re-order things? I've been making small changes and pushing frequently.

As for the triangle plot: it might be useful to show it for the "physical" parameters but doesn't seem totally necessary.

mrtommyb commented 10 years ago

Sounds good. Don't push too hard...

mrtommyb commented 10 years ago

Dan, can I close this issue? Have you updated the text you want updated?

dfm commented 10 years ago

Yup. Looking good.