madlib-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From iyerr3 <...@git.apache.org>
Subject [GitHub] madlib pull request #205: hits and lin regr doc updates
Date Wed, 22 Nov 2017 03:49:40 GMT
Github user iyerr3 commented on a diff in the pull request:

    https://github.com/apache/madlib/pull/205#discussion_r152463751
  
    --- Diff: src/ports/postgres/modules/regress/linear.sql_in ---
    @@ -183,16 +183,15 @@ FROM (
     @par Prediction Function
     The prediction function is as follows:
     <pre class="syntax">
    -linregr_predict(coef, col_ind)
    +linregr_predict(col_ind, coef)
    --- End diff --
    
    This is inconsistent with the actual function, which has `coef` as the first parameter.
I believe the examples incorrectly place the `col_ind` first. 


---

Mime
View raw message