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

Create some common sense/"sanity" checks for datetime ingest #23

Open
jamesrco opened this issue Jun 14, 2018 · 2 comments
Open

Create some common sense/"sanity" checks for datetime ingest #23

jamesrco opened this issue Jun 14, 2018 · 2 comments

Comments

@jamesrco
Copy link
Member

These basic checks could include:

  1. Alert for dates apparently far in the past that might have been mis-recognized when using %Y in tryFormats
  2. Alert for dates in the future (same issue with tryFormats)
    *** could use the Sys.time() as the refernce point

Others?

@jamesrco
Copy link
Member Author

jamesrco commented Jun 14, 2018

Both of these would only trigger a warning() (not a stop()) if the test fails; obviously, there are many instances where old/future dates are valid

@jamesrco
Copy link
Member Author

Could set thresholds at < 1980 or > 24 hr from Sys.time

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

1 participant