#musicbrainz

/

      • ruaok
        k
      • 2007-07-10 19101, 2007

      • ruaok
        will do.
      • 2007-07-10 19128, 2007

      • warp
        ok, i really need to go to bed.
      • 2007-07-10 19154, 2007

      • warp needs to wake up to go to work in about 6 hours.
      • 2007-07-10 19101, 2007

      • warp
        ruaok: good luck! :D
      • 2007-07-10 19118, 2007

      • ruaok
        thanks and sleep well!
      • 2007-07-10 19108, 2007

      • SerialNumber has quit
      • 2007-07-10 19100, 2007

      • Citizen has quit
      • 2007-07-10 19155, 2007

      • acid2_ joined the channel
      • 2007-07-10 19159, 2007

      • yllona
        latersz folks
      • 2007-07-10 19109, 2007

      • yllona has quit
      • 2007-07-10 19117, 2007

      • FauxFaux ponders how to make PicardQt use the cluster-times to do the lookup.
      • 2007-07-10 19150, 2007

      • FauxFaux
        There's not any client-side handling of multiple matches of stuff, which means just adding it to the web-service won't be easy.
      • 2007-07-10 19154, 2007

      • FauxFaux
        Oh, except the cdids..
      • 2007-07-10 19126, 2007

      • acid2_
        svn diff > patchname.patch to create patches, right?
      • 2007-07-10 19104, 2007

      • FauxFaux
        Yeah. Unless mb has a specific format they're wanted in.
      • 2007-07-10 19119, 2007

      • ruaok
        svn diff is a good default
      • 2007-07-10 19107, 2007

      • acid2_
        ok
      • 2007-07-10 19120, 2007

      • acid2_
        should I mark it as fixed when I attach the patch? (it does fix the problem)
      • 2007-07-10 19129, 2007

      • BrianFreud
        Faux: It'd be a pain, but it parses the release, then each track - you could theoretically output the release id on each new release, then times to a file as it hits each, then have someone re-load the same files, and import the times from that temp file on reloaded matching release id
      • 2007-07-10 19149, 2007

      • FauxFaux
        Would (either of) you two consider a patch adding http://faux.no-ip.biz/mbreports/cueread2.php? It's mostly sql this time. :)
      • 2007-07-10 19109, 2007

      • FauxFaux
        BrianFreud: I don't even follow, never mind. :P
      • 2007-07-10 19113, 2007

      • BrianFreud
        lol
      • 2007-07-10 19115, 2007

      • acid2_
        a patch in what way, FauxFaux ?
      • 2007-07-10 19131, 2007

      • BrianFreud
        same thing I said when I last looked at handling releases, instead of tracks, in picard
      • 2007-07-10 19137, 2007

      • FauxFaux
        To get mb_server to do it via web-service or something, acid2_.
      • 2007-07-10 19154, 2007

      • acid2_
        I don't under stand, how is it doing it now then?
      • 2007-07-10 19100, 2007

      • acid2_
        and what needs to be patched?
      • 2007-07-10 19101, 2007

      • FauxFaux
        PHP! :D
      • 2007-07-10 19115, 2007

      • acid2_
        Oh, you mean actually add cueread INTO musicbrainz?
      • 2007-07-10 19122, 2007

      • FauxFaux
        Yeah.
      • 2007-07-10 19136, 2007

      • acid2_
        I don't see why not, dunno where you'd add it though
      • 2007-07-10 19139, 2007

      • BrianFreud
        try to match unknown releases by overall times list, rather than just puid
      • 2007-07-10 19157, 2007

      • FauxFaux
        I'd add it to the release-search web-service, and maybe the "other searches" page.
      • 2007-07-10 19111, 2007

      • FauxFaux
        (probably)
      • 2007-07-10 19128, 2007

      • acid2_
        FauxFaux: Give me the code then :)
      • 2007-07-10 19141, 2007

      • acid2_
        Might as well have a play around in Perl :)
      • 2007-07-10 19112, 2007

      • BrianFreud
        luks: Does PicardQT have any support for global variables, addressable from say taggerscript?
      • 2007-07-10 19116, 2007

      • acid2 has quit
      • 2007-07-10 19124, 2007

      • acid2_
        acid2_ is now known as aCiD2
      • 2007-07-10 19155, 2007

      • FauxFaux
        There's really nothing more than a perl regex and some string concatenation, http://faux.no-ip.biz/svn/mbreports/cueread2.php. I'll have to go dig out the sql. :)
      • 2007-07-10 19156, 2007

      • luks
        BrianFreud: no
      • 2007-07-10 19113, 2007

      • FauxFaux
        Oh hi luks.
      • 2007-07-10 19114, 2007

      • aCiD2
        Cool, looking now FauxFaux
      • 2007-07-10 19135, 2007

      • luks isn't really here :)
      • 2007-07-10 19150, 2007

      • BrianFreud
        I'm imagining creating a matrix on file load, [releaseid, tracknumber, tracktime]. If I ever can get the tracktimes thing patched for the server, it could be used for then submitted a release's worth of times, but also would be good for FauxFaux's lookup function
      • 2007-07-10 19149, 2007

      • FauxFaux
        Yeah!
      • 2007-07-10 19155, 2007

      • ruaok
        ohhh a patch.
      • 2007-07-10 19159, 2007

      • ruaok hugs aCiD2
      • 2007-07-10 19104, 2007

      • aCiD2
        haha
      • 2007-07-10 19110, 2007

      • aCiD2
        thank you rob :P
      • 2007-07-10 19112, 2007

      • BrianFreud
        hmmmm
      • 2007-07-10 19118, 2007

      • ruaok
        :-)
      • 2007-07-10 19118, 2007

      • aCiD2
        I deleted one line, not really hug worthy lol
      • 2007-07-10 19129, 2007

      • BrianFreud
        FauxFaux: Actually, you may be able to do it anyhow
      • 2007-07-10 19133, 2007

      • aCiD2
        (but learn more about how the web service works, which is a good thing :)
      • 2007-07-10 19133, 2007

      • FauxFaux
        I guess it'd be piss-easy in the actual picard code, if I could get the damn thing to build. :P
      • 2007-07-10 19136, 2007

      • ruaok takes the hug back
      • 2007-07-10 19139, 2007

      • aCiD2
        hah
      • 2007-07-10 19105, 2007

      • aCiD2
        FauxFaux: That script has millisecond precision?
      • 2007-07-10 19133, 2007

      • BrianFreud
        the add cluster as release code already pulls all the info from a cluster - you'd just want to do something similar, just pulling the track times and track numbers, then submitting it to a lookup...
      • 2007-07-10 19153, 2007

      • FauxFaux
        Cue files have the microseconds in, and musicbrainz is stored to millisecond, aCiD2. Version 1 of that was trying to guess the discids, whcih is a lot faster without the CACHE. :p
      • 2007-07-10 19104, 2007

      • aCiD2
        :)
      • 2007-07-10 19125, 2007

      • aCiD2
        Man that's some hardcore SQL (to me)... I should learn more SQL :)
      • 2007-07-10 19139, 2007

      • aCiD2
        I never learnt joins... I just kinda stopped there and let Django handle it for me :P
      • 2007-07-10 19153, 2007

      • BrianFreud
        The two problems you could run into: duplicated track #'s (causes issues now too with the ACaR script) and non-extant track # tags
      • 2007-07-10 19102, 2007

      • FauxFaux
        The sql in cueread2 is just a select and grabbing the artist/release name from the id, the nasty stuff is just coming. :p
      • 2007-07-10 19111, 2007

      • ruaok
        aCiD2: go ahead and commit that patch yourself. go get the credit you deserve. :)
      • 2007-07-10 19118, 2007

      • ruaok
        the patch looks good, thanks for catching that.
      • 2007-07-10 19121, 2007

      • aCiD2
        haha, ok :)
      • 2007-07-10 19136, 2007

      • aCiD2
        I subscribed to mb-bugs, so I'm trying to catch the easy stuff as it comes
      • 2007-07-10 19143, 2007

      • aCiD2
        cause I know you lot the hard stuff ;)
      • 2007-07-10 19144, 2007

      • hawke_ joined the channel
      • 2007-07-10 19101, 2007

      • BrianFreud
        lol, I subscribed to that list just so I know what the latest thing to avoid doing is :D
      • 2007-07-10 19118, 2007

      • hawke has quit
      • 2007-07-10 19127, 2007

      • aCiD2
        ruaok: should I commit to trunk, or the RELEASE branch?
      • 2007-07-10 19150, 2007

      • ruaok
        trunk
      • 2007-07-10 19107, 2007

      • aCiD2
        ok
      • 2007-07-10 19115, 2007

      • ruaok
        its almost time to start thinking about another release.
      • 2007-07-10 19141, 2007

      • BrianFreud can't wait... annotation bug fixed!
      • 2007-07-10 19148, 2007

      • aCiD2
        lol
      • 2007-07-10 19158, 2007

      • ruaok plans to finish the tagging branch and then go on a bug hunt.
      • 2007-07-10 19152, 2007

      • BrianFreud
        ruoak: you'd be the one to ask... internally, for the pages that can accept mbids in the url, how do those pages convert the mbids to rowids?
      • 2007-07-10 19118, 2007

      • aCiD2
        BrianFreud: GIve me an example page, and I'll take a look?
      • 2007-07-10 19129, 2007

      • aCiD2
        not that it stops rob answering, but it's one more thing for me to learn :)
      • 2007-07-10 19126, 2007

      • FauxFaux
        BrianFreud: It's just in the database..
      • 2007-07-10 19148, 2007

      • BrianFreud hunts down ruaok's dev post a while back
      • 2007-07-10 19152, 2007

      • FauxFaux
        album { id integer, gid 36characterstring, name string, etc. }
      • 2007-07-10 19127, 2007

      • FauxFaux
        aCiD2: Refresh for s(low)ql.
      • 2007-07-10 19113, 2007

      • ruaok
        (don't look now aCiD2 )
      • 2007-07-10 19126, 2007

      • aCiD2
        ruaok: ha
      • 2007-07-10 19128, 2007

      • ruaok
        BrianFreud: we load the object in question from the DB.
      • 2007-07-10 19130, 2007

      • aCiD2
        FauxFaux: 20 minutes!?
      • 2007-07-10 19134, 2007

      • ruaok
        then we have teh rowid at hand
      • 2007-07-10 19137, 2007

      • FauxFaux
        aCiD2: Heh heh heh.
      • 2007-07-10 19159, 2007

      • aCiD2
        actually, go ahead and change the script to use that... I'll let my friends on digg know >:)
      • 2007-07-10 19114, 2007

      • FauxFaux
        Only needs to be once every so often. :p
      • 2007-07-10 19124, 2007

      • BrianFreud
        ok, under "Changes mainly of interest to MusicBrainz Programmers" at http://blog.musicbrainz.org/archives/2003/09/serv…
      • 2007-07-10 19135, 2007

      • BrianFreud
        any of those pages already support mbid in the url
      • 2007-07-10 19107, 2007

      • BrianFreud
        To pull off a "add times from release" plugin, as I see it, I need to patch editall.html to support time (and I guess trackname) edits in the url, but also add mbid -> release rowid lookup support, as the mbid is avail to picard, but the rowid is not
      • 2007-07-10 19121, 2007

      • luks
        aCiD2: about the patch/bug... please only close tickets once is the patch in svn, otherwise they will get lost
      • 2007-07-10 19133, 2007

      • aCiD2
        Oh ok
      • 2007-07-10 19145, 2007

      • aCiD2
        thanks luks, won't make that mistake again :)
      • 2007-07-10 19154, 2007

      • luks
        I've also created http://bugs.musicbrainz.org/report/33 a couple days ago
      • 2007-07-10 19137, 2007

      • aCiD2
        BrianFreud: Imo, things shouldn't be using rowids
      • 2007-07-10 19137, 2007

      • BrianFreud
        I'm guessing I can copy the track/name edit in url code from edit.html, and the if/elseif/else from http://bugs.musicbrainz.org/browser/mb_server/tru… for the mbid -> rowid lookup
      • 2007-07-10 19147, 2007

      • aCiD2
        I mean, that's what global unique id's are for
      • 2007-07-10 19117, 2007

      • BrianFreud
        yes, well, editall has no support for either, but edit.html has support for edits in urls - but only if addressed via rowid, not mbid
      • 2007-07-10 19137, 2007

      • aCiD2
        Ah, I see
      • 2007-07-10 19144, 2007

      • aCiD2
        well http://bugs.musicbrainz.org/browser/mb_server/tru… is what you want, I think then
      • 2007-07-10 19121, 2007

      • aCiD2
        and make sure you add $mbid to the %args in editall
      • 2007-07-10 19134, 2007

      • BrianFreud
        that would give me $mbid=GUID support in the url?
      • 2007-07-10 19142, 2007

      • aCiD2
        I think so
      • 2007-07-10 19114, 2007

      • BrianFreud loves trying to code in a language he doesn't know :D
      • 2007-07-10 19146, 2007

      • aCiD2
        haha
      • 2007-07-10 19153, 2007

      • aCiD2
        BrianFreud: I recommend you get a copy of Learning Perl btw
      • 2007-07-10 19103, 2007

      • aCiD2
        I'm reading it atm, while it's a bit light for me having coded before, it's a good intro
      • 2007-07-10 19122, 2007

      • BrianFreud
        though I'm guessing this might be better if I want to look up a release, and not a track: http://bugs.musicbrainz.org/browser/mb_server/tru…
      • 2007-07-10 19146, 2007

      • murdos
        luks: how does work the new report (http://bugs.musicbrainz.org/report/33)? does it checks "patch" in keyword and/or summary?
      • 2007-07-10 19104, 2007

      • luks
        exactly
      • 2007-07-10 19117, 2007

      • luks
        (t.summary LIKE '%[patch]%' OR t.summary LIKE '%patch %' OR t.keywords like '%patch%')
      • 2007-07-10 19156, 2007

      • murdos
        k. (I've just found the sql query export ;) )
      • 2007-07-10 19120, 2007

      • FauxFaux
        Lies, learning perl is for girls, real men write php then transliterate it. :p
      • 2007-07-10 19146, 2007

      • FauxFaux
        I believe you're allowed to use a language that isn't php, but I wouldn't recommend it.
      • 2007-07-10 19110, 2007

      • aCiD2
        errr PHP?!
      • 2007-07-10 19115, 2007

      • aCiD2
        hell no
      • 2007-07-10 19124, 2007

      • aCiD2 pokes php with a barge pole
      • 2007-07-10 19144, 2007

      • FauxFaux
        D:
      • 2007-07-10 19150, 2007

      • aCiD2
        python <3
      • 2007-07-10 19138, 2007

      • ruaok
      • 2007-07-10 19147, 2007

      • ruaok
        I'm getting the cannot load tags message there.
      • 2007-07-10 19159, 2007

      • ruaok
        last time it was JSON.pm, but I fixed that.
      • 2007-07-10 19104, 2007

      • ruaok
        how can I debug what the issue is?
      • 2007-07-10 19107, 2007

      • luks checks
      • 2007-07-10 19121, 2007

      • ruaok
        there are no accounts on that machine and the email verification doesnt' work.
      • 2007-07-10 19123, 2007

      • ruaok
        sorry. :)
      • 2007-07-10 19140, 2007

      • ruaok
        but I get no messages in the error log
      • 2007-07-10 19143, 2007

      • luks
        that might be the problem
      • 2007-07-10 19144, 2007

      • luks
      • 2007-07-10 19109, 2007

      • luks
        but this probably shouldn't depend on the email
      • 2007-07-10 19151, 2007

      • luks
        - $m->comp("/comp/checkloggedin", 1, 1)
      • 2007-07-10 19157, 2007

      • luks
        + $m->comp("/comp/checkloggedin", 1, 1, undef, 1)
      • 2007-07-10 19105, 2007

      • luks
      • 2007-07-10 19109, 2007

      • luks
        I think that should fix it
      • 2007-07-10 19119, 2007

      • ruaok
        logins are currently plain broken. :-(
      • 2007-07-10 19124, 2007

      • ruaok works to fix that
      • 2007-07-10 19121, 2007

      • SerialNumber
        SerialNumber has joined #MusicBrainz
      • 2007-07-10 19127, 2007

      • aCiD2
        on all servers, or just yours?
      • 2007-07-10 19155, 2007

      • ruaok
        my dev server
      • 2007-07-10 19127, 2007

      • BrianG has quit