[GitHub] [systemml] Shafaq-Siddiqi commented on a change in pull request #972: [MINOR] MICE now accepts a recoded matrix as input

[GitBox]

Shafaq-Siddiqi commented on a change in pull request #972:
URL: https://github.com/apache/systemml/pull/972#discussion_r438096170



##########
File path: 
src/test/java/org/apache/sysds/test/functions/builtin/BuiltinMiceTest.java
##########
@@ -50,33 +50,47 @@ public void testMiceMixCP() {
                runMiceNominalTest(mask, 1, LopProperties.ExecType.CP);
        }
 
+//     @Test

Review comment:
       The codegen tests in PR are failing here while working fine locally. The 
trace is as follows, any help/hint is appreciated.
   
   [ERROR] Crashed tests:
   [ERROR] org.apache.sysds.test.functions.codegen.RowAggTmplTest
   [ERROR]      at 
org.apache.maven.plugin.surefire.booterclient.ForkStarter.fork(ForkStarter.java:690)
   [ERROR]      at 
org.apache.maven.plugin.surefire.booterclient.ForkStarter.access$600(ForkStarter.java:118)
   [ERROR]      at 
org.apache.maven.plugin.surefire.booterclient.ForkStarter$2.call(ForkStarter.java:447)
   [ERROR]      at 
org.apache.maven.plugin.surefire.booterclient.ForkStarter$2.call(ForkStarter.java:423)
   [ERROR]      at java.util.concurrent.FutureTask.run(FutureTask.java:266)
   [ERROR]      at 
java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
   [ERROR]      at 
java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
   




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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