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

Ranking test #497

Closed
wants to merge 3 commits into from
Closed

Ranking test #497

wants to merge 3 commits into from

Conversation

adrienpoly
Copy link
Owner

NOT to merge

This is an exploration PR to follow #486

It displays the bm25 ranking in the card so that we can understand the effect of the weight for the date

adjusting the date weight to DATE_WEIGHT = 0.000000001 seemed to gave good results

as an example searching for hotwire http://localhost:3000/talks?s=hotwire
give this results

CleanShot 2024-12-06 at 23 58 25@2x

we have all the talk sorted by date with hotwire in the title

then page two after older talks with hotwire in the title we start to see talks without Hotwire in the title (probably found from the description) but from earlier date. So we keep a good relevancy on the title math with some weight for the date

CleanShot 2024-12-06 at 23 59 45@2x

This PR is not really intended to be merged but more as food for thought for finishing #486

poke @aamfahim

@aamfahim
Copy link
Contributor

aamfahim commented Dec 7, 2024

hmm, I went as far as 0.000001 during my testing but didn't really see much difference, but I guess I was missing a couple decimal points lol. But I'm seeing better results with 0.000000001.

@adrienpoly
Copy link
Owner Author

hmm, I went as far as 0.000001 during my testing but didn't really see much difference, but I guess I was missing a couple decimal points lol. But I'm seeing better results with 0.000000001.

yes this is why I displayed the values tu understand the effect of this weight

@adrienpoly adrienpoly closed this Dec 7, 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