-
Notifications
You must be signed in to change notification settings - Fork 11
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
Proposal for common data format #12
Comments
Is there a way to take ASCII option, plus an auxiliary function or perl script to convert? Dan Foreman-Mackey [email protected] wrote:
On the go from my phone... |
ASCII! 👍 |
Yeah. That's a good idea! Sounds like ASCII + auxiliary script is a good idea. What columns do we need? I mentioned |
What about an integer flag, where 0 = this point was used for calculating On Wed, Jun 12, 2013 at 12:25 PM, Dan Foreman-Mackey <
|
Do we have a list of sandbox KICs somewhere on git? If so, I can run Cheers, |
Yes, they are in the detrending/documents directory - a bunch of text files containing all the KICs in the skygroups, all the TCEs identified by the pipeline in Q1-Q12 in the skygroups, three quiet 12th magnitude G stars, three bright variable stars, and a couple of other poster children (Kepler-37 eg). |
Have we decided what will be our test suite will be to compare algorithms On Thursday, June 13, 2013, jessielchristiansen wrote:
|
Depends how your detrending algorithms work. If they need the ensemble of stars to identify common modes, then I would use the whole set of KICs in each skygroup. Otherwise hit up the variable stars! |
We need a way of comparing the output of all of the detrending algorithms. This will (probably) involve something including but not limited to:
Anything that we do will benefit a common output format for the codes (for obvious reasons).
I see 2 main options:
The first option is far easier to implement in any programming language (lowering the barrier to entry) so I'm probably inclined to go with that but the second one seems more useful (and self-contained) for the search phase depending on what we decide to do.
Thoughts?
The text was updated successfully, but these errors were encountered: