thicken() to handle NA values better

EdwinTh / padr

Padding of missing records in time series

https://edwinth.github.io/padr/

Other

132 stars 12 forks source link

thicken() to handle NA values better #47

Closed foundinblank closed 6 years ago

foundinblank commented 6 years ago

I have several date-time columns, and some of them have NA values. Using thicken() breaks when I try it on those columns with NA cells, but works fine on columns with no NA cells.

This is the error I get: Error in if (unique(nchar(x_char)) == 10) { : missing value where TRUE/FALSE needed In addition: Warning message: In if (unique(nchar(x_char)) == 10) { : the condition has length > 1 and only the first element will be used

EdwinTh commented 6 years ago

This is already been fixed for v.0.4.0. You can already install the dev version which is becoming stableish at the moment.

foundinblank commented 6 years ago

Awesome! This is such a great package. Thanks.