#metabrainz

/

      • yvanzo
        zas: now logged into the server, and sudo it, thanks!
      • zas
        voilà, you have as much info we have now ;)
      • BrainzGit
        [musicbrainz-server] 14meziyum opened pull request #2892 (03master…documentation): Update Readme file with MusicBrainz project info https://github.com/metabrainz/musicbrainz-serve...
      • akshaaatt
        jasje I have confirmed that that was pretty much it. Also, I have set a limit to the number of listens we request while fetching
      • alastairp
        hi lucifer, you have a 300gb ~ in wolf, any chance you could clear up a bit?
      • jasje
        akshaaatt: noice i just saw what was the problem
      • prolly use remember for the incoming list too
      • akshaaatt
        we can take the list directly from the viewmodel
      • That shouldn't be a problem
      • Also jasje, I did a cleanup as well in the PR. You can have a look at how we can break things into small pieces to keep the code clen
      • clean*
      • For every UI screen, we should have a preview as well
      • I didn't find enough resources on how to play well with views that require a viewmodel
      • jasje
        akshaaatt: i do agree on both
      • akshaaatt
        docs suggest that we should use mockrepos for it
      • But again, now we have the mocks in the sharedrepo and importing from there was causing some issue
      • jasje
        akshaaatt: just add implementation(sharedTest) in dependencies
      • akshaaatt
        but that's not allowed
      • the sharedTest repo itself depends on our app
      • this isn't inception lol
      • jasje
        xD
      • akshaaatt
        We might have to bring the mocks back to the app for this
      • jasje
        akshaaatt: we'll have to duplicate everything then
      • for the three of em
      • or just duplicate roots connected method
      • lucifer
        alastairp: removed most of it.
      • alastairp
        <3
      • akshaaatt
        Let's see about it jasje after some research on best practices
      • jasje
        akshaaatt: alright
      • BrainzGit
        [listenbrainz-android] 14akshaaatt merged pull request #77 (03main…fix-app-nav-jasje): Converted activities to Composables https://github.com/metabrainz/listenbrainz-andr...
      • akshaaatt
        I am a bit confused with what to do with the PR https://github.com/metabrainz/listenbrainz-andr...
      • jasje should we omit it? It doesn't seem to help much
      • Pratha-Fish
        alastairp: That's 0.59s per file on avg! :D
      • jasje
        he isn't doing any work i poked him alot
      • Pratha-Fish
        !m alastairp
      • BrainzBot
        You're doing good work, alastairp!
      • jasje
        akshaaatt: ill just convert profile screen myself
      • akshaaatt
        Let's avoid logging in from the profile screen
      • I will handle that in the onboarding now
      • alastairp
        Pratha-Fish: do you know something else? Given the original 27 _billion_ rows, there are less than 2m unique canonical MBIDs. So I'll show you some more amazing optimisations if we run this again
      • akshaaatt
        The user has to be signed in
      • zas
        yvanzo: I think we should deploy mediawiki official docker image (https://hub.docker.com/_/mediawiki) on mingus, I installed automysqlbackup on wiki server, so we have db backups in /var/lib/automysqlbackup/
      • alastairp
        I think we can probably process the entire thing in half a day
      • zas
      • Pratha-Fish
        alastairp: half a day? 💀
      • jasje
        akshaaatt: that could work too
      • akshaaatt
        Is the design final from aerozol for the profile page?
      • Pratha-Fish
        only 2 million canonical MBIDs sounds a little sus though don't you think?
      • alastairp
        Pratha-Fish: 20 years of learning and half a day of computation
      • jasje
        akshaaatt: i guess he hasn't taken up the initiative yet
      • alastairp
        nah, I don't think so
      • jasje
        ill have to poke him a bit
      • Pratha-Fish
        Sheeesh
      • akshaaatt
        Let's get on his nerves then :)
      • alastairp
        monkey: mayhem: hi! Lunch next week? I have some things to contar, maybe I'll come and work in the office one day?
      • (nothing bad! just catching up)
      • akshaaatt
      • jasje
        akshaaatt: thak gya hu conflicts se
      • Pratha-Fish
        alastairp: BTW when's the final dataset gonna be ready? I could help with some docs / publishing stuff if you need any help
      • yvanzo
        zas: Opened a PR to document as it is for now.
      • alastairp
        Pratha-Fish: today? maybe?? or early next week
      • zas
        yvanzo: ok, thanks
      • Pratha-Fish
        Noice
      • jasje
        akshaaatt: you moved song view pager
      • akshaaatt
        XD yus
      • zas
        I'll do a migration attempt next week
      • jasje
        akshaaatt: i made alot of changes to that
      • akshaaatt: can you handle that :P
      • akshaaatt
        Just cut and paste it to the new destination then
      • I only updated the location, not the logic
      • Pratha-Fish
        alastairp: Let me know if there's any help you need with the publishing. I couldn't do much in terms of the final script due to some complications, but I can try and help with some basic stuff done as required ^_^
      • jasje
        akshaaatt: okeys
      • akshaaatt
        Yes jasje, you can submit your proposal early on the community forum for gsoc :)
      • jasje
        akshaaatt: oh yeah i forgor
      • alrighty, then here goes nothin
      • akshaaatt: posted
      • akshaaatt
        Awesome work jasje! 💯
      • Let me get back to you on this by next week
      • jasje
        akshaaatt: Thanks!!
      • alastairp
        Pratha-Fish: right, let's see once the archives are created. Regarding the script, in the end we just decided to run what we had (so there is some missing data that we know we can improve). If there's interest from people to have that updated data then maybe we'll do that
      • jasje
        Alrightyy
      • akshaaatt
        Also, I might get hella busy for the next 2 weeks. Have traumatic deadlines coming up
      • Pratha-Fish
        alastairp: sounds good :)
      • jasje
        :((
      • doesn't matter ill keep pinging you anyways haha
      • akshaaatt: conflicts resolved
      • akshaaatt: btw did you see the changes i made?
      • akshaaatt
        Which changes jasje?
      • jasje
        merge this one and see magic
      • real changes
      • you might have noticed the ui changes i guess
      • akshaaatt
        Yus yus
      • jasje
        look noice me proud
      • removed the top bars elevation
      • so it blends
      • akshaaatt
        Kewl
      • jasje
        bottom nav bar and view pager have same color
      • bold text here and there
      • and in the bp fix pr
      • performance improvements and more (checkout the commits)
      • BrainzGit
        [bookbrainz-site] 14MohamedAli00949 closed pull request #952 (03master…bb-718): fix(BB-718): CritiqueBrainz auth endpoint is called server-side https://github.com/metabrainz/bookbrainz-site/p...
      • mayhem
        alastairp: I'm out from the 13th - 24th. I guess there isn't anything urgent that needs telling?
      • alastairp
        not urgent at all
      • mayhem
        ok, then will need to be in april. I'm finally going traveling again. Its been far too long.
      • oh, I have an idea.
      • Adrian, one of the last.fm'ers who is part of our master class series (upcoming) will be in BCN the week of the 20th and some bits later than that.
      • the 24th he plans to work in the metabrainz office. I think you should meet him. Could we have informal drinks at the office late afternoon on the 24th?
      • alastairp
        perfect, I'm putting it in my calendar
      • mayhem
        I'll be a bit jet lagged, but whevs.
      • cool.
      • alastairp
        bon viatge!
      • jivte
        monkey: Hey!!
      • mayhem
        moltes gracies.
      • mayhem needs to program the catalan accents into this key board mapping
      • jivte_ joined the channel
      • jasje
        akshaaatt: should i reply to this: https://github.com/metabrainz/listenbrainz-andr... ?
      • with this:
      • Hello! Thanks for taking our app into consideration. Please consider opening an issue at [LB Android Issue Board](https://tickets.metabrainz.org/secure/RapidBoard.jspa?rapidView=10&projectKey=MOBILE). This helps us keep track of all the issues, hence solving problems faced by our users.
      • mayhem
        monkey: I have some questions on LB#2393
      • BrainzBot
        Fix user name search in PersonalRecommendationModal: https://github.com/metabrainz/listenbrainz-serv...
      • kellnerd joined the channel
      • monkey
        alastairp: Lunch with extreme pleasure ! :) We can do a cantine like the old days of a few months ago :p
      • mayhem
        alastairp: lucifer : how are integration tests run these days? the "int" option has gone from test.sh.
      • (and thus our docs are not up to date)
      • alastairp
        I believe both integration and unit tests happen at the same time with just test.sh (as we significantly sped up the process)
      • mayhem
        ah.
      • mayhem goes to try
      • alastairp
        in which case, docs should be updated to remove that
      • monkey
        aerozol: Thanks, mockups for the mobile version of the menus and navs look great ! That gives me all I need to start implementing that bit I think
      • kellnerd: Thanks for the PR! I made that change locally yesterday too but didn't test thoroughly to PR.
      • alastairp
        monkey: let me know what days this coming week you will be in the office, then!
      • monkey
        Probably most days actually.
      • I think I did every day this week
      • kellnerd
        Hehe, I needed a test DB for my ORM replacement experiments, so I had the same problem to solve yesterday :)
      • jivte
        monkey: Take a look at it when you get time :)
      • monkey
        kellnerd: Just pushed to that PR to fix the test suite. I'll wait for tests to pass then merge and respond on the forum
      • Thanks for th help in confirming the issue and solution :)
      • jasje has quit
      • jivte: Looks pretty good ! Less heavy without the repeating cover art.I think the track number is a little bit too prominent. They could be a tiny bit smaller font size and grey in color like the buttons to the right. In the codebase you can achieve that with the `<small>` html tag with a class of `text-muted`: https://usercontent.irccloud-cdn.com/file/NN6Bl...
      • I also think the rounded corners on the album header are a bit too much (too big). I'd recommend copying what we use on other elements (cards, listencard, etc.)
      • jasje_ has quit
      • Oh, and it could be useful to try it with a much longer album name, I foresee some potential space issues…
      • jivte
        I tried it no issues came to me
      • monkey
      • jivte
      • monkey: there were some space issues and i tried to resolve them
      • BrainzGit
        [musicbrainz-server] 14reosarevok merged pull request #2889 (03master…MBS-12983): MBS-12983: Update to DateTime::Locale 1.37 https://github.com/metabrainz/musicbrainz-serve...
      • jivte
        monkey: Sorry I missed some details this is the updated one