oliver-wyman-actuarial/easyr

atype date conversion issue

Closed this issue · 2 comments

Periodically easyr::read.any will produce an error when auto_convert_dates is set to TRUE (the default handling of read.any). In the atype function, it seems the test.conversion object might get detected as a date in a non-date, string columns, i.e. a comments column where the start of a string includes a date, for example '1/1/2021: here is a comment'.

image

I propose implementing a trycatch and defaulting back to a string if it fails to convert a column to date.

@willdebras I believe Justin found and fixed this. See #39.

Can you install from github to see if that fixes it?

@bryce-chamberlain awesome! Does look like this fixes. Didn't realize Justin had fixed this issue already!