Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 326756 - nameSimilarityMetric should be symmetric
Summary: nameSimilarityMetric should be symmetric
Status: CLOSED FIXED
Alias: None
Product: EMFCompare
Classification: Modeling
Component: Core (show other bugs)
Version: 2.0   Edit
Hardware: PC All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: EMF Compare CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 373245
Blocks:
  Show dependency tree
 
Reported: 2010-10-01 05:49 EDT by Erhard Weinell CLA
Modified: 2013-04-24 07:33 EDT (History)
1 user (show)

See Also:

Attachments
check symmetric name similarity in test (830 bytes, patch)
2010-10-01 05:51 EDT, Erhard Weinell CLA 		no flags Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Erhard Weinell CLA 2010-10-01 05:49:27 EDT 
We stumbled upon the fact that org.eclipse.emf.compare.match.internal.statistic.NameSimilarity.nameSimilarityMetric is not symmetric, e.g. nameSimilarityMetric(a,b) is not equivalent to nameSimilarityMetric(b,a).

Is this intended behavior or is this a bug?
Comment 1 Erhard Weinell CLA 2010-10-01 05:51:58 EDT 
Created attachment 180032 [details]
check symmetric name similarity in test
Comment 2 Laurent Goubet CLA 2011-08-09 05:05:22 EDT 
We can't make this metric symmetric without massively impacting the existing consumers of the project and/or its API. This will only be handled in the future major version of EMF Compare.
Comment 3 Cedric Brun CLA 2012-05-30 05:12:26 EDT 
EMF Compare 2.0 is coming and with it an approach to match elements based on a distance and not on statistical similarity (the difference is subtle but important regarding scalability..)

This kind of similarity metric is useful though and has been kept in EMF Compare 2.0 and made symetric.

See : 
org.eclipse.emf.compare.utils.SimilarityUtil.diceCoefficient(String, String)
Comment 4 Laurent Goubet CLA 2013-01-17 08:32:23 EST 
batch-closing a bunch of "RESOLVED" bugs.