Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge rs_model.cpp into rs_model_impl.hpp #1082

Merged
merged 3 commits into from Aug 25, 2017
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
Expand Up @@ -38,9 +38,9 @@ AugLagrangianFunction<LagrangianFunction>::AugLagrangianFunction(
LagrangianFunction& function,
const arma::vec& lambda,
const double sigma) :
function(function),
lambda(lambda),
sigma(sigma),
function(function)
sigma(sigma)
{
// Nothing else to do.
}
Expand Down
Expand Up @@ -38,8 +38,6 @@ NaiveBayesClassifier<ModelMatType>::NaiveBayesClassifier(
"NaiveBayesClassifier: element type of given data must match the element "
"type of the model!");

const size_t dimensionality = data.n_rows;

// Perform training, after initializing the model to 0 (that is, if Train()
// won't do that for us, which it won't if we're using the incremental
// algorithm).
Expand Down
1 change: 0 additions & 1 deletion src/mlpack/methods/range_search/CMakeLists.txt
Expand Up @@ -8,7 +8,6 @@ set(SOURCES
range_search_stat.hpp
rs_model.hpp
rs_model_impl.hpp
rs_model.cpp
)

# Add directory name to sources.
Expand Down
274 changes: 0 additions & 274 deletions src/mlpack/methods/range_search/rs_model.cpp

This file was deleted.