Album Art Downloader XUI

Topic: Album Art Downloader XUI (Read 2066693 times) previous topic - next topic

0 Members and 2 Guests are viewing this topic.

Album Art Downloader XUI

Reply #875 – 2009-07-18 12:36:15

Cool, thank you.
Just for info, under Windows 7 I get the Program Compatibility Assistant popping up after installation (using Admin account) saying that Installation may not have been successful and if I want to re-run it using compatibility settings for this Windows version.
However it's running fine nonetheless.

Album Art Downloader XUI

Reply #876 – 2009-07-18 13:24:06

Quote from: Adagio on 2009-07-18 12:36:15

under Windows 7 I get the Program Compatibility Assistant popping up after installation

Thanks for the info. I'm planning to upgrade to Win7 once it's released to retail, so I'll look into it then. Or, if it provided any more specific details about exactly why it thought the installation may not have been successful, I could try figuring it out from that.

There's nothing I'm doing specifically to mark the application as compatible or not with Windows 7, and as far as I'm aware there shouldn't be any reason for it not to work 100% correctly without any special settings or compatibility layer.

Alex

Album Art Downloader XUI

Reply #877 – 2009-07-18 15:57:25

As far as I can see the Program Compatibility Assitant does not provide any specifics as to why or what may have failed to install.
But I re-run installer without checking the "Add to Explorer context menu" option and the Compatibility Assistant did not pop up.
Edit: Just realized that in the previous install (where the Program Comp Assistant did appear) the Explorer context menu was apparently not properly installed, afaics.

Album Art Downloader XUI

Reply #878 – 2009-07-18 16:17:35

Quote from: Adagio on 2009-07-18 15:57:25

the Explorer context menu was apparently not properly installed, afaics.

Good to know. For information, checking this option in the installer just adds the registry key "HKEY_CLASSES_ROOT\Folder\shell\AlbumArtDownloader", which adds the Browse for Album Art command to the context menu for folders. I don't know whether Windows 7 just no longer allows commands to be added to folders like this any more, or if it is just that the installer isn't allowed to write to that part of the registry.

Alex

Album Art Downloader XUI

Reply #879 – 2009-07-20 07:44:17

0.28.3 does work far better than 0.28.1, thank you
However, after my initial tests I started working seriously again this weeken (over 800 albums to check ), and I must say that if more than 20-25 albums are queued, the programs does tend to slow down.
To be more precise: I queued 59 albums, and experienced no problem there. I started working and everything went peachy-keen still. However after about 25 albums, the downloads became sluggish, the response was lagging, and even my browsing, foobar and IRC became less responsive.
Closing down Album Art Downloader and starting it again cured this.
For now, it is something I can live with (I managed to do over 200 albums in a lazy saterday afternoon), I just thought I might share it with you

Album Art Downloader XUI

Reply #880 – 2009-07-20 07:57:58

Quote from: Jester-NL on 2009-07-20 07:44:17

after about 25 albums, the downloads became sluggish

Strange... according to my profiling software, once the window is closed it frees up all the resources it was using (with the exception of a few small formatted text strings). It does sound like something is leaking somewhere, though, according to your description. I'll poke around some more and see if I can spot it. To help me rule out one possibility, here's a version which does not cache formatted text strings. In general, this will be slightly slower as it has to continually recreate them - but, it will mean that there should be nothing at all left of a window after it closes. Could you let me know if it fixes the issue you're seeing?

AlbumArtDownloaderXUI-0.28.3-notextcache.zip

Thanks,

Alex

Album Art Downloader XUI

Reply #881 – 2009-07-20 15:01:32

Hi Alex,

I'm still getting an error in XP Pro SP2, .Net 3.5, Compatibility mode.

Code: [Select]

Album Art Downloader has encountered a fatal error, and has had to close.

If you wish to report this error, please include this information, which

has been written to the file: C:\Documents and Settings\WesterfD\AlbumArtDownloader\errorlog.txt



App version: 0.28.3.0, running on Microsoft Windows NT 5.1.2600 Service Pack 2 (32 bit)



System.MissingMethodException: Method not found: 'Void System.Windows.Window.set_ShowActivated(Boolean)'.

at AlbumArtDownloader.SearchQueue.DequeueNextSearchWindow()

at AlbumArtDownloader.SearchQueue.set_SimulataneousWindowsAllowed(Int32 value)

at AlbumArtDownloader.SearchQueue..ctor()

at AlbumArtDownloader.App.ProcessCommandArgs(String[] args)

at AlbumArtDownloader.App.OnStartup(StartupEventArgs e)

at System.Windows.Application.<.ctor>b__0(Object unused)

at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Boolean isSingleParameter)

at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Boolean isSingleParameter, Delegate catchHandler)

at System.Windows.Threading.DispatcherOperation.InvokeImpl()

at System.Windows.Threading.DispatcherOperation.InvokeInSecurityContext(Object state)

at System.Threading.ExecutionContext.runTryCode(Object userData)

at System.Runtime.CompilerServices.RuntimeHelpers.ExecuteCodeWithGuaranteedCleanup(TryCode code, CleanupCode backoutCode, Object userData)

at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)

at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state)

at System.Windows.Threading.DispatcherOperation.Invoke()

at System.Windows.Threading.Dispatcher.ProcessQueue()

at System.Windows.Threading.Dispatcher.WndProcHook(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)

at MS.Win32.HwndWrapper.WndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)

at MS.Win32.HwndSubclass.DispatcherCallbackOperation(Object o)

at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Boolean isSingleParameter)

at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Boolean isSingleParameter, Delegate catchHandler)

at System.Windows.Threading.Dispatcher.InvokeImpl(DispatcherPriority priority, TimeSpan timeout, Delegate method, Object args, Boolean isSingleParameter)

at System.Windows.Threading.Dispatcher.Invoke(DispatcherPriority priority, Delegate method, Object arg)

at MS.Win32.HwndSubclass.SubclassWndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam)

at MS.Win32.UnsafeNativeMethods.DispatchMessage(MSG& msg)

at System.Windows.Threading.Dispatcher.PushFrameImpl(DispatcherFrame frame)

at System.Windows.Threading.Dispatcher.PushFrame(DispatcherFrame frame)

at System.Windows.Threading.Dispatcher.Run()

at System.Windows.Application.RunInternal(Window window)

at System.Windows.Application.Run(Window window)

at System.Windows.Application.Run()

at AlbumArtDownloader.App.AlbumArtDownloader.IPriorInstance.Run()

at AlbumArtDownloader.InstanceMutex.RunAppAsServiceHost(IPriorInstance instance, String channelUri)

at AlbumArtDownloader.App.Main(String[] args)

Album Art Downloader XUI

Reply #882 – 2009-07-20 15:33:28

Quote from: drbeachboy on 2009-07-20 15:01:32

I'm still getting an error in XP Pro SP2, .Net 3.5, Compatibility mode.

Hmm... I'm not quite sure how that can be happening, the line of code in that error report should be skipped over if SP1 is not found. Does the message box warning about SP1 being missing show up?

Album Art Downloader XUI

Reply #883 – 2009-07-20 18:10:53

The .Net box pops up and I press Compatibility mode and the program starts just as like any other update that I've done. At the point where I expect the main window to open, the error log pops up.

Album Art Downloader XUI

Reply #884 – 2009-07-20 18:45:45

Quote from: drbeachboy on 2009-07-20 18:10:53

At the point where I expect the main window to open, the error log pops up.

Oh, so not even dequeuing a queued search window, just starting up the app? OK, that gives me a clue. Could you try this one for me, please?

AlbumArtDownloaderXUI-0.28.4-preSP1experimental.zip

Thanks,

Alex

Album Art Downloader XUI

Reply #885 – 2009-07-20 21:06:35

Hi Alex,

That one did the trick. It's working fine. Thank you.

Album Art Downloader XUI

