#metabrainz

/

      • D4RK-PH0ENiX has quit
      • 2017-05-17 13726, 2017

      • D4RK-PH0ENiX joined the channel
      • 2017-05-17 13700, 2017

      • D4RK-PH0ENiX has quit
      • 2017-05-17 13707, 2017

      • D4RK-PH0ENiX joined the channel
      • 2017-05-17 13735, 2017

      • dseomn has quit
      • 2017-05-17 13701, 2017

      • dseomn joined the channel
      • 2017-05-17 13748, 2017

      • samj1912 joined the channel
      • 2017-05-17 13754, 2017

      • D4RK-PH0ENiX has quit
      • 2017-05-17 13705, 2017

      • Zialus has quit
      • 2017-05-17 13750, 2017

      • Zialus joined the channel
      • 2017-05-17 13734, 2017

      • D4RK-PH0ENiX joined the channel
      • 2017-05-17 13702, 2017

      • kepstin_ joined the channel
      • 2017-05-17 13736, 2017

      • Ether_Man
        Hmm.. The begin field, of the life-span field on musicbrainz. Can it be any format? I've so far seen just a year, and YYYY-MM-DD. Wanna know how many different cases I'm working with here :)
      • 2017-05-17 13755, 2017

      • kuno
        Ether_Man: I _think_ all parts are optional, so for example if you know an artist's birthday is july 23rd, but you don't know their birth year, you'd be able to enter that into MusicBrainz.
      • 2017-05-17 13731, 2017

      • Ether_Man
        Right, but it'll never be in like dd/mm/yyyy? or mm/dd/yyyy?
      • 2017-05-17 13705, 2017

      • kuno
        Ether_Man: the webservice should return it in a standard format
      • 2017-05-17 13720, 2017

      • Ether_Man
        Cheers :)
      • 2017-05-17 13735, 2017

      • kuno
        np :)
      • 2017-05-17 13722, 2017

      • MajorLurker joined the channel
      • 2017-05-17 13758, 2017

      • d4rkie joined the channel
      • 2017-05-17 13748, 2017

      • D4RK-PH0ENiX has quit
      • 2017-05-17 13726, 2017

      • Slurpee has quit
      • 2017-05-17 13724, 2017

      • sanitypassing joined the channel
      • 2017-05-17 13752, 2017

      • pbryan joined the channel
      • 2017-05-17 13759, 2017

      • sanitypassing has quit
      • 2017-05-17 13743, 2017

      • hibiscuskazeneko joined the channel
      • 2017-05-17 13709, 2017

      • CatQuest has quit
      • 2017-05-17 13724, 2017

      • CatQuest joined the channel
      • 2017-05-17 13724, 2017

      • CatQuest has quit
      • 2017-05-17 13724, 2017

      • CatQuest joined the channel
      • 2017-05-17 13713, 2017

      • CatQuest has quit
      • 2017-05-17 13748, 2017

      • CatQuest joined the channel
      • 2017-05-17 13748, 2017

      • CatQuest has quit
      • 2017-05-17 13748, 2017

      • CatQuest joined the channel
      • 2017-05-17 13744, 2017

      • outsidecontext joined the channel
      • 2017-05-17 13734, 2017

      • drsaunder has quit
      • 2017-05-17 13757, 2017

      • Mineo joined the channel
      • 2017-05-17 13750, 2017

      • kepstin_ has quit
      • 2017-05-17 13757, 2017

      • kepstin_ joined the channel
      • 2017-05-17 13719, 2017

      • Mineo has quit
      • 2017-05-17 13721, 2017

      • ferbncode
        alastairp, Gentlecat: Hi :), I wanted some suggestion on the design of the module dealing with the mb related code.
      • 2017-05-17 13722, 2017

      • ferbncode
        I have something similar to this in my mind > https://gist.github.com/ferbncode/11e1632ac8867ac….
      • 2017-05-17 13725, 2017

      • samj1912 has quit
      • 2017-05-17 13733, 2017

      • arbenina_ joined the channel
      • 2017-05-17 13732, 2017

      • reosarevok
        bitmap: does your fix in review also fix not being able to batch-add works with no language in the rel editor? (just noticed that one)
      • 2017-05-17 13722, 2017

      • bitmap
        nope :( that might have to wait another release
      • 2017-05-17 13747, 2017

      • reosarevok
        :( That's kind of a bad one too, but oh well
      • 2017-05-17 13757, 2017

      • reosarevok
        More reason to try and find the language I guess :p
      • 2017-05-17 13716, 2017

      • bitmap
        oops it's late and I read your message wrong
      • 2017-05-17 13739, 2017

      • bitmap
        I read 'multiple languages' instead of 'no languages'
      • 2017-05-17 13746, 2017

      • bitmap
        that does sound bad, yes
      • 2017-05-17 13711, 2017

      • reosarevok
        Yeah. I think it's the same bug as "0 can't be empty" or whatever, so I thought maybe your fix fixed both
      • 2017-05-17 13715, 2017

      • hibiscuskazeneko has quit
      • 2017-05-17 13736, 2017

      • reosarevok
        (well, the same kind of bug at least)
      • 2017-05-17 13753, 2017

      • bitmap
        ah no, that fix was a FormHandler thing which the rel editor doesn't use
      • 2017-05-17 13700, 2017

      • bitmap
        I can look into it tomorrow
      • 2017-05-17 13708, 2017

      • reosarevok
        k :)
      • 2017-05-17 13715, 2017

      • reosarevok
        I'll add a ticket in that case
      • 2017-05-17 13708, 2017

      • reosarevok
        next time force me to test the stuff beforehand more :)
      • 2017-05-17 13738, 2017

      • reosarevok
        (I was super busy with school and didn't, but I would probably have caught some of these if I had :/ )
      • 2017-05-17 13754, 2017

      • reosarevok
        oh yvanzo: do you have a list of STYLE tickets your code will be implementing? (I can go search, but if you have it already I'll save the time)
      • 2017-05-17 13717, 2017

      • reosarevok
        I guess since that'll be a few, I should use the chance to finally do a (heh, "monthly"...) style recap
      • 2017-05-17 13727, 2017

      • yvanzo
        Actually, it is a subset of what is listed in MBS-8393.
      • 2017-05-17 13727, 2017

      • BrainzBot
        MBS-8393: Extend dynamic attributes to all entities https://tickets.metabrainz.org/browse/MBS-8393
      • 2017-05-17 13710, 2017

      • reosarevok
        bitmap: created, assigned. Sleep well! :)
      • 2017-05-17 13735, 2017

      • pbryan has quit
      • 2017-05-17 13752, 2017

      • yvanzo
        reosarevok: Can we check off each point together? (To make sure it fits STYLE expectations too) Then I will update ticket description.
      • 2017-05-17 13711, 2017

      • yvanzo
        “Number of copies” is not a STYLE ticket, “Mono/Stereo” is about STYLE and matches “Number of channels” in gist snippet.
      • 2017-05-17 13718, 2017

      • reosarevok
        Sure :)
      • 2017-05-17 13723, 2017

      • reosarevok
        I started at the to
      • 2017-05-17 13724, 2017

      • reosarevok
        p
      • 2017-05-17 13725, 2017

      • reosarevok
        Of the gist
      • 2017-05-17 13731, 2017

      • reosarevok
        So just closed the MINC ticket
      • 2017-05-17 13736, 2017

      • reosarevok
        SESAC now
      • 2017-05-17 13718, 2017

      • reosarevok
        I'll assign those to you, since technically you did all the work :)
      • 2017-05-17 13727, 2017

      • yvanzo
        STYLE-779
      • 2017-05-17 13727, 2017

      • BrainzBot
        STYLE-779: Add SESAC ID as an artist/label attribute https://tickets.metabrainz.org/browse/STYLE-779
      • 2017-05-17 13712, 2017

      • reosarevok
        MIMO is the Hornbostel-Sachs, right? (STYLE-759)
      • 2017-05-17 13712, 2017

      • BrainzBot
        STYLE-759: Add Hornbostel-Sachs attribute for instruments https://tickets.metabrainz.org/browse/STYLE-759
      • 2017-05-17 13701, 2017

      • reosarevok
        It might be good to include Hornbostel-Sachs in the name there, I think it might make it much clearer for some people
      • 2017-05-17 13708, 2017

      • reosarevok
        Something like MIMO ID (Hornbostel-Sachs) ?
      • 2017-05-17 13710, 2017

      • reosarevok
        Dunno
      • 2017-05-17 13719, 2017

      • yvanzo
        CatQuest may enlight this.
      • 2017-05-17 13742, 2017

      • reosarevok
        (are we validating that free text field in any way right now? Might be good to check it fits the schema, so numbers, or numbers separated by dots or hyphens
      • 2017-05-17 13754, 2017

      • yvanzo
        The PDF link from that ticket description is broken
      • 2017-05-17 13754, 2017

      • reosarevok
        )
      • 2017-05-17 13706, 2017

      • reosarevok
      • 2017-05-17 13750, 2017

      • reosarevok
        Oh. Actually, I wouldn't call it an *ID*. It's a category
      • 2017-05-17 13757, 2017

      • reosarevok
        (it doesn't map 1:1 to instruments)
      • 2017-05-17 13737, 2017

      • reosarevok
      • 2017-05-17 13719, 2017

      • reosarevok
        I think I'd call it "Hornbostel-Sachs (MIMO)" actually but let's wait for CatQuest have a say :)
      • 2017-05-17 13739, 2017

      • reosarevok
        In any case it's being implemented, so closing the ticket
      • 2017-05-17 13706, 2017

      • hibiscuskazeneko joined the channel
      • 2017-05-17 13750, 2017

      • reosarevok
        Freso: https://gist.github.com/yvanzo/f37e70ca354be04efd… adds Matrix but doesn't mention Runout, would you still consider STYLE-781 fixed?
      • 2017-05-17 13751, 2017

      • BrainzBot
        STYLE-781: Add "Matrix" and "Runout" for Releases/Mediums https://tickets.metabrainz.org/browse/STYLE-781
      • 2017-05-17 13701, 2017

      • Freso
        reosarevok: No. But I wouldn't object to making a new ticket for "Runout" and then closing that ticket. Or you can rename it once the admin interface it live (I hope).
      • 2017-05-17 13719, 2017

      • yvanzo afk 5m
      • 2017-05-17 13749, 2017

      • reosarevok
        what is runout anyway?
      • 2017-05-17 13756, 2017

      • reosarevok
        I've heard matrix before, but not that
      • 2017-05-17 13700, 2017

      • reosarevok
        Hmm
      • 2017-05-17 13717, 2017

      • reosarevok
        "The mould SID code identifies the plant where the CD was pressed" - but we don't generally bother keeping pressings separate
      • 2017-05-17 13706, 2017

      • Freso
        I'm honestly not 100% sure of the distinction between the two. :)
      • 2017-05-17 13710, 2017

      • reosarevok
        I guess as long as we support multiple it might be ok, but
      • 2017-05-17 13726, 2017

      • reosarevok
        (oh, https://www.discogs.com/help/submission-guideline… suggests it's just CD vs vinyl, re: runout)
      • 2017-05-17 13752, 2017

      • Freso
        I usually add matrix information for CDs I have physically. I really should link them with Disc IDs though.
      • 2017-05-17 13720, 2017

      • zas
        matrix runouts (or whatever they are called) are very useful to identify "releases", they can differ even for the same "release" (MB definition)
      • 2017-05-17 13735, 2017

      • reosarevok
        I guess matrix is the same?
      • 2017-05-17 13736, 2017

      • reosarevok
        Yeah
      • 2017-05-17 13744, 2017

      • reosarevok
        "One edition of a release may have many different matrix numbers on individual copies - especially for major label releases. These are considered manufacturing variations for the purposes of cataloguing on Discogs, and not unique releases, so one Discogs 'release' may contain multiple variations in matrix numbers etc."
      • 2017-05-17 13747, 2017

      • zas
        for example, some widely available releases can be pressed in more than one factory
      • 2017-05-17 13755, 2017

      • reosarevok
        Guess it's the same for us then
      • 2017-05-17 13708, 2017

      • reosarevok
        (... we probably should have some documentation for this, huh)
      • 2017-05-17 13753, 2017

      • Gentlecat
        ferbncode: one problem is that it might be difficult to maintain both ways of getting MB data
      • 2017-05-17 13718, 2017

      • reosarevok
        yvanzo: can you rename "Matrix code" to "Matrix/Runout" or should I do it from the admin interface eventually? They seem similar enough there's no real reason to keep them separate IMO
      • 2017-05-17 13735, 2017

      • Gentlecat
        and the ways in which they are used might end up looking differently too
      • 2017-05-17 13727, 2017

      • zas
        BTW, i used to scan the "matrix runout" side of CDs, but there is no CAA type for those (apart "medium" + a comment), perhaps this is something to improve in the future, especially if one release can have multiple matrix runouts associated with.
      • 2017-05-17 13725, 2017

      • reosarevok
        Ticket!
      • 2017-05-17 13712, 2017

      • reosarevok
        (not sure what kind though... is that style or CAA? no idea...)
      • 2017-05-17 13719, 2017

      • reosarevok
        Ok, I think that's it for closing tickets :)
      • 2017-05-17 13738, 2017

      • ferbncode
        Gentlecat: The wrappers can be modified to call functions with different arguments, returning data in a similar manner (from both the db and the ws).
      • 2017-05-17 13738, 2017

      • UmkaDK joined the channel
      • 2017-05-17 13743, 2017

      • zas
      • 2017-05-17 13743, 2017

      • BrainzBot
        CAA-98: Add type for "matrix/runout" side of CDs
      • 2017-05-17 13742, 2017

      • Freso
        zas: I usually do that too. Scan that bit.
      • 2017-05-17 13748, 2017

      • reosarevok
        loujine: can you push the update to greasyfork? it still has the April version
      • 2017-05-17 13706, 2017

      • ferbncode
        Gentlecat: also should both the ws and db code need to be present? I am unsure if to keep only the direct db access
      • 2017-05-17 13741, 2017

      • Gentlecat
        that's what I'm not sure about
      • 2017-05-17 13715, 2017

      • Gentlecat
        if there's an easy way to set up MB database in development, then WS access shouldn't be necessary
      • 2017-05-17 13703, 2017

      • ferbncode
        Yes, if we use the musicbrainz test docker image, i think it will be easy. only the import will take some time(from dumps)
      • 2017-05-17 13709, 2017

      • yvanzo is back
      • 2017-05-17 13734, 2017

      • yvanzo
        reosarevok, Freso: In my understanding, Runout/matrix number is the same for CDs and Vinyls.
      • 2017-05-17 13748, 2017

      • reosarevok
        I think so, it's just not called matrix in vinyls
      • 2017-05-17 13727, 2017

      • reosarevok
      • 2017-05-17 13745, 2017

      • yvanzo
        Mastering/Mould SID codes are very clearly defined and can be useful to identify a release.
      • 2017-05-17 13756, 2017

      • Gentlecat
        ferbncode: we probably don't need all the data
      • 2017-05-17 13700, 2017

      • Gentlecat
        like edits
      • 2017-05-17 13732, 2017

      • yvanzo
        reosarevok: thanks for STYLE-788 :)
      • 2017-05-17 13733, 2017

      • BrainzBot
        STYLE-788: Support storing mould and mastering SID codes https://tickets.metabrainz.org/browse/STYLE-788
      • 2017-05-17 13746, 2017

      • reosarevok
        yvanzo: They are, I was just wondering re: having multiple per MB release since we don't want to start separating pressings
      • 2017-05-17 13750, 2017

      • reosarevok
        But that should be fine :)
      • 2017-05-17 13710, 2017

      • yvanzo
        I renamed Matrix/Runout attribute.
      • 2017-05-17 13738, 2017

      • reosarevok
        Thanks! :)
      • 2017-05-17 13745, 2017

      • yvanzo
        We skipped SPARS code.
      • 2017-05-17 13746, 2017

      • ferbncode
        Gentlecat: so we import only the mbdump.tar.bz2 and the mbdump-derived.tar.bz2(if necessary) as they cover most of the data leaving edits history etc
      • 2017-05-17 13713, 2017

      • reosarevok
        Nah I closed it anyway :)
      • 2017-05-17 13718, 2017

      • reosarevok
        (I think?)
      • 2017-05-17 13724, 2017

      • reosarevok
        Yeah
      • 2017-05-17 13702, 2017

      • yvanzo
        There are a few changes regarding initial request: it will be attached to medium rather than release, there are far more allowed values than requested (but most frequent values will be at the top of the list).
      • 2017-05-17 13708, 2017

      • reosarevok
        Medium sounds definitely good
      • 2017-05-17 13751, 2017

      • yvanzo
        We do not have SPARS code at track level for now, but this is a rare case.