barentsen / notebooks

Curated Notebooks from STScI
BSD 3-Clause "New" or "Revised" License
1 stars 0 forks source link

Tutorial: How to combine multiple Quarters of Kepler observations #7

Closed ojhall94 closed 4 years ago

ojhall94 commented 4 years ago

This notebook will form part of the series: Getting started with Kepler and K2 data products.

In this notebook, we will focus on combining multiple quarters of data into a single LightCurve object. This will require previous knowledge of LightCurve and LightCurveFile objects from #3.

We can draw inspiration from existing Lightkurve tutorials:


Learning Goals:

ojhall94 commented 4 years ago

I've put together a draft of this notebook, ready for comments. I'm certain there's lots of great changes we can make to it!

barentsen commented 4 years ago

Thanks @ojhall94. This is great!

My only real comment is that I would provide slightly more details in a few places:

Does that make sense?

ojhall94 commented 4 years ago

Great suggestions, agreed on all counts! I'll get to work on them tomorrow :)

ojhall94 commented 4 years ago

Thanks for all the feedback! Worked them into the tutorial, I think its a lot more fleshed out now. Let me know what you think!

barentsen commented 4 years ago

Thanks @ojhall94! This is going in the right direction!

This tutorial will need a few changes to the code if we end up turning LightCurve into a sub-class of TimeSeries, so we'll want to revisit after a decision is made on https://github.com/KeplerGO/lightkurve/pull/744!

ojhall94 commented 4 years ago

Good idea. In the meantime, I have a comment from Rebekah Hounsell that I think are worth discussing in a bit more detail here:

About Section 1. What is a Kepler Quarter:

Do you want a whole text section discussing the Kepler quarters or maybe a link to a separate page? Too much text could be overwhelming rather than informative.

astrobel commented 4 years ago

This a great tutorial! Everything is really cohesive, and I think the explanation of why quarters occur is useful in this case, especially since you call back to it later in the text. One thing I think could be clearer in this is that the corrections you're doing before stitching (e.g. .normalize()) are quite basic since this is already PDCSAP flux, and that for stitching SAP flux, there are more steps to be taken.

ojhall94 commented 4 years ago

Thanks for your comments here & elsewhere @astrobel!

barentsen commented 4 years ago

I carried out a final review and believe this notebook is ready for copy editing. :+1:

lglattly commented 4 years ago

@ojhall94 and @barentsen I finished copy editing this tutorial "How to Combine Multiple Quarters of Kepler Data with Lightkurve" and it was in great shape, very nice clear writing!

I made some minor copy edits throughout and also combined the summary and companion content text under the Introduction section, as per our finalized tutorial template. The only thing I didn't do was touch the Additional Resources section, which we decided on our last call would be removed and additional resources would be incorporated in the main content in context (which I think is mostly happening in this tutorial with the exception of a few resources listed at the end).

So if someone can take a look at that Additional Resources section and incorporate the list of links, that would be great, otherwise this tutorial is all set on my end and ready to go!

barentsen commented 4 years ago

I reviewed that Additional Resources section. I believe that all the materials that were not already linked are not immediately necessary for the tutorial (e.g. MAST TESS Webpage). Such generic links can better be placed and kept up to date in the environment the notebooks will be integrated into. As a result, I went ahead and removed the Additional Resources section.

Thanks everyone!

barentsen commented 4 years ago

This notebook has successfully been merged into spacetelescope/notebooks. Thanks everyone! Closing the issue...