Reply #886 – 2009-07-21 02:52:48

Thanks for making this. I gotta say as far as time savers go, this is just Godly. I have a few questions about some file naming problems. Few comments and suggestions, also. Currently, I have this setup:

Use File Path: \%artist%\(*) %album%\.
Specify Path: %artist% - %album% (%preset%).%extension%

Presets:
Default %type(Unknown,Front,Back,Inlay,CD)%
CD2 (CD2)
Tray (Tray)
Booklet1-2 (Booklet 1-2)
etc..

This is the only way I could get it to keep %preset% available and let me save to %type% if needed. It's a clunky solution, in that you can't do the single-click save. So, Is it possible to use | to choose between %type(Unknown,Front,Back,Inlay,CD)% or %preset%? from the file browser?

To clarify, for the four main types (Front, Back, Inlay, CD) I want to save those with one click, but also have the the preset submenu available for Unknown & other variants. I just can't seem to get both working. With %type(Unknown,Front,Back,Inlay,CD)%.%extension%|%preset%.%extension% in file browser, the preset submenu arrow does not appear. Reverse it, and without manually clicking Default in presets, your get artist - album ().jpg

This leads me to bugs & suggestions.

1. When using %preset%, one would expect the Default preset to be applied by default It isn't. You still have to choose it from the preset menu, which is a bit tedious. (side note: If this were default behavior, my above problem would be history)

2. The Preset menu doesn't exist in the Preview window, so to apply it, you have to close the preview, find the image (which for unknown size has usually been moved, depending on your sorting settings) and save it from there. Also tedious.

3. The field in the Search/Downloader window that you made read-only. Can you make that optional? I realize it was done to lessen confusion, but that field being editable is actually really useful:

-The easiest way to test how settings from the file browser are applied. With it read-only, you have to either close the window, edit settings in the file browser, reopen and repeat the search, or open and work from an entirely new search window.

-Say you're looking for art for album#1. During the search, art for album#2 (that didn't show up on album#2 search) appears. With this read-only, if you want to save it to the other album's folder, you have go through a clickfest to that path in each save dialog, not to mention manually rename the files. It's much easier to just temporarily substitute \%album% with \album2 in that field.

Third, I saw that someone asked for a feature to pick a default name for Unknown art. Expanding on that, I'd love to see the ability to also set a preferred name for the other types AAD maps elsewhere. Coveralia, for example shows some as Interior Frontal and Interior Trasera, which are both set to Inlay. There's a few more, (LP, Cover, ec) but you get my drift.

Fourth, a question about the actual naming choices. How come this maps the jewel box's back card (the folded one beneath the cd tray, with two side wings showing artist + title) as Back, and the booklet back (or the reverse of a 1-page tray card) as the Inlay? I've designed several cd layouts, and every template and spec guide I've ever seen says inlay = the card stuck in the back of the jewel box (i.e. *inlaid* into the tray). Am I wrong? If so, out of curiosity, what names do you guys use for all the other miscellaneous art types?

I use the following, but am always open to ideas.

Cover = The Front and back of a booklet as a single image
Front = The front of the booklet or the outer package for multidisc or boxsets.
Back = The back of the booklet or outer package. Anything that's not an inlay
Front Inside = Page on the reverse of the Front
Back Inside = Page on the reverse of the Back
Inlay = The back card under the cd tray
Tray = An inlay beneath a clear CD tray that's visible from the front.
Booklet 1-99 = Any pages between Front Inside and Back Inside

Lastly, and semi-off-topic, for those wanting to auto-convert/siize images and hate Imagemagic, check out Photoresize. It's neat in that you renaming the exe will auto-configure the output (i.e. PhotoResizeF800ISNO.exe would be 800px max width, Inplace (overwrite file) Skip smaller, Sharpen, Autoclose console window) Of course you can also pass the same via command line to a plain Photoresize.exe and do the same thing. With a little work, I got it so that one batch file will processes each art type in a folder the way I want (Cover=1600x1200 max, Front = 800x800 max Thumbnail=240x240 fixed, CD=640x640, etc.) Then I added a few shortcuts in SendTo for the most common tasks I do.

