error in common na_values definition

[glaucouri]

Note for 0.14.1: need to add back #N/A N/A to na_values defaults

in pandas 0.12

i found a little bug:

this:
_NA_VALUES = set(['-1.#IND', '1.#QNAN', '1.#IND', '-1.#QNAN',
'#N/A N/A', 'NA', '#NA', 'NULL', 'NaN',
'nan', ''])

must be:
_NA_VALUES = set(['-1.#IND', '1.#QNAN', '1.#IND', '-1.#QNAN',
'#N/A','N/A', 'NA', '#NA', 'NULL', 'NaN',
'nan', ''])

thank you
Glauco

[jreback]

this is not a bug. you can always specify your own na_values if you would like. Or are you advocating N/A be added to the 'common list'?

[jtratner]

No, he's saying that currently one of the values is: '#N/A N/A'
but that should actually be two values '#N/A' and 'N/A' right? in other
words, there's a missing closing single-quote and comma.

[glaucouri]

Exactly . Take a look at the snippet.

Inviato da Aifon

Il giorno 27/nov/2013, alle ore 19:41, Jeff Tratner notifications@github.com ha scritto:

No, he's saying that currently one of the values is: '#N/A N/A'
but that should actually be two values '#N/A' and 'N/A' right? in other
words, there's a missing closing single-quote and comma.
—
Reply to this email directly or view it on GitHub.

Il contenuto di questa e-mail è rivolto unicamente alle persone alle quali è indirizzato e può contenere informazioni la cui riservatezza è tutelata legalmente. Sono vietati la riproduzione, la diffusione e l’uso di questa e-mail in mancanza di autorizzazione del destinatario. Se avete ricevuto questa e-mail per errore vogliate cortesemente contattarci immediatamente.

This e-mail is intended only for the person or entity to which it is addressed and may contain information that is privileged, confidential or otherwise protected from disclosure. Unauthorised reproduction, dissemination or use of this e-mail or of the information herein by anyone other than the intended recipient is prohibited. If you have received this e-mail by mistake, please notify us immediately.

[jtratner]

yeah, it's just easy to miss the point because it's a typo and subtle to
boot :)

[jreback]

#5601 should do it...

[jtratner]

Thanks for catching this!

[djchou]

Actually, it wasn't a typo. Excel returns a '#N/A N/A' value sometimes. Please add this value back to the NA list.

[jreback]

hmm, you are right, that should be their as well. wnat to do a PR for that?

[glaucouri]

Il 26/06/2014 15:37, djchou ha scritto:

Actually, it wasn't a typo. Excel returns a '#N/A N/A' value
sometimes. Please add this value back to the NA list.

did you have a test case?

Gla

[djchou]

Sorry, this is my first time using Github. I'm not sure what you mean by PR.