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

Lemma matches / exact matches are not prioritized in search view #12

Open
trondtynnol opened this issue Nov 20, 2023 · 3 comments
Open
Labels
bug Something isn't working

Comments

@trondtynnol
Copy link
Contributor

When searching for a compound word, the compound is not necessarily prioritized above the dynamically decomposed parts when showing the results. This means it may be hidden below the other analyzes. This has worked previously.

Example: Search for "verkkolanka" in https://saan.oahpa.no/fin/sms. The results for "verkko" and "lanka" are shown above "verkkolanka", completely hiding the latter on my laptop screen before scrolling down.

@trondtynnol trondtynnol added the bug Something isn't working label Nov 20, 2023
@trondtynnol
Copy link
Contributor Author

An especially bad example: https://saan.oahpa.no/fin/sms/?lookup=takaisin, where takaisin is hidden below taata, isä, takka, takka and takainen.

@Phaqui
Copy link
Contributor

Phaqui commented Jan 3, 2024

May be related to, or because of python 3 (when hashing of strings became randomized) - see my full explanation here: https://github.com/giellatekno/neahttadigisanit/blob/restructuring/neahtta/src/nds_lexicon/lexicon.py#L20-L54

(I was working on an unrelated local bug in the restructuring branch.)

@Phaqui
Copy link
Contributor

Phaqui commented Feb 22, 2024

Commit 37d8af9 should at least be on the right path.

fin-sms takaisin looks better, but verkkolanka doesn't. Will have to keep digging.

Edit: nob-sme "for" shows progression. The conjugated "for" of "å fare" and "å fore" is now at the bottom.

@Phaqui Phaqui added the fixed-unverified A fix has been merged, but not verified to 100% work label Feb 26, 2024
@Phaqui Phaqui removed the fixed-unverified A fix has been merged, but not verified to 100% work label Mar 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants