[mlpack-git] [mlpack/mlpack] Spill trees (#747)
MarcosPividori
notifications at github.com
Wed Aug 17 14:07:03 EDT 2016
> @@ -209,12 +245,6 @@ int main(int argc, char *argv[])
> Log::Info << "Loaded kNN model from '" << inputModelFile << "' (trained on "
> << knn.Dataset().n_rows << "x" << knn.Dataset().n_cols << " dataset)."
> << endl;
> -
> - // Adjust singleMode and naive if necessary.
> - knn.SingleMode() = CLI::HasParam("single_mode");
> - knn.Naive() = CLI::HasParam("naive");
> - knn.LeafSize() = size_t(lsInt);
> - knn.Epsilon() = epsilon;
@rcurtin
I removed all that options because they are incorrect:
+ For example if we save a model in `SingleMode`, and we reload that model with no extra option, it will set:
```
knn.SingleMode() = CLI::HasParam("single_mode") ; // false
knn.Naive() = CLI::HasParam("naive") ; // false
```
It will do `Dual Tree Search` instead of `Single Tree Search`.
+ Also, if we set the `LeafSize()` to a different value, we need to call `BuildModel` again, to rebuild the reference tree.
+ Also, in the parameter specifications, it is mentioned that "--leaf_size" "--naive" and "--single_mode" would be ignored when providing an input model file. So, I wanted to be consistent with that comments.
So, it is a bit difficult to properly update a given input model.
For example, if users provide an input model file, without the options "--naive" and "--single_mode", does it means they don't want to modify the model or that they want to update it to do "dual tree search"?
--
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/747/files/fe090ee13c7cad79e2b7eb8b6690628ba3ead1ed#r75175168
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20160817/a7565ffc/attachment-0001.html>
More information about the mlpack-git
mailing list