mlr-org / farff

a faster arff parser
Other
11 stars 6 forks source link

throw error for multi-instance data #21

Closed giuseppec closed 8 years ago

giuseppec commented 8 years ago

This refers to https://github.com/openml/OpenML/issues/230 and #20. PR throws an error if data type is "relational" and includes a unit test for the error.

berndbischl commented 8 years ago

Looks good. I will merge now.

It is now not that important for farff, but please try to squash the commits into 1. I also think in these cases it is good to doc in the function doc that we dont support this. Always try to do this. I will add this now.