ywcb00 commented on code in PR #2525:
URL: https://github.com/apache/systemds/pull/2525#discussion_r3503957799


##########
src/main/java/org/apache/sysds/hops/rewrite/RewriteMatrixMultChainOptimizationSparse.java:
##########
@@ -119,40 +112,30 @@ private static int[][] mmChainDPSparse(double[] dimArray, 
MMNode[] sketchArray,
                                }
 
                                if( LOG.isTraceEnabled() ){
-                                       LOG.trace("mmchainopt 
[i="+(i+1)+",j="+(j+1)+"]: costs = "+dpMatrix[i][j]+", split = 
"+(split[i][j]+1));
+                                       LOG.trace("mmchainoptsparse 
[i="+(i+1)+",j="+(j+1)+"]: costs = "+dpMatrix[i][j]+", split = 
"+(split[i][j]+1));
+                                       System.out.println("mmchainoptsparse 
[i="+(i+1)+",j="+(j+1)+"]: costs = "+dpMatrix[i][j]+", split = 
"+(split[i][j]+1));

Review Comment:
   @Baunsgaard do you have any idea of how to validate this in the 
corresponding test without printing it to System.out?
   Best would be to capture the `LOG.trace` one line before. I already enabled 
the trace log in the test class, do you know a possibility to capture it then?
   Help is very much appreciated. Thank you in advance. :)



-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to