Skip to main content

Notice

Please be aware that much of the software linked to or mentioned on this forum is niche and therefore infrequently downloaded. Lots of anti-virus scanners and so-called malware detectors like to flag infrequently downloaded software as bad until it is either downloaded enough times, or its developer actually bothers with getting each individual release allow listed by every single AV vendor. You can do many people a great favor when encountering such a "problem" example by submitting them to your AV vendor for examination. For almost everything on this forum, it is a false positive.
Topic: Album Art Downloader XUI (Read 1565893 times) previous topic - next topic
0 Members and 2 Guests are viewing this topic.

Re: Album Art Downloader XUI

Reply #3025
I confirm that fanart.tv script is broken again. I can't figure out why though, I tried to update the script myself, I removed the "search" prefix from musicbrainz because that one seems not to work, but even that didn't fix it.

Either there's a problem with the script while searching for the musicbrainz ID or I just don't know what I'm doing. Even tried the update in this thread from last July but still no dice. Changing the settings all over doesn't help either.

Testing the API itself it seems to work so I don't know why the results aren't showing up in AAD. Hoping someone here could help. Thanks!


Re: Album Art Downloader XUI

Reply #3026
I've updated discogs (v0.21) and Mordred's fanarttv (0.7), they should work again now. @jod773 the problem with fanarttv wasn't just the removal of search (although that was the correct first step), it was also that Musicbrainz was forbidding access without providing a meaningful identifying useragent.

 

Re: Album Art Downloader XUI

Reply #3027
it was also that Musicbrainz was forbidding access without providing a meaningful identifying useragent.
Hey! Thanks for clearing that up :) I really thought I did everything but I couldn't think of a way to test the script and see where the error happens.

Thank you for the fix!

Re: Album Art Downloader XUI

Reply #3028
Thanks, Alex!

Re: Album Art Downloader XUI

Reply #3029
Hey Alex, I just discovered today that discogs actually stores images greater than 600x600. For example, this is what discogs returns:
https://img.discogs.com/cTvdhiSkSuFI2Xq_oMJjzwTFLp4=/fit-in/600x609/filters:strip_icc():format(jpeg):mode_rgb():quality(90)/discogs-images/R-629210-1495380053-6160.jpeg.jpg

And this is the full size image:
https://www.discogs.com/image/R-629210-1495380053-6160.jpeg

You'll notice all you need to do from the original image string is:
Code: [Select]
for image in images:
fullImageMatch = Regex("discogs-images/(?<filename>.*jpeg).jpg", RegexOptions.IgnoreCase).Match(image.full)
if fullImageMatch.Success:
imageName = "https://discogs.com/image/" + fullImageMatch.Groups["filename"].Value

However, when I updated discogs.boo to actually implement this, when attempting to retrieve the fullsize image, everything it returns is max 300x300. I can't figure out what's actually going wrong here and was hoping maybe you had some ideas. Being able to get images larger than 600x600 from discogs would be huge.

Re: Album Art Downloader XUI

Reply #3030
Try https://www.discogs.com instead of just https://discogs.com

I've attached a quick test which works for me, but discogs does server-side analytics of useragent and headers and stuff, and if they think you aren't a browser, will serve you the small image size.

Re: Album Art Downloader XUI

Reply #3031
Try https://www.discogs.com instead of just https://discogs.com

I've attached a quick test which works for me, but discogs does server-side analytics of useragent and headers and stuff, and if they think you aren't a browser, will serve you the small image size.
That worked! I had tried changing the UA to resemble a browser and no dice, but hadn't thought about adding www.

BTW, some images are actually smaller than the 600x600 default that discogs serves, them so if you plan to add this to the official version, a better check might be:
Code: [Select]
if (fullImageMatch.Success and (image.width >= 599 or image.height >= 599)):

Re: Album Art Downloader XUI

Reply #3032
BTW, some images are actually smaller than the 600x600 default that discogs serves, them so if you plan to add this to the official version, a better check might be:
Code: [Select]
if (fullImageMatch.Success and (image.width >= 599 or image.height >= 599)):
Hmm, I'm wondering if that might be because the originals are smaller than 600x600, and they just get resized to serve at 600x600? If that's the case, I'd rather have the original at whatever size it is.

Re: Album Art Downloader XUI

Reply #3033
Hmm, I'm wondering if that might be because the originals are smaller than 600x600, and they just get resized to serve at 600x600? If that's the case, I'd rather have the original at whatever size it is.
That's exactly what it is, and you have a fair point. It's a decent resizing algo they're using (as in, I've never noticed that something was OBVIOUSLY upscaled) but certainly photoshop could do a better job.

Re: Album Art Downloader XUI

Reply #3034
Okay, so I did a little more playing around. If you use curl's UA, discogs starts to return 300x300 max size images. With a browser UA they don't appear to ever stop sending fullsize if they exist. I also realized that if the image is less than 600 in width, it's never scaled up to 600. So I added the width/height >600 check back in. No since hitting a non-public end point if it's going to return the same image as before.

