Added lowercased candidates comparison, Damerau distance, Bayesian re…

R1j1t / contextualSpellCheck

✔️Contextual word checker for better suggestions

MIT License

405 stars 56 forks source link

Added lowercased candidates comparison, Damerau distance, Bayesian re… #91

Open fingoldo opened 1 year ago

fingoldo commented 1 year ago

…asoning, GPU inference. Exposed top_n candidates parameter instead of the hardcoded number.

Description

Issue this PR address?
Any other information you think is relevant

Checklist

[ ] Added and/or updated the test (not required for documentation changes)
[ ] My changes don't require a change to the documentation, or if they do, I've added all required information

stale[bot] commented 11 months ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

R1j1t commented 11 months ago

Can you also rebase your branch to pull some recent changes?

R1j1t commented 11 months ago

I have fixed the test pipelines, so post rebase with main the pipeline can check for current behavior. I would like to contribute to your branch, so please let me know if there is any change I can own.

fingoldo commented 11 months ago

I have fixed the test pipelines, so post rebase with main the pipeline can check for current behavior. I would like to contribute to your branch, so please let me know if there is any change I can own.

Sure, but I'm not very experienced with git. How exactly should I proceed?

R1j1t commented 11 months ago

No worries, can you try to follow: https://stackoverflow.com/a/54036195/7630458? The idea is to incorporate recently merged changes to your PR as well.

fingoldo commented 10 months ago

Sorry for the delay, I think I did what's needed, could you check, please?

stale[bot] commented 8 months ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.