[systemds] branch main updated: [SYSTEMDS-3110] Codegen CNodeBinary bug

[baunsgaard]

This is an automated email from the ASF dual-hosted git repository.

baunsgaard pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/systemds.git


The following commit(s) were added to refs/heads/main by this push:
     new c97a3a396b [SYSTEMDS-3110] Codegen CNodeBinary bug
c97a3a396b is described below

commit c97a3a396bd8e81de828187873bbeb28e1bdd99c
Author: baunsgaard <baunsga...@tu-berlin.de>
AuthorDate: Tue May 30 11:47:54 2023 +0200

    [SYSTEMDS-3110] Codegen CNodeBinary bug
    
    Just to verify that there is not a bug anymore,
    and to resolve the bug report.
---
 .../test/functions/federated/codegen/FederatedMultiAggTmplTest.java | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git 
a/src/test/java/org/apache/sysds/test/functions/federated/codegen/FederatedMultiAggTmplTest.java
 
b/src/test/java/org/apache/sysds/test/functions/federated/codegen/FederatedMultiAggTmplTest.java
index 4526c6120b..33a551d28d 100644
--- 
a/src/test/java/org/apache/sysds/test/functions/federated/codegen/FederatedMultiAggTmplTest.java
+++ 
b/src/test/java/org/apache/sysds/test/functions/federated/codegen/FederatedMultiAggTmplTest.java
@@ -117,6 +117,8 @@ public class FederatedMultiAggTmplTest extends 
AutomatedTestBase
        
        private void testFederatedCodegenMultiAgg(ExecMode exec_mode) {
                // store the previous platform config to restore it after the 
test
+               
+               setOutputBuffering(true);
                ExecMode platform_old = setExecMode(exec_mode);
 
                getAndLoadTestConfiguration(TEST_NAME);
@@ -153,7 +155,7 @@ public class FederatedMultiAggTmplTest extends 
AutomatedTestBase
                        "in_rp=" + 
Boolean.toString(row_partitioned).toUpperCase(),
                        "in_test_num=" + Integer.toString(test_num),
                        "out_Z=" + expected(OUTPUT_NAME)};
-               runTest(true, false, null, -1);
+               runTest(null);
 
                // Run actual dml script with federated matrix
                fullDMLScriptName = HOME + TEST_NAME + ".dml";
@@ -164,7 +166,7 @@ public class FederatedMultiAggTmplTest extends 
AutomatedTestBase
                        "in_test_num=" + Integer.toString(test_num),
                        "rows=" + rows, "cols=" + cols,
                        "out_Z=" + output(OUTPUT_NAME)};
-               runTest(true, false, null, -1);
+               runTest(null);
 
                // compare the results via files
                HashMap<CellIndex, Double> refResults  = 
readDMLMatrixFromExpectedDir(OUTPUT_NAME);