hama-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hudson (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HAMA-905) Fix Pi Estimator Example
Date Mon, 26 May 2014 00:23:01 GMT

    [ https://issues.apache.org/jira/browse/HAMA-905?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14008484#comment-14008484
] 

Hudson commented on HAMA-905:
-----------------------------

SUCCESS: Integrated in Hama-Nightly-for-Hadoop-1.x #1256 (See [https://builds.apache.org/job/Hama-Nightly-for-Hadoop-1.x/1256/])
HAMA-905: Fix Pi Estimator Example (millecker: rev 1596782)
* /hama/trunk/CHANGES.txt
* /hama/trunk/examples/src/main/java/org/apache/hama/examples/PiEstimator.java


> Fix Pi Estimator Example
> ------------------------
>
>                 Key: HAMA-905
>                 URL: https://issues.apache.org/jira/browse/HAMA-905
>             Project: Hama
>          Issue Type: Bug
>          Components: examples
>    Affects Versions: 0.6.4
>            Reporter: Martin Illecker
>            Assignee: Martin Illecker
>            Priority: Minor
>              Labels: example, pi_example
>             Fix For: 0.7.0
>
>         Attachments: HAMA-905.patch, HAMA-905_2.patch
>
>
> *Fix Pi Estimator Example*
> I think we should skip the *sqrt* method in \[1] and change the if condition based on
formula 6 on page 2 in \[2]. 
> {code}
> IF (x^2 + y^2 <= 1) THEN
>  SUM = SUM +1
> END IF
> {code}
> Because the Equation of the Circle is 
> {code}
> x^2 + y^2 = r^2
> x^2 + y^2 = 1 (with r=1 in unit circle)
> -> (x^2 + y^2 <= 1)
> {code}
> Thanks!
> \[1] https://github.com/apache/hama/blob/trunk/examples/src/main/java/org/apache/hama/examples/PiEstimator.java#L62-64
> \[2] http://arxiv.org/pdf/1404.1499v2.pdf



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Mime
View raw message