Skip to main content

Notice

Please note that most of the software linked on this forum is likely to be safe to use. If you are unsure, feel free to ask in the relevant topics, or send a private message to an administrator or moderator. To help curb the problems of false positives, or in the event that you do find actual malware, you can contribute through the article linked here.
Topic: foo_openlyrics (not my release) (Read 43811 times) previous topic - next topic
0 Members and 1 Guest are viewing this topic.

Re: foo_openlyrics (not my release)

Reply #125
Thank you for catching that error, and reporting it here, too, @sveakul .

For those who are curious, and don't follow the Github stuff, the dev. answered: "Inspired by your and Paaufo's comments and taking a look at the code, I see it was actually just not doing the correct checks when handling track metadata for internet radio. Thanks for the help!"

He has released v1.10.2, that I can confirm fixes the problem mentioned above.

Re: foo_openlyrics (not my release)

Reply #126
He has released v1.10.2, that I can confirm fixes the problem mentioned above.
Well, it "pretty much" fixes it.  With the "autosearch even when not visible" NOT checked, it will no longer start an autosearch if another panel tab is open.  When a search is started with its panel open, it does start an autosearch. So far all like 1.9.  However if a song is playing and its tab is suddenly opened, no autosearch is started until "Search for Lyrics" is invoked from the context menu, or a new song is started.  This is NOT like 1.9, which while never having the unwanted background search issue, would also START an autosearch on its own when tabbed to, saving two clicks.

However, I can definitely live with the 1.10.2 behavior, where the "starting search with a closed panel even when that option is disabled" problem is indeed FIXED.  BTW all of this nit-picking involves radio stream usage;  I put a couple of GIFs up on the GitHub site illustrating this for the extremely bored.

Re: foo_openlyrics (not my release)

Reply #127
Okay I was using the quick reply function before, so I didn't know this reply box existed. But uhhh, there's no "source" option here.

If as your first image showed, "Metal Archives" is NOT in your left "Search order" box of chosen sources, then the option "Search for options (manually)" must be ignoring the chosen sources and by default doing a search using ALL sources.  I agree that is not expected behavior, and should be considered a bug.

I was told by the dev., on Github, that this is expected behaviour. He showed me he's received reports about this since 2022. Yeah, in my case OL prioritizes empty lyrics from NetEase and relegates other services that I find more useful.

Re: foo_openlyrics (not my release)

Reply #128
Foo_openlyrics 1.11 released.  Bandcamp added as a source, a fix for SongLyrics sometimes stalling a search with its "we don't have the lyrics yet" message.  https://github.com/jacquesh/foo_openlyrics/releases/latest

I did a fresh install of this version after completely removing the previous one.  Tests fine, FAST, and search behavior follows all options.  Works great on streams or files.

It's a shame that Foobar's shortcuts don't include a path to the "About current lyrics" context menu choice present since 1.10, which includes the source of the found lyrics as had been requested.  Anyone know what is required to make it to the Prefs/Keyboard Shortcuts list in Foobar?  There are other OpenLyrics context menu choices there, just not "About current lyrics."

Re: foo_openlyrics (not my release)

Reply #129
First of all, thank you very much for the plugin!

If I can make a small feature request here: Would it be possible to add an option that automatically searched time stamped lyrics that contain only one line are not saved (skipped)?

Because there are many lyrics for my songs that are found and saved even though they only contain one line, like

Code: [Select]
[00:00.00] 作曲 : Linus Munch

This lrc was found for the Song "Fed Rock" by "Island of Love".

I would rather hot have such incomplete lyrics to be saved because maybe when I listen to the song next time in some months, a complete lyric would be available and downloaded automatically then.

The proposed "Skip single line lrc" option could be placed to the "Saving" page of the preferences.

Re: foo_openlyrics (not my release)

Reply #130
https://github.com/jacquesh/foo_openlyrics/issues/202#issuecomment-1802285578 points to the filter in
Preferences -> Tools -> OpenLyrics -> Searching

Configuring it could be a bit tricky. Anyone?

By the way, this thread isn't posted by the author - cf. the "(not my release)"

Re: foo_openlyrics (not my release)

Reply #131
Hi, I see "Nikolaos Spanakis" and also something written in chinese ,  作曲 , in the OpenLyrics window
It appears when there is no lyrics found
Is there a way to remove this please?

 

Re: foo_openlyrics (not my release)

Reply #132
It means it has selected a source that only has composer information, and not the actual lyrics.
You "remove" that by removing that lyric source.

Issues opened on the matter:
https://github.com/jacquesh/foo_openlyrics/issues/301
https://github.com/jacquesh/foo_openlyrics/issues/409