#metabrainz

/

      • drsaunder has quit
      • 2017-06-25 17627, 2017

      • antgel has quit
      • 2017-06-25 17612, 2017

      • drsaunder joined the channel
      • 2017-06-25 17605, 2017

      • D4RK-PH0ENiX has quit
      • 2017-06-25 17645, 2017

      • D4RK-PH0ENiX joined the channel
      • 2017-06-25 17622, 2017

      • Sam__ joined the channel
      • 2017-06-25 17639, 2017

      • Sam__ has quit
      • 2017-06-25 17657, 2017

      • drsaunder has quit
      • 2017-06-25 17615, 2017

      • culinko joined the channel
      • 2017-06-25 17616, 2017

      • culinko has quit
      • 2017-06-25 17616, 2017

      • culinko joined the channel
      • 2017-06-25 17602, 2017

      • dragonzeron joined the channel
      • 2017-06-25 17628, 2017

      • drsaunders joined the channel
      • 2017-06-25 17641, 2017

      • Sam__ joined the channel
      • 2017-06-25 17601, 2017

      • Sam__
        zas online yet?
      • 2017-06-25 17653, 2017

      • Sam__ has left the channel
      • 2017-06-25 17623, 2017

      • Sam__ joined the channel
      • 2017-06-25 17611, 2017

      • zas
        too late
      • 2017-06-25 17632, 2017

      • samj1912 has quit
      • 2017-06-25 17639, 2017

      • Sam__ has quit
      • 2017-06-25 17650, 2017

      • zas
        Is there anyone here having a QNAP nas ? I need some infos about it, and what's running on it.
      • 2017-06-25 17643, 2017

      • ruaok
        iliekcomputers: thanks for the reviews. I totally forgot that you came up with helper functions.
      • 2017-06-25 17634, 2017

      • iliekcomputers
        ruaok: you're welcome :)
      • 2017-06-25 17620, 2017

      • ruaok
        finished most changes. the counts_*py scripts are helpers. not sure I care to keep them
      • 2017-06-25 17609, 2017

      • iliekcomputers
        Methinks not having them would be okay
      • 2017-06-25 17608, 2017

      • ruaok
        I might keep the flush one.
      • 2017-06-25 17615, 2017

      • ruaok
        but I'll fix it up.
      • 2017-06-25 17618, 2017

      • agentsim has quit
      • 2017-06-25 17639, 2017

      • agentsim joined the channel
      • 2017-06-25 17602, 2017

      • agentsim has quit
      • 2017-06-25 17616, 2017

      • agentsim joined the channel
      • 2017-06-25 17626, 2017

      • ruaok
        ok, changes pushed, if you want to look again.
      • 2017-06-25 17658, 2017

      • github joined the channel
      • 2017-06-25 17658, 2017

      • github
        [listenbrainz-server] mayhem closed pull request #210: Import page re-shuffle (master...import-page-layout) https://git.io/vQOm6
      • 2017-06-25 17658, 2017

      • github has left the channel
      • 2017-06-25 17603, 2017

      • github joined the channel
      • 2017-06-25 17603, 2017

      • github
        [listenbrainz-server] mayhem closed pull request #209: Listen counts rebased (master...listen-counts-rebased) https://git.io/vQOfF
      • 2017-06-25 17603, 2017

      • github has left the channel
      • 2017-06-25 17614, 2017

      • ruaok
        listen counts, finally merged. damn, that took long and was painful.
      • 2017-06-25 17630, 2017

      • Mineo has quit
      • 2017-06-25 17635, 2017

      • iliekcomputers
        Just incremental imports left and hopefully that's it for the community beta at least :)
      • 2017-06-25 17616, 2017

      • arthelon
        LordSputnik: Getting a jQuery undefined when running the most up to date from master
      • 2017-06-25 17623, 2017

      • arthelon
        Was this there before?
      • 2017-06-25 17604, 2017

      • LordSputnik
        Hmm that'll be because of my change last week from require to import
      • 2017-06-25 17632, 2017

      • LordSputnik
        I'll need to do some investigating
      • 2017-06-25 17646, 2017

      • arthelon
        found it
      • 2017-06-25 17652, 2017

      • arthelon
        f777e0479b93b32d7c34eec72420a756a6caf877 is the issue
      • 2017-06-25 17610, 2017

      • arthelon
        yeah that makes sense
      • 2017-06-25 17606, 2017

      • arthelon
        May need to use something like https://github.com/thlorenz/browserify-shim
      • 2017-06-25 17610, 2017

      • psolanki sent a long message: psolanki_2017-06-25_09:11:10.txt <https://matrix.org/_matrix/media/v1/download/matrix.org/TMhHOoTOBprAxXrxrDmyHOit>
      • 2017-06-25 17659, 2017

      • agentsim has quit
      • 2017-06-25 17611, 2017

      • arthelon
        ok, adding browserify-shim resolved the issue. Are there any other non-commonjs dependencies?
      • 2017-06-25 17619, 2017

      • arthelon
        asides from jquery
      • 2017-06-25 17656, 2017

      • arthelon
        everything else in package.json looks alright to me
      • 2017-06-25 17652, 2017

      • ruaok
        iliekcomputers: my first full import from last.fm just finished.
      • 2017-06-25 17601, 2017

      • ruaok
        now listening to more music to test incremental imports.
      • 2017-06-25 17609, 2017

      • ruaok
        if that is fine, I'll approve and merge.
      • 2017-06-25 17618, 2017

      • ruaok
        then wipe/deploy
      • 2017-06-25 17638, 2017

      • ruaok
        iliekcomputers: we made changes to the scraper. you should bump the version to 1.7.
      • 2017-06-25 17645, 2017

      • ruaok
        then I'll merge, imports are working. :)
      • 2017-06-25 17658, 2017

      • Sophist-UK has quit
      • 2017-06-25 17624, 2017

      • Sophist-UK joined the channel
      • 2017-06-25 17605, 2017

      • ruaok
        actually, I'll jump bump the version.
      • 2017-06-25 17652, 2017

      • iliekcomputers
        ruaok: okay great, i was doing it but sure
      • 2017-06-25 17653, 2017

      • github joined the channel
      • 2017-06-25 17653, 2017

      • github
        [listenbrainz-server] mayhem closed pull request #207: LB-20 / LB-31 / LB-182: Incremental imports (master...incremental-imports) https://git.io/vQYdR
      • 2017-06-25 17653, 2017

      • github has left the channel
      • 2017-06-25 17651, 2017

      • ruaok
        all merged.
      • 2017-06-25 17613, 2017

      • ruaok
        can you please have a look at the bugs and close things that need closing?
      • 2017-06-25 17631, 2017

      • iliekcomputers
        ok
      • 2017-06-25 17640, 2017

      • ruaok
        only the API docs one is left .
      • 2017-06-25 17648, 2017

      • ruaok checks docstrings
      • 2017-06-25 17651, 2017

      • Sophist-UK has quit
      • 2017-06-25 17601, 2017

      • ruaok
        hmmm.
      • 2017-06-25 17603, 2017

      • ruaok
      • 2017-06-25 17616, 2017

      • Sophist-UK joined the channel
      • 2017-06-25 17618, 2017

      • ruaok
        that should really be under the API, no?
      • 2017-06-25 17623, 2017

      • ruaok
        a little late now....
      • 2017-06-25 17634, 2017

      • ruaok
        but I think that should be under API and then have docstrings added.
      • 2017-06-25 17641, 2017

      • ruaok
        then we can close the API docs bug too
      • 2017-06-25 17640, 2017

      • iliekcomputers
        ruaok: okay, i'll put it in api
      • 2017-06-25 17626, 2017

      • arthelon
        LordSputnik: Only noticed one instance of jQuery being used btw, react-bootstrap doesn't have a dependency on it either
      • 2017-06-25 17641, 2017

      • Sophist-UK has quit
      • 2017-06-25 17605, 2017

      • Sophist-UK joined the channel
      • 2017-06-25 17600, 2017

      • arthelon
        Nvm, it's used quite extensively with select
      • 2017-06-25 17610, 2017

      • LordSputnik
        arthelon: Select2?
      • 2017-06-25 17624, 2017

      • LordSputnik
        If we get rid of that, can we drop jQuery?
      • 2017-06-25 17635, 2017

      • arthelon
        I believe so
      • 2017-06-25 17600, 2017

      • arthelon
        I only see jQuery usage in the EntitySearch and Select2 components
      • 2017-06-25 17607, 2017

      • iliekcomputers
        ruaok: you think the get and update should be different endpoints?
      • 2017-06-25 17622, 2017

      • ruaok
        naw, the same with GET/POST
      • 2017-06-25 17648, 2017

      • iliekcomputers
        okay cool
      • 2017-06-25 17634, 2017

      • arthelon
        LordSputnik: Maybe we can replace select2 with this? http://jedwatson.github.io/react-select/ The largest refactor would be the select2Options that are being passed around
      • 2017-06-25 17606, 2017

      • LordSputnik
        That's what I've started doing with the new entity editor
      • 2017-06-25 17621, 2017

      • LordSputnik
        It has much better performance in react too
      • 2017-06-25 17658, 2017

      • LordSputnik
        Maybe we can make a wrapper component to translate between the Select2 props and the react select props?
      • 2017-06-25 17615, 2017

      • LordSputnik
        Then update to use react select without the wrapper gradually
      • 2017-06-25 17645, 2017

      • arthelon
        Don't think it's too big of a rewrite
      • 2017-06-25 17602, 2017

      • arthelon
        we can try changing all the occurences right away
      • 2017-06-25 17627, 2017

      • arthelon
        Probably less work than going through every occurence, seeing what's being used, and writing out the equivalent in react-select
      • 2017-06-25 17656, 2017

      • arthelon
        It's mainly for the select2Options prop, all the other props seem to be supported in react-select
      • 2017-06-25 17647, 2017

      • arthelon
        Everything in select2Options should be its own prop
      • 2017-06-25 17606, 2017

      • ruaok
        iliekcomputers: XMLHttpRequest cannot load https://beta-api.listenbrainz.org/user/latest-imp…. Response to preflight request doesn't pass access control check: No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'https://beta.listenbrainz.org' is therefore not allowed access.
      • 2017-06-25 17608, 2017

      • ruaok
        heh.
      • 2017-06-25 17639, 2017

      • ruaok
        my personal listen count, the big query listen count and system wide listen counts are all the same.
      • 2017-06-25 17643, 2017

      • ruaok high fives iliekcomputers
      • 2017-06-25 17657, 2017

      • iliekcomputers
        ruaok: why exactly does that error happen for the new endpoint but not for submit-listens? Not familiar with this very much
      • 2017-06-25 17654, 2017

      • ruaok
      • 2017-06-25 17641, 2017

      • ruaok
        the crossdomain decorator emits the Access control header which defines the API end point and allows the web browser to make calls to it.
      • 2017-06-25 17656, 2017

      • ruaok
        without it, you get the error I posted above
      • 2017-06-25 17652, 2017

      • iliekcomputers
        weird, I had added that decorator to the function https://github.com/metabrainz/listenbrainz-server…
      • 2017-06-25 17613, 2017

      • ruaok
        but that was on a different domain.
      • 2017-06-25 17619, 2017

      • ruaok
        beta vs beta-api
      • 2017-06-25 17645, 2017

      • arbenina_ joined the channel
      • 2017-06-25 17633, 2017

      • agentsim joined the channel
      • 2017-06-25 17618, 2017

      • ruaok
        ohhh, check it out!
      • 2017-06-25 17620, 2017

      • ruaok
        win2003.verylegit.link/server1337504warez=64.zip.sh
      • 2017-06-25 17640, 2017

      • agentsim has quit
      • 2017-06-25 17602, 2017

      • github joined the channel
      • 2017-06-25 17602, 2017

      • github
        [listenbrainz-server] paramsingh opened pull request #211: Move latest-import endpoint to api (master...incremental-import/move-to-api) https://git.io/vQOd1
      • 2017-06-25 17602, 2017

      • github has left the channel
      • 2017-06-25 17653, 2017

      • ruaok
        woo
      • 2017-06-25 17615, 2017

      • ruaok
        iliekcomputers: the code for the test or the view didn't really change, did it?
      • 2017-06-25 17627, 2017

      • iliekcomputers
        ruaok: nope no change
      • 2017-06-25 17653, 2017

      • iliekcomputers
        just moved them from one place to another and changed the url accordingly
      • 2017-06-25 17659, 2017

      • ruaok
        k
      • 2017-06-25 17632, 2017

      • github joined the channel
      • 2017-06-25 17632, 2017

      • github
        [listenbrainz-server] mayhem closed pull request #211: Move latest-import endpoint to api (master...incremental-import/move-to-api) https://git.io/vQOd1
      • 2017-06-25 17632, 2017

      • github has left the channel
      • 2017-06-25 17642, 2017

      • D4RK-PH0ENiX has quit
      • 2017-06-25 17607, 2017

      • D4RK-PH0ENiX joined the channel
      • 2017-06-25 17626, 2017

      • Gentlecat
        ferbncode: this might be caused by the DB code changes https://sentry.metabrainz.org/share/issue/342e323…
      • 2017-06-25 17634, 2017

      • ruaok
        iliekcomputers: still getting the CORS error. :(
      • 2017-06-25 17630, 2017

      • iliekcomputers
        weird :/
      • 2017-06-25 17653, 2017

      • ruaok
        XMLHttpRequest cannot load https://beta-api.listenbrainz.org/1/latest-import. Response to preflight request doesn't pass access control check: No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'https://beta.listenbrainz.org' is therefore not allowed access. The response had HTTP status code 405.
      • 2017-06-25 17605, 2017

      • psolanki
        Gentlecat: Hi
      • 2017-06-25 17628, 2017

      • psolanki
        I looked at how to update avg. rating via triggers but I think it can be done through db like MB
      • 2017-06-25 17645, 2017

      • psolanki
        Also, I had started working on it
      • 2017-06-25 17659, 2017

      • psolanki
        Let me know what you think
      • 2017-06-25 17656, 2017

      • ruaok
      • 2017-06-25 17615, 2017

      • ruaok
        the Access-Control headers are different between the two requests.
      • 2017-06-25 17616, 2017

      • iliekcomputers
        But Access-Control-Allow-Origin is * in both of them
      • 2017-06-25 17621, 2017

      • Gentlecat
        psolanki: how does MB do it?
      • 2017-06-25 17656, 2017

      • psolanki
        MB does it using perl only
      • 2017-06-25 17646, 2017

      • psolanki
        I will have it done by tonight, you can review it and see it's workable
      • 2017-06-25 17624, 2017

      • Gentlecat
        well, having a trigger would be better, I think
      • 2017-06-25 17611, 2017

      • Gentlecat
        but if it's easier to do that as a part of the server code then try that
      • 2017-06-25 17643, 2017

      • psolanki
        a part of server code?
      • 2017-06-25 17630, 2017

      • Gentlecat
        like it's done in MB
      • 2017-06-25 17623, 2017

      • psolanki
        I will update the PR and then we will discuss further if that's okay with you