Calculate Cover Size 1.7 - Updated 23/10/2012

Post a reply

Visual Confirmation

To prevent automated access and spam, you are required to confirm that you are human. Please place a check mark next to all images of monkeys or apes. If you cannot see any images, please contact the Board Administrator.

Smilies
:D :) :( :o :-? 8) :lol: :x :P :oops: :cry: :evil: :roll: :wink:
BBCode is ON
[img] is ON
[flash] is OFF
[url] is ON
Smilies are ON
Topic review
   

Expand view Topic review: Calculate Cover Size 1.7 - Updated 23/10/2012

Re: Calculate Cover Size 1.7 - Updated 23/10/2012

Post by trixmoto » Mon Oct 22, 2012 5:23 pm

New version (1.7) is now available to download from my website. I have fixed the "Don't populate..." option not working in some modes.

Re: Calculate Cover Size 1.6 - Updated 12/07/2012

Post by trixmoto » Wed Oct 03, 2012 10:03 pm

Thanks, I'll take a look and fix this in the next version.

Re: Calculate Cover Size 1.6 - Updated 12/07/2012

Post by Nanya » Wed Oct 03, 2012 8:46 pm

Think I may have found a bug...
If you have "don't populate tracks without cover art" checked, the script will only work with "Width x Height" and "Total; W x H".
With much trepidation, I rolled up my sleeves and delved into the code (it's my first time working with MM scripting). The problem seems to be line 316 which reads:

Code: Select all
If (ipop) And (res > 0) And (act <> "") Then


This should probably be either
Code: Select all
If (ipop) And ((res > 0) Or (act <> "")) Then
or even
Code: Select all
If (ipop) And (res > 0) Then


Checking for "act" is superfluous, you can rely on "cnt" (and therefore "res") even in Cases 7 and 8.

Re: Calculate Cover Size 1.6 - Updated 12/07/2012

Post by MMuser2011 » Sun Sep 02, 2012 4:06 am

This script doesn't change your covers. It "only" calculate the size (height, width) and write the numbers to a custom field in the format you choose. It does not adjust the cover itself in any way.

Re: Calculate Cover Size 1.6 - Updated 12/07/2012

Post by vampire0slayer6969 » Sat Sep 01, 2012 6:37 pm

Am a little confuse in how this scripts works....
can you tell me how it work step by step
I got it to go in MM but just not sure how to get my covers all the same size...

Re: Calculate Cover Size 1.6 - Updated 12/07/2012

Post by Vyper » Fri Jul 13, 2012 7:00 am

It's baaack! :D

Re: Calculate Cover Size 1.6 - Updated 12/07/2012

Post by trixmoto » Thu Jul 12, 2012 7:35 am

Yes, there is. My host are currently working to resolve it. Hopefully it'll be back up soon!

Re: Calculate Cover Size 1.6 - Updated 12/07/2012

Post by Vyper » Thu Jul 12, 2012 6:58 am

There seems to be an issue with your site at the moment.

Re: Calculate Cover Size 1.6 - Updated 12/07/2012

Post by trixmoto » Thu Jul 12, 2012 6:25 am

New version (1.6) is now available to download from my website. Changes includes...

- Added option to only store to database
- Fixed "Width x Height" not working with other options

Re: Calculate Cover Size 1.5 - Updated 03/05/2010

Post by trixmoto » Tue Jul 10, 2012 10:32 pm

At the moment the way the script works is that it updates the database for each track individually as it loops through, and then also adds the changed tracks to a list. At the end of the loop, the script calls "UpdateAll" on the list, which is what uses your settings to determine whether or not to update the files as well. I could easily add an option in to not call this final "UpdateAll".

In the meantime you could easily comment out the line, which is line 385, by adding an apostrophe at the front...
Code: Select all
' Call changelist.UpdateAll()

Re: Calculate Cover Size 1.5 - Updated 03/05/2010

Post by MMuser2011 » Tue Jul 10, 2012 1:56 am

PatMcQ wrote:Summary: I suggest storing this "resolution" data in the MM databse only WITHOUT having to disable the MM options "Update tags when editing properties" and "Modify timestamp when updating properties", as those are indespensible options that I don't what to forget to re-enable. And the script should finish much quicker if it's not writing to each of the track files.
The script should respect the setting "Modify timestamp when updating properties" from MM4. I don't want to set it for every add-on or script separately.
I would vote +1 for the option:
a) Write to MM.DB and track
b) Write to MM.DB only

Re: Calculate Cover Size 1.5 - Updated 03/05/2010

Post by PatMcQ » Tue Jul 10, 2012 1:40 am

Groovy script. Build it and they will come!

Since this script is strictly for utility (cleaning up the library art tags) and the data has no permanent value in the track tag, I don't want to alter the file and file's timestamp for it. Besides relying on timestamps for library maintenance and harddrive backups, unnecessarily altering a track file increases the odds (slim as that may be) that the file gets corrupted. Album art resolution is already stored in the Album Art tag and thus this script really should only be writing to the MM database. If the database gets corrupted for some other reason and you lose the temporary album art resolution data generated by this script, no big deal because you have to re-run it anyway, whenever you're editing album art.

Summary: I suggest storing this "resolution" data in the MM databse only WITHOUT having to disable the MM options "Update tags when editing properties" and "Modify timestamp when updating properties", as those are indespensible options that I don't what to forget to re-enable. And the script should finish much quicker if it's not writing to each of the track files.

Re: Calculate Cover Size 1.5 - Updated 03/05/2010

Post by scottycope » Tue Jun 19, 2012 12:22 am

Thanks for the update. I didn't see your post go to the next page and was beginning to feel left out. I appreciate your response. Please let me know if your investigation becomes fruitful as this script will do wonders for my organization.

Thanks again!

Re: Calculate Cover Size 1.5 - Updated 03/05/2010

Post by trixmoto » Sat Jun 16, 2012 8:57 pm

No idea, sorry. I've added it to my list to investigate though.

Re: Calculate Cover Size 1.5 - Updated 03/05/2010

Post by scottycope » Thu Jun 14, 2012 12:09 am

Hi everyone,

I know this is an older thread but I just installed this script today and went to run it on a few albums. I picked Width x Height, stored in Custom 1, and pad 8 characters. I also checked Include images in tags (all my albums should have cover art stored in tags) and to not populate tracks without cover art.

However, when I go to run the script, nothing is written to the Custom 1 field. Yet, if I pick option Total; W x H, values are populated in my selected field.

Any idea why the plain old Width x Height option is not populating my selected field?

Any help is appreciated!

Thanks,
Scott

Top