#metabrainz

/

      • Lotheric_ is now known as Lotheric
      • 2019-05-26 14625, 2019

      • D4RK-PH0ENiX has quit
      • 2019-05-26 14639, 2019

      • Lotheric has quit
      • 2019-05-26 14606, 2019

      • Lotheric joined the channel
      • 2019-05-26 14659, 2019

      • Lotheric has quit
      • 2019-05-26 14624, 2019

      • D4RK-PH0ENiX joined the channel
      • 2019-05-26 14604, 2019

      • Lotheric joined the channel
      • 2019-05-26 14632, 2019

      • Lotheric has quit
      • 2019-05-26 14612, 2019

      • Lotheric joined the channel
      • 2019-05-26 14650, 2019

      • CallerNo6 joined the channel
      • 2019-05-26 14650, 2019

      • CallerNo6 has quit
      • 2019-05-26 14650, 2019

      • CallerNo6 joined the channel
      • 2019-05-26 14606, 2019

      • Cyna
        reosarevok: what does // mean ?
      • 2019-05-26 14609, 2019

      • Cyna
        $split //= '';
      • 2019-05-26 14625, 2019

      • Cyna
        Im trying to convert Diff.diff_side
      • 2019-05-26 14617, 2019

      • reosarevok
        Wasn't that the stuff bitmap told you he was already converting?
      • 2019-05-26 14613, 2019

      • tykling_ is now known as Tykling
      • 2019-05-26 14607, 2019

      • Cyna
        Ohh he was... Didn't know
      • 2019-05-26 14614, 2019

      • Cyna
        So now what do I do ?
      • 2019-05-26 14647, 2019

      • Cyna
        They are used in the .TT file and for converting to React completely I need them
      • 2019-05-26 14618, 2019

      • Cyna
        Guess I'll wait till it's done
      • 2019-05-26 14638, 2019

      • Cyna
        Let me know if you have any other suggestions
      • 2019-05-26 14613, 2019

      • D4RK-PH0ENiX has quit
      • 2019-05-26 14621, 2019

      • reosarevok
        Cyna: he might have a commit you can use. Did you already finish add_remove for alias?
      • 2019-05-26 14636, 2019

      • Cyna
        I made conversions for edit_alias
      • 2019-05-26 14642, 2019

      • D4RK-PH0ENiX joined the channel
      • 2019-05-26 14600, 2019

      • Cyna
        I'll also convert the other part for add and remove alias
      • 2019-05-26 14622, 2019

      • CatQuest
        brb
      • 2019-05-26 14625, 2019

      • CatQuest has left the channel
      • 2019-05-26 14636, 2019

      • Lotheric_ joined the channel
      • 2019-05-26 14622, 2019

      • Lotheric has quit
      • 2019-05-26 14637, 2019

      • CatQuest joined the channel
      • 2019-05-26 14637, 2019

      • CatQuest has quit
      • 2019-05-26 14637, 2019

      • CatQuest joined the channel
      • 2019-05-26 14659, 2019

      • Cyna
        what is html and locale vars ?, are they passed in from controller ?
      • 2019-05-26 14634, 2019

      • Cyna
      • 2019-05-26 14657, 2019

      • reosarevok
        I mean, bitmap has told you like three times to start with add_remove. Your life would be easier if you listened to him :)
      • 2019-05-26 14641, 2019

      • reosarevok
        html is another filter. IIRC you don't need it on React because React escapes html by default, but confirm with bitmal
      • 2019-05-26 14643, 2019

      • reosarevok
        *bitmap
      • 2019-05-26 14659, 2019

      • reosarevok
        You can start the conversion without it and add something if he tells you you need something :)
      • 2019-05-26 14643, 2019

      • Cyna
        I couldnt understand what he meant at that time 😅
      • 2019-05-26 14650, 2019

      • Cyna
        reosarevok: I finished converting the add_remove_alias.tt. How do I test it ?
      • 2019-05-26 14619, 2019

      • reosarevok
        Did you add the edit_template_react to the appropriate Server/Edit files?
      • 2019-05-26 14642, 2019

      • reosarevok
        If so, I think you can just add or remove an alias and then look at the edit it generated
      • 2019-05-26 14656, 2019

      • Cyna
        what is edit_template_react ?
      • 2019-05-26 14603, 2019

      • Cyna
        I didnt do it I guess
      • 2019-05-26 14613, 2019

      • reosarevok
        Look at https://github.com/metabrainz/musicbrainz-server/… again and follow that general idea then
      • 2019-05-26 14650, 2019

      • reosarevok
        If your edit controller has booleans then you should use boolean_to_json like on that PR too :)
      • 2019-05-26 14631, 2019

      • Cyna
        I'll have a look. Thanks :)
      • 2019-05-26 14659, 2019

      • Cyna
        in my case I should edit `lib/MusicBrainz/Server/Edit/Alias/Add.pm` and Delete.pm , right ?
      • 2019-05-26 14615, 2019

      • reosarevok
        Yes, I think so :)
      • 2019-05-26 14613, 2019

      • Cyna
        I made the changes for Add.pm
      • 2019-05-26 14615, 2019

      • Cyna
        Testing it now
      • 2019-05-26 14635, 2019

      • Cyna
        when you have time please test my changes
      • 2019-05-26 14657, 2019

      • Cyna
      • 2019-05-26 14652, 2019

      • Cyna
        I've only made changes to Add.pm for now
      • 2019-05-26 14613, 2019

      • Cyna
        Im getting a weird error while testing
      • 2019-05-26 14614, 2019

      • reosarevok
        What's the error?
      • 2019-05-26 14632, 2019

      • reosarevok is writing some code, maybe can test myself later, but might be able to help with the error sooner :)
      • 2019-05-26 14652, 2019

      • Cyna
      • 2019-05-26 14622, 2019

      • Cyna
        Rendering issue but I dont know why did it occur... I followed steps clearly
      • 2019-05-26 14640, 2019

      • reosarevok
        Cyna: did you add it to server/components?
      • 2019-05-26 14649, 2019

      • reosarevok
        (not sure why that's not on that PR actually, but it's in that file)
      • 2019-05-26 14638, 2019

      • Cyna
        no should I ?
      • 2019-05-26 14648, 2019

      • Cyna
        The PR you sent me didnt have the changes
      • 2019-05-26 14622, 2019

      • Cyna
        And the other files dont have their respective paths either so didnt add it there
      • 2019-05-26 14627, 2019

      • Cyna
        should i
      • 2019-05-26 14627, 2019

      • Cyna
        ?
      • 2019-05-26 14609, 2019

      • reosarevok
        Yes, you should
      • 2019-05-26 14623, 2019

      • reosarevok
        For files that get used in React.Embed you need that
      • 2019-05-26 14652, 2019

      • reosarevok
        I'm really confused why that wasn't in that PR
      • 2019-05-26 14607, 2019

      • Lotheric_ is now known as Lotheric
      • 2019-05-26 14625, 2019

      • Cyna
        Made the changes that fixed the issue
      • 2019-05-26 14647, 2019

      • Cyna
        Now Im comparing it with tt counterpart and checking logical errors
      • 2019-05-26 14632, 2019

      • Lotheric
      • 2019-05-26 14648, 2019

      • Lotheric
        question from ijabz
      • 2019-05-26 14655, 2019

      • Lotheric
        anyone know ?
      • 2019-05-26 14603, 2019

      • Cyna
        reosarevok: what does this do ?
      • 2019-05-26 14602, 2019

      • Cyna
        I've commited the latest changes to my remote branch... Its working fine except there are some rendering bugs
      • 2019-05-26 14612, 2019

      • Cyna
      • 2019-05-26 14631, 2019

      • Cyna
        Issue in rendering after `Sort name`
      • 2019-05-26 14602, 2019

      • reosarevok
        Cyna: " what does this do ?" what this? :)
      • 2019-05-26 14623, 2019

      • reosarevok
        Cyna: also, you have td and/or tr instead of th there
      • 2019-05-26 14635, 2019

      • reosarevok
        Also, you have begin_data instead of begin_date :)
      • 2019-05-26 14632, 2019

      • reosarevok
        But yeah
      • 2019-05-26 14634, 2019

      • reosarevok
      • 2019-05-26 14645, 2019

      • reosarevok
        Original had th for the first of those
      • 2019-05-26 14655, 2019

      • reosarevok
      • 2019-05-26 14603, 2019

      • reosarevok
        Should be th, then td
      • 2019-05-26 14606, 2019

      • reosarevok
        Etc :)
      • 2019-05-26 14648, 2019

      • Cyna
        Ohh will have a look in sometime
      • 2019-05-26 14650, 2019

      • Cyna
        Thanks :)
      • 2019-05-26 14618, 2019

      • rdswift
        zas, I see that outsidecontext isn't around. Do you have any comments regarding https://community.metabrainz.org/t/picard-portabl…
      • 2019-05-26 14627, 2019

      • zas
        rdswift: not really, it seems tickets exist, and patches are welcome ;)
      • 2019-05-26 14651, 2019

      • rdswift
        Okay, thanks.
      • 2019-05-26 14605, 2019

      • zas
        it should be noted there's a command line option to select a different config file
      • 2019-05-26 14622, 2019

      • zas
        and there's a ticket somewhere about "profiles"
      • 2019-05-26 14649, 2019

      • rdswift
        Thanks again. I responded in the discussion with this information.
      • 2019-05-26 14629, 2019

      • candyman joined the channel
      • 2019-05-26 14605, 2019

      • candyman
        hey I'm trying to do some setup/eval of MB and I'm not sure if I'm taking the right route to achieve my goals...
      • 2019-05-26 14614, 2019

      • candyman
        anyone around that could answer a few basic questions?
      • 2019-05-26 14642, 2019

      • Cyna
        reosarevok: Is there some other way i can write this ```IF !edit.display_data.begin_date.is_empty ```
      • 2019-05-26 14603, 2019

      • reosarevok
        candyman: yes, ask
      • 2019-05-26 14611, 2019

      • reosarevok
        Cyna: I can check in a bit
      • 2019-05-26 14617, 2019

      • Cyna
        sure thanks
      • 2019-05-26 14629, 2019

      • candyman
        so the application I'm building is going to both need to pull artist genres (tags) rapidly, and will also be producing new data in the form of artists and their associated genres
      • 2019-05-26 14604, 2019

      • reosarevok
        Cyna: that means " return !(defined $self->year || $self->month || $self->day);" (so it's empty if it's missing those three) FWIW
      • 2019-05-26 14616, 2019

      • candyman
        so my 1st thought is that I would use the musicbrainz-api to both submit new data and pull new data rapidly
      • 2019-05-26 14647, 2019

      • Cyna
        If possible, is there an easy to understand answer
      • 2019-05-26 14651, 2019

      • Cyna
        😅
      • 2019-05-26 14657, 2019

      • candyman
        however the rate-limiting info says that only python-musicbrainz/0.7.3 gets a special limit of 50/sec - would that be true of the node version 'musicbrainz-api' as well?
      • 2019-05-26 14648, 2019

      • candyman
        that is, my application - if implementing the musicbrainz-api - could pull at 50/sec?
      • 2019-05-26 14636, 2019

      • reosarevok
        candyman: I didn't even know we allowed someone to use 50/sec - I thought it was 1/sec for everyone
      • 2019-05-26 14601, 2019

      • reosarevok
        But I might be misremembering something :)
      • 2019-05-26 14621, 2019

      • candyman
      • 2019-05-26 14638, 2019

      • candyman
        and that the IP address limit of 1/sec trumps the user-agent
      • 2019-05-26 14652, 2019

      • reosarevok
        Cyna: actually: import isDateEmpty from '../static/scripts/common/utility/isDateEmpty';
      • 2019-05-26 14659, 2019

      • reosarevok
        Seems to be a thing already :)
      • 2019-05-26 14615, 2019

      • Cyna
        ohh
      • 2019-05-26 14637, 2019

      • reosarevok
        candyman: oh, yeah, that's us *blocking* some agents
      • 2019-05-26 14641, 2019

      • reosarevok
        (more than usual)
      • 2019-05-26 14651, 2019

      • reosarevok
        Because they're associated with bad usage
      • 2019-05-26 14611, 2019

      • candyman
        so if I needed to make fast requests for MB info - there is no API option for this? I would need to setup my own server to peg at my leisure?
      • 2019-05-26 14653, 2019

      • candyman
        I don't need anything crazy but resolving one artist/sec is . . not ideal
      • 2019-05-26 14623, 2019

      • reosarevok
        In general, yes. What's the use case? (can't they be done only once and then cached until needed again?)
      • 2019-05-26 14626, 2019

      • candyman
        its resolving giant batches of artists that come from different sources - spotify playlists, songkick event listings, etc - almost on-demand
      • 2019-05-26 14608, 2019

      • Cyna
        if possible please help me making a PropType for AddRemoveAlias
      • 2019-05-26 14647, 2019

      • BrainzGit
        [musicbrainz-server] reosarevok opened pull request #1083 (master…MBS-9918): MBS-9918: Convert the label public pages to React https://github.com/metabrainz/musicbrainz-server/…
      • 2019-05-26 14649, 2019

      • BrainzBot
        MBS-9918: Convert the label public pages to React https://tickets.metabrainz.org/browse/MBS-9918
      • 2019-05-26 14659, 2019

      • Cyna
        Its functioning perfectly now I'm adding flow to it
      • 2019-05-26 14646, 2019

      • reosarevok
        candyman: how are you planning to do it? :)
      • 2019-05-26 14658, 2019

      • reosarevok
        (say, if you get "Nirvana", how will you map it to the right Nirvana artist)
      • 2019-05-26 14602, 2019

      • candyman
        magic. no lol unfortunately Echonest's Rosetta stone isn't a thing anymore - so resolving Spotify -> MB has been hard but not impossible. I look at their albums and such
      • 2019-05-26 14624, 2019

      • candyman
        Songkick fortunately uses MusicBrainz ids so I can map directly a lot of the time
      • 2019-05-26 14639, 2019

      • reosarevok
        I mean, for some artists we might have the Spotify ID on our side for example, but sadly we don't have anywhere as many as I'd like
      • 2019-05-26 14615, 2019

      • candyman
        right. but assuming I could figure out that link, sounds like setting up a server - which OMG
      • 2019-05-26 14636, 2019

      • candyman
        the indexing has ended, dear lord that took some time
      • 2019-05-26 14601, 2019

      • candyman
        I as of just now have a server setup w/out replication
      • 2019-05-26 14604, 2019

      • reosarevok
        Setting up a server might be the easiest way, yes, unless you can figure good ways of doing cache on your side
      • 2019-05-26 14629, 2019

      • candyman
        alright so part 2 (sorry, you've been a great help, theres no part 3)
      • 2019-05-26 14631, 2019

      • reosarevok
        Is this supposed to be mostly for yourself, or a bigger service? :)
      • 2019-05-26 14603, 2019

      • candyman
        I don't plan on distributing the service super widely, a passion project
      • 2019-05-26 14604, 2019

      • reosarevok
        If you're going to hammer your own server yourself (or with some friends) it probably doesn't matter, but if it'll be a bigger thing you're likely to want some caching anyway :)
      • 2019-05-26 14629, 2019

      • reosarevok
        Part 2? :)
      • 2019-05-26 14646, 2019

      • candyman
        I feel that, thank you. I have an idea already for how to cache the event lookups - just let people 'think' their looking at live data...
      • 2019-05-26 14605, 2019

      • reosarevok
        Tbh we don't get new tags that often, at least not yet
      • 2019-05-26 14625, 2019

      • reosarevok
        Although we'd love to change that, heh. Maybe if ListenBrainz takes off :)
      • 2019-05-26 14640, 2019

      • reosarevok
        Cyna: re: props, what do you have now?
      • 2019-05-26 14650, 2019

      • candyman
        anyways, the new data that I submit. I find that Spotify and MB don't have any genre info, but I manage to figure it out some new genres from other sources
      • 2019-05-26 14614, 2019

      • reosarevok
        Look at the display_data, you'll need to give that proper types.