apaam/mlpack
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix ReadMatrix() (it was trying to write to places that don't exist). I think the implementation in fastlib-armadillo is less bizarre and will end up being used here (arma::vec lends itself better to this task than std::vector). Need to merge those branches...

Browse Source
This commit is contained in:
Ryan Curtin
2010-06-16 22:23:02 +00:00
parent 06c40e5bf0
commit b4dbd8b2fd
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
fastlib/branches/fastlib-stl/fastlib/data/dataset.cc
+4 -4
View File
@@ -424,12 +424,12 @@ success_t DatasetInfo::ReadMatrix(TextLineReader *reader, Matrix *matrix) const
index_t n_points = 0;
success_t retval = SUCCESS_PASS;
bool is_done;
double *tmp_point = new double[n_features];
do {
linearized.push_back(n_features);
double *point = &linearized.back();
retval = ReadPoint(reader, point, &is_done);
retval = ReadPoint(reader, tmp_point, &is_done);
for(int i = 0; i < n_features; i++)
linearized.push_back(tmp_point[i]);
n_points++;
} while (!is_done && !FAILED(retval));