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.

Sign up for GitHub

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

Jump to bottom

Frequency estimation fails with missing data. #582

Open

rneher opened this issue Jun 27, 2020 · 1 comment

Labels

bug priority: low

Member

rneher commented Jun 27, 2020

Current Behavior
when the metadata can't be parsed into numerical dates, frequencies fail as reported here:
nextstrain/ncov#446

We need to make sure that frequency estimation either fails or gracefully handles missing data points.

rneher added the bug label

Member Author

rneher commented Jun 27, 2020

the problem is here:
https://github.com/nextstrain/augur/blob/master/augur/frequencies.py#L97
when numpy attempts to compute a mean of None.
None is returned from here:
https://github.com/nextstrain/augur/blob/master/augur/utils.py#L132

trvrb added the priority: low label

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment