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

Don't override relevance ranking with geo search #280

Closed
acka47 opened this issue Sep 26, 2016 · 2 comments
Closed

Don't override relevance ranking with geo search #280

acka47 opened this issue Sep 26, 2016 · 2 comments
Labels
Projects

Comments

@acka47
Copy link
Contributor

@acka47 acka47 commented Sep 26, 2016

When I type in a specific search (e.g. http://lobid.org/organisations/search?q=usb+k%C3%B6ln) I don't get the expected result as first hit. In my case (via vpn) it even can't be found within the first 20 hits.

There are different options to address this:

  • Use the geo filter only when no query has been entered.
  • Use a combination of geo filter plus relevance ranking (take the first twenty results from the geo query and sort them by relevance.)
  • ?
@acka47
Copy link
Contributor Author

@acka47 acka47 commented Dec 7, 2016

I think we should fix this before the official launch. Adding the respective label.

@acka47 acka47 added the launch label Dec 7, 2016
@fsteeg
Copy link
Member

@fsteeg fsteeg commented Dec 7, 2016

@fsteeg fsteeg self-assigned this Dec 7, 2016
@fsteeg fsteeg added ready and removed ready labels Dec 7, 2016
fsteeg added a commit that referenced this issue Dec 9, 2016
@fsteeg fsteeg added in progress and removed in progress labels Dec 9, 2016
fsteeg added a commit that referenced this issue Dec 9, 2016
fsteeg added a commit that referenced this issue Dec 9, 2016
fsteeg added a commit that referenced this issue Dec 9, 2016
@fsteeg fsteeg added review and removed working labels Dec 9, 2016
@fsteeg fsteeg removed their assignment Dec 9, 2016
@fsteeg fsteeg added review and removed working labels Dec 9, 2016
@fsteeg fsteeg closed this in d01630d Dec 9, 2016
@fsteeg fsteeg removed deploy labels Dec 9, 2016
@acka47 acka47 added this to Deploy in lobid board Apr 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants