cmcuza commented on PR #2200:
URL: https://github.com/apache/systemds/pull/2200#issuecomment-2628991120

   I suggest editing the PR's title to indicate that it is related to the issue 
[[SYSTEMDS-3179]](https://issues.apache.org/jira/browse/SYSTEMDS-3179), for 
example, #2206. Moreover, I suggest giving more details regarding the code in 
this PR the proposed `scripts/builtin/cooccur.dml` has many small 
functionalities. Finally, I suggest adding the proper testing of the proposed 
co-occurrence matrix calculation.  


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