Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DMetaphone has issues with long words #5

Open
jaraco opened this issue Dec 28, 2012 · 2 comments
Open

DMetaphone has issues with long words #5

jaraco opened this issue Dec 28, 2012 · 2 comments

Comments

@jaraco
Copy link
Contributor

jaraco commented Dec 28, 2012

Originally reported by: Brian (Bitbucket: eode, GitHub: eode)


#!python

import fuzzy
fdm = fuzzy.DMetaphone()
fdm10 = fuzzy.DMetaphone(10)

# note that this also trounces the 's' phoneme of 'decent'
>>> fdm('decent')
['TKNT', None]

>>> fdm('decentralization')
['TKNT', None]

>>> fdm10('decentralization')
['TKNT', None]


# ..for comparison:
import metaphone
mdm = metaphone.dm

>>> mdm('decent')
('TSNT', '')

>>> mdm('decentralization')
('TSNTRLSXN', '')

Expected behavior:

  • produce phonemes for the whole word, or for the word up to the length specified.

@jaraco
Copy link
Contributor Author

jaraco commented Dec 28, 2012

Original comment by Brian (Bitbucket: eode, GitHub: eode):


..edited for formatting, and added expected behavior

@jaraco
Copy link
Contributor Author

jaraco commented May 13, 2013

Original comment by Sam Ockman (Bitbucket: NewStart, GitHub: NewStart):


Yes, this has bitten me too...

Here's an example I ran across:

for carbohydrate fuzzy gives KRPH as opposed to KRPHTRT for the original library.

It would be great to get this fixed.

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant