#musicbrainz-devel

/

      • pronik
        navap: I liked those help "baloons" on careers.stackoverflow.com
      • 2010-03-11 07033, 2010

      • pronik
        gotta login and "create new CV" to see it
      • 2010-03-11 07048, 2010

      • ruaok
      • 2010-03-11 07012, 2010

      • aCiD2
        i'm off to snooze
      • 2010-03-11 07013, 2010

      • aCiD2
        bb!
      • 2010-03-11 07054, 2010

      • ruaok
        nn
      • 2010-03-11 07058, 2010

      • ruaok
        I swear. I need a "have to ponder this" status in jira.
      • 2010-03-11 07010, 2010

      • ruaok
        I have too many things that need to be considered in the grand scheme of things.
      • 2010-03-11 07055, 2010

      • ruaok goes to find a lawn to continue working on
      • 2010-03-11 07042, 2010

      • navap
        warp: http://careers.stackoverflow.com/cv/edit has the same kind of stick/unstick div that ac3 needs and it seems quite fast in my slow machine.
      • 2010-03-11 07008, 2010

      • navap
        pronik: We don't have kind of room though, and the icons I'm referring to wouldn't be just for forms.
      • 2010-03-11 07056, 2010

      • alastairp
        aCiD2: pft. perl?
      • 2010-03-11 07004, 2010

      • ruaok joined the channel
      • 2010-03-11 07006, 2010

      • navap
        The footer says "Contact" but the menu says "Contact Us", which should it be?
      • 2010-03-11 07042, 2010

      • pronik` joined the channel
      • 2010-03-11 07003, 2010

      • navap joined the channel
      • 2010-03-11 07056, 2010

      • brianfreud joined the channel
      • 2010-03-11 07001, 2010

      • ijabz joined the channel
      • 2010-03-11 07046, 2010

      • ijabz joined the channel
      • 2010-03-11 07045, 2010

      • ijabz joined the channel
      • 2010-03-11 07003, 2010

      • djce joined the channel
      • 2010-03-11 07031, 2010

      • warp joined the channel
      • 2010-03-11 07047, 2010

      • warp
        navap: the principle is the same (they hook $(window).scroll() in cv-edit.js), but I can't easily check if it feels as slow as ac3... as the other computer is now in the basement.
      • 2010-03-11 07007, 2010

      • warp
        (actually, I can have a look at both on the netbook soonish)
      • 2010-03-11 07045, 2010

      • warp installs deps from cpan.
      • 2010-03-11 07021, 2010

      • brianfreud
        warp, I've been too busy w/work to look... what is it you're actually using the scroll hook to do?
      • 2010-03-11 07054, 2010

      • brianfreud wonders about alternative options that would give the same results
      • 2010-03-11 07021, 2010

      • warp
        brianfreud: try http://frob.nl/~warp/ac3/edit.php and scroll down.
      • 2010-03-11 07037, 2010

      • aCiD2
        morning!
      • 2010-03-11 07039, 2010

      • aCiD2
        fsov morning
      • 2010-03-11 07009, 2010

      • brianfreud
        warp: I see. Why not just use css?
      • 2010-03-11 07054, 2010

      • aCiD2
        brianfreud: how could you do that with css?
      • 2010-03-11 07037, 2010

      • brianfreud
        You have it start as static, then switch to fixed if it would otherwise scroll off the page.
      • 2010-03-11 07002, 2010

      • aCiD2
        oh right, so it still needs JS
      • 2010-03-11 07007, 2010

      • brianfreud
        no
      • 2010-03-11 07011, 2010

      • brianfreud
        that's the current behaviour
      • 2010-03-11 07032, 2010

      • brianfreud
        I assume it only opens when you click on something to open it.
      • 2010-03-11 07032, 2010

      • aCiD2
        How can you change from static to fixed without javascript?
      • 2010-03-11 07005, 2010

      • brianfreud
        So why not just simply 'position: fixed;' position the box instead? Then it'll stay in whatever position, so long as it is open, no matter where you scroll, and no need for any js or capturing the scroll events
      • 2010-03-11 07038, 2010

      • aCiD2
        Because we don't want it to stay fixed in the same loctation
      • 2010-03-11 07054, 2010

      • aCiD2
        We want it to original stay as part of the page, but as you scroll down to stay visible
      • 2010-03-11 07006, 2010

      • brianfreud
        that's the current behaviour, except for at the very top of the page?
      • 2010-03-11 07022, 2010

      • brianfreud
        aCiD2: I guess my question is, esp given the overhead in pulling that trick off, why?
      • 2010-03-11 07023, 2010

      • aCiD2
        but that's not the same, is it :)
      • 2010-03-11 07055, 2010

      • brianfreud
        Anyhow, I don't notice a slowdown on my box.
      • 2010-03-11 07058, 2010

      • brianfreud
        nd cache that for the later checks.
      • 2010-03-11 07012, 2010

      • ijabz has left the channel
      • 2010-03-11 07025, 2010

      • aCiD2
        warp: I fixed your MBS-66 problem... and it's the weirdest solution ever
      • 2010-03-11 07034, 2010

      • aCiD2
        +sub view : Chained('/user/base') PathPart('ratings')
      • 2010-03-11 07036, 2010

      • aCiD2
        that's the diff
      • 2010-03-11 07010, 2010

      • warp
        aCiD2: awesome. that's a pretty subtle typo.
      • 2010-03-11 07042, 2010

      • aCiD2
        Yea
      • 2010-03-11 07048, 2010

      • aCiD2
        copy pasta ftw
      • 2010-03-11 07035, 2010

      • warp
        meh
      • 2010-03-11 07046, 2010

      • aCiD2
        please pull from acid2/mbs-66
      • 2010-03-11 07001, 2010

      • aCiD2
        mbs66*
      • 2010-03-11 07002, 2010

      • warp
        my postgres sorting thing runs out of memory when I try to create an index for track_name.
      • 2010-03-11 07013, 2010

      • aCiD2
        lol
      • 2010-03-11 07024, 2010

      • warp
        I have the feeling I'm leaking memory, but I don't see anything wrong in the code.
      • 2010-03-11 07035, 2010

      • warp
        (and the same thing will happen on test, as it only has 4gb too)
      • 2010-03-11 07024, 2010

      • warp
        aCiD2: how are you at C? :)\
      • 2010-03-11 07046, 2010

      • aCiD2
        about as useful as an elephant trying to thread a needle
      • 2010-03-11 07024, 2010

      • aCiD2
        codereview is down?
      • 2010-03-11 07038, 2010

      • warp
        aCiD2: haha
      • 2010-03-11 07001, 2010

      • warp
        hm, I forgot how to create the test database.
      • 2010-03-11 07003, 2010

      • warp discovers script/create_test_db.sh
      • 2010-03-11 07059, 2010

      • warp
        aCiD2: 2 minutes 40 seconds for a "prove -l t/" run.
      • 2010-03-11 07010, 2010

      • warp
        real 2m41.758s
      • 2010-03-11 07041, 2010

      • aCiD2
        warp: on your new box?
      • 2010-03-11 07053, 2010

      • warp
        yes
      • 2010-03-11 07058, 2010

      • aCiD2
        givvvve
      • 2010-03-11 07058, 2010

      • warp
        so refreshing
      • 2010-03-11 07001, 2010

      • aCiD2
        very nice :)
      • 2010-03-11 07052, 2010

      • navap
        What is the difference between the two ratings paths? The difference is too subtle for me it seems.
      • 2010-03-11 07052, 2010

      • warp does:
      • 2010-03-11 07000, 2010

      • warp
        script/musicbrainz_server -r
      • 2010-03-11 07011, 2010

      • warp
        watch 'curl --silent http://bb:3000/ws/js/artist?q=Shih&limit=10 ; echo ""'
      • 2010-03-11 07014, 2010

      • aCiD2
        navap: one said PathPath the other said PathPart
      • 2010-03-11 07025, 2010

      • warp
        and edit away and stuff happens :)
      • 2010-03-11 07026, 2010

      • navap
        arghh haha
      • 2010-03-11 07041, 2010

      • aCiD2
        navap: yea, I actually pasted it here and thought it was a whitespace thing until warp noticed there was a difference :D
      • 2010-03-11 07051, 2010

      • navap
        warp++ :p
      • 2010-03-11 07001, 2010

      • aCiD2
        :)
      • 2010-03-11 07008, 2010

      • navap
        Codereview is still down :(
      • 2010-03-11 07011, 2010

      • warp
        :(
      • 2010-03-11 07026, 2010

      • warp
        I say when codereview is down we just commit without review, yay!
      • 2010-03-11 07032, 2010

      • warp
        j/k
      • 2010-03-11 07037, 2010

      • aCiD2 goes to make some massive refactoring changes
      • 2010-03-11 07038, 2010

      • aCiD2
        MWAHAHAA
      • 2010-03-11 07042, 2010

      • aCiD2
        oh...
      • 2010-03-11 07049, 2010

      • warp
        haha
      • 2010-03-11 07057, 2010

      • navap
        The diffs could always be just checked manually.
      • 2010-03-11 07059, 2010

      • aCiD2
        i did want to review navap's branch though, I got it running and looked at the code and didn't see anything crazy
      • 2010-03-11 07011, 2010

      • aCiD2
        but I did notice one thing - the edit list looks kind abroke
      • 2010-03-11 07020, 2010

      • aCiD2
        it's missing a lot of padding and looks cramped
      • 2010-03-11 07038, 2010

      • navap
        Yeah I only fixed the content, not the styling on the mass edit page.
      • 2010-03-11 07059, 2010

      • aCiD2
        ah, okies
      • 2010-03-11 07013, 2010

      • navap
        My main focus was on the individual edit page. I didn't want to spend too much time on the mass edit page because I wasn't sure how I wanted it to look.
      • 2010-03-11 07031, 2010

      • navap
        I just wanted to manku sure it was using the proper templates.
      • 2010-03-11 07003, 2010

      • aCiD2
        yea, I think this is enough for one review
      • 2010-03-11 07000, 2010

      • navap
        Here's something that should be quite quick to review. The last commit on that branch is whitespace changes. http://github.com/navap/mb_server/compare/master.…
      • 2010-03-11 07007, 2010

      • warp
        aCiD2: btw, I have to force install Test::WWWW::Mechanize, Catalyst::Plugin::StackTrace and Text::Unaccent this time.
      • 2010-03-11 07012, 2010

      • warp
        s/have/had/
      • 2010-03-11 07039, 2010

      • aCiD2
        warp: Text::Unaccent isn't even needed now
      • 2010-03-11 07001, 2010

      • warp
        it's still 'use 'd somewhere
      • 2010-03-11 07009, 2010

      • aCiD2
        nope.. I removed all that... I'm sure
      • 2010-03-11 07030, 2010

      • aCiD2
        ollie [at debian]:~/musicbrainz-server$ ack-grep 'Text::Unaccent' lib | wc -l
      • 2010-03-11 07033, 2010

      • aCiD2
        0
      • 2010-03-11 07038, 2010

      • aCiD2
        it's probably because your branches aren't on master
      • 2010-03-11 07043, 2010

      • aCiD2
        (they are branched off older commits)
      • 2010-03-11 07055, 2010

      • warp
        Makefile.PL:requires 'Text::Unaccent' => '1.080';
      • 2010-03-11 07058, 2010

      • warp
        lib/MusicBrainz/Server/Validation.pm:use Text::Unaccent qw( unac_string );
      • 2010-03-11 07013, 2010

      • aCiD2
        yea, it's what I said :)
      • 2010-03-11 07019, 2010

      • aCiD2
        and nice machine name :P
      • 2010-03-11 07033, 2010

      • warp
        it's small, black and fast.
      • 2010-03-11 07043, 2010

      • warp
        (all my machines are named after mario characters)
      • 2010-03-11 07054, 2010

      • aCiD2
        that's a cool naming scheme :)
      • 2010-03-11 07058, 2010

      • warp
        the server is slow, heavy and grey. hence thwomp.
      • 2010-03-11 07002, 2010

      • aCiD2
        mine are named after moons
      • 2010-03-11 07003, 2010

      • aCiD2
        boring.
      • 2010-03-11 07044, 2010

      • warp
        my top secret gaming website project is bulbasaur. If that ever takes off I don't have to think about names at all, I just pick them one by one from the pokedex.
      • 2010-03-11 07021, 2010

      • navap
        heh
      • 2010-03-11 07024, 2010

      • warp
        I don't expect to need 400+ hostnames any time soon.
      • 2010-03-11 07027, 2010

      • warp
        ;)
      • 2010-03-11 07048, 2010

      • aCiD2
        hahaha
      • 2010-03-11 07007, 2010

      • warp
        (i could match the last octet of the ip adress to the pokedex number)
      • 2010-03-11 07016, 2010

      • aCiD2
        warp. you're a nerd.
      • 2010-03-11 07020, 2010

      • warp
        no shit :)
      • 2010-03-11 07023, 2010

      • aCiD2
        hehe
      • 2010-03-11 07023, 2010

      • navap
        haha that's quite geeky
      • 2010-03-11 07044, 2010

      • aCiD2
        Yay, XLD finished ripping 1 track off a cd after taking 5 hours
      • 2010-03-11 07036, 2010

      • warp should rip some stuff too.
      • 2010-03-11 07012, 2010

      • warp
        the dvd drive in the previous box wasn't working very well anymore.
      • 2010-03-11 07032, 2010

      • warp fights with roles.
      • 2010-03-11 07039, 2010

      • aCiD2 <3 roles
      • 2010-03-11 07028, 2010

      • luks starts the CPAN dance again
      • 2010-03-11 07038, 2010

      • aCiD2
        how come luks, I thought you were setup?
      • 2010-03-11 07013, 2010

      • luks
        well, mb_server is a moving target
      • 2010-03-11 07014, 2010

      • navap
        You don't dance only once in your file, you gotta keep dancing to get good ;)
      • 2010-03-11 07027, 2010

      • luks
        one week later and I need to download the other half of internet :)
      • 2010-03-11 07048, 2010

      • aCiD2
        Are you sure you need to though? I mean, some version numbers might have changed... but maybe they didn't need to
      • 2010-03-11 07026, 2010

      • luks
        "sudo cpan Test::Mock" installs lots of packages
      • 2010-03-11 07034, 2010

      • aCiD2
        yay for my dependencies
      • 2010-03-11 07036, 2010

      • luks
        plus it fails a test
      • 2010-03-11 07040, 2010

      • aCiD2
        oo, please tell me which
      • 2010-03-11 07016, 2010

      • luks
      • 2010-03-11 07011, 2010

      • aCiD2
        ah, the fix for that hasn't gone to cpan yet
      • 2010-03-11 07030, 2010

      • aCiD2
        make sure you have MooseX::ChainedAccessors::Accessor installed
      • 2010-03-11 07048, 2010

      • luks
        btw
      • 2010-03-11 07050, 2010

      • luks