#metabrainz

/

      • D4RK-PH0ENiX has quit
      • 2019-07-08 18925, 2019

      • Wizzup has quit
      • 2019-07-08 18931, 2019

      • Delemas
        Mineo, thanks for pointing me in the right direction. I've put it in a post as you suggested: https://community.metabrainz.org/t/how-to-delete-…
      • 2019-07-08 18901, 2019

      • Wizzup joined the channel
      • 2019-07-08 18936, 2019

      • D4RK-PH0ENiX joined the channel
      • 2019-07-08 18943, 2019

      • D4RK-PH0ENiX has quit
      • 2019-07-08 18922, 2019

      • D4RK-PH0ENiX joined the channel
      • 2019-07-08 18951, 2019

      • Wizzup has quit
      • 2019-07-08 18959, 2019

      • Wizzup joined the channel
      • 2019-07-08 18942, 2019

      • Wizzup has quit
      • 2019-07-08 18957, 2019

      • Nyanko-sensei joined the channel
      • 2019-07-08 18900, 2019

      • D4RK-PH0ENiX has quit
      • 2019-07-08 18903, 2019

      • Wizzup joined the channel
      • 2019-07-08 18903, 2019

      • ephemer0l has quit
      • 2019-07-08 18950, 2019

      • ephemer0l_ joined the channel
      • 2019-07-08 18940, 2019

      • Nyanko-sensei has quit
      • 2019-07-08 18918, 2019

      • D4RK-PH0ENiX joined the channel
      • 2019-07-08 18916, 2019

      • Wizzup has quit
      • 2019-07-08 18924, 2019

      • Wizzup joined the channel
      • 2019-07-08 18959, 2019

      • G0re joined the channel
      • 2019-07-08 18942, 2019

      • Gore has quit
      • 2019-07-08 18909, 2019

      • Wizzup has quit
      • 2019-07-08 18950, 2019

      • Wizzup joined the channel
      • 2019-07-08 18925, 2019

      • Wizzup has quit
      • 2019-07-08 18923, 2019

      • Wizzup joined the channel
      • 2019-07-08 18918, 2019

      • Wizzup has quit
      • 2019-07-08 18956, 2019

      • Wizzup joined the channel
      • 2019-07-08 18940, 2019

      • BestSteve has quit
      • 2019-07-08 18903, 2019

      • BestSteve joined the channel
      • 2019-07-08 18946, 2019

      • Wizzup has quit
      • 2019-07-08 18958, 2019

      • Wizzup joined the channel
      • 2019-07-08 18917, 2019

      • Lotheric has quit
      • 2019-07-08 18904, 2019

      • Lotheric joined the channel
      • 2019-07-08 18932, 2019

      • Wizzup has quit
      • 2019-07-08 18926, 2019

      • Wizzup joined the channel
      • 2019-07-08 18920, 2019

      • Wizzup has quit
      • 2019-07-08 18945, 2019

      • Wizzup joined the channel
      • 2019-07-08 18921, 2019

      • Wizzup has quit
      • 2019-07-08 18925, 2019

      • Wizzup joined the channel
      • 2019-07-08 18906, 2019

      • yvanzo
        mo’’in’
      • 2019-07-08 18911, 2019

      • Lotheric_ joined the channel
      • 2019-07-08 18916, 2019

      • Lotheric has quit
      • 2019-07-08 18932, 2019

      • Gore|work joined the channel
      • 2019-07-08 18906, 2019

      • G0re has quit
      • 2019-07-08 18951, 2019

      • Nyanko-sensei joined the channel
      • 2019-07-08 18917, 2019

      • D4RK-PH0ENiX has quit
      • 2019-07-08 18927, 2019

      • ruaok
        mooin
      • 2019-07-08 18946, 2019

      • ruaok
        massive thunderstorms rolling over the city today...
      • 2019-07-08 18923, 2019

      • zas
        mo'in
      • 2019-07-08 18918, 2019

      • amCap1712
        hi zas
      • 2019-07-08 18958, 2019

      • amCap1712
        i had opened a few tickets as discussed. can you take a look at those?
      • 2019-07-08 18921, 2019

      • amCap1712
        PICARD-1551 , PICARD-1552, PICAD-1553
      • 2019-07-08 18922, 2019

      • BrainzBot
        PICARD-1551: "compare_to_track" method considers "score" parameter only if track has releases https://tickets.metabrainz.org/browse/PICARD-1551
      • 2019-07-08 18922, 2019

      • BrainzBot
        PICARD-1552: "compare_to_release_parts" considers track count of only first medium https://tickets.metabrainz.org/browse/PICARD-1552
      • 2019-07-08 18945, 2019

      • amCap1712
        PICARD-1553
      • 2019-07-08 18945, 2019

      • BrainzBot
        PICARD-1553: "weights" for various parameters are specified in various files https://tickets.metabrainz.org/browse/PICARD-1553
      • 2019-07-08 18909, 2019

      • zas
        amCap1712: yes, I saw, thanks we'll have a look soon
      • 2019-07-08 18952, 2019

      • amCap1712
        yeah, that would be great
      • 2019-07-08 18955, 2019

      • pristine__
        ruaok: hey!
      • 2019-07-08 18912, 2019

      • ruaok
        hi!
      • 2019-07-08 18939, 2019

      • pristine__
        Our next goal is to get recommendation on Lemmy and test on beta?
      • 2019-07-08 18933, 2019

      • ruaok
        yes.
      • 2019-07-08 18954, 2019

      • pristine__
        Okay.....What should be the approach?
      • 2019-07-08 18951, 2019

      • Nyanko-sensei has quit
      • 2019-07-08 18951, 2019

      • D4RK-PH0ENiX joined the channel
      • 2019-07-08 18930, 2019

      • gr0uch0mars joined the channel
      • 2019-07-08 18930, 2019

      • MajorLurker joined the channel
      • 2019-07-08 18920, 2019

      • BrainzGit
        [listenbrainz-server] paramsingh opened pull request #596 (master…consistency-ensure): Skip listens with inserted_timestamps not in time range https://github.com/metabrainz/listenbrainz-server…
      • 2019-07-08 18935, 2019

      • alastairp
        iliekcomputers: good morning, how are you for looking at https, ratelimiting, and init?
      • 2019-07-08 18941, 2019

      • alastairp
        today, or tomorrow?
      • 2019-07-08 18925, 2019

      • ruaok
        pristine__: let me get caught up on the work you've done last week.
      • 2019-07-08 18930, 2019

      • ruaok
      • 2019-07-08 18908, 2019

      • ruaok
        I feel that this is a bit too specific for the things that have developed since you opened that PR....
      • 2019-07-08 18903, 2019

      • reosarevok
        yvanzo, bitmap
      • 2019-07-08 18923, 2019

      • reosarevok
        Sigh, sneaky enter key. Anyway, any reason not to wontfix https://tickets.metabrainz.org/browse/MBS-3871 ?
      • 2019-07-08 18924, 2019

      • BrainzBot
        MBS-3871: Flag URLs with inappropriate content
      • 2019-07-08 18901, 2019

      • pristine__
        ruaok: yes agreed #578 needs a lot of changes.
      • 2019-07-08 18915, 2019

      • ruaok
        maybe close it for now?
      • 2019-07-08 18928, 2019

      • pristine__
        Yup
      • 2019-07-08 18941, 2019

      • pristine__
        I will close it.
      • 2019-07-08 18948, 2019

      • ruaok
        I think a good first intermediary step is to store the data CF tracks into a new schema on lemmy.
      • 2019-07-08 18948, 2019

      • Lotheric_ is now known as Lotheric
      • 2019-07-08 18917, 2019

      • iliekcomputers
        alastairp: morning, I haven't gotten to redirects now. Let me work on that, we can do a release today
      • 2019-07-08 18931, 2019

      • pristine__
        How should the schema look?
      • 2019-07-08 18944, 2019

      • pristine__
        Two different col/fields for two playlists.
      • 2019-07-08 18912, 2019

      • ruaok
        well right now they should not even be playlists, but just tracks.
      • 2019-07-08 18931, 2019

      • pristine__
        Yes, my bad. Tracks.
      • 2019-07-08 18932, 2019

      • iliekcomputers
        ruaok: I've been testing incremental dumps over the past week. I think they should be ready for release after this batch of PRs.
      • 2019-07-08 18957, 2019

      • ruaok
        iliekcomputers: great. I'm trying to touch up the last bits on the quickbooks stuff so that can finally be merged.
      • 2019-07-08 18903, 2019

      • ruaok
        after that i'll circle back on PRs
      • 2019-07-08 18929, 2019

      • ruaok
        pristine__: I guess the schema might be: row id, user id, track id, last_used (timestamp)
      • 2019-07-08 18948, 2019

      • ruaok
        that way when we use the track, we can keep track of it having been used for a playlist, so we don't overuse it.
      • 2019-07-08 18907, 2019

      • BrainzGit
        [listenbrainz-server] vansika closed pull request #578 (master…recommednation-table): table to store recommended playlist of every user https://github.com/metabrainz/listenbrainz-server…
      • 2019-07-08 18923, 2019

      • pristine__
        ruaok: so we will need to remember the timestamp in Spark when the track was displayed on beta for user to listen, and if the last used is greater than this timestamp, it means user has listened to the track, other wise not.
      • 2019-07-08 18905, 2019

      • iliekcomputers
        how will you get the timestamps to spark?
      • 2019-07-08 18925, 2019

      • ruaok
        my plan wasn't to make spark aware of timestamps.
      • 2019-07-08 18955, 2019

      • ruaok
        I was hoping we can generate loads of tracks and store them in lemmy and keep timestamps there.
      • 2019-07-08 18940, 2019

      • pristine__
        Umm.... Okay. CF spark will just generates loads of tracks and Lemmy will do the other work ( keeping track of tracks listened to by user etc.)
      • 2019-07-08 18923, 2019

      • ruaok
        yep. let's hope that works for now. if we need to feed back which tracks have been used, we'll do that later.
      • 2019-07-08 18917, 2019

      • iliekcomputers
        alright.
      • 2019-07-08 18920, 2019

      • pristine__
        Cool.
      • 2019-07-08 18913, 2019

      • gr0uch0mars has quit
      • 2019-07-08 18929, 2019

      • iliekcomputers
        >There are other GET methods here that we shouldn't redirect on either.
      • 2019-07-08 18936, 2019

      • iliekcomputers
        alastairp: which ones are those exactly?
      • 2019-07-08 18900, 2019

      • alastairp
        iliekcomputers: I was thinking about the legacy endpoints
      • 2019-07-08 18936, 2019

      • iliekcomputers
        Right, all of them, I guess?
      • 2019-07-08 18937, 2019

      • alastairp
        I'm not sure if it makes sense to keep them http-only, or just go ahead and do everything except legacy POST
      • 2019-07-08 18904, 2019

      • iliekcomputers
        Would redirecting break clients?
      • 2019-07-08 18929, 2019

      • alastairp
        it has the potential to break POSTs, which is why I don't want to do it, but I'm not sure about the GETs
      • 2019-07-08 18956, 2019

      • alastairp
        especially since you added the exception to the whole api
      • 2019-07-08 18911, 2019

      • alastairp
        if the legacy endpoints are part of the api, then it makes sense to exclude those too
      • 2019-07-08 18902, 2019

      • iliekcomputers
        Hmm, yeah.
      • 2019-07-08 18913, 2019

      • iliekcomputers
        Let's not redirect for all of them
      • 2019-07-08 18949, 2019

      • alastairp
        I wonder if something like https://stackoverflow.com/questions/38488134/get-… would be useful
      • 2019-07-08 18930, 2019

      • alastairp
        I wonder if there's a version of that which will tell us the blueprint of the view too
      • 2019-07-08 18944, 2019

      • iliekcomputers
        I guess adding exceptions for this will be harder, there's no prefix
      • 2019-07-08 18908, 2019

      • alastairp
        there's no prefix, but it's in a separate blueprint
      • 2019-07-08 18922, 2019

      • iliekcomputers
        Oh, you're looking at the same thing, great minds ;)
      • 2019-07-08 18926, 2019

      • iliekcomputers
        Lemme take a look
      • 2019-07-08 18918, 2019

      • alastairp
      • 2019-07-08 18928, 2019

      • alastairp
        > The endpoint for the :func:`url_for` function is prefixed with the name of the blueprint.
      • 2019-07-08 18955, 2019

      • alastairp
        at the very least, we might just be able to do that - "if urlfor-name.startswith("blueprint.")
      • 2019-07-08 18923, 2019

      • alastairp
        ah - I just noticed also that there are specific blueprint before_request handlers... does that make sense?
      • 2019-07-08 18927, 2019

      • iliekcomputers
        That sounds nicer
      • 2019-07-08 18948, 2019

      • alastairp
        we have quite a few regular BPs, though
      • 2019-07-08 18933, 2019

      • iliekcomputers
        I was thinking of overriding it somehow for the API blueprints, but the startswith thing is probably less complicated and less work
      • 2019-07-08 18914, 2019

      • alastairp
        setting a global one and then overriding it for the api one sounds a bit difficult/complext
      • 2019-07-08 18940, 2019

      • alastairp
        mmm, so. additionally: if you're in a request context, `request.blueprint` is the text name of the blueprint
      • 2019-07-08 18950, 2019

      • alastairp
        no need for any lookups. just match against a string
      • 2019-07-08 18954, 2019

      • BrainzGit
        [musicbrainz-server] reosarevok opened pull request #1136 (master…MBS-3873): MBS-3873: Report: non-matching RG/release titles https://github.com/metabrainz/musicbrainz-server/…
      • 2019-07-08 18955, 2019

      • BrainzBot
        MBS-3873: Report for release groups containing releases with different titles https://tickets.metabrainz.org/browse/MBS-3873
      • 2019-07-08 18959, 2019

      • iliekcomputers
        alastairp: zas: what was the header to check to see the original protocol from flask? `X-Forwarded-Proto` is None
      • 2019-07-08 18935, 2019

      • alastairp
        that's what it should be. maybe it's not set properly on beta?
      • 2019-07-08 18951, 2019

      • BrainzGit
        [musicbrainz-server] reosarevok opened pull request #1137 (master…MBS-9981): MBS-9981 / MBS-10278: PossibleCollaborations improvements https://github.com/metabrainz/musicbrainz-server/…
      • 2019-07-08 18953, 2019

      • BrainzBot
        MBS-9981: conductor & founder relationships should be considered in Artists that may be collaborations https://tickets.metabrainz.org/browse/MBS-9981
      • 2019-07-08 18953, 2019

      • BrainzBot
        MBS-10278: Artists with members listed in PossibleCollaborations if they also have other rels https://tickets.metabrainz.org/browse/MBS-10278
      • 2019-07-08 18922, 2019

      • iliekcomputers
        zas: any idea why flask doesn't have `X-Forwarded-Proto` set for AB beta? https://github.com/metabrainz/docker-server-confi…
      • 2019-07-08 18902, 2019

      • iliekcomputers
      • 2019-07-08 18954, 2019

      • alastairp
        if everything else does and just that one doesn't, I'd say it's an oversight
      • 2019-07-08 18926, 2019

      • D4RK-PH0ENiX has quit
      • 2019-07-08 18919, 2019

      • iliekcomputers
        i'm not sure, i haven't really used that header
      • 2019-07-08 18931, 2019

      • CallerNo6 has quit
      • 2019-07-08 18951, 2019

      • yvanzo
        reosarevok: it is clearly not top priority, but I wouldn't mark it as won't fix, just found it is related to MBS-6686 btw.
      • 2019-07-08 18951, 2019

      • BrainzBot
        MBS-6686: Add a "NSFW" front cover art variable https://tickets.metabrainz.org/browse/MBS-6686
      • 2019-07-08 18959, 2019

      • iliekcomputers
        alastairp: request.url contains http and https correctly, from me seeing. I think it's because right now we only have the one container for the webserver, gonna need zas to confirm
      • 2019-07-08 18905, 2019

      • D4RK-PH0ENiX joined the channel
      • 2019-07-08 18957, 2019

      • iliekcomputers
        alastairp: if you're around, we should release ratelimiting today, i'm don't think this will be ready today.
      • 2019-07-08 18928, 2019

      • iliekcomputers
        not sure why it's redirecting in tests, wtf. i'm gonna look at this tomorrow, this is a bit irritating.
      • 2019-07-08 18949, 2019

      • BrainzGit
        [metabrainz.org] mayhem opened pull request #324 (master…quickbooks-invoices): Add quickbooks invoicing support https://github.com/metabrainz/metabrainz.org/pull…
      • 2019-07-08 18942, 2019

      • Cyna
        bitmap: I need help sorting out flow errors in taskOne branch. After that I could continue my work with place entity