Feature request: allow multiple values for na

[tidyverse]

Feature request: allow multiple values for na

[jakesherman]

Seconded.

[sunitj]

👍 Came here to make the same request. Glad I'm not the only one.

[mattsigal]

I would also love to see this implemented!

[jennybc]

👍

[MichaelChirico]

Also would like this. Can't imagine it's too hard to implement either... any particular hold-up? I'd be happy to help if I was more fluent in C++...

[MichaelChirico]

Seems like the crux of the issue is in the XlsxCell class/methods definition, where an NA is produced with the condition (in the type, asDate, asDouble, and asCharSxp methods):

[MichaelChirico]

The version I've got on my branch is now functional (see Pull Request).

If you're impatient you can use the version on my branch (via install_github("MichaelChirico/readxl@multiple_na")). I'm not sure the procedure for documentation / tests around here, so it may need a bit of polishing before it's officially merged.

[yimingli]

+1