#metabrainz

/

      • _lucifer
        *showing the last passing version
      • 2021-03-18 07724, 2021

      • ruaok
        alastairp: yeah, I had pretty much came to the same conclusion as that blog post -- the wait call is the way to go.
      • 2021-03-18 07735, 2021

      • _lucifer
        However, I asked because the timeline api docs are missing at https://listenbrainz.readthedocs.io/en/production…
      • 2021-03-18 07701, 2021

      • _lucifer
        i just realised that's because we haven't added that blueprint to api.rst
      • 2021-03-18 07754, 2021

      • alastairp
      • 2021-03-18 07705, 2021

      • alastairp
        I didn't know that it was configured like this
      • 2021-03-18 07726, 2021

      • alastairp
        so it appears that production is by default, but master is also available at lastest
      • 2021-03-18 07724, 2021

      • _lucifer
        oh nice :D, i see docs for recommendation apis are missing as well. i'll open a PR.
      • 2021-03-18 07702, 2021

      • alastairp
        I'm just going to merge 1340 now, I made some changes
      • 2021-03-18 07736, 2021

      • _lucifer
        alastairp: i was just going through that PR, we only seem to use code-block at two places
      • 2021-03-18 07757, 2021

      • _lucifer
        at others we just put the json block and it renders fine.
      • 2021-03-18 07705, 2021

      • alastairp
        interesting
      • 2021-03-18 07707, 2021

      • alastairp
        let me look
      • 2021-03-18 07735, 2021

      • _lucifer
        see the `validate-token` endpoint
      • 2021-03-18 07738, 2021

      • alastairp
        yeah, interesting. but the formatting and colours are different if you use ..code-block:: json :)
      • 2021-03-18 07712, 2021

      • _lucifer
        yeah right, should we use the same thing everywhere?
      • 2021-03-18 07742, 2021

      • alastairp
        I think we should just use one way everywhere but it really doesn't matter which one
      • 2021-03-18 07758, 2021

      • _lucifer
        agreed.
      • 2021-03-18 07740, 2021

      • alastairp
      • 2021-03-18 07748, 2021

      • alastairp
        There is a “highlighting language” for each source file. Per default, this is 'python' as the majority of files will have to highlight Python snippets, but the doc-wide default can be set with the highlight_language config value.
      • 2021-03-18 07753, 2021

      • alastairp
        For documents that have to show snippets in different languages, there’s also a code-block directive that is given the highlighting language directly:
      • 2021-03-18 07719, 2021

      • alastairp
        right. so my intuition is that we're using :: (for example in validate-token) and it's highlighting it as a python doct
      • 2021-03-18 07720, 2021

      • alastairp
        dict
      • 2021-03-18 07740, 2021

      • alastairp
        but in the case where we use code-block, it's rendering it the same, but highlighting with json instead of python
      • 2021-03-18 07746, 2021

      • _lucifer
        makes sense
      • 2021-03-18 07758, 2021

      • alastairp
        let's use code-block:: json
      • 2021-03-18 07701, 2021

      • alastairp
        it's nice and explicit
      • 2021-03-18 07703, 2021

      • _lucifer
        👍
      • 2021-03-18 07736, 2021

      • _lucifer
        let's merge this PR then, I'll include the code-block changes in my PR to add the missing api socs
      • 2021-03-18 07735, 2021

      • BrainzGit
        [listenbrainz-server] alastair merged pull request #1340 (master…LB-774): LB-774: Fix Sphinx Documentation Warnings https://github.com/metabrainz/listenbrainz-server…
      • 2021-03-18 07719, 2021

      • alastairp
        thanks
      • 2021-03-18 07714, 2021

      • _lucifer
        alastairp, another thought that came to my mind recently. reviewing code is as much work as writing code so it would be nice if automatically added `Reviewed By` to commit messages on a merge according to users who reviewed the PR.
      • 2021-03-18 07729, 2021

      • BrainzGit
        [musicbrainz-server] reosarevok opened pull request #1996 (master…MBS-10847): MBS-10847 (I): Improve search help box on edit listings https://github.com/metabrainz/musicbrainz-server/…
      • 2021-03-18 07715, 2021

      • _lucifer
        ListenBrainz uses collaborative filtering to generate recording recommendations. These are further to processed to generate playlists for users. These api endpoints allow to fetch the actual recordings programmatically.
      • 2021-03-18 07740, 2021

      • _lucifer
        ruaok: how does that sound for description of get_recommendations edpoint?
      • 2021-03-18 07735, 2021

      • ruaok
        Let me look when we return from lunch, _lucifer
      • 2021-03-18 07746, 2021

      • _lucifer
        👍
      • 2021-03-18 07707, 2021

      • reosarevok
        ruaok: the latest "Re: Please verify your email address" is actually an answer to you (since you might not think of checking it)
      • 2021-03-18 07723, 2021

      • c1e0 has quit
      • 2021-03-18 07729, 2021

      • BrainzGit
        [musicbrainz-server] reosarevok opened pull request #1997 (production…MBS-11472): MBS-11472: Do partition_by before converting attribute to JSON https://github.com/metabrainz/musicbrainz-server/…
      • 2021-03-18 07751, 2021

      • shivam-kapila
        CatQuest: oh so it was actually the extension doing its work ;)
      • 2021-03-18 07741, 2021

      • reosarevok
        bitmap: ^ please check whether there's a simpler way of fixing that when around :)
      • 2021-03-18 07705, 2021

      • CatQuest
        shivam-kapila: well yes, but no :D
      • 2021-03-18 07737, 2021

      • ruaok
        reosarevok: thanks. Interesting.
      • 2021-03-18 07729, 2021

      • reosarevok
        bitmap: also - https://community.metabrainz.org/t/request-timeou… not sure what if anything can be done with this before the schema change, can you take a look tho
      • 2021-03-18 07710, 2021

      • alastairp
        _lucifer: hmm, interesting idea. do you think that the data on the PR on github isn't enough?
      • 2021-03-18 07723, 2021

      • alastairp
        how would you suggest we do this?
      • 2021-03-18 07757, 2021

      • alastairp
        is this to acknowledge the contribution that the reviewers make?
      • 2021-03-18 07709, 2021

      • _lucifer
        yes, the intent is that.
      • 2021-03-18 07705, 2021

      • _lucifer
        currently, i do not have an idea how to approach it but i have seen many other open source projects do it.
      • 2021-03-18 07712, 2021

      • ruaok
        _lucifer: you talking about this endpoint? https://github.com/metabrainz/listenbrainz-server…
      • 2021-03-18 07722, 2021

      • _lucifer
        ruaok, yes
      • 2021-03-18 07704, 2021

      • yubhav joined the channel
      • 2021-03-18 07718, 2021

      • dpmittal_ joined the channel
      • 2021-03-18 07756, 2021

      • ruaok
        "ListenBrainz uses collaborative filtering to generate recording recommendations, which may be further processed to generate playlists for users. This api endpoints allow to fetch the raw collaborative filtered recording IDs."
      • 2021-03-18 07700, 2021

      • ruaok
        it how I would say it.
      • 2021-03-18 07719, 2021

      • _lucifer
        sounds good, i'll use that. thanks :D
      • 2021-03-18 07747, 2021

      • ruaok
        np.
      • 2021-03-18 07712, 2021

      • _lucifer
        when do we have the meeting for Oauth discussion?
      • 2021-03-18 07728, 2021

      • dpmittal_ is now known as dpmittal
      • 2021-03-18 07746, 2021

      • alastairp
        hi dpmittal. I merged your PRs, thank you!
      • 2021-03-18 07732, 2021

      • ruaok
        _lucifer: ready for some task planning for the next few weeks?
      • 2021-03-18 07746, 2021

      • _lucifer
        yup :)
      • 2021-03-18 07730, 2021

      • dpmittal
        hi alastairp Yea I saw that, Thanks a lot for reviewing and merging the changes :D Actually, by the time I had a look at your suggestions you had already committed them, Apologies for that
      • 2021-03-18 07744, 2021

      • ruaok
        ok, we see three distinct projects headed your way.
      • 2021-03-18 07756, 2021

      • ruaok
        1. YouTube player reimplemention
      • 2021-03-18 07700, 2021

      • ruaok
        2. AB work.
      • 2021-03-18 07710, 2021

      • ruaok
        3. MetaBrainz OAuth/user account migration.
      • 2021-03-18 07735, 2021

      • ruaok
        we hope that we can do the youtube stuff in 2 weeks time and then 2 weeks for AB.
      • 2021-03-18 07753, 2021

      • ruaok
        alastair will plan out the AB work and will have more details once you finish up the YT work stuff.
      • 2021-03-18 07715, 2021

      • ruaok
        in the meantime alastairp and I will meet next week (and likely the week thereafter) to work out the details of the OAuth spec.
      • 2021-03-18 07716, 2021

      • shivam-kapila
        I just noticed we now have 10 sponsors on GitHub. Yay!!
      • 2021-03-18 07721, 2021

      • _lucifer
        sounds good
      • 2021-03-18 07745, 2021

      • ruaok
      • 2021-03-18 07752, 2021

      • ruaok
        that is the doc for the YT work.
      • 2021-03-18 07711, 2021

      • ruaok
        Mr_Monkey: it wouldn't allow me to make the doc public without revoking my edit access.
      • 2021-03-18 07722, 2021

      • ruaok
        can you please give edit access to _lucifer, alastairp and myself.
      • 2021-03-18 07723, 2021

      • ruaok
        ?
      • 2021-03-18 07757, 2021

      • Mr_Monkey
        I think it was just you missing ruaok :) Should be good now
      • 2021-03-18 07716, 2021

      • ruaok
        thx
      • 2021-03-18 07722, 2021

      • ruaok
        _lucifer: the first point of work will be to review the doc and to propose a new schema for tracking the player services/user account.
      • 2021-03-18 07735, 2021

      • ruaok
        then update the endpoints that are proposed in the doc.
      • 2021-03-18 07752, 2021

      • ruaok
        there are going to be a lot of open question that we simple don't have an answer for.
      • 2021-03-18 07707, 2021

      • _lucifer
        makes sense
      • 2021-03-18 07715, 2021

      • ruaok
        if you find questions like these, add a section to the doc with open questions and ping Mr_Monkey/me on them.
      • 2021-03-18 07704, 2021

      • ruaok
        now, your work should stop after YT and Spotify are implemented.
      • 2021-03-18 07720, 2021

      • ruaok
        we should open up deezer and apple music as GSoC projects for LB.
      • 2021-03-18 07731, 2021

      • ruaok
        we think that one service would make a decent GSoC project.
      • 2021-03-18 07750, 2021

      • ruaok
        which gives a we bit more than a week to update the LB ideas page.
      • 2021-03-18 07733, 2021

      • Mr_Monkey
        I want to point out that while the document we're looking at is about server-side auth for music services, adding other players on the front-end will most likely not require server-side auth (at least for those music services I looked at)
      • 2021-03-18 07732, 2021

      • Mr_Monkey
        Now if we want to do more than playing music through the front-end on those other services (for example getting users' listening activity), we *will* need server-side auth as a separate matter
      • 2021-03-18 07729, 2021

      • _lucifer
        cool, i'll start work on this asap.
      • 2021-03-18 07702, 2021

      • sumedh joined the channel
      • 2021-03-18 07752, 2021

      • ruaok
        _lucifer: I'm adding the section "open tasks" at the bottom of the doc. you should deffo work on those tasks before diving in too deep.
      • 2021-03-18 07730, 2021

      • RikkoM has quit
      • 2021-03-18 07725, 2021

      • _lucifer
        yes, understood.
      • 2021-03-18 07747, 2021

      • ruaok
        great. thanks for all your efforts!
      • 2021-03-18 07726, 2021

      • _lucifer
        :D
      • 2021-03-18 07706, 2021

      • RikkoM joined the channel
      • 2021-03-18 07723, 2021

      • ruaok
        bwabahahahahah. sweet justice.
      • 2021-03-18 07726, 2021

      • ruaok
      • 2021-03-18 07738, 2021

      • ruaok
        I love that when you search gracenote the MB logo appears.
      • 2021-03-18 07741, 2021

      • ruaok
        !m Meb Team
      • 2021-03-18 07741, 2021

      • BrainzBot
        You're doing good work, Meb Team!
      • 2021-03-18 07709, 2021

      • _lucifer
        🤩
      • 2021-03-18 07715, 2021

      • Mr_Monkey
      • 2021-03-18 07734, 2021

      • ruaok
        wail
      • 2021-03-18 07726, 2021

      • alastairp
        a wild task appeared
      • 2021-03-18 07733, 2021

      • ruaok
        where should client_secrets.json go? docker service configs?
      • 2021-03-18 07752, 2021

      • alastairp
        it should render into config.py
      • 2021-03-18 07716, 2021

      • alastairp
        probably doesn't have to be in the exact format that they give it to you, we can probably just add some more keys to consul?
      • 2021-03-18 07722, 2021

      • shivam-kapila
        alastairp: that sounds like discord welcome message
      • 2021-03-18 07706, 2021

      • ruaok
        ok, lots of uncertainty around this yet. where should I store the json file temporarily in the meantime?
      • 2021-03-18 07718, 2021

      • sumedh has quit
      • 2021-03-18 07758, 2021

      • RikkoM has quit
      • 2021-03-18 07704, 2021

      • alastairp
        docker-server-configs or syswiki sounds reasonable to me
      • 2021-03-18 07744, 2021

      • Mr_Monkey
        ditto
      • 2021-03-18 07724, 2021

      • Mr_Monkey
        ruaok: Here's how it ends up being used with Google's python library:
      • 2021-03-18 07728, 2021

      • Mr_Monkey
      • 2021-03-18 07705, 2021

      • ruaok
        the whole json file then. o.
      • 2021-03-18 07706, 2021

      • ruaok
        ok
      • 2021-03-18 07719, 2021

      • Mr_Monkey
        Looks like
      • 2021-03-18 07723, 2021

      • Mr_Monkey
        Which is a pain
      • 2021-03-18 07733, 2021

      • alastairp
      • 2021-03-18 07727, 2021

      • sumedh joined the channel
      • 2021-03-18 07730, 2021

      • ruaok
        _lucifer: Mr_Monkey: YouTubeAPIKey in syswiki
      • 2021-03-18 07743, 2021

      • Mr_Monkey
        👍
      • 2021-03-18 07757, 2021

      • _lucifer
        👍
      • 2021-03-18 07732, 2021

      • ruaok
        from a dev using our API: "The lookup / search work really well, are a breeze to use and documentation is really good. As a music lover I really appreciate what you're doing."
      • 2021-03-18 07736, 2021

      • ruaok
        !m MB team
      • 2021-03-18 07736, 2021

      • BrainzBot
        You're doing good work, MB team!
      • 2021-03-18 07726, 2021

      • sumedh has quit
      • 2021-03-18 07735, 2021

      • ruaok
      • 2021-03-18 07737, 2021

      • ruaok
        alastairp: ^
      • 2021-03-18 07756, 2021

      • alastairp
        sweet. and then it's just sql
      • 2021-03-18 07703, 2021

      • ruaok
        yerp.
      • 2021-03-18 07709, 2021

      • bitmap
        reosarevok: do you remember what the pages did before we added those has_by_*artist functions? maybe we can revert them for now
      • 2021-03-18 07758, 2021

      • reosarevok
        bitmap: it just made the attempts to load the stuff
      • 2021-03-18 07707, 2021

      • reosarevok
        This was just added to see whether we should show a link...
      • 2021-03-18 07724, 2021

      • reosarevok feels kinda bad about adding that now
      • 2021-03-18 07725, 2021

      • reosarevok
      • 2021-03-18 07730, 2021

      • reosarevok
        Feel free to revert if needed
      • 2021-03-18 07703, 2021

      • _lucifer
        Mr_Monkey: is the custom error boundary we use in line with https://reactjs.org/docs/error-boundaries.html?
      • 2021-03-18 07700, 2021

      • Mr_Monkey
        I have no idea :D
      • 2021-03-18 07708, 2021

      • Mr_Monkey
        But I guess so, at a quick glance?
      • 2021-03-18 07726, 2021

      • Mr_Monkey
        In that guide they set the state in getDerivedStateFromError rather than in componentDidCatch, but 🤷
      • 2021-03-18 07757, 2021

      • BrainzGit
        [bookbrainz-site] akashgp09 opened pull request #578 (master…collection-button): FIX(BB-591): Added margin to buttons on collections page https://github.com/bookbrainz/bookbrainz-site/pul…
      • 2021-03-18 07716, 2021

      • Mr_Monkey
        I guess the main question is: does the sentry errorboundary offer us something more? I read something about user feedback somewhere, for example
      • 2021-03-18 07755, 2021

      • _lucifer
        one thing i'll need to check is whether errors from the custom ErrorBoundary are reported at all to sentry?
      • 2021-03-18 07759, 2021

      • Mr_Monkey
        Yeah, looks like they have more stuff there: https://github.com/getsentry/sentry-javascript/bl…