#metabrainz

/

      • minimal has quit
      • d4rk-ph0enix has quit
      • d4rkie joined the channel
      • MRiddickW joined the channel
      • MRiddickW has quit
      • lusciouslover has quit
      • lusciouslover joined the channel
      • Kladky joined the channel
      • zas[m] joined the channel
      • zas[m]
        Upgrading openresty on gateways
      • aerozol[m] has quit
      • leftmostcatUTC-7 has quit
      • reosarevok[m] joined the channel
      • reosarevok[m]
        aerozol: what do you think about MBS-13698 ? It seems sensible to me, but it's UI so asking first :)
      • BrainzBot
        MBS-13698: improve placement of arrows for reordering uploaded artwork https://tickets.metabrainz.org/browse/MBS-13698
      • reosarevok[m]
        yvanzo: I see more people are asking about STYLE-2402 - and you had asked for something like this before too. I might consider changing the relationship names to "named after artist", "named after work" etc, does that seem sensible? I guess that'd allow us to expand the relationship to have multiple orders even if it sounds a bit silly
      • BrainzBot
        STYLE-2402: extend 'Named after' relationships to more (if not all) entity pairs https://tickets.metabrainz.org/browse/STYLE-2402
      • rimskii[m] joined the channel
      • rimskii[m]
        lucifer: hi! Im sorry for disappearing for a while, i finished with mentoring and back now !
      • so far I finished with adding Soundcloud import to troi, now waiting for its approval.
      • I also finished with Soundcloud import to LB as well (its ready, just need to open PR), so I was thinking to what I should focus now?
      • zas[m]
        reosarevok: about MBS-13698 I tend to agree with the suggestion, those arrows are quite confusing. Also. If one wants to move the last image first in a list of 15 images, re-ordering is quite tedious (and arrows current placement doesn't help). Having drag'n'drop support for image re-ordering would be great (as a complement).
      • BrainzBot
        MBS-13698: improve placement of arrows for reordering uploaded artwork https://tickets.metabrainz.org/browse/MBS-13698
      • reosarevok[m]
        It would, but that's a lot harder :) So better arrows is probably a good start
      • rimskii[m]
        seems like this error is still not resolved
      • aerozol I've noticed that this error seems to be caused by a long description. Perhaps we should either exclude the description while exporting or simply shorten it. wdyt?
      • BrainzBot
        LB-1599: Weekly Mix export to Spotify fails (description too long)
      • yvanzo[m]
        reosarevok: What do you mean with “to expand the relationship to have multiple orders” and why would that “sounds a bit silly”?
      • reosarevok[m]
        Well, we have two options:
      • a) "named after" l_artist_work, for example, which we have now, is "{artist X} is named after {work Y}" - we could add "named after" l_artist_work for "{work X} is named after {artist Y}", but then it would be very hard to pick the right relationship with the way they currently work
      • So the most workable option is
      • b) "named after work" l_artist_work "{artist X} is named after work {work Y}" and "named after artist" "{work X} is named after artist {artist Y}"
      • It seems a bit silly to say "is named after artist {artist}" because well, we know it's an artist without saying it in the link phrase
      • But it's probably the least bad option?
      • yvanzo[m]
        The way you present a) is that it is not a workable solution anyway. So there doesn’t seem to be two options.
      • reosarevok[m]
        That would require changing existing join phrases and breaking translations, but it might be again the least bad option in the situation?
      • Well, it is an option in that the description would still specify which order it is
      • But it could be very confusing if the user hides descriptions
      • yvanzo[m]
        OK, a) would be workable if the code of the relationship editor was improved to identify one relationship type as the other direction of another relationship type.
      • b) doesn't require any code change, it requires a few more new translations.
      • The code changes for a) could be implemented through hard-coded mapping as first implementation, and later on be carried on through the database schema.
      • Like we have parent at the moment, we could have opposite or something like that.
      • reosarevok[m]
        I'd probably do b) now and then possibly even change to a) in a few years if we have made the rels work better natively (same as with MBS-9517 which is kind of a similar concept), unless you feel breaking the translations is really bad
      • BrainzBot
        MBS-9517: Group relationship types by definition https://tickets.metabrainz.org/browse/MBS-9517
      • yvanzo[m]
        reosarevok: Is it fine to release an MBS update today? Do you want to have a try at the new steps?
      • reosarevok[m]
        I think so, and sure, why not?
      • But let me do this before
      • Then we can put beta out too and have the new strings available for translation asap
      • yvanzo[m]
        Sure
      • reosarevok[m]
        Hmm
      • "named after" -> "named after artist" is easy in that one direction. But "inspired the name of" is more annoying - "{artist X] artist inspired the name of" seems super dumb
      • Maybe that one can be left as-is, given "named after artist" should be clear enough on the other direction in order to select the rel?
      • yvanzo: opinion?
      • So rel name "named after artist", "{work} named after artist {artist}", "{artist} inspired the name of {work}", "{work} is named after artist {artist}"
      • yvanzo[m]
        reosarevok: `{artist} inspired the name of work {work}`?
      • reosarevok[m]
        Hmm. I guess?
      • Will do that then
      • mayhem[m] joined the channel
      • mayhem[m]
        ansh: monkey : there is a support@ email asking about a login problem on LB. Could you please take a look?
      • ansh[m] joined the channel
      • ansh[m]
        Sure! Can you please forward that email
      • mayhem[m]
        done. if you're interested in being on the support@ list, let me know.
      • ansh[m]
        Sure! Add me on that list
      • mayhem[m]
        done. the etiquette for the list is that if you respond to an email, do a reply to all, so that we see the response and can toss that thread.
      • reosarevok[m]
        yvanzo: Ok, done with that. I also updated https://github.com/metabrainz/musicbrainz-serve... to fix conflicts - would you rather it also accepted URLs without `/charts` and added it to them since I understand they fixed the lack of redirects?
      • I'm going out with the dog + lunch, but after that I can release (before the meeting) :)
      • (dunno if you feel that PR would be ok to release today too, or if you would rather have it on beta for a bit that's also fine)
      • yvanzo[m]
        reosarevok: Replied to the PR.
      • minimal joined the channel
      • lucifer[m] joined the channel
      • lucifer[m]
        <rimskii[m]> "lucifer: hi! Im sorry for..." <- > <@kubrimskii:matrix.org> lucifer: hi! Im sorry for disappearing for a while, i finished with mentoring and back now !... (full message at <https://matrix.chatbrainz.org/_matrix/media/v3/...>)
      • SigHunter has quit
      • SigHunter joined the channel
      • reosarevok[m]
        yvanzo: now back :)
      • Amended the commit message for offizielle
      • Time to release prod then I guess? : )
      • LupinIII
        afaik, there is a setting in matrix to prevent long messages going into auto "full message at (url)" which is super annoying for regular text. while still enabling the ability to do a paste with things like code-snipplets..
      • yvanzo[m]
        reosarevok: You probably didn’t push it.
      • reosarevok[m]
      • I did not update the PR text tho
      • I can do that and the ticket, let me see
      • yvanzo[m]
        OK, weird that it wasn’t in my notifications.
      • LupinIII
        btw about reordering with drag and drop. that's a good idea. but it can get kinda finicky with a list like this. , arrows fixing is easy, but as a long term goal, being able to put in number of order in a text boox would be neat?. like each get's assigned a number automatically in the list, and thne that shows in a text box which can be edited which will automatically reorder the image in thelist...
      • so if I write in the text box for image 10, "08" it gets the 08 position, and 08, 09 and 11 automatically gets pushed down with the new numbers 09, 10 and 11
      • 11 not getting a change because it's logic
      • for things with images in the 20, 40 or more category.. doing it like this if you have an image atthe bottom that should be at the top.. that would be good
      • reosarevok[m]
        yvanzo: but I release without this and then put it in beta then? :)
      • yvanzo[m]
        Yes, release at first, without it.
      • And yes, beta can be updated afterwards.
      • reosarevok[m]
        Ok, perfect, on it!
      • monkey[m] joined the channel
      • monkey[m]
        ansh, mayhem I looked into the support email about signing into LB. Missing email verification for the MB account results in the user simply being redirected to the LB homepage when they try to log in, with no error message
      • Will create ticket now to improve that
      • reosarevok[m]
        That does seem bad, yup
      • BrainzGit
        [listenbrainz-server] 14MonkeyDo merged pull request #2943 (03master…LB-1606): LB-1606, LB-1601: Standalone recordings cannot be manually added as listen https://github.com/metabrainz/listenbrainz-serv...
      • reosarevok[m]
        monkey: lucifer had already answered the same you did, effectively, a while ago - I guess you didn't see it because you're not on the support mail or?
      • <BANG>
      • Welcome to another MetaBrainz Monday Meeting!
      • Hi from Asturias from your host, who's falling asleep because of an excess of cachopo but will battle it for you!
      • We have two mailed in reviews:
      • LupinIII
        <3
      • reosarevok[m]
        ericd says:
      • 'Last week I implemented feed for individual playlist. This week is about recommendation playlist and testings.'
      • aerozol says:
      • '''
      • Just staying on top of the usual duties this week.
      • As part of that I try to keep the userscripts wiki page updated, and recently made updates to a couple of related pages. This is the full trio: https://wiki.musicbrainz.org/Guides/Userscripts / https://wiki.musicbrainz.org/External_Resources / https://wiki.musicbrainz.org/Picard_Resources
      • I refer to them v. often, so I assume others do too, and I took a bit of time to table-fy them all recently.
      • Apart from that… I sent this update on Friday because I am on hooooooliday! I didn’t even bring the laptop :-0 So hopefully I am on a beach right now. See you all next week!
      • '''
      • mayhem[m]
        reosarevok[m]: #jealous
      • reosarevok[m]
        You get summer holidays in there, aerozol? It's not even the summer! :p
      • On my list for today: jasje, MonkeyPython, akshaaatt, kellnerd, twodoorcoupe, yvanzo, rimskii, ansh, reosarevok, bitmap, pranav, theflash_, mayhem, JadedBlueEyes, zas, yellowhatpro, monkey, lucifer, atj, outsidecontext
      • LupinIII
        ara?
      • reosarevok[m]
        Not sure if that was asking to go first, but go LupinIII (IRC) :D
      • LupinIII
        Arararara~! hi!
      • lots of offline bits and pieces, getting some booster shots/vaccines tomorrow!
      • looking into some listenbrainz user things, testing and I made a couple of tickets... also I'm working on setting up work environment, so soon I will be doing tickets again!
      • it looks like I'm not doing much, but I was out all day and also saturday and friday, tomorrow is another day of STUFF :D
      • anyway, that's it for me,next can be.. reosarevok[m] ?
      • reosarevok[m]
        Hi!
      • Last week I looked into MBS-6741 which turned out to be a lot trickier than it sounds
      • BrainzBot
        MBS-6741: Include subdivision and country in the webservice when returning an area https://tickets.metabrainz.org/browse/MBS-6741
      • reosarevok[m]
        I also finally merged a big eslint PR which should make our future code nicer!
      • Plus a bunch of small things, such as MBS-13682
      • BrainzBot
        MBS-13682: Add language filter to Works tab https://tickets.metabrainz.org/browse/MBS-13682
      • LupinIII
        oooh
      • reosarevok[m]
        I also updated the timeline with more events, including the hiring of a bunch of you
      • And blocked using more invisible characters as edit notes, because people sometimes suck. Invisibly.
      • Plus the usual support and community bits
      • That's it for me, what about ansh ?
      • On my list for today: jasje, akshaaatt, kellnerd, twodoorcoupe, yvanzo, rimskii, bitmap, pranav, theflash_, mayhem, JadedBlueEyes, zas, yellowhatpro, monkey, lucifer, atj, outsidecontext
      • ansh[m]
        Hi Everyone!
      • Last week, I mostly spent time on the MusicPlayer UI for BrainzPlayer.
      • Other than that, I worked on some tickets and upgraded Sentry on LB
      • And reviewed GSoC PRs. And also spent time on researching about LB-1596.
      • BrainzBot
        LB-1596: Listen Along/Sync Play https://tickets.metabrainz.org/browse/LB-1596
      • ansh[m]
        That’s it for me.
      • rimskii: next?
      • rimskii[m]
        yep, thanks!
      • hi everyone!
      • well for the last week couldnt do much, unfortunately
      • finally finished my mentoring, also got into some huawei competition and even flew back to home !
      • starting from this day I am working on changes on some of my PRs
      • finally started working on exporting functions for apple and soundcloud, for now doing it at troi. Hopefully, I can finished them for both since they are pretty similar
      • also working on ticket LB-1599, waiting aerozol from his holiday to discuss haha
      • BrainzBot
        LB-1599: Weekly Mix export to Spotify fails (description too long) https://tickets.metabrainz.org/browse/LB-1599
      • theflash__ joined the channel
      • rimskii[m]
        thats it for me