#musicbrainz-devel

/

      • ruaok
        its interesting… its hard to find releases that we dont have a match for, that is also in Spotify.
      • with only 250k matching releases.
      • Gentlecat
      • spinza joined the channel
      • ruaok
        thanks.
      • ok, the add code works. finally.
      • (took long enough)
      • now to do the add vote and then to deploy this to the new server.
      • once I have it running, I’ll start deploying CB as well.
      • do you think there is any point to ever voting FOR a mapping?
      • what would we ever do with the knowledge that a mapping is correct?
      • Gentlecat
        I don't think people are going to vote for that much
      • ruaok
        exactly.
      • Gentlecat
        maybe just add a small link under embed "Spotify link is incorrect" or something like that
      • ruaok
        the only point to vote is if a match is wrong.
      • yes, exactly.
      • ok, I’ll just skip the vote type column.
      • there is only one reason to record a vote and its to indicate a bad match.
      • Gentlecat
        right
      • JesseW joined the channel
      • JesseW joined the channel
      • spinza joined the channel
      • spinza joined the channel
      • ruaok
        ok, voting works.
      • I’m going to dump the db and spool it up on sakura.
      • Nyanko-sensei joined the channel
      • spinza_ joined the channel
      • so, I can work on the deployment bits for mbspotify.
      • Gentlecat
        I might need some help with nginx setup since I never done anything with python there
      • if you want me to do that
      • ruaok
        don’t sweat it. I can do that.
      • I’ve got things laying around that I can crib from.
      • ianmcorvidae notes to use uwsgi-pass
      • not sure if the domain transfer will go through in time for our demo tomorrow.
      • ianmcorvidae
        (look at what I did for geordi on rocko, at least in part)
      • ruaok
        ianmcorvidae: yep.
      • ianmcorvidae: actually, can you also please setup critiquebrainz.musicbrainz.org to also go to sakura:80?
      • ianmcorvidae
        sure
      • ruaok
        that will be my failback in case we don’t have the finished domain transfer.
      • they claimed it might take 5-8 days. ;(
      • I should’ve thought about this last week apparently.
      • Gentlecat
        so mbspotify is complete, right?
      • ruaok
        in theory, yes.
      • Gentlecat
        I'll finish integration with CB then
      • ruaok
        let me update the gist from earlier
      • Gentlecat
        which one?
      • I think I can just look at the sources
      • ruaok
      • faster this way.
      • it would be nice to finish this today.
      • Gentlecat
        mapping stuff and deployment?
      • ruaok
        especially since you’re going to be sleeping right up until the hack day is over.
      • whatever we can get done.
      • but, yeah.
      • ruaok gets the DNS change underway
      • ianmcorvidae
        anything running on sakura:80 right now? I have the lenny bits up
      • ruaok
        I dont think so.
      • critiquebrainz.musicbrainz.org has been ordered from DWNI
      • we’ll need to use that for the time being.
      • let me get nginx up on sakura
      • ianmcorvidae: I’ve nginx answering on port 80, but it doesn’t seem to work
      • 403 forbidden
      • ianmcorvidae
        yeah, I'm not sure where that's coming from
      • ruaok goes to hunt a slice of pizza
      • spinza joined the channel
      • Gentlecat
        looks like someone is having fun with my box :)
      • ruaok: we probably want some kind of confirmation dialog for mapping submission, right?
      • ruaok
        I think that is probably a good idea, yeah.
      • Gentlecat: do you remeber the command line you used to restore the .sql file?
      • ianmcorvidae
        how did you dump it?
      • Gentlecat
        which one?
      • ruaok
        pg_dump
      • ianmcorvidae
        pg_dump has its own format, if you used that, which requires using pg_restore
      • (you can dump as sql too, but usually the special format is faster)
      • ruaok
        pg_restore: [archiver] input file does not appear to be a valid archive
      • :-(
      • I did pg_dump without arguments.
      • ianmcorvidae
        no <
      • Gentlecat
        I unpacked it and piped into psql
      • ruaok
        ah, thanks ianmcorvidae
      • same error.
      • ianmcorvidae
        then I'm guessing the input file isn't valid, heh
      • oh, but it's a plain-sql one
      • ruaok
        I’ll just redirect it hen
      • into a hen, that is.
      • Gentlecat
        I can make one for you
      • ruaok
        squawk.
      • ianmcorvidae
        check the file and see if it includes the creating of the DB, if so you can just psql it in, anyway
      • ruaok
        Gentlecat: no worries.
      • I’ll create it by hand
      • ianmcorvidae
        (I've mostly only ever used pg_dump --schema-only, for testing schema change stuff, so :P)
      • ruaok
        ok, that worked. wicked fast too.
      • ianmcorvidae
        I can't imagine that DB is exactly huge
      • Gentlecat
        about 250k rows iirc
      • ianmcorvidae
        yeah, heh, 264.5k rows in one table
      • nothing
      • Gentlecat
        ruaok: AssertionError: View function mapping is overwriting an existing endpoint function: add
      • ianmcorvidae
        yup, two add functions
      • heh
      • ruaok
        lame copy pasta
      • fix pushed
      • now to deploy this, then to deploy CB itself. then that will probably be it for the day.
      • tomorrow, I’ll try and create a userscript to use the mapping layer for mb.org
      • Gentlecat
        CB might me a good source of mapping submissions after some time
      • ruaok
        any preferences on where virtual envs should live on the server?
      • yeah.
      • Gentlecat
        good way to fill the DB
      • ruaok
        but we have enough for a good demo now.
      • which is all that is needed to get Spotify’s attention.
      • Gentlecat
        for sure
      • ruaok
        which, I think, we already have.
      • ianmcorvidae
        considering they invited us to have a summit at their headquarters I imagine they're fairly aware of us :P
      • Gentlecat
        did they?
      • ianmcorvidae
        at the last summit, yeah XD
      • Gentlecat
        wow, that's cool
      • ruaok
        ianmcorvidae: sure they are, but they are not moving fast enough on the MBIDs bit.
      • ianmcorvidae
        yeah, that much is fair
      • ruaok
        I’m trying to create more external pressure.
      • separate user account for mbspotify and critiquebrainz?
      • then /home/<user>/ve ?
      • user name mbspotify and cb?
      • or spell out cb?
      • are two letter usernames even allowed?
      • ianmcorvidae
        I'd spell it out anyway
      • Gentlecat
        I'm going to send user ids on CB
      • I believe that's all we have there apart from user id on musicbrainz
      • oh, you are talking about different thing
      • sorry
      • ruaok
        :)
      • ok, long names used
      • Gentlecat
      • ianmcorvidae
        don't use % with cur.execute
      • you're not actually using string formatting
      • should probably be ('''SELECT ...''', id_tuple) other than I'm not completely sure how it deals with tuples
      • if you have the proper array options set, mbid = any(%s) should behave properly though
      • (related, I hate the psycopg2 decided to quasi-override the string formatting syntax, since *actually* using string formatting there is a big sql injection risk)
      • ruaok
        habit, thanks for catching that.
      • ruaok thinks to the doc not about not ever doing that, even at gunpoint.
      • ianmcorvidae
        yeah, heh
      • ruaok
        at least I got right elsewhere.
      • ianmcorvidae
        I really wish they'd use a different syntax so that it'd fail harder if you do it wrong
      • ruaok
        +1 ianmcorvidae
      • Gentlecat: pull
      • (my finger)
      • which is interesting too. when you to a pull, the bits come from two floors above me.
      • github is an odd, crazy, opulent place.
      • ianmcorvidae: did you see the pic I posted to FB?
      • ianmcorvidae
        I did
      • I have a low opinion of github's self-indulgence