Allow Excel formulas (=...) to be processed in CSV (MLDB-2170)

mldbai / mldb

MLDB is the Machine Learning Database

http://mldb.ai

Apache License 2.0

663 stars 102 forks source link

Allow Excel formulas (=...) to be processed in CSV (MLDB-2170) #870

Closed jeremybarnes closed 7 years ago

jeremybarnes commented 7 years ago

Allows for the ="123" found in some CSV exports from Excel to be properly parsed (by ignoring it). This was seen in the wild.

mailletf commented 7 years ago

I'm not sure it makes sense to put this in import.text; feels like overfitting a dataset. Did the import fail without this option? Shouldn't this be done in a transform and a builtin function ?

jeremybarnes commented 7 years ago

@mailletf There are lots of datasets out there produced by Excel with this exact format. It's not overfitting a single dataset.

mailletf commented 7 years ago

allrighty +1