#metabrainz

/

      • shivam-kapila
        as iliekcomputers said
      • 2020-09-12 25654, 2020

      • pristine___
        yeah,.
      • 2020-09-12 25637, 2020

      • pristine___
        What?
      • 2020-09-12 25658, 2020

      • shivam-kapila
        you dont need the call then
      • 2020-09-12 25608, 2020

      • shivam-kapila
        just maipulate the state as iliekcomputers said
      • 2020-09-12 25615, 2020

      • pristine___
        Which call? API call?
      • 2020-09-12 25619, 2020

      • shivam-kapila
        yes
      • 2020-09-12 25631, 2020

      • pristine___
        I don't remember know what he said :(
      • 2020-09-12 25633, 2020

      • shivam-kapila
        since all the data is already on frontend
      • 2020-09-12 25647, 2020

      • yokel has quit
      • 2020-09-12 25651, 2020

      • shivam-kapila
        you have all the listens in props.listens
      • 2020-09-12 25654, 2020

      • pristine___
        Oh. I didn't see iliekcomputers 's message. Lol
      • 2020-09-12 25625, 2020

      • shivam-kapila
        you just need tyo set the desired 25 in the state
      • 2020-09-12 25615, 2020

      • shivam-kapila
        RecentListens will soon need a refactor. infact our whole react codebase needs little polish
      • 2020-09-12 25627, 2020

      • pristine___
        shivam-kapila: if that number would have been big maybe 2k or even bigger, then storing every thing in listen array would have been a bad idea no? Causing lag or something
      • 2020-09-12 25615, 2020

      • shivam-kapila
        if you render only 25 then wont affect much
      • 2020-09-12 25650, 2020

      • pristine___
        > RecentListens will soon need a refactor. infact our whole react codebase needs little polish
      • 2020-09-12 25608, 2020

      • iliekcomputers
        What's wrong with the react codebase?
      • 2020-09-12 25628, 2020

      • pristine___
        shivam-kapila: I just want to merge this PR then I will be writing a recommendation component at least.
      • 2020-09-12 25659, 2020

      • shivam-kapila
        iliekcomputers: just segregation. like the stats portion is proper but earlier code is little mixed up
      • 2020-09-12 25609, 2020

      • shivam-kapila
        pristine___: gotcha
      • 2020-09-12 25633, 2020

      • yokel joined the channel
      • 2020-09-12 25605, 2020

      • iliekcomputers
        Earlier code is mostly RecentListens :)
      • 2020-09-12 25624, 2020

      • pristine___
        shivam-kapila: so will try this next/previous thing. I will surely mess up. May need you again :p
      • 2020-09-12 25645, 2020

      • shivam-kapila
        no issues at all
      • 2020-09-12 25643, 2020

      • sumedh has quit
      • 2020-09-12 25638, 2020

      • sumedh joined the channel
      • 2020-09-12 25638, 2020

      • yokel has quit
      • 2020-09-12 25613, 2020

      • pristine___
        ruaok: labs api for recording mbid loopkup return 200 for an empty response. Is this desirable? Shouldn't it be 204?
      • 2020-09-12 25614, 2020

      • yokel joined the channel
      • 2020-09-12 25610, 2020

      • pristine___
        > Store the listens you're displaying in the state
      • 2020-09-12 25620, 2020

      • pristine___
      • 2020-09-12 25633, 2020

      • pristine___
        They are already stored in the state in here iliekcomputers , no?
      • 2020-09-12 25610, 2020

      • shivam-kapila
        thats a type definition
      • 2020-09-12 25617, 2020

      • shivam-kapila
      • 2020-09-12 25626, 2020

      • shivam-kapila
        what you want is here
      • 2020-09-12 25658, 2020

      • pristine___
        shivam-kapila: oh, cool.
      • 2020-09-12 25626, 2020

      • chaban has quit
      • 2020-09-12 25630, 2020

      • reosarevok has quit
      • 2020-09-12 25633, 2020

      • Mineo has quit
      • 2020-09-12 25626, 2020

      • joshuaboniface has quit
      • 2020-09-12 25632, 2020

      • chaban joined the channel
      • 2020-09-12 25652, 2020

      • Mineo joined the channel
      • 2020-09-12 25600, 2020

      • reosarevok joined the channel
      • 2020-09-12 25626, 2020

      • reg[m] has quit
      • 2020-09-12 25655, 2020

      • D4RK-PH0ENiX has quit
      • 2020-09-12 25641, 2020

      • goldenshimmer has quit
      • 2020-09-12 25628, 2020

      • D4RK-PH0ENiX joined the channel
      • 2020-09-12 25655, 2020

      • D4RK-PH0ENiX has quit
      • 2020-09-12 25647, 2020

      • D4RK-PH0ENiX joined the channel
      • 2020-09-12 25605, 2020

      • goldenshimmer joined the channel
      • 2020-09-12 25647, 2020

      • joshuaboniface joined the channel
      • 2020-09-12 25610, 2020

      • reg[m] joined the channel
      • 2020-09-12 25627, 2020

      • yokel has quit
      • 2020-09-12 25650, 2020

      • yokel joined the channel
      • 2020-09-12 25625, 2020

      • yokel has quit
      • 2020-09-12 25654, 2020

      • yvanzo
        nelgin: it cannot be up to date as the search.index queue is not empty yet.
      • 2020-09-12 25641, 2020

      • yvanzo
        but I notice that there are much lesser messages than the previous time.
      • 2020-09-12 25612, 2020

      • ruaok
        pristine___: sure, if you want to be really pedantic about it. :)
      • 2020-09-12 25616, 2020

      • pristine___
        ruaok: it will be help to debug the data we send to the api. no hurries :)
      • 2020-09-12 25626, 2020

      • ruaok
        make a ticket?
      • 2020-09-12 25656, 2020

      • ruaok
        shivam-kapila: did you make a ticket or PR for the one query that was buggy?
      • 2020-09-12 25634, 2020

      • yokel joined the channel
      • 2020-09-12 25622, 2020

      • yokel has quit
      • 2020-09-12 25657, 2020

      • yokel joined the channel
      • 2020-09-12 25625, 2020

      • sumedh has quit
      • 2020-09-12 25602, 2020

      • Gazooo794 has quit
      • 2020-09-12 25619, 2020

      • shivam-kapila
        ruaok: not yet. will do after lunch
      • 2020-09-12 25643, 2020

      • Gazooo794 joined the channel
      • 2020-09-12 25653, 2020

      • pristine___
        shivam-kapila: I tried writing a function for 25 recs on a page.
      • 2020-09-12 25628, 2020

      • pristine___
        when i click on the next button, the page keeps loading with the following error in console
      • 2020-09-12 25651, 2020

      • pristine___
      • 2020-09-12 25658, 2020

      • pristine___
        have you ever seen this?
      • 2020-09-12 25624, 2020

      • yokel has quit
      • 2020-09-12 25659, 2020

      • shivam-kapila
        yep
      • 2020-09-12 25600, 2020

      • v6lur joined the channel
      • 2020-09-12 25640, 2020

      • v6lur has quit
      • 2020-09-12 25650, 2020

      • v6lur joined the channel
      • 2020-09-12 25609, 2020

      • sumedh joined the channel
      • 2020-09-12 25605, 2020

      • yokel joined the channel
      • 2020-09-12 25614, 2020

      • pristine___
        shivam-kapila: what does it mean? There is no line number specified therefore I don't understand which part of the code the error is
      • 2020-09-12 25645, 2020

      • shivam-kapila
        pristine___: can you share the props.listens once
      • 2020-09-12 25633, 2020

      • yokel has quit
      • 2020-09-12 25636, 2020

      • pristine___
        props.listens?
      • 2020-09-12 25601, 2020

      • shivam-kapila
        A screenshot from components tab
      • 2020-09-12 25652, 2020

      • pristine___
      • 2020-09-12 25601, 2020

      • pristine___
        This?
      • 2020-09-12 25642, 2020

      • shivam-kapila
        copy the listens array and paste in gist
      • 2020-09-12 25630, 2020

      • pristine___
        Oh. All the 50 listens are identical?
      • 2020-09-12 25650, 2020

      • shivam-kapila
        yep thats the problem
      • 2020-09-12 25654, 2020

      • pristine___
        Is that the prob? I was testing so I just copied the same listen 50 times :(
      • 2020-09-12 25609, 2020

      • shivam-kapila
        when we render a list of components then each one has a different key
      • 2020-09-12 25619, 2020

      • pristine___
        I will try with unique recs.
      • 2020-09-12 25659, 2020

      • pristine___
        But listens is not a component right?
      • 2020-09-12 25600, 2020

      • shivam-kapila
      • 2020-09-12 25607, 2020

      • pristine___
        It's an array
      • 2020-09-12 25616, 2020

      • shivam-kapila
        the key is generated like this
      • 2020-09-12 25632, 2020

      • shivam-kapila
        pristine___: yeah it should be an array
      • 2020-09-12 25604, 2020

      • pristine___
        Ah cool. I get your point.
      • 2020-09-12 25617, 2020

      • yokel joined the channel
      • 2020-09-12 25629, 2020

      • pristine___
        shivam-kapila: one more thing
      • 2020-09-12 25638, 2020

      • shivam-kapila
        shoot
      • 2020-09-12 25643, 2020

      • pristine___
        The function I have written for next, previous works but
      • 2020-09-12 25602, 2020

      • pristine___
        Since all the recs are stored in 'listens', we have 200 recs when the page is loaded for the first time.
      • 2020-09-12 25626, 2020

      • pristine___
      • 2020-09-12 25608, 2020

      • pristine___
        I think I will have to do something to have only 25 recs in the listens array initially.
      • 2020-09-12 25612, 2020

      • pristine___
        no?
      • 2020-09-12 25628, 2020

      • shivam-kapila
        yesyou need to manipulate the state
      • 2020-09-12 25643, 2020

      • shivam-kapila
      • 2020-09-12 25647, 2020

      • shivam-kapila
        crop the array here
      • 2020-09-12 25603, 2020

      • shivam-kapila
        but do that for mode == cf_recs only
      • 2020-09-12 25640, 2020

      • pristine___
        Right. I will test these two things.
      • 2020-09-12 25647, 2020

      • pristine___
        shivam-kapila: thankssssssssssss
      • 2020-09-12 25629, 2020

      • v6lur has quit
      • 2020-09-12 25644, 2020

      • v6lur joined the channel
      • 2020-09-12 25614, 2020

      • shivam-kapila
        yop
      • 2020-09-12 25619, 2020

      • shivam-kapila
        yo*
      • 2020-09-12 25618, 2020

      • yokel has quit
      • 2020-09-12 25625, 2020

      • snazz joined the channel
      • 2020-09-12 25609, 2020

      • yokel joined the channel
      • 2020-09-12 25604, 2020

      • yokel has quit
      • 2020-09-12 25633, 2020

      • yokel joined the channel
      • 2020-09-12 25652, 2020

      • nelgin
        yvanzo, the reason the queue is smaller is because I created a new VM with fixed disk instead of dynamic and it's on a new ssd.
      • 2020-09-12 25641, 2020

      • nelgin
        If I was to do this again, I think I would do it natively on my linux box rather than using docker, if that was at all possible.
      • 2020-09-12 25603, 2020

      • nelgin
        I'm sure the disk speed is suffering due to the virtualization layer.
      • 2020-09-12 25646, 2020

      • yokel has quit
      • 2020-09-12 25629, 2020

      • yokel joined the channel
      • 2020-09-12 25641, 2020

      • BrainzGit
        [critiquebrainz] amCap1712 opened pull request #300 (master…CB-361): CB-361: Group member list shows None if relationship has end date and no start date https://github.com/metabrainz/critiquebrainz/pull…
      • 2020-09-12 25642, 2020

      • BrainzBot
        CB-361: Group member list shows None if relationship has end date and no start date https://tickets.metabrainz.org/browse/CB-361
      • 2020-09-12 25613, 2020

      • yokel has quit
      • 2020-09-12 25629, 2020

      • yokel joined the channel
      • 2020-09-12 25620, 2020

      • snazz has quit
      • 2020-09-12 25638, 2020

      • snazz joined the channel
      • 2020-09-12 25611, 2020

      • snazz has quit
      • 2020-09-12 25633, 2020

      • snazz joined the channel
      • 2020-09-12 25631, 2020

      • pristine___
        shivam-kapila: next/prev buttons and everything is working as expected :)
      • 2020-09-12 25633, 2020

      • pristine___
        Thanksssss
      • 2020-09-12 25650, 2020

      • nelgin
        yvanzo, in 11 hours I'm down to 109848 from 142387. After the next replication, if it's not caught up, I'm going to turn off the live indexing until you come up with something that's more suitable for mirror servers that don't have as many resources. Feel free to reach out to me if you want me to try something.
      • 2020-09-12 25655, 2020

      • v6lur has quit
      • 2020-09-12 25654, 2020

      • v6lur joined the channel
      • 2020-09-12 25609, 2020

      • yvanzo
        nelgin: thanks for your feedback
      • 2020-09-12 25622, 2020

      • _lucifer
        yvanzo: reosarevok: are broadcasts shown in release groups or not?
      • 2020-09-12 25647, 2020

      • _lucifer
        release groups for an artist
      • 2020-09-12 25629, 2020

      • _lucifer
      • 2020-09-12 25639, 2020

      • _lucifer
      • 2020-09-12 25656, 2020

      • _lucifer
        *show in
      • 2020-09-12 25623, 2020

      • yvanzo
        _lucifer: it doesn't show compilation by default, see “show various artists release groups” at the bottom
      • 2020-09-12 25606, 2020

      • _lucifer
        oh ok thanks yvanzo
      • 2020-09-12 25655, 2020

      • yokel has quit
      • 2020-09-12 25639, 2020

      • yvanzo
        _lucifer: but I cannot find it even then, so there might be something about broadcasts too.