DavidSandruTU commented on a change in pull request #1481:
URL: https://github.com/apache/systemds/pull/1481#discussion_r774806125



##########
File path: scripts/perftest/genALSData.sh
##########
@@ -21,30 +21,31 @@
 #-------------------------------------------------------------
 
 CMD=$1
-DATADIR=$2
+DATADIR=$2/als
 
-FORMAT="binary" # can be csv, mm, text, binary
+FORMAT="text" # can be csv, mm, text, binary
 DENSE_SP=0.9
 SPARSE_SP=0.01
 
-BASEPATH=$(dirname $0)
+EXTRADOT=""
+if [ "${CMD}" == "systemds" ]; then EXTRADOT="." ; fi
 
 #generate XS scenarios (80MB)
-${CMD} -f ${BASEPATH}/../datagen/genRandData4ALS.dml --nvargs 
X=${DATADIR}/X10k_1k_dense rows=10000 cols=1000 rank=10 nnz=`echo "scale=0; 
10000 * 1000 * $DENSE_SP" | bc` sigma=0.01 fmt=$FORMAT
-${CMD} -f ${BASEPATH}/../datagen/genRandData4ALS.dml --nvargs 
X=${DATADIR}/X10k_1k_sparse rows=10000 cols=1000 rank=10 nnz=`echo "scale=0; 
10000 * 1000 * $SPARSE_SP" | bc` sigma=0.01 fmt=$FORMAT
+${CMD} -f ${EXTRADOT}./datagen/genRandData4ALS.dml --nvargs 
X=${DATADIR}/X10k_1k_dense rows=10000 cols=1000 rank=10 nnz=`echo "scale=0; 
10000 * 1000 * $DENSE_SP" | bc` sigma=0.01 fmt=$FORMAT

Review comment:
       Do you mean one folder further in, in the fed folder?
   I've changed that script to be used by the general ALS benchmarks, as it was 
previously used by the federated ALS benchmark. The federated benchmarks now 
have a new genALS_FedData.sh in the fed folder.




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