lucifer: seems better, thanks. I think we will need to tweak this over time -- still doesn't feel amazing. but I have to remind myself that the list of tracks is not a finished product, but input to the next step.
2021-10-07 28059, 2021
lucifer
agreed on both points.
2021-10-07 28002, 2021
ruaok
lucifer: #1645 is ready for review.
2021-10-07 28043, 2021
ruaok
I think I am going to so some mapping cleanup and then finish the work to emit MBIDs with listens.
2021-10-07 28011, 2021
alastairp
ruaok: once you have some time I have 2 or 3 things that I want to run past you
2021-10-07 28031, 2021
ruaok
I think after/during lunch might be good for that.
btw the summit is next weekend, we should put up some agenda/session info up.
2021-10-07 28049, 2021
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, MeB Summit Planning (lucifer)
personally I'd prefer a vm because it'd just be a matter of cloning the sentry repo and running docker-compose up. any other option requires faffing around
2021-10-07 28039, 2021
ruaok
exactly that. stop faffing, code more.
2021-10-07 28058, 2021
alastairp
is it worth going up to a CX41 for memory overhead?
2021-10-07 28009, 2021
alastairp
hence why I stopped and sent the email
2021-10-07 28024, 2021
ruaok
if you think it is necessary, then lets do it.
2021-10-07 28044, 2021
ruaok
pick the specs you want, not the specs you think we'll approve. :)
2021-10-07 28013, 2021
ruaok
I told zas a long time ago that if we needed resources, we should use them. but lets make sure we don't waste any -- that's dumb.
2021-10-07 28028, 2021
alastairp
the only guidelines I have to go on is what sentry says is required, and I don't know if they include overhead, or are bare minimum requirements
2021-10-07 28052, 2021
alastairp
let's go higher. as I said, there are 30 containers in a sentry setup!
2021-10-07 28059, 2021
ruaok
then go one step above the minimum and then we'll evaluate. we can always upgrade.
2021-10-07 28013, 2021
alastairp
OK, let's provision a CX41 in Finland
2021-10-07 28038, 2021
ruaok will let zas do the honors and initial setup.
lucifer: regarding the recommendations we provide, can we make sure that we don't include the already listened songs by the user?
2021-10-07 28045, 2021
lucifer
akshat: we just removed that step earlier today because it doesn't work as expected. that is the raw set of tracks though. troi could always do that while creating the playlist.
monkey: did you manage to have a look at my PR yet? I want to confirm I have done the submit listens block correctly
2021-10-07 28018, 2021
tandy[m]
i didnt include all of the error catches that are in the APIService, should I include these?
2021-10-07 28016, 2021
ruaok is being buried in BS faster than he can vanquish it
2021-10-07 28037, 2021
tandy[m]
<tandy[m]> "i didnt include all of the error..." <- i'm half tempted to just copy in the whole proc from the APIService ..
2021-10-07 28039, 2021
akshat
ruaok: I see that the MusicBrainz twitter handled isn't much active. Can we do something to automate a tweet as soon as a blog is posted on blog.meb, a post at community.meb becomes very popular or a release is made?
2021-10-07 28056, 2021
akshat
Else someone can take the responsibility to manage the account
2021-10-07 28000, 2021
ruaok
monkey: when I use brainzplayer, its only a matter of time until that page ends up spinning hard and taking up 100% of one core. are you aware of this?
2021-10-07 28034, 2021
alastairp
akshat: we've been trying harder to tweet and blog more, but sometimes we stop doing it because we have other things to do
2021-10-07 28053, 2021
alastairp
of course, to tweet a blog post we need to blog more too :)
2021-10-07 28057, 2021
akshat
My point being, I want to embed the Twitter id on the mb web revamp. That could be our source of updates for everyone somewhat like how lb does
2021-10-07 28020, 2021
ruaok
akshat: yes, please tweet more! can you access the syswiki?
2021-10-07 28037, 2021
akshat
alastairp: I do feel we have enough content! :)
2021-10-07 28040, 2021
ruaok
syswiki is where we have internal documentation.
2021-10-07 28046, 2021
akshat
Okayy ruaok
2021-10-07 28003, 2021
alastairp
akshat: specifically for that, we'll need to be careful - the twitter embed on LB and AB sets a twitter cookie, which is a big no no for us, and we need to do something about it on those sites
2021-10-07 28038, 2021
ruaok
yeah, twitter embeds are a thing of the past for us. Listing our handle, sure. but embedding recent tweets, lets no do that.
2021-10-07 28040, 2021
alastairp
so we'd need a way to show a twitter feed without setting 3rd party cookies. If you can find a way to do that, it'd be really helpful!
2021-10-07 28047, 2021
ruaok
(we need to remove it from the LB home page)
2021-10-07 28034, 2021
tandy[m]
alastairp: RSS embed could work without the nastiness?
tandy[m]: yes, though that'd probably require us to apply some kind of formatting to make it look nice. If you have any knowledge on how to do that please tell us!
alastairp: we could probably make a react thing to do that so we could use it across metabrainz
2021-10-07 28046, 2021
alastairp
akshat: tandy[m]: That sounds like a good option, as long as we can confirm that this process sets no twitter.com cookies (I just checked the demo above and I have some cookes set, not sure from which demo)
2021-10-07 28047, 2021
alastairp
ah, it's possible those were just my regular "you're logged into twitter" cookies, because now after deleting them I'm logged out
2021-10-07 28023, 2021
tandy[m]
if this is a long term idea ill make a Meta ticket
2021-10-07 28030, 2021
akshat
I think this package is a good option
2021-10-07 28044, 2021
tandy[m]
cos its best to keep dependencies off services like twitter when we already have an rss feed
2021-10-07 28031, 2021
alastairp
tandy[m]: yes, please create a ticket
2021-10-07 28034, 2021
tandy[m]
done
2021-10-07 28053, 2021
alastairp
the react-twitter-widgets app still makes many requests to twitter.com from the user's web browser, so we can't use it. maybe there are other options
