-
Notifications
You must be signed in to change notification settings - Fork 5
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
check for entirely missing columns in guess_dates #109
base: master
Are you sure you want to change the base?
Conversation
@aspina7, can you check this with |
so it works for a character NA variable (returns only NAs). But i think the default for an entirely NA var is to be logical.... |
Okay, I've set it so that it will return the unmodified NA column before it panics about vector types. Try it now. |
works a charm - maybe just specify in the warning which variable? |
Again, difficult to do because the context of this function is on individual vectors. It doesn't know that these vectors come from a data frame. |
ah true true sorry! |
This will return the column unchanged if encountered and will fix #108