Closed coolinx closed 1 month ago
There is/was a deeper root cause to the failure rate. I too often get what seems like books or plays, e.g. Shakespeare's Hamlet.
At some point I intend on upgrading the underlying genius python library which I think fixes or vastly improves this behavior. Sorry I have not gotten to it in, well, a long time. "Soon" I need to upgrade my HA from v0.106.6 to the much evolved latest. I make no promises to get to it really soon, but I will get to it before the end of this year :)
I have a nice example to test: The Rolling Stones Sympathy For The Devil - 50th Anniversary Edition Instead of song text this gives me some Karl Marx text. similar to if I query genius.com: https://genius.com/search?q=The Rolling Stones Sympathy For The Devil - 50th Anniversary Edition But without the part after the hyphen https://genius.com/search?q=The Rolling Stones Sympathy For The Devil I get the correct song returned. I have this with nearly all song with a hyphen in the song title. Most of the time the part after the hyphen is in fact not part of the song title at all and can be truncated. Would there be a workaround possible where the part after the hyphen gets ignored in the genuis.com query?
I have a nice example to test: The Rolling Stones Sympathy For The Devil - 50th Anniversary Edition Instead of song text this gives me some Karl Marx text. similar to if I query genius.com: https://genius.com/search?q=The Rolling Stones Sympathy For The Devil - 50th Anniversary Edition But without the part after the hyphen https://genius.com/search?q=The Rolling Stones Sympathy For The Devil I get the correct song returned. I have this with nearly all song with a hyphen in the song title. Most of the time the part after the hyphen is in fact not part of the song title at all and can be truncated. Would there be a workaround possible where the part after the hyphen gets ignored in the genuis.com query?
If I agree with you that in English titles you can safely truncate after an hyphen, it's not the case in some other languages, as French, where hypens are use to make link between some words. so I think it's safer to juste remove the hyphen.
I've not forgotten about this...I have done some testing and I do not believe simply removing a hyphen is the solution. There are still varying results and I think I need to dive deeper into the lyricsgenius package to investigate.
Awesome that you're looking into this. Very much appreciated!
I've not forgotten about this...I have done some testing and I do not believe simply removing a hyphen is the solution. There are still varying results and I think I need to dive deeper into the lyricsgenius package to investigate.
Were you able to make any progress on this? Still getting rather peculiar results for songs with hyphens (which on my current playing playlist is most of them!)
hi got a strange box at the top how can I remove it?
I think this is the toughest bug of them all, and it's going to need to be addressed in the library used by this integration. In that library too I recently discovered a bug I intend to "improve" by adding an artist name comparison when a song hit is returned -- I found that some songs would get non-english translation returned because of assumptions and ordering in the http response data handling.
In regards to improving hits, this is one improvement. I need to test and see if those odd scenarios where long play scripts (like Hamlet) are returned go away with this change, as well as possibly increasing the chance for correct song, aside from translation.
@appsfinder2 yes, fixed in new refactored branch and applies to #18
ultimately created a wrapper class to deal with handling results from song search. The library we depend on is not really getting updated as desired...so...at least we have a way forward.
added support for song title to cleverly clean using regex... enjoy!
First of all I want to express my gratitude for the creator of this very nice piece of software. I have used it now for a year or so and love it.
I do get a fair number of false hits, where quite common songs are not found in the genius database. Instead it finds some non song text pieces of text which appear book chapters or lists. I noticed this is almost always the case when the song title had a hyphen "-" followed by some text like "live at..." or "featuring...", "Remastered", etc. When searching manually on the Genius webpage, I get the same results.
I think if the software uses only the first part of the song title until the hyphen, and would ignore the rest, the success rate of finding a proper song text would increase a lot.