contextualSpellCheck icon indicating copy to clipboard operation
contextualSpellCheck copied to clipboard

Added lowercased candidates comparison, Damerau distance, Bayesian re…

Open fingoldo opened this issue 2 years ago • 7 comments

…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

fingoldo avatar Jul 25 '23 08:07 fingoldo

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

stale[bot] avatar Sep 16 '23 21:09 stale[bot]

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

R1j1t avatar Sep 28 '23 06:09 R1j1t

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.

R1j1t avatar Oct 02 '23 06:10 R1j1t

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?

fingoldo avatar Oct 02 '23 07:10 fingoldo

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.

R1j1t avatar Oct 02 '23 19:10 R1j1t

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

fingoldo avatar Oct 21 '23 23:10 fingoldo

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

stale[bot] avatar Dec 15 '23 04:12 stale[bot]