#metabrainz

/

      • reosarevok
        Pretty!
      • 2020-09-09 25304, 2020

      • bitmap
        anyway, do you think we could look at one more? https://github.com/metabrainz/musicbrainz-server/…
      • 2020-09-09 25331, 2020

      • bitmap
        this one doesn't seem blocked by any future UI redesign
      • 2020-09-09 25357, 2020

      • yvanzo
        this one needs clarification, I'm pretty sure I got confused somewhere as bitmap wrote.
      • 2020-09-09 25354, 2020

      • bitmap
        I think the cases I wrote in the ticket are still the only places we show the Writers
      • 2020-09-09 25319, 2020

      • BrainzGit
        [listenbrainz-server] release v-2020-09-09.1 has been published by paramsingh: https://github.com/metabrainz/listenbrainz-server…
      • 2020-09-09 25353, 2020

      • yvanzo
        reosarevok: is that correct? if so, can the ticket's description be updated accordingly?
      • 2020-09-09 25304, 2020

      • reosarevok
        Let me see
      • 2020-09-09 25317, 2020

      • reosarevok
        Yes, I mean work writers, isn't that what the description says? :D
      • 2020-09-09 25342, 2020

      • bitmap
        any ideas on how we can make the description more clear?
      • 2020-09-09 25332, 2020

      • yvanzo
        reosarevok: no, it's not
      • 2020-09-09 25357, 2020

      • yvanzo
        reosarevok: it's difficult to tell the scope of the change from the ticket's description.
      • 2020-09-09 25300, 2020

      • reosarevok
        I guess it can specify it's not only shown on searches, but
      • 2020-09-09 25301, 2020

      • reosarevok
        Ok
      • 2020-09-09 25335, 2020

      • yvanzo
        If your patch is applied, this page won't display “Australian Chamber Orchestra” anymore, right? https://musicbrainz.org/search?query=artist%3A%22…
      • 2020-09-09 25318, 2020

      • reosarevok
        Not under writers, no
      • 2020-09-09 25328, 2020

      • bitmap
        right
      • 2020-09-09 25333, 2020

      • reosarevok
        IMO a search shouldn't find it either
      • 2020-09-09 25338, 2020

      • reosarevok
        But that's a search server change I guess
      • 2020-09-09 25345, 2020

      • yvanzo
        I can hardly understand how that would be an improvement though, just changing the column label woudn't do?
      • 2020-09-09 25316, 2020

      • reosarevok
        Because the writers column wasn't meant to be a list of all relationships to a work, it was supposed to be for writer-related rels
      • 2020-09-09 25324, 2020

      • reosarevok
        It's not that bad in here (although still distracting)
      • 2020-09-09 25300, 2020

      • yvanzo
        But at that time, there was no interest/effort put in dedication and all, maybe now it's different?
      • 2020-09-09 25330, 2020

      • reosarevok
        But it's super weird in an inline search, where while searching for, say, Tavener's Eternal Memory you'll get "John Tavener, Australian Chamber Orchestra, Steven Isserlis"
      • 2020-09-09 25333, 2020

      • reosarevok
        eh.
      • 2020-09-09 25336, 2020

      • yvanzo
        Returning less results doesn't look like an improvement to me.
      • 2020-09-09 25343, 2020

      • reosarevok
        "Writers: John Tavener, Australian Chamber Orchestra, Steven Isserlis"
      • 2020-09-09 25355, 2020

      • yvanzo
        That is a presentation issue.
      • 2020-09-09 25307, 2020

      • reosarevok
        At that time we didn't realize we would have artist-work relationships that are not writers
      • 2020-09-09 25321, 2020

      • reosarevok
        ... yes? All of this is about presentation. To be precised, this stuff shouldn't be presented there
      • 2020-09-09 25325, 2020

      • reosarevok
        *precise
      • 2020-09-09 25351, 2020

      • reosarevok
        I mean, we could instead rename that column and the inline search to "artists related to the work" rather than "writers"
      • 2020-09-09 25352, 2020

      • yvanzo
        but changing search results isn't just changing presentation
      • 2020-09-09 25321, 2020

      • reosarevok
        Well, given the search is "artists" and not "writers", maybe that can be kept, they just wouldn't appear under writers nor artists because they're neither
      • 2020-09-09 25342, 2020

      • bitmap
        I'm not sure where/if we should show the non-writers but removing them from the writers columns makes sense to me
      • 2020-09-09 25301, 2020

      • bitmap
        we do already have an "artists" column though which it may be confused with
      • 2020-09-09 25340, 2020

      • yvanzo
        "composition & misc"?
      • 2020-09-09 25333, 2020

      • yvanzo
        getting search results without the relationship to the queried artist would be weird
      • 2020-09-09 25329, 2020

      • reosarevok
        I expect most classical editors (the case where these other rels are most common) don't want the writers column full of irrelevant stuff
      • 2020-09-09 25334, 2020

      • yvanzo
        column "artists" can be renamed "recording artists"
      • 2020-09-09 25337, 2020

      • reosarevok
        Renaming the column doesn't help with the usability issue
      • 2020-09-09 25351, 2020

      • reosarevok
        The inline search issue is especially bad
      • 2020-09-09 25315, 2020

      • reosarevok
        I need to be able to see at a glance that this is by Tavener, not try to guess it from a bunch of crap
      • 2020-09-09 25344, 2020

      • bitmap
        right, it's a bigger issue there
      • 2020-09-09 25346, 2020

      • yvanzo
        there is nothing about "the inline search issue" in the ticket
      • 2020-09-09 25346, 2020

      • reosarevok
        I would personally rather drop these reltypes than keep them there tbh, and I added them because I wanted them in the first place. But not at that price
      • 2020-09-09 25314, 2020

      • reosarevok
        It's the same issue, only there's not even info about what kind of role each writer had
      • 2020-09-09 25349, 2020

      • reosarevok
        And that there it was always meant to tell the composers and whatnot at a glance - all the other info is completely irrelevant there
      • 2020-09-09 25307, 2020

      • reosarevok
        I can't think of a single case where you're looking in the inline search for a work dedicated to X
      • 2020-09-09 25325, 2020

      • bitmap
        we have limited space there anyway so I would def. remove the non-writers there
      • 2020-09-09 25352, 2020

      • bitmap
        should we add another column for the non-writing rels to the search results?
      • 2020-09-09 25326, 2020

      • yvanzo
        this seems to be specific to the inline search
      • 2020-09-09 25339, 2020

      • reosarevok
        No, this is most important in the inline search
      • 2020-09-09 25354, 2020

      • reosarevok
        But the expectation of our editors is that they'll find writers in there
      • 2020-09-09 25301, 2020

      • reosarevok
        If you want we can have a misc column, yes
      • 2020-09-09 25316, 2020

      • reosarevok
        Eh. " in there" -> in the writers column
      • 2020-09-09 25340, 2020

      • yvanzo
        it's already confusing to have "writers" and "artists" to be honest
      • 2020-09-09 25345, 2020

      • reosarevok
        I guess that would fit fine, so if you want a third column with "misc. relationships"... :)
      • 2020-09-09 25308, 2020

      • reosarevok
        I mean, it's cool info, if it doesn't get in the way
      • 2020-09-09 25313, 2020

      • bitmap
        we could put them in the existing artists column with small performers/misc. headings?
      • 2020-09-09 25318, 2020

      • bitmap
        dunno
      • 2020-09-09 25347, 2020

      • reosarevok
        The existing artists column is already sometimes absurdly big (did we already merge the code collapsing it in those cases? I think we did)
      • 2020-09-09 25305, 2020

      • bitmap
        not sure if we did
      • 2020-09-09 25304, 2020

      • yvanzo
        reosarevok: I mean the solution is specific to the inline search: it seems legit to drop these dedications from the inline search if the rel type can't be displayed, but not from the work search page where it can be displayed.
      • 2020-09-09 25314, 2020

      • bitmap
        slapping on a "Misc. Relationships" column is fine with me for now too, as long as we're all happy with it
      • 2020-09-09 25326, 2020

      • reosarevok
        yvanzo: they're not writing-related. They shouldn't be under writers
      • 2020-09-09 25332, 2020

      • reosarevok
        I don't care what happens to them other than that
      • 2020-09-09 25335, 2020

      • reosarevok
        Third column works for me
      • 2020-09-09 25350, 2020

      • yvanzo
        reosarevok: I agree, not under writers either
      • 2020-09-09 25359, 2020

      • reosarevok
      • 2020-09-09 25317, 2020

      • yvanzo
        can we rename that vague column title "artists" too btw?
      • 2020-09-09 25329, 2020

      • bitmap
        cool. so either another column, or merge them into Artists and add small sections
      • 2020-09-09 25341, 2020

      • reosarevok
        yvanzo: to recording artists? Would work for me
      • 2020-09-09 25347, 2020

      • reosarevok
        (here, in inline search it would be too long I think)
      • 2020-09-09 25324, 2020

      • yvanzo
        in inline search it could be: "composed by" and "recorded by"
      • 2020-09-09 25320, 2020

      • bitmap
        I think the idea behind using Writers is that they're all writing related but not necessarily composers
      • 2020-09-09 25336, 2020

      • bitmap
        and merging them into a single list saves space
      • 2020-09-09 25347, 2020

      • yvanzo
        "wrote by" if you prefer
      • 2020-09-09 25309, 2020

      • yvanzo
        I used "composed by" because it's the top relationship type
      • 2020-09-09 25319, 2020

      • yvanzo
      • 2020-09-09 25301, 2020

      • bitmap
        why change Writers to wrote by tho?
      • 2020-09-09 25324, 2020

      • v6lur joined the channel
      • 2020-09-09 25329, 2020

      • yvanzo
        to change "recording artists" to "recorded by"
      • 2020-09-09 25352, 2020

      • yvanzo
        currently just "artists" which doesn't tell about the relationship type
      • 2020-09-09 25350, 2020

      • reosarevok
        Well, if the recording artist is still set to the composer, claiming "recorded by" is a lie :p
      • 2020-09-09 25307, 2020

      • reosarevok
        They're recording artists, but that doesn't necessarily mean more than that
      • 2020-09-09 25319, 2020

      • reosarevok
        (since they're not taken from relationships that actually say *something* meaningful)
      • 2020-09-09 25335, 2020

      • yvanzo
        if recording artist is set to the composer, it should be fixed, no?
      • 2020-09-09 25301, 2020

      • reosarevok
        Well, yes, but it's accepted in the guideline as a middle point
      • 2020-09-09 25308, 2020

      • yvanzo
        ?
      • 2020-09-09 25325, 2020

      • reosarevok
        We don't force people adding releases to change recording artists, because of how much of a pain it is :p
      • 2020-09-09 25332, 2020

      • reosarevok
        In general, though, especially outside classical, "recording artist" can be anything
      • 2020-09-09 25341, 2020

      • reosarevok
        It's *often* the performer, but...
      • 2020-09-09 25344, 2020

      • yvanzo
        but when the data is defective
      • 2020-09-09 25330, 2020

      • yvanzo
        :) I got your point and I know that can be the case for a lot of classical recordings but no more, right?
      • 2020-09-09 25324, 2020

      • yvanzo
        bitmap, reosarevok: 2h30 already, any other point that should be discussed quickly?
      • 2020-09-09 25329, 2020

      • bitmap
        not sure we can be any more specific than "recording artists" there
      • 2020-09-09 25342, 2020

      • bitmap
        since it's just who it's mainly credited to
      • 2020-09-09 25301, 2020

      • bitmap
        not today, I'd mostly just like to come to a resolution on this one
      • 2020-09-09 25338, 2020

      • bitmap
        if we add another column to the search results is that sufficient?
      • 2020-09-09 25355, 2020

      • yvanzo
        sure, the "artists" column was almost beyond the scope of this change
      • 2020-09-09 25315, 2020

      • bitmap
        yeah, if we can fix the bug first and improve strings later that'd be ideal for getting the PR closed, I think
      • 2020-09-09 25343, 2020

      • reosarevok
        yvanzo: can you leave a quick note on the ticket or the PR about this? :)
      • 2020-09-09 25352, 2020

      • reosarevok
        Wherever you think is better of the two
      • 2020-09-09 25332, 2020

      • yvanzo
        Ok, will make a comment.
      • 2020-09-09 25323, 2020

      • bitmap
        thanks!
      • 2020-09-09 25353, 2020

      • bitmap
        that's probably enough for now but I'll take a look at the OAuth stuff and the smaller PRs reosarevok linked at the beginning of the meeting later
      • 2020-09-09 25302, 2020

      • reosarevok
        Thanks!
      • 2020-09-09 25306, 2020

      • yvanzo
        thanks!
      • 2020-09-09 25303, 2020

      • BrainzGit
        [musicbrainz-android] amCap1712 opened pull request #60 (master…refactor-lookup-activity): Refactor lookup activity https://github.com/metabrainz/musicbrainz-android…
      • 2020-09-09 25313, 2020

      • BrainzGit
        [musicbrainz-android] amCap1712 merged pull request #60 (master…refactor-lookup-activity): Refactor lookup activity https://github.com/metabrainz/musicbrainz-android…
      • 2020-09-09 25329, 2020

      • piano_it_guy joined the channel
      • 2020-09-09 25308, 2020

      • piano_it_guy has quit
      • 2020-09-09 25336, 2020

      • piano_it_guy joined the channel
      • 2020-09-09 25347, 2020

      • Nyanko-sensei has quit
      • 2020-09-09 25321, 2020

      • Nyanko-sensei joined the channel
      • 2020-09-09 25313, 2020

      • v6lur has quit
      • 2020-09-09 25308, 2020

      • Nyanko-sensei has quit
      • 2020-09-09 25305, 2020

      • Nyanko-sensei joined the channel
      • 2020-09-09 25327, 2020

      • supersandro2000 has quit
      • 2020-09-09 25336, 2020

      • supersandro2000 joined the channel