[mlpack-git] [mlpack/mlpack] Optimize load csv 00 (#681)

[Tham]

>does the fast CSV parser outperform boost::spirit?

Will give it a test later on, it is not an easy task to give a fair
test.Because fast csv use their own buffer to store the data, instead of
reading the line one by one, fast csv read a lot of lines at once, this
reduce the overhead of IO. boost spirit can do the same thing too, need
some adjustment. Besides, boost spirit support semantic actions, fast csv
do not support.

>now we have to be sure to keep it up to date, but maybe git submodules or
something can be used here

I think this cannot work, because the codes need some changes, unless the
author agree with the change(I can ask him). However, the codes are not
very complicated and well written, it is not hard to maintain it(of course,
not as easy as boost::spirit).


---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/mlpack/mlpack/pull/681#issuecomment-234695403
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20160722/91c430d8/attachment.html>