-
Notifications
You must be signed in to change notification settings - Fork 1.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
embedart: Resize images only once per album #1028
Comments
Indeed; it would be better to do the resize only once per album. Shouldn't be hard; contributions would be welcome here. |
Added a possible fix for this into my existing PR, #1033. This is only triggered when embed_album is called, ie. when tagging an entire album at once. If embedart is used to tag multiple tracks from one album, but not the whole album, the image will still be resized each time. Open to ideas on how to fix that edge case. |
I have had a bit of a busy week this week at work. I planned on getting started/taking a look tonight.
On Oct 24, 2014, at 3:06 PM, kerobaros notifications@github.com wrote:
|
I actually think that's fine—the most egregious waste here was when embedding for a whole album. Embedding different tracks from the same album separately is less obvious & it's probably okay to leave that un-fixed for simplicity's sake. |
Re: issue #1028: "embedart: Resize images only once per album
I've noticed this a few times now but I updated recently and I don't leave verbose on for multiple albums. Today was the first time I've run it on a few albums individually.
Every time the same thing:
The resized for the album art image is processing it many times. That is to say, for every track, it's converting the same image again and again. It's not a huge task for some images, but if the album art is huge it's a lot of processing and it adds quite a bit of time to imports.
It happened here as well:
And again here:
Just for added stuff, I'm running: beets version 1.3.8
After updating to 1.3.9:
I don't think this is necessary behavior as the album cover should cover the album, so one conversion should be necessary for copying into all tracks yes?
The text was updated successfully, but these errors were encountered: