GitHub user FavioVazquez opened a pull request:

    https://github.com/apache/spark/pull/19190

    [SPARK-21976][DOC]

    ## What changes were proposed in this pull request?
    
    Fixed wrong documentation for Mean Absolute Error.
    
    Even though the code is correct for the MAE:
    
    ```scala
    @Since("1.2.0")
      def meanAbsoluteError: Double = {
        summary.normL1(1) / summary.count
      }
    ```
    In the documentation the division by N is missing.
    
    ## How was this patch tested?
    
    All of spark tests were run.
    
    Please review http://spark.apache.org/contributing.html before opening a 
pull request.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/FavioVazquez/spark mae-fix

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/spark/pull/19190.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #19190
    
----
commit b2e2f8caef4f43d62cb48fc09a0fe17e71f3f4dd
Author: FavioVazquez <favio.vazqu...@gmail.com>
Date:   2015-04-30T19:46:40Z

    Merge remote-tracking branch 'upstream/master'

commit edab1ef07312cf44e26fccb7fca8f4a4977ad3ee
Author: FavioVazquez <favio.vazqu...@gmail.com>
Date:   2015-05-05T14:16:02Z

    Merge remote-tracking branch 'upstream/master'

commit 9af7074235b6c13001924e037772195b640115b8
Author: FavioVazquez <favio.vazqu...@gmail.com>
Date:   2015-05-15T13:58:04Z

    Merge remote-tracking branch 'upstream/master'

commit f27a20b9f53b643d5c963729f56ee548d0c8e263
Author: FavioVazquez <favio.vazqu...@gmail.com>
Date:   2015-06-04T16:10:00Z

    Merge remote-tracking branch 'upstream/master'

commit ad882a378e24458832b961fd97eb4b7662203ef9
Author: FavioVazquez <favio.vazqu...@gmail.com>
Date:   2015-06-09T12:47:59Z

    Merge remote-tracking branch 'upstream/master'

commit 424a92853f44c34f310e0a9e8dd927d246bd9171
Author: FavioVazquez <favio.vazqu...@gmail.com>
Date:   2015-06-17T14:56:35Z

    Merge remote-tracking branch 'upstream/master'

commit 5311719db61454eee5e1715f44507c294509ec1c
Author: FavioVazquez <favio.vazqu...@gmail.com>
Date:   2015-09-21T06:43:29Z

    Merge remote-tracking branch 'upstream/master'

commit d6b551b1d25cef07096b7e1fc22b659ed753d9dc
Author: faviovazquez <favio.vazqu...@gmail.com>
Date:   2017-09-11T13:36:27Z

    Merge remote-tracking branch 'upstream/master'

commit a95cfc6c5e88f44429319b52462b537ae0bc1857
Author: Favio André Vázquez <favio.vazqu...@gmail.com>
Date:   2017-09-11T13:49:03Z

    Fix doc for MAE

----


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to