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

Feature/lws 88 boosting #1531

Merged
merged 12 commits into from
Dec 11, 2024
Merged

Feature/lws 88 boosting #1531

merged 12 commits into from
Dec 11, 2024

Conversation

kwahlin
Copy link
Contributor

@kwahlin kwahlin commented Dec 2, 2024

Boost same fields in Libris Search as in the cataloging.

  • Code adapted from mainly ESQueryLensBoost.groovy and some parts from ESQuery.groovy. Should produce exactly the same boosted fields.
  • Set _debug=esScore to include scores + explanation in the API response. Works with both the "old" query style (using q param) and the "new" (_q). Thought it might facilitate working with improved relevance.
  • Use _boost for custom boosting, e.g. _boost=hasTitle.mainTitle^200,prefLabel^100....
  • A bug was discovered in the existing code: When computing boosting from cards a field would not be boosted if another field containing the same nested subfield was already boosted. E.g. boosting publication.year would block production.year from being boosted. See fix: bdb3d08.

Copy link
Contributor

@jannistsiroyannis jannistsiroyannis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wont claim to understand much of it, but I saw nothing obviously problematic!

@kwahlin kwahlin merged commit 4c997a9 into develop Dec 11, 2024
1 check passed
@kwahlin kwahlin deleted the feature/lws-88-boosting branch December 11, 2024 14:04
olovy added a commit that referenced this pull request Dec 11, 2024
olovy added a commit that referenced this pull request Dec 11, 2024
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

Successfully merging this pull request may close these issues.

2 participants