search

mmaul / clml

Common Lisp Machine Learning Library
Other
259 stars 36 forks source link

Arff reader #36

Closed neil-lindquist closed 6 years ago

neil-lindquist commented 6 years ago

I've finally gotten a reader created for arff file format, as talked about in issue #32. It was added as an option to clml.hjs.read-data:read-data-from-file and clml.hjs.read-data:read-data-from-stream (which also got some cleaning up) and clml.utility.arff:read-arff-stream is the function to read arff files specifically.

This is developed off the code in pull request #34, so that I could make sure I wasn't causing new tests to fail. So, accepting this pull request adds that code and has the same notes about causing travis to indicate failure due to failing tests being properly tested.

mmaul commented 6 years ago

Again very nice, it is doing the right thing. Not just need to fix test nmf for CCL build.