[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