Try this out and let me know if you encounter any issues.

Re: Album Art Downloader XUI

Reply #3035
Thanks for that. I've just changed the UA to match the one used for downloading the full sized images (from util.boo GetPageStream with useFirefoxHeaders) - probably best if all the requests have the same UA.

I've uploaded this now as v0.22, if anyone notices any issues then please do report them as usual.

Re: Album Art Downloader XUI

Reply #3036
11 MAR 21

Win 7 x64,  just did the March updates as offered by WU [w7x64 rollup and .net 4.8 update]

Now Album Art DL does not diaplay any art, the "frames" are just grey.  If I click the disk icon to save the file it saves fine and can be displayed in irfanview fine, so the files are fine, AA-DL is having issues displaying them on screen.

I assumed it was the .net updates.  did a complete uninstall of  dotNet and then a clean reinstall of 4.8 - same issue.

then took WU offered update [again] and still same issue.  WU says I am up to date.

So maybe just the W7 update did it, not dotNet?  not sure, and didn't roll back [no restore pt - dumb]

Anyway, thought I'd put this out there, anyome else see this issue today???

Thanks...



Re: Album Art Downloader XUI

Reply #3037
11 MAR 21   

    Part 2:    I did figure out how to uninstall  the March W7 Rollup, KB5000841, and it backed my system to around JAN 2021  time frame.  [it was fully up to date thru Feb].

   AA-DL  is now working just fine again  -- it is the W7 Rollup and NOT dotNet update. 

   SO BE  FORE-WARNED!!!    W7 update for March is not ready for public consumption!!!    Never be a pioneer !

Re: Album Art Downloader XUI

Reply #3038
11 MAR 21

now Part 3...

applied the Feb W7 Rollup, my system is now back to where it was prior to  this morning, with the addition of March dotNet 4.8 update,  and AA-DL is working just fine.  Confirmation that W7x64 March 2021 Rollup was the culprit.

Thanks for listening.... 

I hope someone learns from this experience, I know I did - never install updates without waiting several weeks !!

-=Paul=-

Re: Album Art Downloader XUI

Reply #3039
javacat:  you might try changing the DPI settings in the Properties/Compatibility/Change High DPI Settings tab of AlbumArt.exe to "Scaling performed by System" and see if that fixes your problem.

Re: Album Art Downloader XUI

Reply #3040
Thanks, but on my W7 setup the compatability tab doesn't have that BUTTON,  just the older check box to "disable display scaling on high dpi settings".   And as I've reverted things I can't test that now.  I will not be trying the March Rollup again for many weeks and hope it will eventually be resolved. 

Re: Album Art Downloader XUI

Reply #3041
Nope, sorry. That feature requires Windows 10.


Re: Album Art Downloader XUI

Reply #3043
Help?
You probably don't have permission to write to that directory because your user account isn't the "owner".

Re: Album Art Downloader XUI

Reply #3044
Thanks for that. I've just changed the UA to match the one used for downloading the full sized images (from util.boo GetPageStream with useFirefoxHeaders) - probably best if all the requests have the same UA.

I've uploaded this now as v0.22, if anyone notices any issues then please do report them as usual.
Apparently 5 days ago discogs pulled the plug on the ability to get high-res artwork through this method (wondering if it was due to increased bandwidth from this script? Timing seems suspect). It seems unlikely to come back in the near future so probably need to roll back the source to what it was for 0.21.

Re: Album Art Downloader XUI

Reply #3045
That's a shame. I've put 0.21 back again, thanks for the heads-up

Re: Album Art Downloader XUI

Reply #3046
Alex,
  I posted 10 days ago about an issue with the Win 7x64 March 2021 Rollup causing a problem with AADL displaying images.  Only grey boxes were displayed in their place.  

   Out of curiosity, has anyone else reported this issue to you?  I've looked around the web and I seem to be alone in having this problem, which is weird.  uninstalling the rollup definately brought image display back, I'm confident it was something in the rollup that the dotNet framework was built upon, my guess is ieframe.dll dated 12feb21, or another dll with that date.

  [FWIW I did an uninstall of the rollup itself, not a restore point recover, so I think it was the rollup and not something else].

Thanks...  -=PAUL=-

Re: Album Art Downloader XUI

Reply #3047
No, no one else has reported the issue. It's quite possible you are the only person using AAD on Windows 7, though! To be honest I didn't think they were still putting out updates for that.

As Windows 7 is EOL, support for it can best be described as "probably works". If anyone else reports grey boxes on Win7, though, I shall direct them to your post and advise uninstalling the rollup.


Re: Album Art Downloader XUI

Reply #3049
Good, good. You volunteer to test and develop on Windows 7, then.

 
SimplePortal 1.0.0 RC1 © 2008-2021