DRAFT: Multi-player, Multi-lyric source enhancements (IN PROGRESS) #40
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
… is the last option. Also, local now checks if the mp3 path from xesam has a lrc at that exact path.
…g to decide whether to log. Also, loads multiple players now (mpris, browser, spotify, ...) but I don't think it's pushing all the events to the same channel? Need to write something so that it handles the first item in the player list when its playing and sends data, that way we can say 'get something from the browser if they browser is playing, but if a local app is sending MPRIS data, then use that data, etc., etc.,' This will allow users to specify that they want Spotify as an app, Strawberry locally, and Jellyfin in a browser to all provide information, and to use the top listed one first if they all send data. Also, it can now provide lyrics from multiple sources, local, hosted, spotify, etc.
|
Looking forward to this. I especially love the part "if the path of the local MPRIS player specifies a specific file path, check that file path for a matching .lrc file." The current local provider is a bit too naive for my library, and this should solve a large part of that. I was considering to create a PR for that myself when I discovered this one. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I'm not a Go developer, so I'm still working on how to do what I want to do, but generally the idea is to list multiple players in the configuration and multiple lyric source configurations, so that if the hosted or Spotify lyric sources don't provide lyrics, but the user has .lrc files present, it will use those as a backup.
Also, looking into changing how the query data is passed to the lyrics providers, so that we can check the local file sources to make sure that the album, artist, and song title are present, and also if the path of the local MPRIS player specifies a specific file path, check that file path for a matching .lrc file.
If any Go developers have ideas on how to make it listen to multiple players, please let me know, I'm a C# guy.