min-guk commented on code in PR #2103:
URL: https://github.com/apache/systemds/pull/2103#discussion_r1748069092
##########
src/test/java/org/apache/sysds/test/AutomatedTestBase.java:
##########
@@ -1263,16 +1267,98 @@ protected void runRScript(boolean newWay) {
}
}
- private static String buildOutputStringR(String standardOut, String
standardError){
+ /**
+ * Runs an Python script
+ */
+ protected void runPythonScript() {
Review Comment:
I have written the `runPythonScript()` function by following the structure
of the existing `runRScript()` function to run Python scripts. However, I would
like to ask for clarification on how you would like me to proceed with further
modifications.
Would you prefer that I refactor the code to combine the `runRScript()` and
`runPythonScript()` functions into a single function (e.g.,
`runBaselineScript()`), and also consolidate class variables such as `rCmd`?
--
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
