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

Augur filter does not support the integer.float input date format for --min/max-date filtering #748

Open
corneliusroemer opened this issue Jul 12, 2021 · 1 comment · May be fixed by #938
Labels
bug Something isn't working

Comments

@corneliusroemer
Copy link
Member

Current Behavior
Despite the documentation mentioning augur style integer dates (e.g. 2019.2413) as being accepted as a parameter of --min/max-date, this format is not supported as input data format to augur filter. When trying to filter a dataset that uses index dates as dates, an error such as the following is thrown:

WARNING: MT343860 has an invalid data string: 2020.1338797814208

Expected behavior
I would expect integer dates to be accepted as input format, because integer dates are accepted as parameters.

Related to this discussion: #662

@huddlej
Copy link
Contributor

huddlej commented Aug 19, 2021

@corneliusroemer, do you have an example dataset that uses these floating point dates in the metadata?

@rneher pointed out that the use case for these dates is for analyses of ancient DNA/RNA sequencing where dates like -11220-04-12 will break the standard Python datetime library.

I'm curious if we didn't catch this issue before because no one has tried to use Augur for these types of analyses or they did and gave up when it immediately failed. If this type of input is actually more of an edge case (relative to the primary use cases), we might consider this as a feature request instead of a bug.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
2 participants