parzer icon indicating copy to clipboard operation
parzer copied to clipboard

erroring

Open sckott opened this issue 7 years ago • 2 comments

not sure how to error. some methods

  • if method gives back single output numeric/int/etc. then can error straight away
  • when method gives back data.frame: error straight away when found? or throw warnings? or put error messages in an "error" column in the output, and fill with NAs?

sckott avatar Feb 19 '19 00:02 sckott

not sure if this is the appropriate ticket to comment on, but it would be nice if parse_lat/parse_lon did not throw out a warning for detecting NA values in the vector. E.g.

> parzer::parse_lon(c(42.5, NA, 43.6, 56.3, -53, 0))

[1]  42.5   NaN  43.6  56.3 -53.0   0.0
Warning message:
In base::.Call(...) : no digits detected, got: NA

popovs avatar Dec 15 '22 20:12 popovs

Seems to be a good place to me. Thanks! We are in the process of migrating from Rcpp to cpp11 so this might be something directly affected and if not, I agree there is room for improvement so I'll keep it in mind.

AlbanSagouis avatar Dec 16 '22 10:12 AlbanSagouis