#musicbrainz-devel

/

      • ianmcorvidae
        it is
      • 2014-07-01 18226, 2014

      • ianmcorvidae
        so one sec and you can try again
      • 2014-07-01 18238, 2014

      • ianmcorvidae
        dunno if it'd need some sort of change to make it use the py2.7 dev headers
      • 2014-07-01 18206, 2014

      • Gentlecat
        thanks
      • 2014-07-01 18235, 2014

      • ianmcorvidae
        that's installed if you want to try again, anyway
      • 2014-07-01 18221, 2014

      • Gentlecat
        yes, it worked
      • 2014-07-01 18226, 2014

      • ianmcorvidae
        yay
      • 2014-07-01 18246, 2014

      • Gentlecat
      • 2014-07-01 18201, 2014

      • ianmcorvidae
        hm
      • 2014-07-01 18207, 2014

      • ianmcorvidae
        I guess psycopg2 defaults to TCP auth
      • 2014-07-01 18227, 2014

      • ianmcorvidae
        use 127.0.0.1 rather than localhost, since localhost defaults to ipv6 localhost which oidentd doesn't listen on
      • 2014-07-01 18249, 2014

      • nikki
        oh jeez, that again
      • 2014-07-01 18207, 2014

      • nikki
        the number of times it's caused me problems, and I don't even use python much :P
      • 2014-07-01 18213, 2014

      • ianmcorvidae
        heh
      • 2014-07-01 18219, 2014

      • ianmcorvidae
        the first time it came up it was java IIRC
      • 2014-07-01 18232, 2014

      • nikki
        it's always been python for me, afaik
      • 2014-07-01 18224, 2014

      • ianmcorvidae
        anyway, yeah, add host=127.0.0.1 to the connection bit I Think
      • 2014-07-01 18225, 2014

      • ianmcorvidae
        think*
      • 2014-07-01 18213, 2014

      • ianmcorvidae
        hm, though it claims it defaults to unix socket if not provided
      • 2014-07-01 18215, 2014

      • ianmcorvidae
        weird
      • 2014-07-01 18248, 2014

      • Gentlecat
        it worked, I'm trying to compile styling
      • 2014-07-01 18200, 2014

      • ianmcorvidae
        alright
      • 2014-07-01 18213, 2014

      • ianmcorvidae
        npm install from the top directory and then it'll be in node_modules/less/bin/lessc
      • 2014-07-01 18229, 2014

      • ianmcorvidae
        though you have to define input/output files directly right now, no script for that
      • 2014-07-01 18235, 2014

      • Gentlecat
      • 2014-07-01 18237, 2014

      • ianmcorvidae
        but it's just turn geordi.less into geordi.css anyway
      • 2014-07-01 18250, 2014

      • ianmcorvidae
        hm
      • 2014-07-01 18254, 2014

      • ianmcorvidae
        too old a less version, looks like?
      • 2014-07-01 18208, 2014

      • Gentlecat
        it worked fine with JS
      • 2014-07-01 18211, 2014

      • Gentlecat
        maybe
      • 2014-07-01 18217, 2014

      • ianmcorvidae
        yeah, needs 1.5.0 I guess
      • 2014-07-01 18222, 2014

      • ianmcorvidae
        which I should fix in the dependencies, but
      • 2014-07-01 18204, 2014

      • ianmcorvidae
        and yeah, the less.js one is from 1.7.something I think
      • 2014-07-01 18221, 2014

      • ianmcorvidae
        1.7.1 I guess
      • 2014-07-01 18247, 2014

      • KillDaBOB
        (sorry to interrupt your CB discussions) this is my daily plea to the Picard devs to release 1.3. evidence that this is a good thing: http://forums.musicbrainz.org/viewtopic.php?id=50… people want to use Picard, but 1.2 is vastly out of date compared to 1.3. most people don’t even know where to find the daily builds. please finalize anything that needs to be done and release 1.3.
      • 2014-07-01 18201, 2014

      • Gentlecat
        ianmcorvidae: I don't think I can update it
      • 2014-07-01 18213, 2014

      • Gentlecat
        can you help me with that?
      • 2014-07-01 18226, 2014

      • nikki notes that picard is maintained by volunteers, and asking them daily to do something is more likely to annoy them than anything
      • 2014-07-01 18252, 2014

      • ianmcorvidae
        Gentlecat: I don't see a node_modules folder in the geordi folder at all, where do you have it installed?
      • 2014-07-01 18214, 2014

      • Gentlecat
        I didn't install anything node-related
      • 2014-07-01 18228, 2014

      • ianmcorvidae
        lessc is a nodejs app
      • 2014-07-01 18245, 2014

      • Gentlecat
        I was using lessc that is installed right now on rika
      • 2014-07-01 18248, 2014

      • ianmcorvidae
        I'm not sure where you're getting lessc from is what I'm wondering
      • 2014-07-01 18249, 2014

      • ianmcorvidae
        hm
      • 2014-07-01 18251, 2014

      • Gentlecat
        yeah, I know
      • 2014-07-01 18206, 2014

      • ianmcorvidae
        hm
      • 2014-07-01 18213, 2014

      • ianmcorvidae
        yeah, I think installing it local to the directory would be best
      • 2014-07-01 18224, 2014

      • ianmcorvidae
        from the top directory of the geordi checkout, 'npm install'
      • 2014-07-01 18234, 2014

      • ianmcorvidae
        and then you'll have a lessc in node_modules/less/bin/lessc
      • 2014-07-01 18250, 2014

      • KillDaBOB
        nikki: who’s in charge of Picard? like I said yesterday, more and more over the past few weeks, people are coming to the forums with questions and problems that are resolved by telling them to obtain the daily builds of Picard. It’s time for 1.3 to be released. Who would be the person to bug to get this to happen?
      • 2014-07-01 18213, 2014

      • CatQuest
        the answer is not to bug people :(
      • 2014-07-01 18230, 2014

      • Gentlecat
      • 2014-07-01 18255, 2014

      • ianmcorvidae
        haha whoo, you reintegrated museo slab
      • 2014-07-01 18258, 2014

      • ianmcorvidae
        anyway yay!
      • 2014-07-01 18209, 2014

      • Gentlecat
        I created PR. will go eat something now :)
      • 2014-07-01 18211, 2014

      • nikki
        I'm not sure who's in charge any more. luks used to do the windows builds, but afaik he's looking for someone else to take it over
      • 2014-07-01 18222, 2014

      • Gentlecat
        thanks for helping
      • 2014-07-01 18212, 2014

      • nikki
        but yes, as CatQuest said, the answer isn't to bug people
      • 2014-07-01 18247, 2014

      • ianmcorvidae
        hm, logout seems not to work? weird
      • 2014-07-01 18203, 2014

      • ianmcorvidae
        or the front page just won't interact properly with my cache or something, bah
      • 2014-07-01 18226, 2014

      • Gentlecat
        works fine here
      • 2014-07-01 18243, 2014

      • KillDaBOB
        if people are never bugged, they will never see this is a problem. people who are new to MB and all of its goodness will continue to download Picard 1.2 and continue to have problems and continue using an inferior product. 1.2 doesn’t have covert art download capibilities (except through a plugin that barely works). 1.2 automatically tags any release with more than one artist as a compilation, which is wrong. 1.2 cras
      • 2014-07-01 18244, 2014

      • KillDaBOB
        or won’t even load for some Windows 8 users (i’m not sure of the conditions) while 1.3 works just find for them. some of the devs, like bitmap, have stated they rarely read the forums, so they do not know about these issues. i’m championing for those on the forums whose voices are not heard.
      • 2014-07-01 18247, 2014

      • ianmcorvidae
        I can get the login button to reappear on other pages, but the front page will only show me logged in
      • 2014-07-01 18230, 2014

      • Gentlecat
        ianmcorvidae: custom-variables.less was a replacements for variables.less, right?
      • 2014-07-01 18247, 2014

      • ianmcorvidae
        yeah, it was variables.less with all the paddings/margins cut in half (basically)
      • 2014-07-01 18253, 2014

      • Gentlecat
        I thought that this file came from bootstrap and just replaced it
      • 2014-07-01 18257, 2014

      • ianmcorvidae
        yeah
      • 2014-07-01 18207, 2014

      • ianmcorvidae
        I'm not sure if there's a better way to do it just overriding the things we care about
      • 2014-07-01 18227, 2014

      • ianmcorvidae
        and now logout works for whatever reason
      • 2014-07-01 18228, 2014

      • ianmcorvidae
        weird
      • 2014-07-01 18200, 2014

      • ianmcorvidae
        anyway seems fine, I just wanted to look at the login-button styling again to make sure I liked it :)
      • 2014-07-01 18227, 2014

      • Gentlecat
        does checkbox look good?
      • 2014-07-01 18205, 2014

      • ianmcorvidae
        yeah -- I think I liked the slide-out a bit better, but the popup isn't bad
      • 2014-07-01 18220, 2014

      • ianmcorvidae
        is the popup thing provided by bootstrap.js, or?
      • 2014-07-01 18236, 2014

      • Gentlecat
        yes, that's bootstrap.js
      • 2014-07-01 18247, 2014

      • Gentlecat
        it's also used to collapse navbar
      • 2014-07-01 18210, 2014

      • Gentlecat
        try changing window width
      • 2014-07-01 18230, 2014

      • Gentlecat
        there's a small issue with checkbox block
      • 2014-07-01 18236, 2014

      • Gentlecat
        not sure how to fix this
      • 2014-07-01 18258, 2014

      • ianmcorvidae
        the width thing when it's collapsed, I guess?
      • 2014-07-01 18210, 2014

      • Gentlecat
        yes
      • 2014-07-01 18212, 2014

      • dufferzafar
        nikki: I created a PR for the iTunes importer: https://bitbucket.org/96187/userscripts/pull-requ…
      • 2014-07-01 18217, 2014

      • nikki
        thanks
      • 2014-07-01 18233, 2014

      • dufferzafar
        np :)
      • 2014-07-01 18247, 2014

      • nikki
        KillDaBOB: we're already aware that a new release is long overdue. as I said, the people who work on picard are volunteers, they're doing it in their own time for nothing. coming here and telling people to do a release and repeatedly talking about "bugging" people doesn't come across well, and isn't likely to make people want to have anything to do with it
      • 2014-07-01 18214, 2014

      • nikki
        I mean, who would want to admit to working on picard when you've just said you want to bug them daily until they do a release? :P
      • 2014-07-01 18227, 2014

      • Gentlecat
        ianmcorvidae: tooltip stops working after I move JS to the bottom of the page
      • 2014-07-01 18241, 2014

      • ianmcorvidae
        yeah, I noticed that was an inline script
      • 2014-07-01 18259, 2014

      • ianmcorvidae
        the inline script can presumably be put in block js to move it down though
      • 2014-07-01 18213, 2014

      • Gentlecat
        oh, right
      • 2014-07-01 18224, 2014

      • Gentlecat
        ianmcorvidae: do you want me to add inline comments there https://github.com/metabrainz/geordi/pull/13#disc…
      • 2014-07-01 18210, 2014

      • ianmcorvidae
        it probably doesn't need to be in the code itself, I just don't understand bootstrap terribly well so I wanted to confirm I understood the right thing (i.e., when it's under a certain breakpoint, put it to the right (not sure exactly what pull-right does, but) and pull it to width 100%)
      • 2014-07-01 18234, 2014

      • Gentlecat
        okay, I'll add a comment in PR
      • 2014-07-01 18251, 2014

      • Gentlecat
        ianmcorvidae: I decreased the gap between navbar and h1
      • 2014-07-01 18256, 2014

      • Gentlecat
        is it better now?
      • 2014-07-01 18214, 2014

      • ianmcorvidae
        yeah, that looks better
      • 2014-07-01 18223, 2014

      • KillDaBOB
        nikki: point taken. my voice has been heard. a lot (not all) of the people who contribute to Picard hang out in this channel, so my hope is that they will read their logs and see this. i’ll keep quiet- for now.
      • 2014-07-01 18202, 2014

      • ianmcorvidae
        heh, I guess I should write matching soon so the login actually has a purpose
      • 2014-07-01 18239, 2014

      • Gentlecat
        I was thinking about trying to migrate to sqlalchemy
      • 2014-07-01 18208, 2014

      • Gentlecat
        to make code cleaner and development easier in general
      • 2014-07-01 18212, 2014

      • ianmcorvidae
        probably would be good, would make it easier to do the API stuff too
      • 2014-07-01 18213, 2014

      • ianmcorvidae
        yeah
      • 2014-07-01 18233, 2014

      • ianmcorvidae
        I just went more with what I know :)
      • 2014-07-01 18252, 2014

      • chirlu` joined the channel
      • 2014-07-01 18212, 2014

      • chirlu`
        “Ned g’schimpft is g’lobt g’nug!”
      • 2014-07-01 18219, 2014

      • chirlu`
        (re: pep talk)
      • 2014-07-01 18210, 2014

      • Mineo wonders if that's a "nicht"-ned or "nett"-ned
      • 2014-07-01 18209, 2014

      • Gentlecat
        ianmcorvidae: are issues here http://tickets.musicbrainz.org/browse/GEORDI from pre-rewrite period?
      • 2014-07-01 18228, 2014

      • Gentlecat
        are they still relevant?
      • 2014-07-01 18232, 2014

      • chirlu`
        Mineo: I understand it as “nicht”.
      • 2014-07-01 18231, 2014

      • Mineo
        fair enough :)
      • 2014-07-01 18238, 2014

      • chirlu` thinks it should have been “isch”, actually.
      • 2014-07-01 18201, 2014

      • ianmcorvidae
        Gentlecat: they're from the pre-rewrite period, though some of them might be applicable still, or solved by the rewrite
      • 2014-07-01 18218, 2014

      • ianmcorvidae
        anything data-source-specific is certainly not applicable until we add those data sources back
      • 2014-07-01 18253, 2014

      • ianmcorvidae
        GEORDI-17 is half-solved by the test dataset, GEORDI-20 is definitely still applicable
      • 2014-07-01 18224, 2014

      • ianmcorvidae
        the stuff about importing subitems (no longer a concept that exists in the new version) and other entities are things the rewrite's supposed to help with, but there's no importer yet
      • 2014-07-01 18217, 2014

      • ianmcorvidae
        the automatic matching stuff is a can of worms we haven't really opened with the new code, though the intention is to move them into the geordi codebase rather than having them be external things talking to an API (which should help)
      • 2014-07-01 18230, 2014

      • ianmcorvidae
        I think a lot of the rest is data source specific
      • 2014-07-01 18240, 2014

      • Gentlecat
        okay
      • 2014-07-01 18219, 2014

      • ianmcorvidae
        lol the ninjatune data
      • 2014-07-01 18233, 2014

      • ianmcorvidae
        has 'NOT FOR RELEASE OR EXPLOITATION' in a barcode field somewhere, it appears
      • 2014-07-01 18216, 2014

      • ianmcorvidae
        bitmap: updated that release events/geordi PR
      • 2014-07-01 18250, 2014

      • bitmap
        cool
      • 2014-07-01 18202, 2014

      • bitmap just got the emails and is still reading them :P
      • 2014-07-01 18213, 2014

      • ianmcorvidae
        if you have a better idea for the no_manip thing I'm all ears, heh, that was definitely a hack
      • 2014-07-01 18234, 2014

      • ianmcorvidae
        but it's not hopelessly repetitive anymore, which means even though I didn't do named functions it's probably not as insane to parse all the lambdas and whatever any more :P
      • 2014-07-01 18258, 2014

      • ianmcorvidae wonders if I should break list(chain.from_iterable(foo)) into a utility
      • 2014-07-01 18202, 2014

      • ianmcorvidae
        probably
      • 2014-07-01 18230, 2014

      • bitmap
        it's definitely easier to read now
      • 2014-07-01 18201, 2014

      • bitmap
        where in the code is the PathPart normalization process you mentioned in the one comment?
      • 2014-07-01 18234, 2014

      • bitmap is still wrapping his head around that part
      • 2014-07-01 18225, 2014

      • ianmcorvidae
      • 2014-07-01 18230, 2014

      • ianmcorvidae
      • 2014-07-01 18252, 2014

      • bitmap
        ah
      • 2014-07-01 18200, 2014

      • ianmcorvidae
        which initializes a PathInserter class with the path as provided and then uses the source_value() of each path part it creates to go back
      • 2014-07-01 18220, 2014

      • ianmcorvidae
        SimplePathPart's source_value is what uses no_manip, around https://github.com/metabrainz/geordi/blob/master/…
      • 2014-07-01 18236, 2014

      • bitmap
        and the no_manip means the last source_value remains a pathpart
      • 2014-07-01 18258, 2014

      • ianmcorvidae
        it means that whichever one it's set on remains a SimplePathPart, yeah
      • 2014-07-01 18216, 2014

      • ianmcorvidae
        which means the code at https://github.com/metabrainz/geordi/blob/master/… won't append an OrderedPathPart() instance to the end
      • 2014-07-01 18257, 2014

      • ianmcorvidae
        as mentioned I'm pretty sure there's a better way to be doing this, but I don't want to force every path to stick "('some fake name',)" at the end or whatever
      • 2014-07-01 18216, 2014

      • ianmcorvidae
        oh, I guess it'd just be (,) but that's still silly
      • 2014-07-01 18217, 2014

      • ianmcorvidae
        one option is to stop accepting non-array things, i.e. let it be possible for a single release event thing to list more than one date and more than one country
      • 2014-07-01 18239, 2014

      • ianmcorvidae
        but that seems silly given you can/should just add another release event thing instead
      • 2014-07-01 18208, 2014

      • ianmcorvidae
        it'd be useful if you don't have the connection between dates and countries, but there's a separate place in the data format for so-called split data like that
      • 2014-07-01 18208, 2014

      • ruaok runs off to bed
      • 2014-07-01 18209, 2014

      • ruaok
        nn!
      • 2014-07-01 18215, 2014

      • ianmcorvidae
        scaring you away, eh
      • 2014-07-01 18218, 2014

      • ianmcorvidae
        aw, he left :P
      • 2014-07-01 18258, 2014

      • bitmap
        I think I'm confusing tuples in the source path and tuples in the destination path