#metabrainz

/

      • Nyanko-sensei has quit
      • Nyanko-sensei joined the channel
      • supersandro2000 has quit
      • supersandro20003 joined the channel
      • Chinmay3199 joined the channel
      • v6lur has quit
      • CatQuest has quit
      • CatQuest joined the channel
      • CatQuest has quit
      • CatQuest joined the channel
      • supersandro20003 has quit
      • supersandro2000 joined the channel
      • MusicbrainzB0T1 joined the channel
      • MusicbrainzB0T has quit
      • MusicbrainzB0T joined the channel
      • MusicbrainzB0T1 has quit
      • dolina has quit
      • shivam-kapila
        Morning!!
      • prabal
        GoodMorningg!
      • yvanzo
        mo’’in’
      • diru1100
        Morning!!!
      • Mr_Monkey
        Moin !
      • iliekcomputers
        moin
      • ishaanshah[m]: i won't be able to make our meeting normal time today either, i pushed it an hour earlier.
      • v6lur joined the channel
      • Chinmay3199 has quit
      • reosarevok
        Argh
      • I hate the "authority control" name
      • People keep thinking we're the "owners" of some Wikipedia pages because they have authority control -> musicbrainz
      • yvanzo
        Do they want to pay us taxes or something?
      • Gazooo has quit
      • Gazooo joined the channel
      • CatQuest
        huh.
      • BrainzBot
        BB-472: The Neverending Identifierslist
      • CatQuest
        (yes my ticekt names are refrences to some books.)
      • mzfr joined the channel
      • ruaok
        Mr_Monkey: on the playlist page, which is behaving well for me (woo!), the timestamps for tracks are still shown.
      • I'd love to also show some metadata for the playlist.
      • I've thought more about your suggestion of giving playlists UUIDs and then saving them locally. I think that sounds like a good first step, since that allows me to go back to the playlist in question and look at the comments to find relevant data.
      • what should we do with the timestamps? can we put something else useful there?
      • CatQuest
        Mr_Monkey: this is probably a schema change. no? https://tickets.metabrainz.org/browse/BB-473
      • BrainzBot
        BB-473: Create a comprehensive relationship-type editing interface
      • CatQuest
        (also sorry for focusing on the bb thask things now that you're 50% on LB. it's jsut that i was planning on doing all these tickets before we heard of he priority change somewhat :D I hope lots of bb tickets now does not stress you)
      • Mr_Monkey is back
      • Mr_Monkey
        Thanks CatQuest ! I'll have a look later. More tickets is always good !
      • ruaok: Right, timestamps.
      • I'm not sure we need to put anything there.
      • ruaok
        and playlist metadaa. lets noodle on that too.
      • Mr_Monkey
        But definitely removing them
      • ruaok
        ok. I'm sure we'll find other stuff to put there in time.
      • Mr_Monkey
        Playlist metadata we should add to the html flask template. Since we're reusing the existing react components as much as possible that makes the most sense
      • ruaok
        hmm. ok.
      • then I'll change the structure of the data that is posted and do the parsing for you, then pass you nothing but the list of listens and then add new fields to template.
      • Mr_Monkey
        👍
      • Let me know where you're pushing that, I'll do the timestamp change there too
      • ruaok
        k, let me tuck away a couple of things I'm working on first.
      • CatQuest
        oh ho that reminds me. i actually have some listenbrainz ticket ( ^ิ ॢᗨ^ิ゚) ohohoho
      • reosarevok
        Oh no.
      • jmp_music joined the channel
      • jmp_music
        Moiiin
      • alastairp: I have studied your papers. I have to mentions that it is an incredible work. Could we please arrange a meeting e.g. in the middle of the next week? I have some questions about the implementation process, especially on the training phase, as the evaluation on the validation sets is obvious I think.
      • *mention
      • ruaok
        pristine__: is #846 ready to merge?
      • pristine__
        Yeah. Once you approve. I will test on the cluster then.
      • ruaok
        is there no way to test it before review?
      • pristine__
        I have tested on locally
      • ruaok
        ok.
      • alastairp
        jmp_music: sure. all days are good for me. How about Tuesday/Wednesday, the same time that we talked the other week? (3:30 CEST, 4:30 EEST)
      • jmp_music: by the way, I didn't send you a confirmation, but I also uploaded audio for the datasets to the same drive folder
      • shivam-kapila
        pristine__: Hi. I was willing to jump into the Spark stuff for recommendations. Can you help get into it? I am quite much interested in recommendations :)
      • pristine__
        shivam-kapila: let's talk about it tomorrow?
      • shivam-kapila
        Sure thing. Thanks.
      • iliekcomputers
        shivam-kapila: when will your endsems take place?
      • i'm wondering if it's worth it to get a start on the gsoc project instead
      • shivam-kapila
        iliekcomputers: No notification yet. The earlier notification said to open college on july 1 and then exams in July end. But I think they are gonna change things.
      • * july 1 -> June 1
      • ruaok
        pristine__: gah. *another* giant PR. :(
      • when you do cleanup AND code development in on PR it makes the PRs a nightmare to go through.
      • can you please please please not do this again?
      • shivam-kapila
        > i'm wondering if it's worth it to get a start on the gsoc project instead.
      • I also had that thought. I had some points in mind. I will discuss it with the team and if its okay I will get it started.
      • ruaok
        we can get started anytime you want.
      • shivam-kapila
        Hm. I guess then I should start from the next week itself.
      • I will see and plan the early step things this weekend.
      • jmp_music
        alastairp: Cool! thanks! yeah, Wednesday at 3:30 CEST sounds good
      • alastairp
        👍
      • pristine__
        ruaok: haven't done cleanup. The scripts are inter connected, so the tests fail so I had to do it in one PR. Clean up is a result of PEP-8 . I have pushed a commit for every script to make it a lil easy to review.
      • BrainzGit
        [listenbrainz-server] mayhem closed pull request #846 (vansika/candidate-recordings…candidate-recordings-tar): generate candidate recordings for recommendation toolkit https://github.com/metabrainz/listenbrainz-serv...
      • ruaok
        then break that PR into at least 2 PRs, please.
      • with iliekcomputers' lead we're all working to submit smaller PRs, so please do that as well.
      • pristine__
        Okay
      • ruaok
        it might be best to do the PEP-8 cleanup first, then the other changes.
      • it will make the changes easier to read, since the code overall is easier to read due to the PEP-8 cleanup.
      • iliekcomputers
        my general thinking these days is that if you're doing changes to existing code and the PR is more than 250 lines, it can probably be broken up.
      • pristine__
        Right.
      • ishaanshah[m]
        iliekcomputers: We will be calculating all the stats at once right?
      • Like, all_time, last_year, last_month. last_week
      • iliekcomputers
        ishaanshah[m]: make a seperate command for the month thing for now. I'll have to see how long it takes.
      • ishaanshah[m]
        I have done that
      • I was thinking we should load all the listens and go on filtering
      • step by step
      • iliekcomputers
        we'll have to experiment, doing that for month, week makes sense because they're in the same file
      • for the others, i'm not so sure.
      • ishaanshah[m]
        Right now time taken to load a months data is almost instatanous
      • Hmm, I have created seperate functions for all right now
      • Each functions loads the data it needs and filters it out accordingly
      • I will let it be for now then
      • We can figure the optimisations later
      • iliekcomputers
        sounds reasonable to me
      • prabal
        Mr_Monkey: ping
      • jmp_music has quit
      • jmp_music joined the channel
      • dolina joined the channel
      • Mr_Monkey
        prabal: pong
      • prabal
        Hi
      • Do you know how to mock res and req ? I need them for writing tests
      • I found a `mock-req-res` package
      • Will try that
      • alastairp
        iliekcomputers: ishaanshah[m]: Hi, can I download artist stats?
      • ishaanshah[m]
        alastairp: Yes you can
      • Just send a request to the api endpint
      • alastairp
        is there a dump for all users?
      • ishaanshah[m]
        no
      • not yet
      • alastairp
        ah. I have a great idea that I'd like to try out today
      • obviously the data is being copied from spark to listenbrainz though. I'll wait for iliekcomputers to jump in and tell me where it is, and I'll get my own copy
      • ishaanshah[m]
        its store in a pg database
      • stored*
      • shivam-kapila
        alastairp: Does SQL query work for you for the time being?
      • ishaanshah[m]
        statistics.user
      • shivam-kapila
        yes that ^^
      • NataliVon joined the channel
      • NataliVon has quit
      • iliekcomputers
        alastairp: this dump should have the table ideally: http://ftp.musicbrainz.org/pub/musicbrainz/list...
      • it's a bit old, but new dump should be coming up next week, i fixed cron.
      • the table is statistics.user
      • alastairp
        iliekcomputers: perfect, that's plenty for now
      • BrainzGit
        [listenbrainz-server] vansika opened pull request #852 (vansika/candidate-recordings…candidate-set-for-all): Generate candidate sets for all users at once. https://github.com/metabrainz/listenbrainz-serv...
      • ishaanshah[m]
        Any way to clear RMQ queue
      • I sent a wrong message and my RAM is full 😬
      • shivam-kapila
        ishaanshah[m]: Try to bash into rabbitmq container
      • docker exec -it listenbrainz_rabbitmq_1 bash
      • then `rabbitmqctl purge_queue <queue_name>`
      • alastairp
        iliekcomputers: is that a postgres dump format. or tsv? I tried to just read it manually and parse the data, but json module in python fails to load the data column
      • iliekcomputers
        postgres
      • prabal
        Mr_Monkey: I found a way. mock-req-res wasn't required :)
      • iliekcomputers