You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@systemds.apache.org by GitBox <gi...@apache.org> on 2022/01/04 12:03:06 UTC

[GitHub] [systemds] lederb edited a comment on pull request #1496: [SYSTEMDS-3260] Builtin for Matthews Correlation Coefficient

lederb edited a comment on pull request #1496:
URL: https://github.com/apache/systemds/pull/1496#issuecomment-1004750279


   there is a typo in the comment at the moment (* instead of +), the actual computation is correct  
   
   https://github.com/apache/systemds/blob/0cb9d586e586460de1b64de9f93fbaac8919716a/scripts/builtin/mcc.dml#L106
   
   to avoid rerunning all the tests on the system again I'll wait to fix it with a new push till there is some feedback in case there are other changes required. 


-- 
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-unsubscribe@systemds.apache.org

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