-
Notifications
You must be signed in to change notification settings - Fork 27
Spellcheck #5
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
base: main
Are you sure you want to change the base?
Spellcheck #5
Conversation
Bloeckchengrafik
commented
Apr 22, 2025
|
I would unironically merge this, if it also integrated autocorrection for more words. This can be done just like It contains a |
|
i'll work on it |
|
LGTM, will check it out tomorrow afternoon. Thank you! |
|
I've done some testing and it partially works. There are a ton of edge cases where it gets stuff wrong:
The code inside I've pushed a change to cache database results, as it was querying everything on each function call. I suppose we could attempt to create a more resource-intensive checker for |
|
I'll build a small dictionary-based one later |
|
I believe the best approach at this would be using a large language model and storing the autocorrections, similar to how we currently handle autocomplete. However, I've attempted this with the "ai summaries" feature, and it was pretty costly. |
|
That's what i'm trying to circumvent: My current idea is embedding the old and new token as well as the query to compare the vectors to get to the relevant meaning. Real-Time-AI is too expensive for this |
|
That's my last attempt at an algorithm for this since I sadly got other stuff to do as well and this should originally be a quick and simple PR for laughs and giggles. But anyways, here we go! The current approach works as follows:
Doing grammar correction (where os john pork -> where is john pork) is out-of-scope for this PR! |