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

make claims multiple declaration of LRSDP #312

Closed
rcurtin opened this Issue Dec 29, 2014 · 1 comment

Comments

Projects
None yet
1 participant
@rcurtin
Member

rcurtin commented Dec 29, 2014

Reported by birm on 23 Aug 44106208 12:20 UTC

== Input ==


arma::mat c(40, 40);
c.randn();
const size_t b=3;
mlpack::optimization::LRSDP d(b,c);
== Output: ==


CMakeFiles/mlpack_test.dir/to_string_test.cpp.o: In function mlpack::optimization::AugLagrangianFunction<mlpack::optimization::LRSDP>::Evaluate(arma::Mat<double> const&) const': /scratch/strings/trunk/src/mlpack/../mlpack/core/optimizers/lrsdp/lrsdp_impl.hpp:22: multiple definition ofmlpack::optimization::AugLagrangianFunctionmlpack::optimization::LRSDP::Evaluate(arma::Mat const&) const'
CMakeFiles/mlpack_test.dir/lrsdp_test.cpp.o:/scratch/strings/trunk/src/mlpack/../mlpack/core/optimizers/lrsdp/lrsdp_impl.hpp:22: first defined here
CMakeFiles/mlpack_test.dir/to_string_test.cpp.o: In function mlpack::optimization::AugLagrangianFunction<mlpack::optimization::LRSDP>::Gradient(arma::Mat<double> const&, arma::Mat<double>&) const': /scratch/strings/trunk/src/mlpack/../mlpack/core/optimizers/lrsdp/lrsdp_impl.hpp:62: multiple definition ofmlpack::optimization::AugLagrangianFunctionmlpack::optimization::LRSDP::Gradient(arma::Mat const&, arma::Mat&) const'
CMakeFiles/mlpack_test.dir/lrsdp_test.cpp.o:/scratch/strings/trunk/src/mlpack/../mlpack/core/optimizers/lrsdp/lrsdp_impl.hpp:62: first defined here
collect2: ld returned 1 exit status

@rcurtin rcurtin self-assigned this Dec 29, 2014

@rcurtin rcurtin added this to the mlpack 1.0.9 milestone Dec 29, 2014

@rcurtin rcurtin closed this Dec 29, 2014

@rcurtin

This comment has been minimized.

Show comment
Hide comment
@rcurtin

rcurtin Dec 30, 2014

Member

Commented by rcurtin on 3 Nov 44114064 20:40 UTC
Fixed in r16261 with what I realized was probably the most worthless commit message ever ("Move things around so LRSDP can actually be included in stuff."). Anyway, it works now...

Member

rcurtin commented Dec 30, 2014

Commented by rcurtin on 3 Nov 44114064 20:40 UTC
Fixed in r16261 with what I realized was probably the most worthless commit message ever ("Move things around so LRSDP can actually be included in stuff."). Anyway, it works now...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment