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

Improvements to synonyms #33

Open
ojwb opened this issue Jul 15, 2024 · 1 comment
Open

Improvements to synonyms #33

ojwb opened this issue Jul 15, 2024 · 1 comment

Comments

@ojwb
Copy link
Contributor

ojwb commented Jul 15, 2024

fastcardriver on #xapian ran into this not working:

db.add_synonym( "Treppe", "Treppenkörper" );

The issue is the term needs to be lowercase (otherwise T is treated as a prefix), but we don't actually seem to explain that.

Also they noted that it seems wrong that add_synonym() is called from the search code. Perhaps it would be better to have separate scripts for indexing, adding synonyms, and searching to make it clear that these are essentially separate steps.

@jaylett
Copy link
Contributor

jaylett commented Jul 28, 2024

Agree on both counts. It's weird that search_synonym has to use a writable database, where none of the other searches do; splitting out synonyms would allow us to resolve that.

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

2 participants