It's not as full-featured as ImageMagic, but is also free and works well. A less-nuclear solution, if you will.

Sorry for the novel, and thanks in advance if anyone can answer any of the above questions.

Kel

Album Art Downloader XUI

Reply #887 – 2009-07-21 04:20:33

Quote from: AlexVallat on 2009-07-20 18:45:45

Quote from: drbeachboy on 2009-07-20 18:10:53
At the point where I expect the main window to open, the error log pops up.

Oh, so not even dequeuing a queued search window, just starting up the app? OK, that gives me a clue. Could you try this one for me, please?

AlbumArtDownloaderXUI-0.28.4-preSP1experimental.zip

Thanks,

Alex

Fixed the problem for me as well.

Album Art Downloader XUI

Reply #888 – 2009-07-21 08:36:35

Quote from: Kelani on 2009-07-21 02:52:48

Thanks for making this. I gotta say as far as time savers go, this is just Godly. I have a few questions about some file naming problems. Few comments and suggestions, also.

Thanks for your comments and suggestions, I'll try and give them all answers:

Quote from: Kelani on 2009-07-21 02:52:48

1. When using %preset%, one would expect the Default preset to be applied by default

Yes, that was the intention. Looks like I screwed it up somewhere along the line, I'll fix that for 0.29.

For your setup, then, %preset%.%extension% in the file browser, and a default preset of %type% should do the trick.

Quote from: Kelani on 2009-07-21 02:52:48

2. The Preset menu doesn't exist in the Preview window

Good suggestion. I'll probably stick that in 0.29 too. Feature Request

Quote from: Kelani on 2009-07-21 02:52:48

3. The field in the Search/Downloader window that you made read-only. Can you make that optional?

Sorry, but I think that on balance it's better off as read-only. What I would suggest is just to kick off a new search window by tweaking the search terms (just add a space, or change letter capitalisation if they really are exactly right already). With "Open search results in new window" checked, that will open a new window that uses the default path pattern, but should have the same results.

Quote from: Kelani on 2009-07-21 02:52:48

Third, I saw that someone asked for a feature to pick a default name for Unknown art. Expanding on that, I'd love to see the ability to also set a preferred name for the other types AAD maps elsewhere

You can set the names of any of the types using the same method, just replace the name in the %type% placeholder. %type(Unknown,Front,Back,Inlay,Face)% would call CD "Face", for example.

As for the actual choice of types, what it comes down to is what is available from the sources. The majority have only the front cover, of course, but the other types AAD knows about reflect what was available from sources at the time. I don't know about the official terms, but I've always thought of Front and Back as referring to the front and back of the jewel case, and Inlay being the part inside. What I'd suggest is to rename "Inlay" as "Inside" for the next version (Feature Request), which is a more general term for anything that isn't on the outside, or printed onto the CD. I'd rather not add more types at this point, as sources can't be relied upon to be more specific than that.

Thanks also for the tip about Photoresize. I'm sure people here will find it handy. I have vague plans to eventually add some sort of image post-processing system (including resizing) to AAD, but it's certainly not going to happen soon.

Alex

Album Art Downloader XUI

Reply #889 – 2009-07-22 22:57:32

Hello, I just upgraded from 0.28.1 to 0.28.3 and I'm getting a consistent crash when I run AAD with "/fileBrowser" argument. It was working ok in 0.28.1, also if I run AAD normaly (without arguments) and then do Ctrl+B the browser opens up just fine. I tried with both the external hard drive connected and disconected (default browser path) but it didn't make a difference, it would always crash.

Heres the crash log (hope it helps):

Code: [Select]

Album Art Downloader has encountered a fatal error, and has had to close.
If you wish to report this error, please include this information, which
has been written to the file: C:\Program Files\AlbumArtDownloader\errorlog.txt

App version: 0.28.3.0, running on Microsoft Windows NT 6.0.6002 Service Pack 2 (32 bit)

