[mlpack-git] [mlpack] LoadSave Unit test fails Windows x64 (#468)

Joseph Mariadassou notifications at github.com
Wed Nov 11 04:41:00 EST 2015


podarray<unsigned char> data(N);

data[N-1] could be anything. We could initialise it to char(20)  and this
problem can be fixed. As I understood, the problem is that it could come up
in other cases where the file is not opened as binary. MLPack appears to be
doing the right thing when it comes to reading a binary file. When a file
is opened as a text file Armadillo is treating it like a binary file. This
is where the problem arises.

Conrad S:






*I looked at the documentation for C++, and I don't think it's possible to
detect how a stream was opened. I'll update Armadillo's documentation to
state that the streams must be opened in binary. *

If you want me to run any test on Windows do let me know.
Joe


---
Reply to this email directly or view it on GitHub:
https://github.com/mlpack/mlpack/issues/468#issuecomment-155717037
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20151111/a64b6024/attachment.html>


More information about the mlpack-git mailing list