kev-inn commented on a change in pull request #874: Add SystemDSContext for systemds operations URL: https://github.com/apache/systemml/pull/874#discussion_r406407145
########## File path: src/main/python/docs/source/simple_examples.rst ########## @@ -62,36 +72,43 @@ Remember to first start up a new terminal:: m2 = np.array(np.random.randint(5, size=5 * 5) + 1, dtype=np.double) m2.shape = (5, 5) - # element-wise matrix multiplication, note that nothing is executed yet! - m_res = Matrix(m1) * Matrix(m2) - # lets do the actual computation in SystemDS! We get an numpy array as a result - m_res_np = m_res.compute() - print(m_res_np) + # Create a context + with SystemDSContext() as sds: + # element-wise matrix multiplication, note that nothing is executed yet! + m_res = sds.matrix(m1) * sds.matrix(m2) + # lets do the actual computation in SystemDS! We get an numpy array as a result Review comment: Are you referring to the use of "we" or to the comment itself. The comment refers to the line below (where we get a numpy array). ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