System.IndexOutOfRangeException: Index was outside the bounds of the array.
   at AlbumArtDownloader.App.PathFix(String pathParam)
   at AlbumArtDownloader.App.ProcessCommandArgs(String[] args)
   at AlbumArtDownloader.App.OnStartup(StartupEventArgs e)
   at System.Windows.Application.<.ctor>b__0(Object unused)
   at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Boolean isSingleParameter)
   at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Boolean isSingleParameter, Delegate catchHandler)
   at System.Windows.Threading.Dispatcher.WrappedInvoke(Delegate callback, Object args, Boolean isSingleParameter, Delegate catchHandler)
   at System.Windows.Threading.DispatcherOperation.InvokeImpl()
   at System.Windows.Threading.DispatcherOperation.InvokeInSecurityContext(Object state)
   at System.Threading.ExecutionContext.runTryCode(Object userData)
   at System.Runtime.CompilerServices.RuntimeHelpers.ExecuteCodeWithGuaranteedCleanup(TryCode code, CleanupCode backoutCode, Object userData)
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
   at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state)
   at System.Windows.Threading.DispatcherOperation.Invoke()
   at System.Windows.Threading.Dispatcher.ProcessQueue()
   at System.Windows.Threading.Dispatcher.WndProcHook(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
   at MS.Win32.HwndWrapper.WndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
   at MS.Win32.HwndSubclass.DispatcherCallbackOperation(Object o)
   at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Boolean isSingleParameter)
   at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Boolean isSingleParameter, Delegate catchHandler)
   at System.Windows.Threading.Dispatcher.WrappedInvoke(Delegate callback, Object args, Boolean isSingleParameter, Delegate catchHandler)
   at System.Windows.Threading.Dispatcher.InvokeImpl(DispatcherPriority priority, TimeSpan timeout, Delegate method, Object args, Boolean isSingleParameter)
   at System.Windows.Threading.Dispatcher.Invoke(DispatcherPriority priority, Delegate method, Object arg)
   at MS.Win32.HwndSubclass.SubclassWndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam)
   at MS.Win32.UnsafeNativeMethods.DispatchMessage(MSG& msg)
   at System.Windows.Threading.Dispatcher.PushFrameImpl(DispatcherFrame frame)
   at System.Windows.Threading.Dispatcher.PushFrame(DispatcherFrame frame)
   at System.Windows.Threading.Dispatcher.Run()
   at System.Windows.Application.RunDispatcher(Object ignore)
   at System.Windows.Application.RunInternal(Window window)
   at System.Windows.Application.Run(Window window)
   at System.Windows.Application.Run()
   at AlbumArtDownloader.App.AlbumArtDownloader.IPriorInstance.Run()
   at AlbumArtDownloader.InstanceMutex.RunAppAsServiceHost(IPriorInstance instance, String channelUri)
   at AlbumArtDownloader.App.Main(String[] args)

Album Art Downloader XUI

Reply #890 – 2009-07-23 07:42:48

Quote from: blackspawn on 2009-07-22 22:57:32

I'm getting a consistent crash when I run AAD with "/fileBrowser" argument.

Oops, thanks for reporting that. I'll have it fixed for the next version. Until then, to work around this, add a path to the fileBrowser argument, like: /fileBrowser "f:\music"

Alex

Album Art Downloader XUI

Reply #891 – 2009-07-25 11:43:28

Album Art Downloader XUI v0.29 Released

Although this is mostly bug fixing, I've also added a couple of minor requested features. It may also have improved long-term memory usage (after opening and closing several search windows), but that's a bit theoretical at the moment.

Full list of changes since 0.28.3:

Further pre-SP1 compatibility fixes.
Clicking the Save button now uses the Default (first) preset, if presets are used.
Added Presets dropdown to save button in Preview window
Fixed crash when /fileBrowser argument is used without path parameter
Renamed term "Inlay" to "Inside", as it applies more generically to any artwork from the inside of the jewel case, not just the inlay.

