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