[
https://issues.apache.org/jira/browse/MAHOUT-1602?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14387949#comment-14387949
]
Hudson commented on MAHOUT-1602:
--------------------------------
SUCCESS: Integrated in Mahout-Quality #3032 (See
[https://builds.apache.org/job/Mahout-Quality/3032/])
MAHOUT-1602: Euclidean Distance Similarity Math, this also closes #60
(suneel.marthi: rev 57429b176a94514acce5f6985a1299fccb8d115f)
*
mrlegacy/src/main/java/org/apache/mahout/cf/taste/impl/similarity/EuclideanDistanceSimilarity.java
* CHANGELOG
> Euclidean Distance Similarity Math
> -----------------------------------
>
> Key: MAHOUT-1602
> URL: https://issues.apache.org/jira/browse/MAHOUT-1602
> Project: Mahout
> Issue Type: Bug
> Components: Collaborative Filtering, Math
> Affects Versions: 0.9
> Reporter: Leonardo Fernandez Sanchez
> Assignee: Stevo Slavic
> Priority: Minor
> Labels: legacy
> Fix For: 0.10.0
>
>
> Within the file:
> /mrlegacy/src/main/java/org/apache/mahout/cf/taste/impl/similarity/EuclideanDistanceSimilarity.java
> Mentions that the implementation should be sqrt(n) / (1 + distance).
> Once the equation is simplified, should be:
> 1 / ((1 + distance) / sqrt(n))
> Coded:
> return 1.0 / ((1.0 + Math.sqrt(sumXYdiff2)) / Math.sqrt(n));
> But instead is (missing grouping brackets):
> 1 / (1 + distance / sqrt (n))
> Coded:
> return 1.0 / (1.0 + Math.sqrt(sumXYdiff2) / Math.sqrt(n));
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
