#musicbrainz-devel

/

      • Gentlecat
        ianmcorvidae: have you seen my messages?
      • 2014-05-01 12141, 2014

      • Gentlecat
        looks like you've got some connection problems :)
      • 2014-05-01 12126, 2014

      • ianmcorvidae
        ah, sorry, I went for a walk
      • 2014-05-01 12131, 2014

      • ianmcorvidae
        and apparently had connection issues as well
      • 2014-05-01 12108, 2014

      • Gentlecat
      • 2014-05-01 12140, 2014

      • CallerNo6 gets a 500
      • 2014-05-01 12150, 2014

      • ianmcorvidae
        presumably since there's no DB, yeah
      • 2014-05-01 12157, 2014

      • Gentlecat
        well, yes
      • 2014-05-01 12112, 2014

      • Gentlecat
        all i need now is db and figure out how to properly run both server and client in background :)
      • 2014-05-01 12135, 2014

      • ianmcorvidae
        I think we just ran them in screen before
      • 2014-05-01 12142, 2014

      • ianmcorvidae
        oh
      • 2014-05-01 12146, 2014

      • ianmcorvidae
        there's a thing to make sure they keep running
      • 2014-05-01 12150, 2014

      • ianmcorvidae
        lemme grab the one used by reports.mbsandbox
      • 2014-05-01 12123, 2014

      • Gentlecat
        i'm not a linux guru yet heh
      • 2014-05-01 12144, 2014

      • ianmcorvidae
        for that I just use: while true; do ./venv/reports/bin/python reports.fcgi & sleep 1200; ps aux | grep reports.fcgi | grep -v grep | awk '{print $2}' | xargs kill; done
      • 2014-05-01 12100, 2014

      • ianmcorvidae
        which is 100% hack, in a real system we'd deploy either of these with uwsgi
      • 2014-05-01 12104, 2014

      • ianmcorvidae
        but it works for this :)
      • 2014-05-01 12135, 2014

      • ianmcorvidae
        obviously change the location of the virtualenv python binary and the name of the .fcgi file
      • 2014-05-01 12139, 2014

      • ianmcorvidae
        (in both places)
      • 2014-05-01 12149, 2014

      • Gentlecat
        right
      • 2014-05-01 12128, 2014

      • ianmcorvidae
        grepping for something including your username is also a good idea since of course mjjc's is still running
      • 2014-05-01 12107, 2014

      • ianmcorvidae
        his seems to be in tmux rather than screen, but same idea :)
      • 2014-05-01 12129, 2014

      • Gentlecat
      • 2014-05-01 12158, 2014

      • Gentlecat
        something like this?
      • 2014-05-01 12106, 2014

      • ianmcorvidae
        you won't want that, since both things will include 'gentlecat'
      • 2014-05-01 12111, 2014

      • ianmcorvidae
        so the client one will kill the server and vice versa
      • 2014-05-01 12110, 2014

      • ianmcorvidae
        I'd have do /home/gentlecat/...venv/bin/python (i.e., use the full path), and then replace 'grep gentlecat' with 'grep client.fcgi | grep gentlecat'
      • 2014-05-01 12115, 2014

      • ianmcorvidae
        (or server.fcgi, for that one, obviously)
      • 2014-05-01 12140, 2014

      • ianmcorvidae
        probably could be done with one grep but it doesn't really matter, so :)
      • 2014-05-01 12150, 2014

      • Gentlecat
        updated pastebin
      • 2014-05-01 12120, 2014

      • Gentlecat
        oh, also full path
      • 2014-05-01 12124, 2014

      • ianmcorvidae
        yeah
      • 2014-05-01 12130, 2014

      • ianmcorvidae
        the DB is created, incidentally
      • 2014-05-01 12144, 2014

      • ianmcorvidae
        seems to still be 500ing, not sure why
      • 2014-05-01 12106, 2014

      • ianmcorvidae
        oh, I guess I haven't actually done the table-making and such, though you can probably do that
      • 2014-05-01 12118, 2014

      • ianmcorvidae
        I've done what python manage.py create_db would do, anyway, if somewhat more manually
      • 2014-05-01 12132, 2014

      • Gentlecat
        okay
      • 2014-05-01 12122, 2014

      • ianmcorvidae
        the pastebin looks right at this point as well
      • 2014-05-01 12159, 2014

      • ianmcorvidae
        if you don't know screen: 'screen' creates a new one, then you can run one of them, then "ctrl-a c" will create a second window in which to run the other, ctrl-a n/p to move to next/previous window and ctrl-a d to detach; screen -ls will list current screens and screen -rd will reconnect if there's only one; if there's more than one you'll have to specify a process number or such, which you can get from screen -ls
      • 2014-05-01 12122, 2014

      • Gentlecat
        that's some cool magic, looks like i've got both server and client running
      • 2014-05-01 12142, 2014

      • Gentlecat
        still getting 500 though
      • 2014-05-01 12153, 2014

      • ianmcorvidae
        yup, getting errors rather than 502s on both
      • 2014-05-01 12112, 2014

      • ianmcorvidae
        I assume tables/fixtures still need inserting, anyway, but as mentioned I think you can do that without me, so
      • 2014-05-01 12131, 2014

      • Gentlecat
        yep, i'll try that now
      • 2014-05-01 12107, 2014

      • ianmcorvidae
        heh, no error logging
      • 2014-05-01 12153, 2014

      • Nyanko-sensei joined the channel
      • 2014-05-01 12105, 2014

      • Gentlecat
        looks like my server is not set up properly
      • 2014-05-01 12133, 2014

      • Gentlecat
      • 2014-05-01 12133, 2014

      • ianmcorvidae
        yeah
      • 2014-05-01 12143, 2014

      • ianmcorvidae
        hm
      • 2014-05-01 12156, 2014

      • ianmcorvidae
        'gentlecat' is mispelled in your client config.py
      • 2014-05-01 12101, 2014

      • ianmcorvidae
        but that's the only thing I can see
      • 2014-05-01 12111, 2014

      • Gentlecat
        ah
      • 2014-05-01 12116, 2014

      • ianmcorvidae
        and I guess the server's also set to debug mode
      • 2014-05-01 12134, 2014

      • Gentlecat
        yeah, i just changed it
      • 2014-05-01 12142, 2014

      • Gentlecat
        experimenting a bit
      • 2014-05-01 12116, 2014

      • Gentlecat
        yay! it's alive
      • 2014-05-01 12129, 2014

      • Gentlecat
        ianmcorvidae: is it me or you changed something?
      • 2014-05-01 12153, 2014

      • Gentlecat
        anyway, it works now http://gentlecat.mbsandbox.org/
      • 2014-05-01 12132, 2014

      • CatQuest has left the channel
      • 2014-05-01 12101, 2014

      • ianmcorvidae joined the channel
      • 2014-05-01 12133, 2014

      • CallerNo6
        huh. project gutenberg URLs auto-detect as "lyrics". I was wondering how that would work. i.e. wondering if one could use "misc" or something for audiobooks.
      • 2014-05-01 12110, 2014

      • CallerNo6
        well, okay, "misc" is deprecated , bad example
      • 2014-05-01 12155, 2014

      • ianmcorvidae
        Gentlecat: I hadn't changed anything, but good it's working I guess!
      • 2014-05-01 12125, 2014

      • Gentlecat
        I switched to branch with cover art, should look a bit better now
      • 2014-05-01 12144, 2014

      • ianmcorvidae
        cool
      • 2014-05-01 12100, 2014

      • ianmcorvidae
        I see a review on the front page that doesn't seem to exist? I guess the 'Random user' user is fake anyway though
      • 2014-05-01 12124, 2014

      • ianmcorvidae
        oh, or it just went away and it was my local HTTP cache, heh
      • 2014-05-01 12105, 2014

      • Gentlecat
        yeah, I just removed one
      • 2014-05-01 12133, 2014

      • Gentlecat
        I guess we need more ways to access review creation and editing stuff
      • 2014-05-01 12115, 2014

      • ianmcorvidae
        yeah, a prominent 'create review' button or such would be nice (when you aren't over your limits, at least, though probably just disabling it in that case makes sense)
      • 2014-05-01 12116, 2014

      • Gentlecat
        it's a bit annoying that I need to open /profile/review/ just to edit reviews or add a new one
      • 2014-05-01 12120, 2014

      • ianmcorvidae
        yeah
      • 2014-05-01 12154, 2014

      • ianmcorvidae
        the onerror for the images isn't getting fired for me in some cases
      • 2014-05-01 12113, 2014

      • Gentlecat
        yep, I noticed that too
      • 2014-05-01 12128, 2014

      • ianmcorvidae
        which may be because you need to do what musicbrainz-server does and not add the src until the onerror is there (i.e., add an empty <img> and then add the src with javascript)
      • 2014-05-01 12138, 2014

      • Gentlecat
        mostly when tab is in background
      • 2014-05-01 12110, 2014

      • Gentlecat
        can you link me an example?
      • 2014-05-01 12107, 2014

      • ianmcorvidae
        of what MBS does? the issue is of course transient, it depends on how caches interact mostly
      • 2014-05-01 12145, 2014

      • Gentlecat
        yes, on mbs
      • 2014-05-01 12148, 2014

      • ianmcorvidae
        the MBS onerror/src binding stuff happens around https://github.com/metabrainz/musicbrainz-server/…
      • 2014-05-01 12122, 2014

      • ianmcorvidae
        we replace with a non-image when it fails, rather than changing the src, but same idea
      • 2014-05-01 12137, 2014

      • Gentlecat
        ianmcorvidae: can you reproduce this case?
      • 2014-05-01 12102, 2014

      • ianmcorvidae
        probably not deliberately
      • 2014-05-01 12134, 2014

      • Gentlecat
        I modified my function a bit http://pastebin.com/qkVXSU94
      • 2014-05-01 12132, 2014

      • ianmcorvidae
        if that's still the onerror callback that won't help, because the issue is the onerror callback not firing
      • 2014-05-01 12104, 2014

      • Gentlecat
        right, I need to call that for every image
      • 2014-05-01 12115, 2014

      • ianmcorvidae
        yeah, but if you already have a src set, that's the issue
      • 2014-05-01 12147, 2014

      • ianmcorvidae
        you need to set the error callback before setting the src at all; it looks like what you're doing there is setting the src to the same thing after setting the error callback
      • 2014-05-01 12114, 2014

      • ianmcorvidae
        which won't work because the browser won't rerequest the image if it already knows it's missing, and thus won't fire the error callback
      • 2014-05-01 12102, 2014

      • Gentlecat
        I updated last pastebin
      • 2014-05-01 12134, 2014

      • ianmcorvidae
        yeah, that looks right
      • 2014-05-01 12139, 2014

      • Gentlecat
        but that seems kind of hacky
      • 2014-05-01 12100, 2014

      • ianmcorvidae
        it definitely is hacky, yeah :/ not much to do about it
      • 2014-05-01 12103, 2014

      • Gentlecat
        I guess we'll have to wait for API fixes
      • 2014-05-01 12118, 2014

      • ianmcorvidae
        there's other methods of trying to see if an image loaded correctly (img.complete and image.naturalWidth seem to pop up fairly often, but neither is supported everywhere), none that are events quite the way you'd probably want
      • 2014-05-01 12145, 2014

      • ianmcorvidae
        img error events don't bubble, as I recall, which prevents the potentially-nice option of putting it on <body> or something
      • 2014-05-01 12152, 2014

      • ianmcorvidae
        another option would be something server-side, I guess, that tries to load the image, and if it works return a redirect to it, otherwise a redirect to the not-found image?
      • 2014-05-01 12134, 2014

      • ianmcorvidae
        arguably that'd be a good thing to add to caa.org, even, just an extra parameter for something to 307 to in case of a 404, rather than just 404ing
      • 2014-05-01 12152, 2014

      • Gentlecat
        but wouldn't server side checking increase page load time?
      • 2014-05-01 12119, 2014

      • ianmcorvidae
        probably, if it's not within caa.org
      • 2014-05-01 12120, 2014

      • Gentlecat
        or you mean mirroring cover art archive in some way?
      • 2014-05-01 12157, 2014

      • Gentlecat
        ok, got it
      • 2014-05-01 12159, 2014

      • ianmcorvidae
        by server-side checking I mean just for the image, so it'd not block the main page loading
      • 2014-05-01 12128, 2014

      • ianmcorvidae
        set the src of the image to something like gentlecat.mbsandbox.org/rg-image/<uuid> and have that endpoint do the checking I mention
      • 2014-05-01 12150, 2014

      • ianmcorvidae
        but doing it within coverart_redirect would probably be best, since that's already designed to return 307s to the image
      • 2014-05-01 12101, 2014

      • rvedotrc joined the channel
      • 2014-05-01 12148, 2014

      • Th_rez_ joined the channel
      • 2014-05-01 12108, 2014

      • UmkaDK joined the channel
      • 2014-05-01 12157, 2014

      • JonnyJD joined the channel
      • 2014-05-01 12123, 2014

      • ijabz joined the channel
      • 2014-05-01 12153, 2014

      • ruaok joined the channel
      • 2014-05-01 12147, 2014

      • Gentlecat
        ruaok: good news! http://gentlecat.mbsandbox.org/ is up and running :)
      • 2014-05-01 12113, 2014

      • bandtrace joined the channel
      • 2014-05-01 12121, 2014

      • ruaok
        Nice.
      • 2014-05-01 12132, 2014

      • ruaok plays with it
      • 2014-05-01 12107, 2014

      • rvedotrc joined the channel
      • 2014-05-01 12138, 2014

      • ruaok
        there is no clear way how to add a new review from the home page.
      • 2014-05-01 12151, 2014

      • ianmcorvidae
        we were talking about that earlier :)
      • 2014-05-01 12157, 2014

      • ruaok
        ok. :)
      • 2014-05-01 12116, 2014

      • ruaok
        Gentlecat: when I rated your review the highlighting is quite clear
      • 2014-05-01 12118, 2014

      • ruaok
        Did you find this review useful? Yes No (delete your vote)
      • 2014-05-01 12130, 2014

      • ruaok
        the Yes, seems selected and enabled and the no is greyed out.
      • 2014-05-01 12142, 2014

      • Gentlecat
      • 2014-05-01 12145, 2014

      • ruaok
        I selected No, so this is a little confusing.
      • 2014-05-01 12105, 2014

      • Gentlecat
        let me take a look at voting
      • 2014-05-01 12116, 2014

      • ruaok
        good good.
      • 2014-05-01 12122, 2014

      • ruaok
        I quite like how this is coming along. :)
      • 2014-05-01 12109, 2014

      • Gentlecat
        me too! exciting stuff
      • 2014-05-01 12112, 2014

      • Gentlecat
        how about we highlight selected vote with a specific color: Yes = green, No = red?
      • 2014-05-01 12151, 2014

      • Gentlecat
        to make it more obvious which option has been selected
      • 2014-05-01 12128, 2014

      • ruaok
        green and red are intuitive, but garish.
      • 2014-05-01 12150, 2014

      • ruaok
        the selected option should be one of the non-gray colors that are already used.
      • 2014-05-01 12159, 2014

      • ruaok
        and the not selected option should be gray
      • 2014-05-01 12113, 2014

      • Gentlecat
        so just swap them?
      • 2014-05-01 12125, 2014

      • Gentlecat
        styles i mean
      • 2014-05-01 12128, 2014

      • UmkaDK joined the channel
      • 2014-05-01 12146, 2014

      • ruaok
        lets try that. it might be enough
      • 2014-05-01 12115, 2014

      • bandtrace joined the channel
      • 2014-05-01 12144, 2014

      • jesus2099 joined the channel
      • 2014-05-01 12108, 2014

      • Gentlecat
        ruaok: I updated it, take a look
      • 2014-05-01 12143, 2014

      • ruaok
        did you change how to get to a review?
      • 2014-05-01 12153, 2014

      • ruaok
        I can’t get to reivews anymore
      • 2014-05-01 12124, 2014

      • Gentlecat
        from what page?
      • 2014-05-01 12125, 2014

      • Gentlecat
        if you are on album page then yeah, there's an icon missing
      • 2014-05-01 12146, 2014

      • Gentlecat
        i fixed it in one of PRs
      • 2014-05-01 12148, 2014

      • xBytez joined the channel
      • 2014-05-01 12156, 2014

      • bandtrace joined the channel
      • 2014-05-01 12108, 2014

      • ruaok
        grrr.
      • 2014-05-01 12116, 2014

      • ruaok
        thanks for not sending me notifications, bitbucket.