#metabrainz

/

      • ritiek has quit
      • 2021-07-08 18958, 2021

      • BrainzGit has quit
      • 2021-07-08 18936, 2021

      • BrainzGit joined the channel
      • 2021-07-08 18919, 2021

      • opal joined the channel
      • 2021-07-08 18951, 2021

      • DragonsChild joined the channel
      • 2021-07-08 18934, 2021

      • DragonsChild has quit
      • 2021-07-08 18956, 2021

      • reosarevok
        Jeez. Does LB really need Google stuff? Because those emails where they're never happy sure do look like a pain in the ass
      • 2021-07-08 18953, 2021

      • yyoung
        yvanzo: I've updated PR 2151
      • 2021-07-08 18920, 2021

      • d4rk joined the channel
      • 2021-07-08 18958, 2021

      • d4rkie has quit
      • 2021-07-08 18925, 2021

      • ruaok
        that is exactly the question I was just asking myself.
      • 2021-07-08 18931, 2021

      • ruaok
        I'm going to mail my contact at YouTube and ask him for advice.
      • 2021-07-08 18947, 2021

      • lucifer
        now we are just going around in circles...
      • 2021-07-08 18947, 2021

      • lucifer
        i do not understand which point of the list we do not meet.
      • 2021-07-08 18947, 2021

      • ruaok
        yeah. all for something we don't actually want. which is the worst part.
      • 2021-07-08 18947, 2021

      • lucifer
        exactly.
      • 2021-07-08 18946, 2021

      • lucifer
      • 2021-07-08 18909, 2021

      • lucifer
        their last email about Privacy Policy did not have this section.
      • 2021-07-08 18927, 2021

      • ruaok
        I think each reviewer we get, applies their own standards.
      • 2021-07-08 18927, 2021

      • ruaok
        seems intractable.
      • 2021-07-08 18927, 2021

      • lucifer
        yeah :/
      • 2021-07-08 18934, 2021

      • lucifer
      • 2021-07-08 18930, 2021

      • lucifer
        oh well. did you see the security assessment BS down that page?
      • 2021-07-08 18951, 2021

      • ruaok
        can we request NO SCOPES?
      • 2021-07-08 18954, 2021

      • lucifer
        that was how things workedd till last year. google deprecated an removed that.
      • 2021-07-08 18955, 2021

      • ruaok
        are there any non-restricted scopes at all?
      • 2021-07-08 18922, 2021

      • lucifer
        not for youtube i think but i'll recheck.
      • 2021-07-08 18942, 2021

      • alastairp
        morning
      • 2021-07-08 18959, 2021

      • alastairp
        ruaok: sorry, I didn't reply to that tweet. just done it now
      • 2021-07-08 18926, 2021

      • alastairp
        yesterday wasn't going my way, I turned off all of the computers at midday and took a day off, it was definitely needed
      • 2021-07-08 18946, 2021

      • alastairp
        lucifer: agreed, we should structure the training process such that if an error happens, we propagate an exception to a single top-level error handler, report it to sentry, and then log it in a way to show the user that there was a problem
      • 2021-07-08 18908, 2021

      • lucifer
        hi!
      • 2021-07-08 18912, 2021

      • lucifer
        yes, makes sense.
      • 2021-07-08 18952, 2021

      • lucifer
        i made some more progress on the sklearn stuff, i think its mostly ready except for the exception handling part.
      • 2021-07-08 18920, 2021

      • alastairp
        cool, great! Let me concentrate on finishing off these two AB PRs that I'm currently working on, then I'll start with testing this one
      • 2021-07-08 18910, 2021

      • ruaok
        alastairp: thanks!
      • 2021-07-08 18952, 2021

      • Lotheric has quit
      • 2021-07-08 18916, 2021

      • Lotheric joined the channel
      • 2021-07-08 18921, 2021

      • ritiek joined the channel
      • 2021-07-08 18955, 2021

      • BrainzGit
        [listenbrainz-server] 14amCap1712 opened pull request #1542 (03master…youtube-again): Query Youtube API using api key only https://github.com/metabrainz/listenbrainz-server…
      • 2021-07-08 18944, 2021

      • ruaok
        thanks param !
      • 2021-07-08 18904, 2021

      • BrainzGit
        [listenbrainz-server] 14mayhem merged pull request #1538 (03master…Pinned-Recording-Modal): Add Pinned Recording Modal https://github.com/metabrainz/listenbrainz-server…
      • 2021-07-08 18934, 2021

      • param
        sorry for the late review, I only just saw it today. :(
      • 2021-07-08 18900, 2021

      • ruaok
        all good.
      • 2021-07-08 18915, 2021

      • ruaok
        lucifer: another thought I had... I think we need to build content resolution into lb-server.
      • 2021-07-08 18939, 2021

      • ruaok
        having it in brainzplayer makes BP far more complicated than it needs to be.
      • 2021-07-08 18920, 2021

      • reosarevok
        Jesus
      • 2021-07-08 18930, 2021

      • reosarevok
        We have a week and a half of 30+ degrees coming
      • 2021-07-08 18946, 2021

      • reosarevok
        Maybe I should get a plane to somewhere cooler, like... the Mediterranean :p
      • 2021-07-08 18951, 2021

      • lucifer
        yeah makes sense. so send 1 query to LB server and then LB sends back the link to play?
      • 2021-07-08 18940, 2021

      • ruaok
        lucifer: or even on a whole playlist at a time. so that BP can just do playback and nothing else.
      • 2021-07-08 18957, 2021

      • ruaok
        reosarevok: Is 24C here. :)
      • 2021-07-08 18958, 2021

      • lucifer
        yup agreed.
      • 2021-07-08 18928, 2021

      • reosarevok
        ruaok: sadly we are only 7 days past the second vaccine, so a bit early to travel anyway :(
      • 2021-07-08 18906, 2021

      • ruaok
        and you don't want to come here. a huge number of youngsters, who never cared about covid are getting infected in record numbers.
      • 2021-07-08 18920, 2021

      • reosarevok
        Well, hopefully the vaccine would help with that... but yeah
      • 2021-07-08 18925, 2021

      • BrainzGit
        [listenbrainz-server] 14amCap1712 merged pull request #1542 (03master…youtube-again): Query Youtube API using api key only https://github.com/metabrainz/listenbrainz-server…
      • 2021-07-08 18958, 2021

      • lucifer
        ruaok: let's release?
      • 2021-07-08 18903, 2021

      • ruaok
        go!
      • 2021-07-08 18919, 2021

      • alastairp
        meanwhile: Francia aconseja no veranear en España y Portugal
      • 2021-07-08 18926, 2021

      • ruaok
        yerp.
      • 2021-07-08 18926, 2021

      • BrainzGit
        [listenbrainz-server] release 03v-2021-07-08.0 has been published by 14github-actions[bot]: https://github.com/metabrainz/listenbrainz-server…
      • 2021-07-08 18934, 2021

      • alastairp
        veranear, what a verb
      • 2021-07-08 18951, 2021

      • shivam-kapila
        30 degress? Its 50 degrees here
      • 2021-07-08 18951, 2021

      • ruaok
        I can't wait to see what the foreign ministry of germany says for next week.
      • 2021-07-08 18905, 2021

      • ruaok
        I'll be coming vaccinated and with a PCR test, but still.
      • 2021-07-08 18919, 2021

      • reosarevok
        shivam-kapila: I'd like to think your buildings are built taking the heat into account though, at least?
      • 2021-07-08 18920, 2021

      • ruaok
        shivam-kapila: reosarevok is a wimp
      • 2021-07-08 18923, 2021

      • reosarevok
        (if not, yeesh)
      • 2021-07-08 18947, 2021

      • reosarevok
        Ours aren't, since we're a cold country - they're mostly built to keep the heat in :p
      • 2021-07-08 18955, 2021

      • shivam-kapila
        Not at all reosarevok
      • 2021-07-08 18958, 2021

      • lucifer
        nice of you to think buildings are planned before building in india
      • 2021-07-08 18902, 2021

      • reosarevok
        haha
      • 2021-07-08 18916, 2021

      • reosarevok
        Sorry I had hopes for your country, should have known better (?) :p
      • 2021-07-08 18921, 2021

      • ruaok
        your insulated house is good for keeping heat out... for a while.
      • 2021-07-08 18934, 2021

      • shivam-kapila
        On point lucifer. Plus mine is a rented place. Why would the landlord care
      • 2021-07-08 18910, 2021

      • reosarevok
        At least we're "only" at almost constant 27 degrees inside
      • 2021-07-08 18919, 2021

      • reosarevok
        Some people are at 35, so I guess I should be happy :p
      • 2021-07-08 18947, 2021

      • shivam-kapila
        My mac is so heated that I can cook chapattis on it
      • 2021-07-08 18948, 2021

      • ruaok
        I'm just glad that evenings still cool to 21C.
      • 2021-07-08 18921, 2021

      • reosarevok
        It's a more normal temperature in Lapland, so that'd be an option - sadly it's death by mosquitoes in there in the summer AFAICT :D
      • 2021-07-08 18937, 2021

      • reosarevok
        Do they make laptops you can use while sitting in a river yet?
      • 2021-07-08 18905, 2021

      • shivam-kapila
        Then offices will be in swimming pools rather than buildings :p
      • 2021-07-08 18916, 2021

      • reosarevok
        Sounds like a plan
      • 2021-07-08 18953, 2021

      • param
        lol someone at work got github copilot to solve an interview problem we give
      • 2021-07-08 18930, 2021

      • shivam-kapila
        Its trending right now
      • 2021-07-08 18955, 2021

      • shivam-kapila
        How copilot can help crush leetcode questions
      • 2021-07-08 18904, 2021

      • lucifer
        LB prod updated.
      • 2021-07-08 18939, 2021

      • akshaaatt[m]
        <param "lol someone at work got github c"> Oh my😂😂
      • 2021-07-08 18908, 2021

      • param starts reading up on those ML books
      • 2021-07-08 18939, 2021

      • lucifer
        alastairp: the best model pickle file should be the history path in db, right?
      • 2021-07-08 18917, 2021

      • alastairp
        lucifer: right, they're equivalent, but perhaps we could use a different key for the sklearn one, maybe just "model" ?
      • 2021-07-08 18915, 2021

      • lucifer
        we could do that but that only causes to add more ifs elsewhere.
      • 2021-07-08 18948, 2021

      • lucifer
        whenever we want to access the model. i don't think we do it anywhere currently though.
      • 2021-07-08 18918, 2021

      • ritiek has quit
      • 2021-07-08 18909, 2021

      • lucifer
        i'll do the model key then, if we need to change we can do that later anyways.
      • 2021-07-08 18944, 2021

      • lucifer
      • 2021-07-08 18957, 2021

      • lucifer
        alastairp: there's also this warning in sklearn logs
      • 2021-07-08 18952, 2021

      • alastairp
        lucifer: how many items are in your dataset?
      • 2021-07-08 18957, 2021

      • alastairp
        345?
      • 2021-07-08 18957, 2021

      • lucifer
        345
      • 2021-07-08 18913, 2021

      • alastairp
        yeah, looks like the warning is pretty harmless, then
      • 2021-07-08 18921, 2021

      • lucifer
        👍
      • 2021-07-08 18938, 2021

      • yvanzo has quit
      • 2021-07-08 18914, 2021

      • yvanzo joined the channel
      • 2021-07-08 18927, 2021

      • akashgp09 joined the channel
      • 2021-07-08 18907, 2021

      • ritiek joined the channel
      • 2021-07-08 18938, 2021

      • BrainzGit
        [listenbrainz-server] 14amCap1712 opened pull request #1543 (03master…revert-privacy-policy): Remove LB specific privacy policy https://github.com/metabrainz/listenbrainz-server…
      • 2021-07-08 18940, 2021

      • agatzk has quit
      • 2021-07-08 18948, 2021

      • yyoung has quit
      • 2021-07-08 18913, 2021

      • yyoung joined the channel
      • 2021-07-08 18930, 2021

      • BrainzGit
        [musicbrainz-server] 14yvanzo closed pull request #2081 (03master…MBS-10211): MBS-10211: Formatting Tweet links for recordings https://github.com/metabrainz/musicbrainz-server/…
      • 2021-07-08 18946, 2021

      • BrainzGit
        [musicbrainz-android] 14akshaaatt opened pull request #82 (03master…asset_attributions): Asset Attributions https://github.com/metabrainz/musicbrainz-android…
      • 2021-07-08 18914, 2021

      • akshaaatt[m]
        Hi lucifer !
      • 2021-07-08 18921, 2021

      • lucifer
        hi!
      • 2021-07-08 18959, 2021

      • akshaaatt[m]
        <BrainzGit "[musicbrainz-android] akshaaatt "> If you could confirm quickly whether this format is good?
      • 2021-07-08 18915, 2021

      • akshaaatt[m]
        For attrubtions, I have basically prepared a table
      • 2021-07-08 18934, 2021

      • akshaaatt[m]
        attributions^
      • 2021-07-08 18947, 2021

      • lucifer
        sure looks good to me. I would suggest to use markdown file instead though so that formatting is easier.
      • 2021-07-08 18910, 2021

      • akshaaatt[m]
        Oh Okay!
      • 2021-07-08 18912, 2021

      • lucifer
        and link the master version of this file in the about page of the app.
      • 2021-07-08 18929, 2021

      • akshaaatt[m]
        Cool Sounds great!
      • 2021-07-08 18958, 2021

      • yvanzo
        yyoung: are you able to run selenium tests atm?
      • 2021-07-08 18952, 2021

      • yyoung
        yvanzo: I haven't tried to run it locally since then, I just found that CI combining with local Selenium IDE plugin works fine
      • 2021-07-08 18902, 2021

      • yvanzo
        Ok thanks, I will look into running selenium tests locally, because it would be nice to add tests for your last fix in 2151.
      • 2021-07-08 18929, 2021

      • yyoung
        If I was able to import json5 files into it, then it would work perfectly
      • 2021-07-08 18957, 2021

      • yyoung
        BTW, I just found that the saved file of selenium IDE is in json format, maybe I could write a convertion script
      • 2021-07-08 18903, 2021

      • yvanzo
        yyoung: If you’d like to explain how to use that plugin with CI, we could probably add it to HACKING.md notes.
      • 2021-07-08 18949, 2021

      • yyoung
        Currently I have to manually enter every action from json5 to the IDE
      • 2021-07-08 18929, 2021

      • yyoung
        If I found a way to convert, I would be glad to write the notes :)
      • 2021-07-08 18901, 2021

      • lucifer
        alastairp: i'll start work on the model testing part meanwhile in a differrent PR. this only is already too big. sounds good?
      • 2021-07-08 18913, 2021

      • alastairp
        lucifer: yeah, I think that's fine. this is the intermediate tables for doing evaluation?
      • 2021-07-08 18950, 2021

      • lucifer
        alastairp: yes, that and in general for external validation of models.
      • 2021-07-08 18901, 2021

      • alastairp
        cool
      • 2021-07-08 18924, 2021

      • alastairp
        up to you - but we could even do this without the sklearn code integrated for now
      • 2021-07-08 18936, 2021

      • alastairp
        in order to prevent us getting into trouble with multiple large prs that merge into each other
      • 2021-07-08 18946, 2021

      • lucifer
        yes, right. i'll base the PR from master.
      • 2021-07-08 18900, 2021

      • lucifer
        or should i do it from Py3 branch?
      • 2021-07-08 18905, 2021

      • alastairp
        cool. gaia only, but designed so that adding sklearn in later would be fine
      • 2021-07-08 18907, 2021

      • alastairp
        mmm
      • 2021-07-08 18911, 2021

      • alastairp
        the highlevel extractor runs with subprocess.run, so we could probably get away with py3
      • 2021-07-08 18951, 2021

      • lucifer
        yeah indeed but it uses future module.
      • 2021-07-08 18918, 2021

      • lucifer
        i saw a ticket on gaia repo that py2 futures backport is different from py3 inbuilt futures module.
      • 2021-07-08 18950, 2021

      • lucifer
      • 2021-07-08 18933, 2021

      • alastairp
        note that this is during the training process
      • 2021-07-08 18940, 2021

      • alastairp
        ideally we'd still keep this in py 2
      • 2021-07-08 18948, 2021

      • lucifer
        ah ok.
      • 2021-07-08 18959, 2021

      • lucifer
        py2 + six then?
      • 2021-07-08 18934, 2021

      • alastairp
        yep
      • 2021-07-08 18956, 2021

      • lucifer
        +1