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. 


---

Reply via email to