#metabrainz

/

      • zas
        Leo_Verto[m]: there were changes, but none of those are "offficial" (like X-Rate-Limit headers)
      • 2019-03-14 07328, 2019

      • Leo_Verto[m]
        thanks zas, when is new-ratelimit expected to be released?
      • 2019-03-14 07331, 2019

      • zas
        but ws usage recommendations are still the same (proper UA, and reasonable use)
      • 2019-03-14 07303, 2019

      • Leo_Verto[m]
        amCap1712: that's wonderful to hear, what do you want to work on?
      • 2019-03-14 07328, 2019

      • amCap1712
        Leo_Verto[m]: I am working on MusicBrainz for Android.
      • 2019-03-14 07331, 2019

      • Leo_Verto[m]
        ah yeah, I saw your reply to the community thread about the app needing a maintainer!
      • 2019-03-14 07338, 2019

      • zas
        which new-ratelimit ? I'm not aware of anything going on, we had plans to move to an API manager (and/or so called ws/2.5) but for now nothing changed, apart the fact we are more or less deprecating XML (we recommend to use json ws, but that's not mandatory)
      • 2019-03-14 07302, 2019

      • zas
        Picard uses json only now
      • 2019-03-14 07320, 2019

      • Leo_Verto[m]
        you talked about some kind of adaptive rate limiting during the summit, is that part of the ws/2.5 changes?
      • 2019-03-14 07334, 2019

      • zas
        Yes. Is there anything wrong with current stuff ?
      • 2019-03-14 07334, 2019

      • Leo_Verto[m]
        Not at all, I've spent some time working on the Jellyfin project (which is a recent for of Emby) and been asked to review this PR https://github.com/jellyfin/jellyfin/pull/1105 so I was wondering if we were still doing everything the right way
      • 2019-03-14 07346, 2019

      • zas
        does the code checks for http error codes ? in case of 503, it is recommended to delay, and try again (rather than giving up, or retrying without delay), and of course, 1 req/s (in fact you can often do faster reqs in practice, as we tolerate "bursts") but anything permanently hitting our servers above 1 req/s might be banned without notice.
      • 2019-03-14 07349, 2019

      • zas
        and a user agent string allowing app identification (ideally with url or email, so we can contact people if needed)
      • 2019-03-14 07346, 2019

      • Leo_Verto[m]
        Roger that, I'll relay that info
      • 2019-03-14 07340, 2019

      • amCap1712
        gr0uch0mars:
      • 2019-03-14 07317, 2019

      • amCap1712
        updated the proposal. after it I felt, a lot of changes can be made. Much of everything was explained.the technical aspect are covered but it lacked as to the practical aspect.
      • 2019-03-14 07342, 2019

      • amCap1712
        as to what changes would enable us to do what.
      • 2019-03-14 07340, 2019

      • alastairp
        yvanzo: no, I don't have permissions to see project settings
      • 2019-03-14 07331, 2019

      • yvanzo
        neither do I for AB, bitmap might have global access
      • 2019-03-14 07355, 2019

      • bitmap
        I approved the request that just came in if that helps
      • 2019-03-14 07308, 2019

      • yvanzo
        Thank you, it allowed me to enable JIRA plugin for AB project on Sentry.
      • 2019-03-14 07323, 2019

      • yvanzo
        bitmap: I renamed mb-sentry to dev-sentry on JIRA, so that every project can use it.
      • 2019-03-14 07302, 2019

      • bitmap
        not sure what that is :)
      • 2019-03-14 07347, 2019

      • yvanzo
      • 2019-03-14 07306, 2019

      • yvanzo
        Updated credentials in MB project on Sentry too.
      • 2019-03-14 07352, 2019

      • ruaok
        Mr_Monkey: did you add debuggin info to #549?
      • 2019-03-14 07354, 2019

      • bitmap
        oh ok. that always confused me because I couldn't figure out who was leaving the comment
      • 2019-03-14 07324, 2019

      • yvanzo
        alastairp: It is now set up for AB project on Sentry, there is a JIRA button in the toolbar when you review an issue on Sentry.
      • 2019-03-14 07333, 2019

      • alastairp
        yvanzo: I see it now, thanks!
      • 2019-03-14 07351, 2019

      • alastairp
        and just linked a ticket
      • 2019-03-14 07304, 2019

      • alastairp
        annoying that it doesn't link to your username (the github one does)
      • 2019-03-14 07309, 2019

      • Mr_Monkey
        ruaok: Afirmative. You'll have to set your browser console's level so that you see the 'debug' level
      • 2019-03-14 07311, 2019

      • yvanzo
        bitmap: yeah, that’s clumsy but JIRA credentials are per-project and not per-user on Sentry.
      • 2019-03-14 07315, 2019

      • aliamo has quit
      • 2019-03-14 07323, 2019

      • ruaok
        great.
      • 2019-03-14 07337, 2019

      • ruaok
        lets see if I have enough brainpower to review it.
      • 2019-03-14 07341, 2019

      • aliamo joined the channel
      • 2019-03-14 07343, 2019

      • Mr_Monkey
        Also this ruaok: Yes, I can get the current state of the spot player: https://developer.spotify.com/documentation/web-p…
      • 2019-03-14 07305, 2019

      • Mr_Monkey
        So if we decide to go for a watchdog, we have that option
      • 2019-03-14 07313, 2019

      • ruaok
        ohhh, and it does not require an HTTP call?
      • 2019-03-14 07323, 2019

      • Mr_Monkey
        Correct
      • 2019-03-14 07328, 2019

      • ruaok
        well, let's see what this PR brings and then we'll see further.
      • 2019-03-14 07306, 2019

      • Mr_Monkey
        Well, I think it doesn't. Haven't peered at the implementation. It returns a promise, so it's possible there's an HTTP request behind it
      • 2019-03-14 07334, 2019

      • Mr_Monkey
        iliekcomputers: With this new PR I also hope to solve the issue you were seeing with the 404s. It should recover a bit better.
      • 2019-03-14 07346, 2019

      • Mr_Monkey
        Let me know if you see any difference
      • 2019-03-14 07336, 2019

      • bitmap
        yvanzo: it's perfectly fine to me if people sign their comments :)
      • 2019-03-14 07340, 2019

      • ruaok
        iliekcomputers: https://github.com/metabrainz/listenbrainz-server… can you please throw a quick look at that?
      • 2019-03-14 07355, 2019

      • iliekcomputers
        Yes, on it.
      • 2019-03-14 07359, 2019

      • ruaok
        <3
      • 2019-03-14 07307, 2019

      • aliamo has quit
      • 2019-03-14 07331, 2019

      • yvanzo
        oh right, should do that
      • 2019-03-14 07301, 2019

      • djwhitey joined the channel
      • 2019-03-14 07338, 2019

      • iliekcomputers
        lgtm
      • 2019-03-14 07304, 2019

      • iliekcomputers
        let me merge and deploy
      • 2019-03-14 07325, 2019

      • amCap1712
        can embed a video of the working instead of the mockups
      • 2019-03-14 07340, 2019

      • amCap1712
        since I have a working version, I suppose its better
      • 2019-03-14 07311, 2019

      • amCap1712
        I could upload a video to youtube and post the link in the proposal
      • 2019-03-14 07324, 2019

      • CatQuest
        [16:38] <Freso> amCap1712: I changed `post edit time limit` to 120 days now; I'm open to extend it further (or disable it entirely) too,
      • 2019-03-14 07324, 2019

      • CatQuest
        ah actually that's a good idea (disable entierly) since there actually was some old instrument post I wanted to update/edit but couldn't since I couldn't edit it
      • 2019-03-14 07331, 2019

      • iliekcomputers
        >npm WARN using --force I sure hope you know what you are doing.
      • 2019-03-14 07333, 2019

      • amCap1712
        would that be fine ruaok or yvanzo?
      • 2019-03-14 07333, 2019

      • iliekcomputers
        lol
      • 2019-03-14 07329, 2019

      • ruaok
        amCap1712: can you render the film on chocolate? or somehow apply an organge filter?
      • 2019-03-14 07334, 2019

      • ruaok
        that would most certainly help.
      • 2019-03-14 07337, 2019

      • CatQuest
        amCap1712: when it comes to artist links: (or instrument, album etc) I am more interested in wikidata, bandcamp, dicogs, thna facebook spotofy or twitter
      • 2019-03-14 07344, 2019

      • ruaok
        :-p
      • 2019-03-14 07302, 2019

      • ruaok
        (yes, videos are great. working code wins. always)
      • 2019-03-14 07318, 2019

      • amCap1712
        thanks ruaok. I
      • 2019-03-14 07320, 2019

      • CatQuest
        also plox no "icons instead of text" type links please. it's difficult ot know what it is if it doesnt hav a text
      • 2019-03-14 07336, 2019

      • iliekcomputers
      • 2019-03-14 07302, 2019

      • amCap1712
        got it CatQuest
      • 2019-03-14 07304, 2019

      • ruaok
        ohhh, users at the bottom. :)
      • 2019-03-14 07346, 2019

      • CatQuest
        as for apparence. ou sohld als ogoo look at chhavi_'s interface ideas on the ticket tracker (with component "design" I belive) we'd want the website and the app to be congruent in apparence (thought the colours of her mockups are not set in stone)
      • 2019-03-14 07330, 2019

      • ruaok
        filter mode "verbose", Mr_Monkey? I see no debug.
      • 2019-03-14 07348, 2019

      • CatQuest
        for links i'd say have the name of websitte (and (also?) icon, (ther shoudl be fetchable from the mb api))
      • 2019-03-14 07300, 2019

      • CatQuest
        and on longpress the full url cna be shown
      • 2019-03-14 07301, 2019

      • CatQuest
        maybe
      • 2019-03-14 07304, 2019

      • Mr_Monkey
        ruaok: Yes, verbose. Should appear in blue
      • 2019-03-14 07338, 2019

      • CatQuest
        also if there are the indian streaming sites yo ushould suggest them to be auto-thingy in mb, (that way the icon cna be stored by us too)
      • 2019-03-14 07340, 2019

      • reosarevok
        Your mom is verbose
      • 2019-03-14 07317, 2019

      • CatQuest
        oh a separate page. then alright, the url full there
      • 2019-03-14 07322, 2019

      • iliekcomputers
        we just follow each other and listen to the same stuff lol
      • 2019-03-14 07341, 2019

      • iliekcomputers
        Mr_Monkey and ruaok are listening to the same song.
      • 2019-03-14 07346, 2019

      • Mr_Monkey
        iliekcomputers: Hah! I try to inject some new music once every couple of days :D
      • 2019-03-14 07348, 2019

      • reosarevok
        Yay echo chambers :p
      • 2019-03-14 07354, 2019

      • ruaok
        I wonder why.
      • 2019-03-14 07358, 2019

      • iliekcomputers
        :)
      • 2019-03-14 07310, 2019

      • ruaok
        I wonder if we can make it so that all of us listen to only one song. convergence.
      • 2019-03-14 07319, 2019

      • ruaok
        🤬
      • 2019-03-14 07320, 2019

      • reosarevok
      • 2019-03-14 07333, 2019

      • reosarevok
        There, someone play some Estonian electro folk. That'll confuse them
      • 2019-03-14 07347, 2019

      • iliekcomputers
        Mr_Monkey: ooooh alive 2007
      • 2019-03-14 07349, 2019

      • iliekcomputers
        noice
      • 2019-03-14 07314, 2019

      • Mr_Monkey
        Yeah! They never did anything better, IMO
      • 2019-03-14 07355, 2019

      • CatQuest
        oh i thoguht yo uwhere saying *I* was too verbose D:
      • 2019-03-14 07306, 2019

      • CatQuest
        I mea, yea that's true but
      • 2019-03-14 07331, 2019

      • CatQuest
        reosarevok: tell me these lell'o things are on bandcmap!
      • 2019-03-14 07344, 2019

      • ruaok
      • 2019-03-14 07353, 2019

      • ruaok
        so far so good, but the sort order is borked.
      • 2019-03-14 07314, 2019

      • reosarevok
        CatQuest: no, but their albums are https://tradattackmusic.bandcamp.com/
      • 2019-03-14 07325, 2019

      • CatQuest
        reosarevok: :D
      • 2019-03-14 07340, 2019

      • iliekcomputers
        i think, initial sort order is by username and then later by timestamp.
      • 2019-03-14 07342, 2019

      • iliekcomputers
        not sure though
      • 2019-03-14 07345, 2019

      • reosarevok
        iliekcomputers: is that the pyramid tour? I was on that tour, it was cool
      • 2019-03-14 07303, 2019

      • iliekcomputers
        reosarevok: wow
      • 2019-03-14 07308, 2019

      • Mr_Monkey
        Yeha, sort order is as sent by the server, which is probably what iliekcomputers describes
      • 2019-03-14 07312, 2019

      • CatQuest
        ugh pages that don't allow you to scroll
      • 2019-03-14 07313, 2019

      • reosarevok
        Yeah, I'm old
      • 2019-03-14 07317, 2019

      • iliekcomputers
        from all i've read about it, it seems awesome.
      • 2019-03-14 07319, 2019

      • iliekcomputers
        lucky
      • 2019-03-14 07324, 2019

      • iliekcomputers jealous
      • 2019-03-14 07325, 2019

      • Mr_Monkey
        Second that
      • 2019-03-14 07330, 2019

      • Mr_Monkey jealous too
      • 2019-03-14 07301, 2019

      • Mr_Monkey
        All right, let me chuck in an initial sort by time, then, and I'll be done for the day
      • 2019-03-14 07305, 2019

      • CatQuest
        reosarevok: how many songs are on that esto electro folk thing?
      • 2019-03-14 07315, 2019

      • CatQuest
        (becasue i have on idea becasue the page literally wont let me scroll)
      • 2019-03-14 07317, 2019

      • reosarevok
        It's around 1h long I think?
      • 2019-03-14 07319, 2019

      • iliekcomputers
        should it be fixed in the backend?
      • 2019-03-14 07329, 2019

      • CatQuest
        I only get 30 secs. how many *songs*
      • 2019-03-14 07339, 2019

      • reosarevok
        17 or so
      • 2019-03-14 07344, 2019

      • CatQuest
        ah
      • 2019-03-14 07351, 2019

      • CatQuest
        damn this album wasn't on bandcamp
      • 2019-03-14 07327, 2019

      • iliekcomputers
        hmm, there is an order by time in the query already.
      • 2019-03-14 07341, 2019

      • CatQuest
        wait... that thigyo ulinked was that an album?
      • 2019-03-14 07345, 2019

      • ruaok
        yes, but users get returned in batches.
      • 2019-03-14 07351, 2019

      • ruaok
        each user is sorted.
      • 2019-03-14 07357, 2019

      • iliekcomputers
        right
      • 2019-03-14 07304, 2019

      • CatQuest
        ruaok: you want true shuffle right?
      • 2019-03-14 07321, 2019

      • iliekcomputers
        tbh, i'd expect the entire list to be sorted by time from that query...
      • 2019-03-14 07324, 2019

      • iliekcomputers
        weird.
      • 2019-03-14 07332, 2019

      • CatQuest would want it shuffled
      • 2019-03-14 07339, 2019

      • ruaok
        iliekcomputers: look at how they are collated.
      • 2019-03-14 07353, 2019

      • ruaok
        CatQuest: no, I want seqential sort order.
      • 2019-03-14 07355, 2019

      • iliekcomputers
        oooh
      • 2019-03-14 07357, 2019

      • iliekcomputers
        ok.
      • 2019-03-14 07359, 2019

      • iliekcomputers
        right.
      • 2019-03-14 07314, 2019

      • CatQuest
        ¯\_(ツ)_/¯
      • 2019-03-14 07344, 2019

      • CatQuest
        i thikn this might wary for differnt people? let users be albe to sort the list/shuffle it and tnhe jsut play liek that? idk
      • 2019-03-14 07327, 2019

      • gr0uch0mars
        amCap1712: great! Now that everything is written down we can estimate each part. I would add more time to each task as a prevention, like the LiveData challenge now
      • 2019-03-14 07334, 2019

      • ruaok
        we just removed the sort order -- it didn't do what we wanted.
      • 2019-03-14 07341, 2019

      • ruaok
        we're going to keep it simple and release this.
      • 2019-03-14 07347, 2019

      • ruaok
        people will tell us what they want.
      • 2019-03-14 07301, 2019

      • gr0uch0mars
        I started a change on the code before at work, and it looked good. But I did not complete it
      • 2019-03-14 07314, 2019

      • gr0uch0mars
        I will try to make it work and pass to you
      • 2019-03-14 07319, 2019

      • amCap1712
        ok
      • 2019-03-14 07322, 2019

      • iliekcomputers
        ruaok: yeah, i like that idea :)
      • 2019-03-14 07329, 2019

      • ruaok
        iliekcomputers: thank you for playing something different. :)
      • 2019-03-14 07345, 2019

      • iliekcomputers decided to add some Indian music in there
      • 2019-03-14 07354, 2019

      • gr0uch0mars
        Besides, if we want to progress quick in the refactor part (to leave as much time as possible to Picard feature), we should focus on keeping the code clean and manageable
      • 2019-03-14 07303, 2019

      • gr0uch0mars
        That would allow quick changes