[systemds] branch main updated: [MINOR] Change ytest to an optional input to lmPredict

[arnabp20]

This is an automated email from the ASF dual-hosted git repository.

arnabp20 pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/systemds.git


The following commit(s) were added to refs/heads/main by this push:
     new b065677  [MINOR] Change ytest to an optional input to lmPredict
b065677 is described below

commit b0656773231d2514e2e9c97e457233a634f7dc0f
Author: arnabp <arnab.ph...@tugraz.at>
AuthorDate: Tue Dec 14 00:06:15 2021 +0100

    [MINOR] Change ytest to an optional input to lmPredict
    
    This patch reverts a change that made ytest a mandatory input
    to lmPredict builtin. ytest is unavailable for unseen data.
    
    Closes #1484
---
 scripts/builtin/lmPredict.dml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/builtin/lmPredict.dml b/scripts/builtin/lmPredict.dml
index f53e326..0570f50 100644
--- a/scripts/builtin/lmPredict.dml
+++ b/scripts/builtin/lmPredict.dml
@@ -20,7 +20,7 @@
 #-------------------------------------------------------------
 
 m_lmPredict = function(Matrix[Double] X, Matrix[Double] B, 
-  Matrix[Double] ytest, Integer icpt = 0, Boolean verbose = FALSE) 
+  Matrix[Double] ytest = matrix(0,1,1), Integer icpt = 0, Boolean verbose = 
FALSE) 
   return (Matrix[Double] yhat)
 {
   intercept = ifelse(icpt>0 | ncol(X)+1==nrow(B), as.scalar(B[nrow(B),]), 0);