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

Mismatch in lookup method if compared to python's SymSpell #19

Open
RevadiSai opened this issue Feb 21, 2025 · 3 comments
Open

Mismatch in lookup method if compared to python's SymSpell #19

RevadiSai opened this issue Feb 21, 2025 · 3 comments

Comments

@RevadiSai
Copy link

RevadiSai commented Feb 21, 2025

Earlier I am using python Symspell which can easily installed through python's pip. And later i have created a fast-api server to use Symspell as an API. Currently for faster performance I wanted to use go lang HTTP server instead python's fast-api. For this, I am using this go-symspell library.

However, I am observing a mismatch between these two implementations for lookup method even if i am using same dictionaries ( both unigram and bigram ). Can someone please help me here with any suggestions?

pip module reference: symspellpy

@SamMHD
Copy link

SamMHD commented Feb 23, 2025

Hello dear @RevadiSai
Can you please share your examples?
Currently we have made some improvements to vanilla Symspell here, I just want to take a look at examples to see if it's intended or it's a bug.

@RevadiSai
Copy link
Author

Hi @SamMHD
I am using LookupCompound function in both python's Library and this go-symspell library with same unigram and bigram dictionaries and getting different results in both of them. Here are few examples

input symspellpy go-symspell
chhattpati shuvaji maharaj chhatrpati shivaji maharaj chhat pati shivaji maharaj
ganagama thimmaiah conventional hall gangama thimmaiah conventional hall janagama thimmaiah conventional hall
Kemopegowda International Terminal 2 kempegowda international terminal 2 kemp gowda international terminal 2

@sepehrsoh
Copy link
Collaborator

Hi @RevadiSai ,
We have most of the tests of symspellpy package here.
we use option pattern to set symspell config, i suugest firt look at tests, but if your issue continues we can check it implementation and compare with what you do in symspellpy

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

No branches or pull requests

3 participants