e-strauss commented on code in PR #2103:
URL: https://github.com/apache/systemds/pull/2103#discussion_r1751681352


##########
docker/testsysds.Dockerfile:
##########
@@ -78,6 +78,13 @@ RUN apt-get install -y --no-install-recommends \
        && rm -rf installDependencies.R \
        && rm -rf /var/lib/apt/lists/*
 
+RUN apt-get install -y --no-install-recommends \
+       python3 python3-pip && \
+       apt-get clean && \
+       python3 -m pip install --upgrade pip && \
+       pip3 install numpy scipy && \
+    echo "Python3 installed"

Review Comment:
   jep, in the Python test you can compare to numpy, you can take a look at the 
test in #2085 . (#2097 would be an example for an operator with two matrices as 
input, but since you got one matrix as input #2085 fits better), 
   so you can extend the 
https://github.com/apache/systemds/blob/main/src/main/python/systemds/operator/nodes/matrix.py
 with your operator.
   You can also take a look at the quantile operator there on how to pass 
scalars to your function call (for your shift paramerter)



-- 
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: dev-unsubscr...@systemds.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to