T0biasCZe / Windows-Media-Player-Discord-RPC

Discord rich presence for windows media player
Mozilla Public License 2.0
11 stars 0 forks source link

Will not show album art despite being setup correctly #6

Open ZachjuKamashi opened 7 months ago

ZachjuKamashi commented 7 months ago

I am having several issues with album arts.

Screenshot (711)

it will not show the album art despite it being setup correctly for this specific song but everything else works fine? I even copy pasted the text to make sure it matches but it fails. I've already tried restarting the program and going as far as restarting explorer which actually fixed another song that also didn't show album art but not the one I am trying to fix for some bizarre reason?

I've already tried to rule out the three dots and that ain't it. it still will not work.

Then I actually have one more issue, I have 4 songs with the same title but all have different album arts, I am not able to differentiate them with whats provided. I tried the "Doesn't contain" string section but that doesn't work for artist names and there is nothing else I can do except name the title differently. so if possible it would be great as well to have another text field for artist name.

T0biasCZe commented 7 months ago

in the app dialog, does it show filename of the albumart, or does it just show wmp_empty? image also did you correctly enter your app id into the textbox?

ZachjuKamashi commented 7 months ago

in the app dialog, does it show filename of the albumart, or does it just show wmp_empty? also did you correctly enter your app id into the textbox?

As I previously stated "for this specific song but everything else works fine?" so yes, I have correctly entered the app id because otherwise I would see no album art at all. But the dialog in fact does show as wmp_empty so you correct on that assumption

Screenshot (717)

T0biasCZe commented 7 months ago

Could you send me the affected audio file so I can debug why it doesn't work properly?

ZachjuKamashi commented 7 months ago

Could you send me the affected audio file so I can debug why it doesn't work properly?

Sure! Had to put it into a zip file though as github doesn't want mp3's apparently. AffectedAudioFile.zip

T0biasCZe commented 7 months ago

image

image

i tried adding it as "title contains" and it correctly detected it... (note, after you add the pair in the album art adder, you need to restart the app for the updated list to be loaded)

T0biasCZe commented 7 months ago

And I could add pair for artist name and file name... Gimme 20 minutes

ZachjuKamashi commented 7 months ago

i tried adding it as "title contains" and it correctly detected it... (note, after you add the pair in the album art adder, you need to restart the app for the updated list to be loaded)

If you take a look at the very first screenshot I posted, you can see the "albumsarts.csv" open and the 2nd to last at the bottom you can see the song that has this issue. As I have previously mentioned, I have gone as far as literally copy pasting it to make sure it will work and it just refuses to do so. On the right of that screenshot you can also see the album art used with that name, you can double check as many times as you please but I know I've set it correctly. Oh and I know you need to restart the app, I mentioned that on the first post that I have just tried everything.

The fact that it works fine for you, but not me is very bizarre.

T0biasCZe commented 7 months ago

I copied it from the file name

so if possible it would be great as well to have another text field for artist name.

Added ability to use artist's name or audio file name is added in release v2.1.1b

ZachjuKamashi commented 7 months ago

I copied it from the file name

So did I and it refuses to work. Bizarre

Added ability to use artist's name or audio file name is added in release v2.1.1b

Thank you! :-)

T0biasCZe commented 7 months ago

Added ability to use artist's name or audio file name is added in release v2.1.1b

it was actually broken in v2.1.1b, it now actually works in v2.1.1c (third time is the charm)

ZachjuKamashi commented 7 months ago

Added ability to use artist's name or audio file name is added in release v2.1.1b

it was actually broken in v2.1.1b, it now actually works in v2.1.1c (third time is the charm)

When I did mentioned "so if possible it would be great as well to have another text field for artist name." on my post I though you would add that as an optional text field for "Finding art based on title". Because as it is, it's only useful if that specific artist only used the exact same art, you still can't really differentiate them.

However you did come up with "Finding art based on filename (inc" (Yes the "(inc" bit by the way is cutoff from there in the program) which does solve the issue I was having. I still don't know why it refused to work with title but works fine filename. Is there a max capacity for "finding art based on title"? That's the next best thing I can guess.

You are very dedicated though, I appreciate the effort! :)