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

Search: Reinstate source selection #906

Closed
4 of 10 tasks
DavisNT opened this issue May 30, 2023 · 16 comments
Closed
4 of 10 tasks

Search: Reinstate source selection #906

DavisNT opened this issue May 30, 2023 · 16 comments
Labels
feature request P1 High priority (next release)

Comments

@DavisNT
Copy link
Contributor

DavisNT commented May 30, 2023

Iris version

3.67.0

Operating system(s) affected

  • Windows
  • MacOS
  • iOS
  • Android
  • Linux
  • Other

Browser(s) affected

  • Firefox
  • Chrome
  • Edge
  • Other

What happened?

In the Search screen (URL: /iris/search) the Sources button is not shown. Searching always happens in all sources.
The sources selection is a very handy feature to refine search results.

Logs

No response

@DavisNT DavisNT added the bug label May 30, 2023
@cpt1gl0
Copy link

cpt1gl0 commented Jun 18, 2023

I am encountering the same issue.

@jaedb
Copy link
Owner

jaedb commented Jun 25, 2023

This is the result of some simplification of how the search functionality works. Essentially the intent is to put as much of the search state into the URL so back-navigation (and shortcuts) can more easily restore a previous search.

I understand the use case to filter each search by its sources, but I'm tending towards a global set of search providers. You could pick-and-mix which of your providers are used when searching under Settings but it wouldn't be selectable per search term.

@kingosticks
Copy link
Contributor

I find selecting the provider on a per-search basis extremely useful but I think it really depends on what backends you regularly use. For example, the YouTube backend has stuff that's not available on Spotify (or similar) so sometimes it's your only choice when searching, but most of the time the search results are awful and you want everything except YouTube selected. Having to jump in and out of settings each time to make this change is awkward. I appreciate you have to draw the line somewhere but if others have the same use case, could you reconsider including it in the state?

@DavisNT
Copy link
Contributor Author

DavisNT commented Jun 25, 2023

@jaedb I completely agree with @kingosticks that in many use cases ability to select search providers is crucial.
In my case Spotify is the main source and Soundcloud is something I use sometimes. I assume there are many more users with similar setup that mine and @kingosticks's.
Currently I have reverted to using Iris version 3.66.1.

Could it be possible to include the search sources (e.g. abbreviated to a few letters or as numeric flags) in the search URL and make the source selection available again?

@curiousercreative
Copy link

You could pick-and-mix which of your providers are used when searching under Settings but it wouldn't be selectable per search term.

@jaedb is this a proposed feature? I don't see anything in settings about toggling which providers (backends) should be searched.

@curiousercreative
Copy link

Am I the only one that struggles with the relevancy of search results even with a single backend/provider enabled? I posted here about it, but I bet with better search results relevancy we'd have less need to filter by provider.

@orontee
Copy link
Contributor

orontee commented Jul 17, 2023

From my pov selection of backends before the search is done is non-sense.

Why not search every backend but allow users to filter out search results according to their source? And memorise the user choice for later searches.

@kingosticks
Copy link
Contributor

So ask the sever to search everything (slower) and do client-side filtering of everything. That's twice the work for the same result. I don't see the advantage in this approach.

That aside, the real-world problem with this approach is some backends are very slow to provide their search results (through no fault of their own). With your suggestion, you waste that time on every search for no reason.

@orontee
Copy link
Contributor

orontee commented Jul 17, 2023

I thought backends were requested one at a time and asynchronously, and search results updated when a request for a peculiar backend ends.

Do we expect end user to have knowledge on which backend is slow so they stop using them?

@kingosticks
Copy link
Contributor

That's not how Mopidy's search results work. Anything that Iris is requesting separately from elsewhere might work like that, I don't know.

No, we don't expect that. But it should remain possible to experiment and work it out.

@andy-lang
Copy link
Contributor

andy-lang commented Jul 24, 2023

I would definitely support being able to select search backends on an opt-in/opt-out basis. I use Bandcamp, Tidal, and Local as providers. If I search with all three, Bandcamp tends to be the fastest, but also returns a heap of unrelated results that I don't care about; but it's really useful as a backup if the music I'm looking for isn't available on Local or Tidal. As it stands, my Local results tend to get buried in the noise of Bandcamp and Tidal results, even though my Local collection is my preferred source.

So at the very least I would appreciate being able to choose the priority of results - e.g. show me all my Local results as highest priority, then Tidal, then Bandcamp as a last resort.

@jaedb jaedb changed the title Sources button missing in Search Search: Reinstate source selection Aug 10, 2023
@jaedb jaedb added feature request and removed bug labels Aug 10, 2023
@jaedb jaedb added the P1 High priority (next release) label Sep 5, 2023
@jaedb jaedb mentioned this issue Sep 5, 2023
@jaedb jaedb pinned this issue Sep 10, 2023
@jaedb
Copy link
Owner

jaedb commented Oct 24, 2023

Released since 3.69.0

@jaedb jaedb closed this as completed Oct 24, 2023
@jaedb jaedb unpinned this issue Oct 24, 2023
@PureTryOut
Copy link
Contributor

Eh this doesn't seem to work. The menu is there again but clicking any entry in it makes it show a white page (with sidebar and player controls though). This is with 3.69.2.

jaedb added a commit that referenced this issue Jan 2, 2024
- Always display "no results" when a query is present
- Placeholder path for no sources selected (ensures URL retains structure)
- Fixes comment #906 (comment)
@jaedb
Copy link
Owner

jaedb commented Jan 2, 2024

🤔 right you are! Bugfix release coming out shortly.

@DavisNT
Copy link
Contributor Author

DavisNT commented Jan 3, 2024

@jaedb Could it be possible to persist the selected search sources on the client (i.e. when the search will be opened from the menu it would have selected the same sources that were used in previous search)?

@bendschs
Copy link

bendschs commented Feb 4, 2024

@jaedb Could it be possible to persist the selected search sources on the client (i.e. when the search will be opened from the menu it would have selected the same sources that were used in previous search)?

it used to be like thatin earlier versions, in 3.69.3 all sources are preselected by default when navigating to the search page. would be very useful to make source selection persistent again.

cheers
bendsch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request P1 High priority (next release)
Projects
None yet
Development

No branches or pull requests

9 participants