[mlpack-git] master: Remove unnecessary comments from tests. (56b53a0)
gitdub at mlpack.org
gitdub at mlpack.org
Mon Apr 18 13:46:46 EDT 2016
Repository : https://github.com/mlpack/mlpack
On branch : master
Link : https://github.com/mlpack/mlpack/compare/7e0e6ea1e3a7d242305a5a6a6e9730c5c3bfdc98...56b53a09e2d46b65f4d80560964487f1e193d345
>---------------------------------------------------------------
commit 56b53a09e2d46b65f4d80560964487f1e193d345
Author: Ryan Curtin <ryan at ratml.org>
Date: Mon Apr 18 10:46:46 2016 -0700
Remove unnecessary comments from tests.
>---------------------------------------------------------------
56b53a09e2d46b65f4d80560964487f1e193d345
src/mlpack/tests/activation_functions_test.cpp | 4 ----
src/mlpack/tests/convolution_test.cpp | 4 ----
src/mlpack/tests/init_rules_test.cpp | 6 +-----
src/mlpack/tests/layer_traits_test.cpp | 4 ----
src/mlpack/tests/recurrent_network_test.cpp | 4 ----
5 files changed, 1 insertion(+), 21 deletions(-)
diff --git a/src/mlpack/tests/activation_functions_test.cpp b/src/mlpack/tests/activation_functions_test.cpp
index 90e653d..34ec009 100644
--- a/src/mlpack/tests/activation_functions_test.cpp
+++ b/src/mlpack/tests/activation_functions_test.cpp
@@ -32,10 +32,6 @@ using namespace mlpack::ann;
BOOST_AUTO_TEST_SUITE(ActivationFunctionsTest);
-// Be careful! When writing new tests, always get the boolean value and store
-// it in a temporary, because the Boost unit test macros do weird things and
-// will cause bizarre problems.
-
// Generate dataset for activation function tests.
const arma::colvec activationData("-2 3.2 4.5 -100.2 1 -1 2 0");
diff --git a/src/mlpack/tests/convolution_test.cpp b/src/mlpack/tests/convolution_test.cpp
index f7d012d..368f32d 100644
--- a/src/mlpack/tests/convolution_test.cpp
+++ b/src/mlpack/tests/convolution_test.cpp
@@ -20,10 +20,6 @@ using namespace mlpack::ann;
BOOST_AUTO_TEST_SUITE(ConvolutionTest);
-// Be careful! When writing new tests, always get the boolean value and store
-// it in a temporary, because the Boost unit test macros do weird things and
-// will cause bizarre problems.
-
/*
* Implementation of the convolution function test.
*
diff --git a/src/mlpack/tests/init_rules_test.cpp b/src/mlpack/tests/init_rules_test.cpp
index 451ee8a..3d09c26 100644
--- a/src/mlpack/tests/init_rules_test.cpp
+++ b/src/mlpack/tests/init_rules_test.cpp
@@ -2,7 +2,7 @@
* @file init_rules_test.cpp
* @author Marcus Edel
*
- * Tests for the various weight initialize methods.
+ * Tests for the various weight initialize methods.
*/
#include <mlpack/core.hpp>
@@ -21,10 +21,6 @@ using namespace mlpack::ann;
BOOST_AUTO_TEST_SUITE(InitRulesTest);
-// Be careful! When writing new tests, always get the boolean value and store
-// it in a temporary, because the Boost unit test macros do weird things and
-// will cause bizarre problems.
-
// Test the RandomInitialization class with a constant value.
BOOST_AUTO_TEST_CASE(ConstantInitTest)
{
diff --git a/src/mlpack/tests/layer_traits_test.cpp b/src/mlpack/tests/layer_traits_test.cpp
index 8096958..0a0e3cc 100644
--- a/src/mlpack/tests/layer_traits_test.cpp
+++ b/src/mlpack/tests/layer_traits_test.cpp
@@ -20,10 +20,6 @@ using namespace mlpack::ann;
BOOST_AUTO_TEST_SUITE(LayerTraitsTest);
-// Be careful! When writing new tests, always get the boolean value and store
-// it in a temporary, because the Boost unit test macros do weird things and
-// will cause bizarre problems.
-
// Test the defaults.
BOOST_AUTO_TEST_CASE(DefaultsTraitsTest)
{
diff --git a/src/mlpack/tests/recurrent_network_test.cpp b/src/mlpack/tests/recurrent_network_test.cpp
index 2fedd69..355aa81 100644
--- a/src/mlpack/tests/recurrent_network_test.cpp
+++ b/src/mlpack/tests/recurrent_network_test.cpp
@@ -28,10 +28,6 @@ using namespace mlpack::optimization;
BOOST_AUTO_TEST_SUITE(RecurrentNetworkTest);
-// Be careful! When writing new tests, always get the boolean value and store
-// it in a temporary, because the Boost unit test macros do weird things and
-// will cause bizarre problems.
-
/**
* Construct a 2-class dataset out of noisy sines.
*
More information about the mlpack-git
mailing list