#metabrainz

/

      • reosarevok
        That said, I think the CAA numbers are big enough that it might not matter
      • So I'm ok with dropping
      • Freso
        ^
      • rdswift
        Also more incentive for people to upload to CAA.
      • reosarevok
        Yeah
      • KassOtsimine
        it's rlated in that the "this ocveris from amazon" + the actual image (iirc atleast) used to actually link directly with the mb referrer in
      • Freso
        (But then, I’ve said that for more than 5 years. :p)
      • reosarevok
        KassOtsimine: the amounts we ever got from that are tiny enough to be irrelevant IIRC anyway
      • KassOtsimine
        so does this mean we can get BAA soon too or?
      • reosarevok
        ruaok would know but :)
      • ruaok runs away
      • KassOtsimine
        (book art, not sheep)
      • reosarevok
        KassOtsimine: bitmap is still working on the event one, IIRC :)
      • KassOtsimine
        :DD
      • sigh
      • reosarevok
        Hopefully after that?
      • Freso
        Anyway. I think this concludes this topic. :)
      • reosarevok
        I wouldn't mind a sheep art archive, too
      • KassOtsimine
        bitmap: what's an eta on event art?
      • reo: :P
      • Freso
        I hear no complaints about that motion, so I’ll round up. :p
      • yvanzo supports reosarevok covering these stats!
      • TOPIC: MetaBrainz Community and Development channel | MusicBrainz non-development: #musicbrainz | BookBrainz: #bookbrainz | Channel is logged; see https://musicbrainz.org/doc/IRC for details | Agenda: Reviews
      • bitmap
        KassOtsimine: I hope it should be 'live' by the end of the year at least
      • KassOtsimine
        :O
      • bitmap
        there isn't much work left
      • it's just been postponed by working on other MBS things
      • KassOtsimine
        and hopefully yo ucna also use that work to base BAA on?
      • of course :)
      • Freso
        Thank you all for your time! Be sure to stay hydrated and take care of yourselves and those around you!
      • </BANG>
      • bitmap
        yea, the new indexer is a bit more extensible
      • reosarevok
        Well, the actual BAA code needs to be worked on by monkey I guess, but the containers and a fair amount of backend code I'd expect can be reused
      • I added MBS-11829 for the other topic, ruaok and bitmap please control it makes sense :)
      • BrainzBot
        MBS-11829: Allow redacting birth date off edit data https://tickets.metabrainz.org/browse/MBS-11829
      • KassOtsimine
        (i as spesifically becasue i 've got like 7 gigs of art scanned of books i'd liek to uh pload instead of storing on my computer :D)
      • yvanzo
        reosarevok: Don’t be specist, let's have an Animal Art Archive, AAA can only be a blockbuster! ;)
      • ruaok
        👍 to the ticket
      • KassOtsimine definitly wants cats
      • KassOtsimine
        but archives of catphotos aren'r rare
      • so the thing about AAA it woudl be dominated by cats (and soem dogs)
      • ruaok
        lucifer: is is allowed to have an array in a pandas dataframe? parqet?
      • reosarevok
        Can we have a pandas art archive?
      • reosarevok shuts up already
      • KassOtsimine
        reosarevok: Paa
      • Pä
      • ruaok
        lucifer: seems so, so I would be included in add a new column to the mbid mapping data frame: artist_mbids array[uuid]. how does that sound to you?
      • lucifer
        ruaok: yes. +1
      • ruaok
        k
      • yvanzo
        reosarevok: no, PAA stands for Projects Aborted Archive already.
      • reosarevok
        bitmap: sigh, https://github.com/metabrainz/musicbrainz-serve... is already +-500
      • I guess I should add flow to Input, then do Output in the same way as Input at least
      • bitmap
        that's fine, it's not too bad to read
      • reosarevok
        But after that maybe we should consider doing the next on a different PR. I dunno yet :) Let's see how bad it gets
      • Etua joined the channel
      • bitmap
        is wordListLength always equal to wordList.length? we should probably get rid of that then
      • reosarevok
        I think so?
      • In fact, it gets manually updated every time we change wordList, lol
      • bitmap
        yeah...
      • and stuff like insertWordAtEnd should really just use .push
      • reosarevok
        haha
      • As in
      • https://www.irccloud.com/pastebin/WQn2EH7Y/
      • Or as in "replace uses of insertWordAtEnd with .push"
      • I guess the former?
      • I'm also confused about
      • bitmap
        we can start with that
      • reosarevok
        https://www.irccloud.com/pastebin/XulRwM2o/
      • bitmap
        though searching for insertWordAtEnd, I don't see any uses of it :P
      • reosarevok
        Seems it just takes currentWord to see if wordIndex is in the wordList?
      • lol, me neither. Ok, off it goes
      • bitmap
        yes I think that's the intent behind that
      • reosarevok
        Isn't it the same as checking whether wordIndex is bigger than wordList.length??
      • bitmap
        maybe some entries can be nully?
      • reosarevok
        But then why wouldn't you want to update them?
      • bitmap
        these are good questions
      • reosarevok
        I'll make the change, then see if the tests pass :p
      • bitmap
        a lot of the gc code has a defensive coding style so it might be checking for things that don't happen in practice
      • adding types will help find more stuff like that
      • reosarevok
        So that would be
      • https://www.irccloud.com/pastebin/OPL7mfdS/
      • ?
      • lol, yeah, mainly defense by obfuscation :p
      • bitmap
        remove the -1?
      • reosarevok
        Yeah, that'd do it
      • Ok, time to stop for the day! Thanks! I think I can figure further stuff myself for now tomorrow
      • bitmap
        okay :) np