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 2021/04/30 10:11:47 UTC

[GitHub] [systemds] Baunsgaard opened a new pull request #1244: [MINOR] Force ytest argument in lmPredict

Baunsgaard opened a new pull request #1244:
URL: https://github.com/apache/systemds/pull/1244


   This is not the ideal solution, but just to make the test pass the LMPredict now require ytest argument,
   The reason i simply because the python generators does not like default parameters on matrice arguments yet.


-- 
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:
users@infra.apache.org

[GitHub] [systemds] Baunsgaard commented on pull request #1244: [MINOR] Force ytest argument in lmPredict

Posted by GitBox <gi...@apache.org>.

Baunsgaard commented on pull request #1244:
URL: https://github.com/apache/systemds/pull/1244#issuecomment-830695286


   merged


-- 
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:
users@infra.apache.org

[GitHub] [systemds] Baunsgaard closed pull request #1244: [MINOR] Force ytest argument in lmPredict

Posted by GitBox <gi...@apache.org>.

Baunsgaard closed pull request #1244:
URL: https://github.com/apache/systemds/pull/1244


   


-- 
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:
users@infra.apache.org