[GitHub] astonzhang commented on a change in pull request #7678: Clean amalgamation ws and skip failing test

2017-08-31 Thread git
astonzhang commented on a change in pull request #7678: Clean amalgamation ws 
and skip failing test
URL: https://github.com/apache/incubator-mxnet/pull/7678#discussion_r136480471
 
 

 ##
 File path: tests/python/unittest/test_loss.py
 ##
 @@ -63,6 +63,9 @@ def get_net(num_hidden):
 
 
 def test_ce_loss():
+# Skipping this test to have nightly build passing.
+# There seems to be an issue and tracked at - issue 7677
+return
 
 Review comment:
   The unittest fix is now available in a separate pr #7693 
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] astonzhang commented on a change in pull request #7678: Clean amalgamation ws and skip failing test

2017-08-30 Thread git
astonzhang commented on a change in pull request #7678: Clean amalgamation ws 
and skip failing test
URL: https://github.com/apache/incubator-mxnet/pull/7678#discussion_r136233692
 
 

 ##
 File path: tests/python/unittest/test_loss.py
 ##
 @@ -63,6 +63,9 @@ def get_net(num_hidden):
 
 
 def test_ce_loss():
+# Skipping this test to have nightly build passing.
+# There seems to be an issue and tracked at - issue 7677
+return
 
 Review comment:
   #7659 proposed some change in the unittest
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services