search

berndbischl / BBmisc

Other
20 stars 7 forks source link

convertListOfRowsToDataFrame converts NULL to NA #68

Closed jakob-r closed 7 years ago

jakob-r commented 8 years ago 
> BBmisc::convertListOfRowsToDataFrame(list(list(a = 1, b = NULL)))
  a  b
1 1 NA

I know, that a NULL in a data.frame cell is not verry common. But we could issue a warning about this conversion.

berndbischl commented 7 years ago

well the docs said that "missing values" are encoded as NA in the result. "missing" here meaning: not in the list.

NULL is basically the same thing. i will just update docs and add a few tests for this. i think this is actually good, consistent behavior.

and warnings suck.

berndbischl commented 7 years ago

did what i said