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

Palladio recognizes dates in the YYYY format as numbers #142

Open
elizawe opened this issue Feb 9, 2016 · 3 comments
Open

Palladio recognizes dates in the YYYY format as numbers #142

elizawe opened this issue Feb 9, 2016 · 3 comments
Assignees

Comments

@elizawe
Copy link
Contributor

elizawe commented Feb 9, 2016

No description provided.

@esjewett
Copy link
Member

I'm a little torn about what to do with this one. We can prioritize recognition of 4-digit numbers as dates over recognition as numbers. Is that preferred?

@cncoleman
Copy link
Member

That is tricky. I'm going to say that, at this point, since we are primarily building for historical data, we should assume dates. If we could run some checks to determine likely dates, that might help:

  1. If the first digit changes by more than one increment, treat it as number not date.
    or if
  2. If the second digit changes by more than three increments, treat it as number not date.

@esjewett
Copy link
Member

This is probably something I can implement.

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

No branches or pull requests

3 participants