#musicbrainz-devel

/

      • ocharles

        • 2012-09-19 26304, 2012

      • ocharles
        :D

        • 2012-09-19 26315, 2012

      • ocharles
        bayesian classifiers ftw

        • 2012-09-19 26324, 2012

      • voiceinsideyou joined the channel

        • 2012-09-19 26301, 2012

      • ocharles
        Gotta head off now, but I might try this again more seriously with better edit note sampling

        • 2012-09-19 26345, 2012

      • MBJenkins
        Project musicbrainz-server_beta build #118: STILL FAILING in 22 min: http://ci.musicbrainz.org/job/musicbrainz-server_…

        • 2012-09-19 26346, 2012

      • MBJenkins
        warp: Upgrade Relationship Editor tests to use QUnit.equal instead of QUnit.equals.

        • 2012-09-19 26338, 2012

      • warp
        still :(

        • 2012-09-19 26347, 2012

      • djce joined the channel

        • 2012-09-19 26338, 2012

      • jdamcd joined the channel

        • 2012-09-19 26354, 2012

      • kepstin-work joined the channel

        • 2012-09-19 26310, 2012

      • nikki
        I'm getting errors in my error console on beta

        • 2012-09-19 26321, 2012

      • nikki
        http://pastebin.com/b4jY8jEU that's from opera next (no user scripts, so they can't be the problem)

        • 2012-09-19 26336, 2012

      • Leftmost joined the channel

        • 2012-09-19 26311, 2012

      • uk
        Relationship editor isn't working for me, no recordings are shown (tried several releases).

        • 2012-09-19 26345, 2012

      • Prophet5 joined the channel

        • 2012-09-19 26346, 2012

      • ruaok
        heh. I was just chatting with spotify and when I mentioned to them that we're about to release a JSON api, one of them said: "oh, you're joining the 21st century?"

        • 2012-09-19 26349, 2012

      • ruaok
        snap. :)

        • 2012-09-19 26352, 2012

      • CallerNo6 joined the channel

        • 2012-09-19 26359, 2012

      • jdamcd
        does somebody know if it's a bug that release groups in a browse request are no longer returned in first-release-date order?

        • 2012-09-19 26305, 2012

      • jdamcd
        or is that intended?

        • 2012-09-19 26351, 2012

      • ruaok
        sounds like a bug, but I am not sure.

        • 2012-09-19 26354, 2012

      • ruaok
        nikki: ping

        • 2012-09-19 26342, 2012

      • Freso joined the channel

        • 2012-09-19 26307, 2012

      • bitmap
        ocharles, warp: the js file for the relationship editor is empty on beta: http://beta.musicbrainz.org/static/0813e9b890f519…

        • 2012-09-19 26313, 2012

      • bitmap
        any idea why that is?

        • 2012-09-19 26330, 2012

      • nikki
        ruaok: pong

        • 2012-09-19 26336, 2012

      • warp
        bitmap: probably has a trailing comma then :)

        • 2012-09-19 26337, 2012

      • warp
        21:28 warp [at bullet-bill]:~/code/mb/root/static/scripts/relationship-editor (beta)$ closure-verify *js

        • 2012-09-19 26340, 2012

      • warp
        Dialog.js:128: ERROR - Parse error. Internet Explorer has a non-standard intepretation of trailing commas. Arrays will have the wrong length and objects will not parse at all. };

        • 2012-09-19 26344, 2012

      • warp
        bitmap: yep. I'll fix them on beta.

        • 2012-09-19 26318, 2012

      • bitmap
        ooh, didn't know those would break the minifier

        • 2012-09-19 26324, 2012

      • bitmap
        warp: thanks

        • 2012-09-19 26345, 2012

      • warp
        bitmap: well, they break in internet explorer.

        • 2012-09-19 26351, 2012

      • warp
        bitmap: and that is why the minifier complains.

        • 2012-09-19 26324, 2012

      • warp
        bitmap: it should be better now.

        • 2012-09-19 26355, 2012

      • bitmap
        warp: it seems to have generated a new js file, but it's still empty :(

        • 2012-09-19 26304, 2012

      • warp
        bitmap: hrm

        • 2012-09-19 26321, 2012

      • warp
        yeah, I missed one.

        • 2012-09-19 26343, 2012

      • warp
        bitmap: ok, how about now? :)

        • 2012-09-19 26307, 2012

      • bitmap
        it works! \o/

        • 2012-09-19 26314, 2012

      • bitmap
        thanks :)

        • 2012-09-19 26340, 2012

      • warp
        you're welcome.

        • 2012-09-19 26340, 2012

      • uk goes adding some relationships.

        • 2012-09-19 26333, 2012

      • adhawkins
        Don't suppose there's anyone here with access to a cross-compilation toolchain who can test something for me?

        • 2012-09-19 26357, 2012

      • MBJenkins
        Project musicbrainz-server_beta build #119: STILL FAILING in 25 min: http://ci.musicbrainz.org/job/musicbrainz-server_…

        • 2012-09-19 26358, 2012

      • MBJenkins
        warp: Remove trailing commas in relationship-editor/Dialog.js

        • 2012-09-19 26310, 2012

      • bitmap
        warp: also, I committed a fix for the QUnit tests (MBS-5321) to my relationship-editor-gsoc branch

        • 2012-09-19 26310, 2012

      • mb-chat-logger
        http://tickets.musicbrainz.org/browse/MBS-5321

        • 2012-09-19 26338, 2012

      • bitmap
        apparently if you pass it a number/undefined to print, it no longer converts it to a string for you o_O

        • 2012-09-19 26311, 2012

      • djce joined the channel

        • 2012-09-19 26353, 2012

      • MBJenkins
        warp: Remove trailing commas in relationship-editor/Entity.js

        • 2012-09-19 26305, 2012

      • jesus2099 joined the channel

        • 2012-09-19 26349, 2012

      • uk
        bitmap: What to do with bug reports/improvement requests to the relationship editor?

        • 2012-09-19 26313, 2012

      • bitmap
        uk: I think they should go on jira now

        • 2012-09-19 26314, 2012

      • bitmap
        assign to me (Michael Wiencek)

        • 2012-09-19 26320, 2012

      • uk
        Affected version is Beta, or 2012-10-01?

        • 2012-09-19 26354, 2012

      • bitmap
        Beta, I suppose

        • 2012-09-19 26335, 2012

      • ianmcorvidae
        hm

        • 2012-09-19 26347, 2012

      • ianmcorvidae
        Beta is usually for things that should be fixed before it ships

        • 2012-09-19 26352, 2012

      • ianmcorvidae
        so I'd say don't set a fix version at all

        • 2012-09-19 26322, 2012

      • hawke_
        affected version. :-)

        • 2012-09-19 26349, 2012

      • ianmcorvidae
        that we don't use at all :P

        • 2012-09-19 26307, 2012

      • ianmcorvidae
        and setting it to beta will just mean it's inaccurate in two weeks :P

        • 2012-09-19 26336, 2012

      • nikki
        if we don't use it, we should just remove it from the screen when you report things

        • 2012-09-19 26348, 2012

      • ianmcorvidae
        probably :/

        • 2012-09-19 26300, 2012

      • nikki
        we should probably also turn off time tracking if we're not using that now either

        • 2012-09-19 26306, 2012

      • ijabz joined the channel

        • 2012-09-19 26308, 2012

      • nikki
        jira's complicated enough without having extra stuff we don't need :P

        • 2012-09-19 26309, 2012

      • warp
        bitmap: great, I'll merge it in.

        • 2012-09-19 26301, 2012

      • bitmap
        yay, first relationship editor bug report

        • 2012-09-19 26340, 2012

      • Prophet5 joined the channel

        • 2012-09-19 26354, 2012

      • nikki
        oh well, I reported the error at http://tickets.musicbrainz.org/browse/MBS-5325

        • 2012-09-19 26318, 2012

      • uk
        bitmap: That was quick. :)

        • 2012-09-19 26315, 2012

      • bitmap
        yeah, it was a small (and dumb) mistake. I'd never tested auto-editing with it

        • 2012-09-19 26301, 2012

      • bitmap
        I'm not sure how the beta process works, should I nag on someone for every bug fix I need merged?

        • 2012-09-19 26357, 2012

      • MBJenkins
        Project musicbrainz-server_beta build #121: STILL FAILING in 22 min: http://ci.musicbrainz.org/job/musicbrainz-server_…

        • 2012-09-19 26358, 2012

      • MBJenkins
        mwtuea: Fix two relationship editor QUnit tests

        • 2012-09-19 26357, 2012

      • ianmcorvidae
        search servers are doing something weird

        • 2012-09-19 26350, 2012

      • ruaok
        what are they doing?

        • 2012-09-19 26355, 2012

      • ianmcorvidae
        I'm getting read timeouts from rika

        • 2012-09-19 26302, 2012

      • ianmcorvidae
        and sometimes they aren't loading anything on the main site

        • 2012-09-19 26311, 2012

      • ianmcorvidae
        er, from beta

        • 2012-09-19 26314, 2012

      • ianmcorvidae
        though beta seems to be working now

        • 2012-09-19 26328, 2012

      • ruaok
        main site seems fine

        • 2012-09-19 26352, 2012

      • ianmcorvidae
        from beta I've just gotten a "sorry, we got an error we don't understand" error

        • 2012-09-19 26321, 2012

      • ianmcorvidae
        and it's back on rika

        • 2012-09-19 26322, 2012

      • ianmcorvidae
        weird

        • 2012-09-19 26335, 2012

      • uk
        bitmap: Yes, I think warp or ocharles have to merge any fixes into beta. Not sure if reviews are required.

        • 2012-09-19 26324, 2012

      • ianmcorvidae
        I'm able to merge things now, I don't really know when/if I should though yejt :)

        • 2012-09-19 26314, 2012

      • ruaok
        I love how google asks to translate the beta.mb.org pages.

        • 2012-09-19 26322, 2012

      • ianmcorvidae
        haha

        • 2012-09-19 26324, 2012

      • ianmcorvidae
        yup

        • 2012-09-19 26329, 2012

      • ruaok
        and at least for the spanish on it does quite a good job if you say yes.

        • 2012-09-19 26341, 2012

      • ruaok
        damn, we missed having pirate for speak like a pirate day.

        • 2012-09-19 26345, 2012

      • ruaok
        oh well. next year.

        • 2012-09-19 26317, 2012

      • bitmap
        ianmcorvidae: okay, well if you decide it's okay to merge https://github.com/mwiencek/musicbrainz-server/co… it would allow auto-editors to actually auto-edit :)

        • 2012-09-19 26351, 2012

      • ianmcorvidae
        bitmap: what branch is that on?

        • 2012-09-19 26357, 2012

      • bitmap
        relationship-editor-gsoc

        • 2012-09-19 26357, 2012

      • ianmcorvidae
        man, we have a lot of artists with 'cream' in their name

        • 2012-09-19 26301, 2012

      • ianmcorvidae
        anyway, I'll merge that now

        • 2012-09-19 26344, 2012

      • ianmcorvidae
        hm

        • 2012-09-19 26348, 2012

      • ianmcorvidae
        apparently pushing didn't work?

        • 2012-09-19 26349, 2012

      • ianmcorvidae confused

        • 2012-09-19 26306, 2012

      • ianmcorvidae
        bah

        • 2012-09-19 26311, 2012

      • ianmcorvidae
        what is beta using as its upstream, this is silly

        • 2012-09-19 26325, 2012

      • ianmcorvidae
        bah

        • 2012-09-19 26334, 2012

      • ianmcorvidae
        had to manually do the pull but whatever

        • 2012-09-19 26357, 2012

      • ianmcorvidae
        bitmap: it's pushed and deployed

        • 2012-09-19 26318, 2012

      • bitmap
        awesome, thanks

        • 2012-09-19 26346, 2012

      • tommy_o joined the channel

        • 2012-09-19 26323, 2012

      • tommy_o
        I'm having trouble with a bit of plugin code not properly saving tracks on an album. can someone take a look at it on pastebin?

        • 2012-09-19 26343, 2012

      • tommy_o
        I

        • 2012-09-19 26354, 2012

      • tommy_o
        I'm just modifying an existing plugin to save then do the regular action

        • 2012-09-19 26318, 2012

      • ijabz_ joined the channel

        • 2012-09-19 26329, 2012

      • bitmap
        tommy_o: are you going to link to the pastebin? :)

        • 2012-09-19 26324, 2012

      • ianmcorvidae
        meh, still getting random search server errors from rika

        • 2012-09-19 26341, 2012

      • tommy_o
        http://pastebin.com/P4L7bzGH

        • 2012-09-19 26346, 2012

      • tommy_o
        shoot, sorry -- got distracted

        • 2012-09-19 26331, 2012

      • bitmap
        I'm guessing since it saves the files in a separate thread you're removing the files from picard (with remove_album) before they

        • 2012-09-19 26334, 2012

      • bitmap
        're saved

        • 2012-09-19 26338, 2012

      • tommy_o
        I was looking at the code from the "Autosave Complete Albums" plugin, but the code is not so great (https://raw.github.com/encukou/picard-plugins/mas… )

        • 2012-09-19 26358, 2012

      • tommy_o
        oh, didnt realize it was saving in a separate thread

        • 2012-09-19 26335, 2012

      • tommy_o
        sorry, I don't write much more than procedural utility scripts in python

        • 2012-09-19 26338, 2012

      • bitmap
        the autosave.py code has its way of dealing with that, so it might be best to copy that

        • 2012-09-19 26307, 2012

      • tommy_o
        oh ok, thats why it does that

        • 2012-09-19 26343, 2012

      • hawke_ joined the channel

        • 2012-09-19 26337, 2012

      • MBJenkins
        Project musicbrainz-server_beta build #122: STILL FAILING in 25 min: http://ci.musicbrainz.org/job/musicbrainz-server_…

        • 2012-09-19 26338, 2012

      • MBJenkins
        mwtuea: Add auto-editor checkbox to rel editor

        • 2012-09-19 26318, 2012

      • ijabz joined the channel

        • 2012-09-19 26314, 2012

      • bitmap
        uk: that's odd, it works fine on my sandbox

        • 2012-09-19 26306, 2012

      • uk
        Not on beta, though. :)

        • 2012-09-19 26336, 2012

      • bitmap
        I wonder if the javascript hasn't been updated

        • 2012-09-19 26314, 2012

      • bitmap
        ctrl+f as_auto_editor 0 of 0

        • 2012-09-19 26353, 2012

      • bitmap
        ianmcorvidae: does the js have to be re-minified manually or something?

        • 2012-09-19 26322, 2012

      • uk files another ticket while waiting. ;-)

        • 2012-09-19 26307, 2012

      • bitmap
        ouch, the language switcher breaks the rel editor too

        • 2012-09-19 26318, 2012

      • Freso joined the channel

        • 2012-09-19 26331, 2012

      • ruaok
        ianmcorvidae: on MBH-279 -- did djce give you interim access to the our private git repos?

        • 2012-09-19 26331, 2012

      • mb-chat-logger
        http://tickets.musicbrainz.org/browse/MBH-279

        • 2012-09-19 26326, 2012

      • ianmcorvidae
        ruaok: yeah, I'm set as an admin

        • 2012-09-19 26332, 2012

      • ruaok
        ok.

        • 2012-09-19 26341, 2012

      • ianmcorvidae
        bitmap: the deployment scripts recompile resources

        • 2012-09-19 26357, 2012

      • ruaok
        because there are so many things on my plate for the coming weeks/months that I see no immediate chance for me to finish the github backup.

        • 2012-09-19 26318, 2012

      • ianmcorvidae
        yup

        • 2012-09-19 26329, 2012

      • ianmcorvidae
        I'll do the DNS thing, been meaning to look at that

        • 2012-09-19 26344, 2012

      • ruaok
        kewl.

        • 2012-09-19 26339, 2012

      • ianmcorvidae
        bitmap: you might check if it's in the beta branch, at least

        • 2012-09-19 26309, 2012

      • ianmcorvidae
        bitmap: the deployment scripts recompile resources but they *were* doing some funky things -- so, if you could check that the correct code made it into the branch, and then we can check the resources

        • 2012-09-19 26305, 2012

      • nikki
        ruaok: what did you want earlier?