Download:
AlbumArtDownloaderXUI.exe (Installer. Recommended)
AlbumArtDownloaderXUI.zip (zip archive for those who don't like installers)
.NET Framework 3.5 SP1 (required)

Album Art Downloader XUI

Reply #892 – 2009-07-27 10:55:09

Quote from: AlexVallat on 2009-07-25 11:43:28

Fixed crash when /fileBrowser argument is used without path parameter

Thanks for the quick fix!

Album Art Downloader XUI

Reply #893 – 2009-07-30 21:47:19

I am sorry if this was answered before, but 35 pages are lot to read. How can I limit it's search to folder.jpg only files? Now it finds front.jpg and back.jpg, and I don't want that, I want to have front.jpg for album art.

Album Art Downloader XUI

Reply #894 – 2009-07-31 08:18:12

Quote from: hlloyge on 2009-07-30 21:47:19

I am sorry if this was answered before, but 35 pages are lot to read. How can I limit it's search to folder.jpg only files? Now it finds front.jpg and back.jpg, and I don't want that, I want to have front.jpg for album art.

I don't quite understand. Do you mean that, when searching for a cover for an album, you only want to see results which have the type Front or Unknown? Or that when you are using the File Browser, you only want it to consider an album to have art already present if it is called exactly "folder.jpg"? Or that you always want to save results as exactly "folder.jpg"?

Alex

Album Art Downloader XUI

Reply #895 – 2009-07-31 09:12:39

I am sorry... I see it is hard to see what I want from the way I wrote question when I search in File Browser for albums which already has cover art, it searches directories, and marks them green if there is ANY coverart, be it front.jpg, folder.jpg, or nrg948hdidf8.jpg. I want it to search just for folder.jpg, and nothing else. But I haven't found any options for that.
Right now I have to manually enter these folders and rename and resize existing cover art, and it would be much easier for me to just download it from internet.

Album Art Downloader XUI

Reply #896 – 2009-07-31 15:54:28

I signed up to these forums just to say THANKYOU! Between this and foo_discogs you get simply the best music library ever. This app makes even itunes art look low res.. so big love <3

Album Art Downloader XUI

Reply #897 – 2009-07-31 17:54:10

Quote from: hlloyge on 2009-07-31 09:12:39

when I search in File Browser for albums which already has cover art, it searches directories, and marks them green if there is ANY coverart, be it front.jpg, folder.jpg, or nrg948hdidf8.jpg. I want it to search just for folder.jpg, and nothing else.

In the File Browser window, click the Options link, then in the "Specify path to find images" box, put just: folder.jpg

Quote from: SwiftDestiny101 on 2009-07-31 15:54:28

I signed up to these forums just to say THANKYOU!

Thanks, always nice to know when my work is appreciated!

Alex

Album Art Downloader XUI

Reply #898 – 2009-07-31 22:55:10

Thank you, Alex! I would expect this as a button, or an option from the menus, not as a link... that's why I couldn't find it!

Album Art Downloader XUI

Reply #899 – 2009-07-31 23:45:08

Hi, I am new here. I have been a light user of Foobar for several years now.
I am currently using version 0.9.5.2 with fooAvA 1.05.

I downloaded Net 3.5 framework, then Album Art Downloader, and Foo_run.
The Album Art Downloader program and foo_run work fine. My Foobar works fine.
They just don't seem to have hooked up...

In other words, none of the covers that AAD has found & that i have added, are showing up when i run Foobar. For example, I have a Beethoven Piano Concerto that I was amazed AAD found the right cover for, since I never could. Here is what happens: I click on the image. AAD momentarily displays the message, "Downloading image" or something like that. And then on to the next one. I did this for my very small library of music. Then I re-loaded Foobar. Voila! Nothing was any different. All the covers that were missing before, are still missing. Maybe there is some obvious mistake or oversight i have made, or maybe an incompatibility? I'm wise enough to admit i really need some help.
Any suggestions/ advice/ from anyone, would be much appreciated!

Notice