#metabrainz

/

      • Mr_Monkey
        prabal: about that typo Yeah, I'm surprised too ! I must have read it a hundred times…wrong.
      • SomalRudra: What can I help you with?
      • reosarevok
        yvanzo: we should consider upgrading Jira again :) Either to 8.5 or 8.11ç
      • "The End of Life date for JIRA version 8.0 is Feb 9, 2021. To ensure you're still fully supported, you should upgrade before this date."
      • yvanzo
        reosarevok: right, I will check add-ons compatibility
      • ishaanshah
      • iliekcomputers: fixed it
      • ruaok
        ishaanshah: well done!
      • BrainzGit
        [musicbrainz-server] reosarevok opened pull request #1641 (master…MBS-11025): MBS-11025: Update the SoundCloud logo https://github.com/metabrainz/musicbrainz-serve...
      • BrainzBot
        MBS-11025: Update the SoundCloud logo used in the sidebar https://tickets.metabrainz.org/browse/MBS-11025
      • alastairp
        good morning
      • hi jmp_music_, yes, I'm around. how about in 1 hour?
      • white_snack joined the channel
      • white_shadow has quit
      • outsidecontext
        rdswift: are you around? We are preparing the Picard 2.4 release today. What's needed for the docs site? I think you had the changes already lined up, right?
      • prabal
        Mr_Monkey: https://test.bookbrainz.org/collection/caf53b17... you made me a collaborator of this collection. I should have an option to remove myself as a collaborator. I am thinking of adding a button `remove yourself as collaborator` in this page.
      • I am confused whether I should re-use the `delete-collection-modal` here or should i make a new `remove-collaborator` modal.
      • https://github.com/bookbrainz/bookbrainz-site/b... . The structure of `remove-collaborator-modal` will be pretty much same as `delete-modal` which makes me think I should re-use the modal but there are few changes - postUrl, body-text, header, button-text - which will make reusing it little messy
      • what do you think?
      • Mr_Monkey
        prabal: I think you could refactor to resuse the same modal. for the modal body, you could pass the contents to the component using `{children}` inside the modal component so you'd call it like this: `<RefactoredModal> <div>the modal body</div></RefactoredModal>
      • That could save you a bit of trouble. The rest of the elements could be passed as props without problem I think
      • prabal
        yeahh okayy
      • iliekcomputers
        ishaanshah: awesome!
      • when in doubt, create your own component :D
      • ishaanshah
        Yeah I just looked up the source code and reimplemented it with slight changes
      • ruaok
      • jmp_music_
        alastairp: nice!
      • ruaok
        I decided to split it into two PRs. ok to merge that one?
      • next one coming right up.
      • zas: ^^
      • zas
        yes better
      • ruaok
        ok
      • and I'll let you and bitmap finish this one off -- the DB on paco is already restarted with the right setting, but bitmap may need to do more work on it.
      • abhinavohri joined the channel
      • sumedh joined the channel
      • white_snack has quit
      • white_shadow joined the channel
      • white_shadow has quit
      • abhinavohri
        iliekcomputers: Can u help me with the` url_for`method? For my test ,i want that it does not return a trailing slash at the end of the url. How can i do it?
      • iliekcomputers
        just put in the string directly, instead of using url_for
      • i think it's ok for this case
      • BrainzGit
        [listenbrainz-server] ishaanshah opened pull request #1026 (master…ishaan/artist_map_500_fix): [WIP] LB-692: Fix 500s on artist-map endpoint https://github.com/metabrainz/listenbrainz-serv...
      • BrainzBot
        LB-692: 500s on the artist-map endpoint https://tickets.metabrainz.org/browse/LB-692
      • iliekcomputers
        self.client.get('/user/blahdeblah/')
      • abhinavohri
        ok thanks
      • BrainzGit
        [musicbrainz-server] reosarevok opened pull request #1642 (master…MBS-11028): MBS-11028: Convert historic Edit Release Language edit to React https://github.com/metabrainz/musicbrainz-serve...
      • BrainzBot
        MBS-11028: Convert historic Edit Release Language edit to React https://tickets.metabrainz.org/browse/MBS-11028
      • abhinavohri
        What is `self.assertContext('user', self.user)` doing?
      • I am not using 'url_for' method so should i keep it or omit it?
      • iliekcomputers
        ishaanshah: will have to cancel our meeting today. Apologies!
      • rdswift
        outsidecontext, zas: I've been collecting the new version documentation changes in a separate branch, so all I should need to do is rebase it and merge.
      • zas
        ok, thanks, we are preparing the release, binaries are built, website updated (but not deployed yet)
      • rdswift
        Just waiting for the release before doing that, but I can start a pr for it now. Thanks for the "heads up".
      • outsidecontext
        rdswift: yep, already found it :) I did some additions regarding the WAVE files and opened https://github.com/rdswift/picard-docs/pull/21
      • rdswift
        Beat me to it. Thanks.
      • zas
        we expect to release later today
      • rdswift
        Super!
      • sumedh has quit
      • pristine___
        ruaok: user_name [ rdswift, Mr_Monkey, ishaanshah, zas]
      • List of user name
      • ruaok
        on it
      • pristine___
        Cool.
      • ruaok
        any code changes?
      • pristine___
        Nope
      • ruaok
        request_dataframes and request_model as well?
      • pristine___
        ruaok escaped *master*
      • :p
      • > request_dataframes and request_model as well?
      • ruaok
        or request_candidate_sets and request_recommendations?
      • pristine___
        Yeah. No need to run that
      • > request_candidate_sets and request_recommendations?
      • Just these two
      • ruaok
        ok. heh, I forgot the -- in front of user-name .. I think I may have just requested *all* users. 🙄
      • if so I will kill the request consumer and start over.
      • but first some stats are running.
      • pristine___
        :p
      • Cool
      • ( just fyi, it is good to give list name with candidate_sets, and run recommendation as such)
      • BrainzGit
        [bookbrainz-site] prabalsingh24 opened pull request #488 (UserCollection…remove-collaborator): Feature: Remove Collaboration https://github.com/bookbrainz/bookbrainz-site/p...
      • shivam-kapila
        I am removed from list.
      • :p
      • v6lur has quit
      • v6lur joined the channel
      • kieto has quit
      • ruaok
        gah, what a busy day. but, PRs submitted, board meeting prepared, interview given, beer in hand. phew.
      • iliekcomputers
        interview?
      • alastairp
        german beer at that?
      • ruaok
      • iliekcomputers
        noice
      • i have the https://www.pythonpodcast.com/ thing in ~45 min
      • ruaok
      • alastairp: black forest beer is really quite good.
      • alastairp
        beer _and_ woods
      • jealous
      • ruaok
        Waldhaus Naturtrüb that I got from the honor system beer manger in the woods was hands down the best beer in weeks.
      • alastairp
        !!
      • ruaok
      • alastairp
        iliekcomputers: hey, I guess you're working and then interview? I had some thoughts about spark that I wanted to run past you
      • got some time around meeting-time?
      • ruaok
        its an honor system drinks station. each drink 2€. some chap was there swapping out cooling packs.
      • only in germany.
      • alastairp
        ruaok: did you see the recent photos of the mini wine windows in italy?
      • iliekcomputers
        alastairp: probably not today, is tomorrow ok?
      • alastairp
        any day is fine
      • ruaok
        yes, resurrected because of covid? great!
      • alastairp
        let's talk tomorrow. thanks
      • iliekcomputers
        sounds good.
      • pristine___ probably has more context on the collaborative filtering stuff re spark btw
      • alastairp
        it's more about the workflow, rather than tools/algorithms
      • iliekcomputers
        ok. cool. that's probably me then. :P
      • ruaok
        iliekcomputers: "[2020-08-10 14:57:44,631] DEBUG in request_consumer: Pushing result to RabbitMQ..."
      • is that stats being pushed? it takes a looong time....
      • iliekcomputers
        Which stat is it?
      • Note that because spark is lazy, it only does the actual computations when it needs to start pushing stuff which means the logs aren't exact. It's calculating the stat after it logs the pushing message.
      • Still don't expect anything to take hours though...
      • ruaok
        I dunno. I can't tell from the logs.
      • and I requested some recs, buts its been stuck like this for a while.
      • alastairp
        hey, so this is actually my question. maybe I'll open the discussion now, and we can continue it whenever
      • I understand that after you send a message to compute something, it's sent back to the listenbrainz side over rabbitmq
      • I was just thinking through this - is there a reason why we don't save it to hdfs and just return a filename? and then have listenbrainz download it from hdfs and do whatever with it
      • ruaok
        alastairp: they are hosted in two different places.
      • consider the machines at hetzner to be "permanent machines". We can expect them to be there at all times.
      • alastairp
        right, so that would require opening hdfs up on the public internet?
      • ruaok
        whereas the spark cluster is more flexible. right now its stored on 4 machines (dirt cheap, the stuff that someone has used before) at hetzner.
      • if someone were to offer us a better cluster for free, we'd move there.
      • MFCR_ColbyRay joined the channel
      • I'd love to have a 16 node cluster with loads of disk space, but alas, we're have a tiny 4 node cluster.
      • alastairp
        I'm not sure that that strictly discounts my suggestion, but it seems likely that it introduces additional complexity to the idea
      • ruaok
        this arrangement allows us to pass data back and forth using a mechanism we already rely on with minimal config exceptions.
      • it doesn't... yet.
      • the spark cluster is considered "disposable" and "batch oriented".
      • whereas our productions servers are considered to be "stable" and "per request fast".
      • alastairp
        right. it was just a thought that I had after looking through this part of the code, and after remembering these types of discussions happening a few times before (e.g. "does all of the data that we want to return fit inside rabbitmq?")
      • ruaok
        so, results being stored in PG for fast response to the user.
      • alastairp
        sure, I'm not suggesting removing results from pg
      • ruaok
        loads of discussions, yes.
      • alastairp
        the idea would be for the pg writer to request from hdfs, and then write to pg. instead of sending the results directly back in a rabbitmq message
      • ruaok
        and its a shitty balance between what I am willing to open our wallets for and what we need to work extra for.
      • alastairp
        instead, rabbitmq would just be a signaling mechanism
      • ruaok
        I see where you're going with that.
      • except docker. docker swarm in particular.
      • its.. special.
      • if you publish a port from a service, then docker goes and opens those ports for that server to the WORLD.
      • and you CANNOT say don't do that.
      • alastairp
        where does rabbitmq live? it's the main mb cluster, and is publicly accessible for spark to connect to?
      • right, got it
      • ruaok
        hetzer. lemmy has an opening that allows the spark leader to connect to rabbitmq. but lemmy only allows that from that one IP of the leader.
      • alastairp
        got it
      • iliekcomputers
        ~lemmy~ trille