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

TDAR calculation #2

Closed
allopole opened this issue Apr 25, 2019 · 4 comments
Closed

TDAR calculation #2

allopole opened this issue Apr 25, 2019 · 4 comments

Comments

@allopole
Copy link

confirmed tdar output of get_stats() plot is linear:

image

@e3bo
Copy link
Owner

e3bo commented Apr 25, 2019 via email

@allopole
Copy link
Author

Spencer fixed a bug in the TDAR calculation inside get_stats() in his fork of spaero. See:

SpencerDH@477d42c

@e3bo
Copy link
Owner

e3bo commented Apr 30, 2019

I have pushed this fix to the tdar branch of e3bo/spaero. I want to make sure we follow up on the tau issue. Can you reproduce the negative tau with a buggy tdar output like above?

@allopole
Copy link
Author

We reproduced it. Problem was in how ktseq() function in analysis.R handled missing values. Eamon replaced with R corr function that handles missing values properly.

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