#musicbrainz-devel

/

      • reosarevok
        So, I have tickets!
      • 2012-10-01 27552, 2012

      • reosarevok
        :D
      • 2012-10-01 27506, 2012

      • reosarevok
        Quite convenient and only slightly over my budget
      • 2012-10-01 27542, 2012

      • reosarevok
        ruaok will be happy to know I am flying Lufthansa after all :)
      • 2012-10-01 27553, 2012

      • MBJenkins
        Project libcoverart build #6: SUCCESS in 19 sec: http://ci.musicbrainz.org/job/libcoverart/6/
      • 2012-10-01 27521, 2012

      • dinog joined the channel
      • 2012-10-01 27539, 2012

      • ruaok joined the channel
      • 2012-10-01 27540, 2012

      • bitmap
        I'm getting nothing but 504 gateway timeouts on beta
      • 2012-10-01 27516, 2012

      • ianmcorvidae
        that would seem like a problem
      • 2012-10-01 27531, 2012

      • ianmcorvidae
        I got a 502, but same idea :P
      • 2012-10-01 27534, 2012

      • ianmcorvidae
        huh
      • 2012-10-01 27545, 2012

      • ianmcorvidae
        beta somehow got reverted to where it was the beginning of this week?
      • 2012-10-01 27553, 2012

      • ianmcorvidae
        ocharles, warp: either of you awake?
      • 2012-10-01 27559, 2012

      • ianmcorvidae
        er, or someone checked out master
      • 2012-10-01 27533, 2012

      • ianmcorvidae
        meh
      • 2012-10-01 27539, 2012

      • ianmcorvidae
        bitmap: beta should be back, I don't know what happened (ignore my last 5 lines or so :P)
      • 2012-10-01 27511, 2012

      • bitmap
        yeah, works now for me, thanks :)
      • 2012-10-01 27508, 2012

      • ianmcorvidae
        the servers got killed somehow
      • 2012-10-01 27534, 2012

      • ianmcorvidae
        may have something to do with the big spikes in load average on hobbes though, hm
      • 2012-10-01 27543, 2012

      • warp
        ianmcorvidae: yes.
      • 2012-10-01 27513, 2012

      • warp
        (awake)
      • 2012-10-01 27537, 2012

      • ianmcorvidae
        beta went down for some reason -- because the fabric script merges in master that's happened, so it now has the hotfix from last week
      • 2012-10-01 27545, 2012

      • ianmcorvidae
        I haven't determined any real reason why
      • 2012-10-01 27510, 2012

      • warp
        merging master in beta should be safe.
      • 2012-10-01 27556, 2012

      • ianmcorvidae
        it brought in my fix for shutting up modbot :)
      • 2012-10-01 27502, 2012

      • ianmcorvidae
        so jenkins is running
      • 2012-10-01 27507, 2012

      • ianmcorvidae
        but that shouldn't be an issue
      • 2012-10-01 27531, 2012

      • MBJenkins
        * ianmcorvidae: MBS-5369: make sure that EditReleaseLabel edits to remove labels still work
      • 2012-10-01 27532, 2012

      • MBJenkins
        * ianmcorvidae: MBS-5369: Add a test to ensure removing a label name works correctly
      • 2012-10-01 27555, 2012

      • warp
        SUCCESS \o/
      • 2012-10-01 27519, 2012

      • ianmcorvidae
        yup
      • 2012-10-01 27546, 2012

      • ianmcorvidae
        unfortunately we have to revert the release editor or that'd basically be a success for master too
      • 2012-10-01 27512, 2012

      • ianmcorvidae
        er relationship editor
      • 2012-10-01 27513, 2012

      • ianmcorvidae
        heh
      • 2012-10-01 27545, 2012

      • warp
        both the MBS-4855 ticket and your question to mb-i18n are a bit vague
      • 2012-10-01 27546, 2012

      • mb-chat-logger
      • 2012-10-01 27516, 2012

      • ianmcorvidae
        MBS-4822 you mean?
      • 2012-10-01 27517, 2012

      • mb-chat-logger
      • 2012-10-01 27524, 2012

      • warp
        yes
      • 2012-10-01 27531, 2012

      • ianmcorvidae
        so at present we have two strings, e.g. "Artist" and "Artist:"
      • 2012-10-01 27546, 2012

      • ianmcorvidae
        and like wise label, release group, etc.
      • 2012-10-01 27559, 2012

      • ianmcorvidae
        there's a lot of pairs of strings that vary only in terms of punctuation
      • 2012-10-01 27503, 2012

      • ianmcorvidae
        colon and period being the big offenders
      • 2012-10-01 27521, 2012

      • ianmcorvidae
        so the question is would it be okay to keep only "Artist" by adding "{item}:" for adding the colon
      • 2012-10-01 27537, 2012

      • warp
        for the specific case mentioned in the ticket, it seems line 215 is just missing a period.
      • 2012-10-01 27513, 2012

      • ianmcorvidae
        ah, yes
      • 2012-10-01 27515, 2012

      • ianmcorvidae
        for that one that's true
      • 2012-10-01 27517, 2012

      • warp
        (which would then make these two equal)
      • 2012-10-01 27530, 2012

      • ianmcorvidae
        however it's the more general question of "is it okay to simply add a colon to an otherwise already-translated string"
      • 2012-10-01 27555, 2012

      • ianmcorvidae
        i.e. "will you translate 'Artist' differently from 'Artist:' differently from 'Artist.' other than the trailing character"
      • 2012-10-01 27501, 2012

      • warp
        I expect it would not be OK, but I don't speak any languages which use colon differently from english.
      • 2012-10-01 27513, 2012

      • ianmcorvidae
        and additionally "will you translate 'Artist:' differently from 'Release:' (for example)"
      • 2012-10-01 27528, 2012

      • ianmcorvidae
        (looking *only* at the trailing punctuation, in that case)
      • 2012-10-01 27538, 2012

      • ianmcorvidae
        yeah, I'm not sure
      • 2012-10-01 27540, 2012

      • ianmcorvidae
        hence the email :)
      • 2012-10-01 27514, 2012

      • warp
        e.g. for japanese subtitles foo: bar is foo ~bar~, so for all I know they don't use colon for this case either :)
      • 2012-10-01 27534, 2012

      • ianmcorvidae
        ah, but not using colon is fine, so long as they use whatever-they-use the same places we use a colon
      • 2012-10-01 27503, 2012

      • ianmcorvidae
        because they'd then just translate, say, '{title}: {subtitle}' as '{title} ~{subtitle}~'
      • 2012-10-01 27510, 2012

      • warp
        you mean have just the colon as a translateable string?
      • 2012-10-01 27523, 2012

      • ianmcorvidae
        not just the colon, it would include the placement of the thing-to-interpolate
      • 2012-10-01 27526, 2012

      • ianmcorvidae
        but pretty much, yes
      • 2012-10-01 27529, 2012

      • warp nods.
      • 2012-10-01 27536, 2012

      • ianmcorvidae
        the proposed one was for "{item}:" as a translatable string
      • 2012-10-01 27545, 2012

      • ianmcorvidae
        adding a macro 'add_colon' or such
      • 2012-10-01 27548, 2012

      • warp
        right, that seems to be lacking context
      • 2012-10-01 27551, 2012

      • warp
        but that's all the opinion I have on this. I'm only fluent in english/dutch/frisian, and they all use colon the same way.
      • 2012-10-01 27557, 2012

      • ianmcorvidae
        yeah
      • 2012-10-01 27508, 2012

      • ianmcorvidae
        mostly I suspect v6lur or someone will come along and throw a wrench in things
      • 2012-10-01 27516, 2012

      • warp
        :)
      • 2012-10-01 27516, 2012

      • ianmcorvidae
        with our current set of translators it's usually Estonian that makes things hard :P
      • 2012-10-01 27521, 2012

      • warp
        haha
      • 2012-10-01 27502, 2012

      • ianmcorvidae
        (because of the case system, generally)
      • 2012-10-01 27516, 2012

      • ianmcorvidae
        (the romance languages have genders but people are used to working around those -- less so with case languages)
      • 2012-10-01 27538, 2012

      • ianmcorvidae
        can't even imagine how much brokenness we'd get with something that has weirder categories :)
      • 2012-10-01 27502, 2012

      • warp stops reading wikipedia to figure out what "case language" means and continues on the schema change ticket.
      • 2012-10-01 27509, 2012

      • ianmcorvidae
        heh
      • 2012-10-01 27532, 2012

      • ianmcorvidae
      • 2012-10-01 27527, 2012

      • ianmcorvidae
        german also has cases, but the system's a lot less complicated than estonian's
      • 2012-10-01 27522, 2012

      • warp
        ah, then a german example would probably help me understand it better.
      • 2012-10-01 27529, 2012

      • ianmcorvidae
        but for example "{age} ago" doesn't work in german because the preposition you have to use to translate 'ago' changes the thing into a different case than what you'd use for an age alone
      • 2012-10-01 27542, 2012

      • ianmcorvidae
        i.e. 'am 7 Jahren' rather than '7 Jahre'
      • 2012-10-01 27559, 2012

      • ianmcorvidae
        er, not am, vor
      • 2012-10-01 27539, 2012

      • ianmcorvidae
        and the case ending isn't always just 'add n', so you can't just make it 'vor {age}n', which would of course be a gross hack anyway :P
      • 2012-10-01 27552, 2012

      • ianmcorvidae
        I could easily see estonian translating "Artist" and "Artist:" differently, because the context provided by the ':' might change the case
      • 2012-10-01 27550, 2012

      • warp
        Apparantly dutch still has some remnants of that case stuff. (from reading https://nl.wikipedia.org/wiki/Naamval )
      • 2012-10-01 27518, 2012

      • warp
        (that still doesn't help me really grok the concept :)
      • 2012-10-01 27534, 2012

      • Leftmost joined the channel
      • 2012-10-01 27535, 2012

      • Leftmost joined the channel
      • 2012-10-01 27546, 2012

      • ijabz joined the channel
      • 2012-10-01 27514, 2012

      • ruaok joined the channel
      • 2012-10-01 27501, 2012

      • warp
        ocharles: ayt? do we have a standard way to do an upsert/merge?
      • 2012-10-01 27506, 2012

      • warp
        (I've probably asked for that before :)
      • 2012-10-01 27505, 2012

      • jesus2099 joined the channel
      • 2012-10-01 27513, 2012

      • jesus2099
        Did something change on when we receive email notifications of edit notes ?
      • 2012-10-01 27518, 2012

      • jesus2099
        I don’t think I received notification for « I tried to fix it. » in unvotable edit http://musicbrainz.org/edit/19163596
      • 2012-10-01 27524, 2012

      • jesus2099
        maybe I did receive but forgot but I don’t think so
      • 2012-10-01 27529, 2012

      • jesus2099
        Although I did receive on abstained edit http://musicbrainz.org/edit/19111101
      • 2012-10-01 27534, 2012

      • jesus2099
        Previously if you talked in an edit you would receive notifications…
      • 2012-10-01 27535, 2012

      • ijabz joined the channel
      • 2012-10-01 27501, 2012

      • ijabz_ joined the channel
      • 2012-10-01 27532, 2012

      • ijabz
        warp, ping
      • 2012-10-01 27533, 2012

      • stefans joined the channel
      • 2012-10-01 27555, 2012

      • warp
        ijabz: ack
      • 2012-10-01 27516, 2012

      • reosarevok joined the channel
      • 2012-10-01 27533, 2012

      • ocharles
        good morning
      • 2012-10-01 27536, 2012

      • ocharles
        warp: no standard way
      • 2012-10-01 27541, 2012

      • ocharles
        and yes, you've asked before :)
      • 2012-10-01 27510, 2012

      • warp
        ocharles: ok
      • 2012-10-01 27525, 2012

      • ocharles
        warp: the 'standard' postgres way is to update, check the row count returned, insert if it's not what you expected, and do all this in a loop; you break if the insert completed without exceptions, and you loop if there was a primary key violation
      • 2012-10-01 27515, 2012

      • warp
        what I have now is:
      • 2012-10-01 27556, 2012

      • warp
        INSERT INTO cover_art_archive.release_group_cover_art (release_group, release)
      • 2012-10-01 27559, 2012

      • warp
        (SELECT ? AS release_group, ? AS release WHERE NOT EXISTS
      • 2012-10-01 27502, 2012

      • warp
        (SELECT 1 FROM cover_art_archive.release_group_cover_art
      • 2012-10-01 27504, 2012

      • warp
        WHERE release_group = ?));",
      • 2012-10-01 27547, 2012

      • ocharles
        that's what an approach
      • 2012-10-01 27548, 2012

      • warp
        (which I haven't tested yet)
      • 2012-10-01 27557, 2012

      • ocharles
        you can simplify it with a function in postgresql and use try/catch
      • 2012-10-01 27510, 2012

      • ocharles
        but you can't do that outside a postgresql function without causing the exception to be logged, which i would like to avoid :)
      • 2012-10-01 27531, 2012

      • warp
        ah
      • 2012-10-01 27515, 2012

      • warp
        I'll stick to this, assuming it works. you can offer alternatives on the code review if you come up with something better :)
      • 2012-10-01 27532, 2012

      • ocharles
        ocharles has changed the topic to: Arctic week! | http://musicbrainz.org/#devel | Agenda: replication packets (ocharles), schema change progress, nes3 (ocharles), github (ocharles), translations (ian), MBS-2377 DR (ocharles), MBS-3059 DR (ocharles), echoprint "partnership" (reotab)
      • 2012-10-01 27539, 2012

      • ianmcorvidae
        ianmcorvidae has changed the topic to: Arctic week! | http://musicbrainz.org/#devel | Agenda: replication packets (ocharles), schema change progress, nes3 (ocharles), github (ocharles), MBS-2377 DR (ocharles), MBS-3059 DR (ocharles), echoprint "partnership" (reotab), translations (ian)
      • 2012-10-01 27555, 2012

      • ianmcorvidae puts myself at the end of the line, everything else seems more important
      • 2012-10-01 27536, 2012

      • reosarevok
        Not mine :p
      • 2012-10-01 27512, 2012

      • reosarevok
        (and heh, sounds like another looong meeting)
      • 2012-10-01 27509, 2012

      • warp
        ocharles: I guess we should release today.
      • 2012-10-01 27531, 2012

      • reosarevok
        Remember to revert the relationship editor first, if it's not done yet
      • 2012-10-01 27549, 2012

      • ocharles
        i know
      • 2012-10-01 27551, 2012

      • ocharles
        i'm doing it now
      • 2012-10-01 27513, 2012

      • warp
        ok :)
      • 2012-10-01 27524, 2012

      • ocharles
        well that was to reosarevok, but I guess it applies to your comment too :)
      • 2012-10-01 27533, 2012

      • warp
        haha
      • 2012-10-01 27536, 2012

      • warp
        >_<
      • 2012-10-01 27554, 2012

      • ianmcorvidae hopes jenkins for master is fixed?
      • 2012-10-01 27526, 2012

      • ocharles ponders how to revert the relationship editor
      • 2012-10-01 27508, 2012

      • ocharles
        ianmcorvidae: not that i know
      • 2012-10-01 27516, 2012

      • nikki
        is that why beta is so slow then?
      • 2012-10-01 27523, 2012

      • DremoraLV joined the channel
      • 2012-10-01 27527, 2012

      • ocharles
        possibly
      • 2012-10-01 27537, 2012

      • ocharles
        but i don't think so right now, cause jenkins isn't doing anything
      • 2012-10-01 27506, 2012

      • ocharles questions how well this revert is going to work
      • 2012-10-01 27512, 2012

      • ocharles
        doing it on the day of the release is fun
      • 2012-10-01 27516, 2012

      • warp
        :)
      • 2012-10-01 27543, 2012

      • ocharles
        All tests successful.
      • 2012-10-01 27549, 2012

      • ocharles
        ok, the site will be perfect. yay
      • 2012-10-01 27503, 2012

      • ocharles
        ianmcorvidae: what exactly am I testing on the edit search?
      • 2012-10-01 27523, 2012

      • ianmcorvidae
        ocharles: do a search, get results, then try to add a new property and hit search again
      • 2012-10-01 27539, 2012

      • ocharles
        ok
      • 2012-10-01 27547, 2012

      • ianmcorvidae
        before my fix that wouldn't work, but it had to do with the new jquery version, so I want to make sure my fix doesn't need reverting too.
      • 2012-10-01 27512, 2012

      • warp
        hm, perhaps new jquery should stay merged in?