#musicbrainz

man, picard save ogg vorbis files slowly :(

yeah

It's almost as if OGG Vorbis is a terrible terrible terrible file format.

vorbis is fine, it's just the ogg ;)

I guess they just didn't bother to put support for padding into the format?

huh, it looks like there are some patches to some tools to use padding in the comments.

there's an annoying thing with the flac tag writer in picard - if it thinks there's /too much/ padding for the comments in a flac file, it'll rewrite the flac file to reduce the padding.

and the amount of padding that the flac encoder includes by default is above the limit that picard has :/

so picard completely rewrites every freshly encoded flac I tag with it :(

enter a ticket then :P

bitmap would probably know

or luks, but he's probably asleep

kepstin-laptop: perhaps http://mutagen.sourcearchive.com/documentation/... or so is to blame? It's not supposed to change the size of PADDING blocks but perhaps the way picard's writing tags results in multiple blocks that then must be merged?

ianmcorvidae: I dunno, but the flac encoder writes 8kb of padding by default, and mutagen always reduces that to 1kb or less padding (after tag writing)

hmm. looks like http://mutagen.sourcearchive.com/documentation/... is probably responsible, but i'm not sure exactly how it is...

huh

that thing where it appends 1020 bytes of padding (I think?) is strange

that's 1020 bytes of padding plus a four-byte header, so a 1kb padding block.

ah

yeah

since the documentation evidently doesn't tell you :/

well, you're looking at the public api docs, that's a private function

well, yes

even so :)

ianmcorvidae: it appears to find the offset of the first audio frame in the file

every block before that is metadata blocks

ah

(except the first 4 bytes, which are "fLaC"

)

http://code.google.com/p/mutagen/source/browse/... will probably tell me the same soon

http://flac.sourceforge.net/documentation_forma... :)

oh, huh

wait, never mind

I thought that _load maybe wasn't reading in padding but I was wrong

meh, probably better to just ask luks/bitmap later, I don't know this codebase :)

heh

it looks like there shouldn't ever be a case where it makes the header section smaller, there simply isn't code to do that

it has code to handle the existing space in the header being too small, that's what the call to insert_bytes on line 674 is for.

so the code path should be, given a simple example of "old vorbiscomment=1kb old padding=7kb", doing an update that causes new vorbiscomment=2kb, it should add 1kb padding to the end, giving 2kb comment + 8kb padding, notice that that's over the available space, and cut the padding down to 6kb.

yeah :/\

possibly would be worth going through this at the python command-line with mutagen

yeah.

there is literally no place in flac.py that could cause the flac file to get smaller

at least isolate the problem to mutagen vs. picard, since we've been assuming mutagen but it looks like mutagen is clean

the insert_bytes() function will give an assertion failure if passed a negative number, and there's no call to delete_bytes

of course, I don't see anything in http://bazaar.launchpad.net/~musicbrainz-develo... either :/

and in all of the example files I have, the total size of the metadata blocks in the header is /less/ after tagging with picard.

I can't figure this out, it looks like the mutagen code itself is working fine when I run through it by hand in a python shell :/

kepstin-laptop, you around?

(In the unlikely case it is not sold out there, of course)

reosarevok: I'm in Canada, not US :)

we got the $100 TouchPad a day before the US did, and it sold out everywhere by yesterday.

hi, i'm not receiving emails for new releases of my watched artists

is that a known issue, or is there anything i can do about it?

do you have an example?

let me find one

the most obvious one would be glee cast

did that feature actually ever work?

it didn't before

i've got that on my watched list, soundtracks are enabled in my preferences, yet i never receive any emails for any of their rather frequent releases

I don't know about in ngs

Are there any auto-editors out there who can approve this edit for me please? http://musicbrainz.org/edit/15052199

I only ask because it's blocking me from doing the merge...

done

Thanks

if the tracks on the other release are mistakenly swapped, won't they also need fixing first?

That would be nice...

The correct track list reference is here: http://www.discogs.com/Mike-Oldfield-The-Collec...

Could an auto-editor expediate this process somehow? Either by first accepting the merge before the track list change, or by accepting the track list change before the merge?

auto-editors can only accept things they can auto-edit themselves, so not merges

the latter should work though

I'll give it a go, and post the edit here

they should probably be fixed first anyway, since otherwise merging will merge the wrong recordings

Do I have to rename the tracks (and swap recordings) or is there a re-order tracks option?

cosares, in the advanced traclist, just change the track numbers

Done: Edit #15052260

I tried both ways before committing and the resultant edit was the same - it's not a very clear message: http://musicbrainz.org/edit/15052260

Does this edit need to be approved before the merge is entered, otherwise there would be a track list change conflict / failure?

Gotta go, I'll check the logs the later, ciao and thanks all

Is ModBot in a sad place again?

I've not been getting mails from him.

or her.

I've never checked. :)

this is what Gender: Other was _made_ for, I figure :P

lol, srsly

ianmcorvidae: ah, but has modbot ever stated that he/she is not male/female and doesn't want to be known as male/female? :P

:P

Someone program modbot to do so

someone fix all the bugs I keep finding :P

Someone bring the summer back

It's far more important to have ModBot identify their gender.

anyone here using opera?

Hmm. "Falco vs. the Young Canoeist" or "Falco vs. The Young Canoeist" as a song title?

someone reporting bugs I find and don't report anymore