Yomguithereal / talisman

Straightforward fuzzy matching, information retrieval and NLP building blocks for JavaScript.
https://yomguithereal.github.io/talisman/
MIT License
704 stars 47 forks source link

Fixed regex for words containing "aine" in sonnex.js #184

Open knokknok opened 2 months ago

knokknok commented 2 months ago

sonnex("capitaine") -> kE Should be: sonnex("capitaine") -> kapitEn

Yomguithereal commented 2 months ago

@knokknok looks sensible indeed. Can you add a unit test validating your change please?

Also, if you are interested in French phonetic algorithms I have a way better one here:https://github.com/Yomguithereal/phonogram although undocumented.