Need an emoji for sir-lancebot#661 
#dev-contrib
sleek steppe
dusky shoreBOT
vale ibex
Need an emoji for sir-lancebot#661 <:lemon_grimace:754601078561046558>
Do you have an image?
sleek steppe
Sure but it might not be the best quality
vocal prairie
Emojis are small, so it should be fine.
brazen charm
desaturating the current emoji will have the best result
vale ibex
got this, just getting it put onto the emoji server now
vale ibex
Sure but it might not be the best quality
brazen charm
Does that color fit the embed background?
vale ibex
!embed
stable mountainBOT
sleek steppe
<:PRDraft:829755345425399848>`<:PRDraft:829755345425399848>`
thanks
vale ibex
It's contrasting enough imo
placid ermine
how's this looking
thorny obsidian
oooh, spiffy. What does something like $e^3 - x_2$ look like?
placid ermine
gritty wind
neat
thorny obsidian
Niiiiice
tawdry vapor
That sounds like a heavy dependency for just latex rendering
placid ermine
well it was matplotlib or an api
or well sympy which is even heavier
or a local latex installation which is even even heavier
heh
tawdry vapor
Is a local install really that heavy if it's just pure latex without any extra packages
placid ermine
heavier than matplotlib, surely?
we would also have to worry about untrusted input, etc
tawdry vapor
Matplotlib requires a latex installation anyway
https://github.com/JelteF/PyLaTeX is an alternative python interface
Oh I see it has its own parser but also support for latex, the latter requires it installed
placid ermine
Matplotlib requires a latex installation anyway
not necessarily...
sleek steppe
lol
fallen patrol
The real reason I took lunch here
sleek steppe
ah you threw me under the bus
fallen patrol
no one said you had to make sir-lancebot#665
dusky shoreBOT
fallen patrol
wait
.issue
wat that's a command
uhm
that should be whitelisted in this channel, no?
placid ermine
where are the error handlers
oh nvm
should i make a custom exception for latex compilation errors?
sleek steppe
normally a try and except should be fine
vocal prairie
Why? It's just a link to an issue?
fallen patrol
sir-lancsdrbot#1 for example
dusky shoreBOT
fallen patrol
that regex is bad
vocal prairie
Yeah, that could be better. But why is linking bad?
fallen patrol
linking in itself is not bad.
brazen charm
I don't think there are many cases where you'd accidentally invoke it, maybe could be whitespace terminate
fallen patrol
but making it work in every channel with this regex is a bad idea imo
vocal prairie
Oh, got it.
brazen charm
does the in message handling include other orgs?
fallen patrol
seems like most of the current issues have been patched
fallen patrol
lol sad
vale ibex
We could easily put a \b either side of the regex so it only looks for full words
I think @green oriole is currently fixing/refactoring that cog, what are your thoughts Ak?
fallen patrol
I *think* <@!407110650217627658> is currently fixing/refactoring that cog, what ...
he is that's the issue above
green oriole
Yes I am
I can do that
!remind 13h Add word boundaries around the regex if you haven't already
stable mountainBOT
vale ibex
👍
green oriole
I don't mind PRs to my branch either if you want to take credit for it
fallen patrol
wut
vale ibex
I'll push something now
fallen patrol
I don't *mind* PRs to my branch either if you want to take credit for it
confusidied grammar
green oriole
lmao
vale ibex
probably just mobile things 😛
green oriole
Nah, just my brain being smooth, there is a missing word
vale ibex
lol
brazen charm
What are the gh api limits?
vale ibex
I'll open an issue on your branch and chris can add the pr
no need for an issue, I'll just PR onto Ak's branch
thorny obsidian
Probably for the esoteric coding challenges we had for a bit
brazen charm
and eval
green oriole
no need for an issue, I'll just PR onto Ak's branch
Can directly push it tbh
Eval is allowed in eso, yeah
brazen charm
The channel's category was special and it doesn't look like the config was updated so it's "special"
vale ibex
Can directly push it tbh
good point, but too late I've already got the branch up now lol
green oriole
What are the gh api limits?
We use authenticated requests everywhere, so I think around 50k/h? I don't fully remember
vale ibex
What are the gh api limits?
not sure of the exact numbers, but I know its quite forgiving if you supply an api key
fallen patrol
We use authenticated requests everywhere, so I think around 50k/h? I don't fully...
where's the code for that?
.src issue
green oriole
For .issue?
fallen patrol
.source should really be allowed in this channel.….
brazen charm
does the in message handling include other orgs?
Was thinking of this, where'd it'd either handle <pydis_repo>#<issue_no> or <org>/<repo>#<issue_no>
green oriole
It doesn't
fallen patrol
that would be nice pls add
green oriole
That was part of the original requirements
The staff decided to not include them
Dunno why, I wasn't there haha
brazen charm
Curious about the reason if someone has it, not really a format that's prone to being used accidentally
vale ibex
Oh, branch protection extends to non-main branches 😦
That's weird, since it doesn't on bot afaik
fallen patrol
The staff decided to not include them
reasoning?
green oriole
Dunno why, I wasn't there haha
^
green oriole
Oh, branch protection extends to non-main branches 😦
It doesn't though 
thorny obsidian
Dunno why, I wasn't there haha
You can read the meeting minutes from that staff meeting
vale ibex
It doesn't though <:lemon_glass:754441879935778867>
ahh, review bot is still there, but I can merge anyway
green oriole
Well there is not the full reasoning
placid ermine
PRed latex, will respond to any comments tomorrow morning, need sleep 😅
green oriole
I always read minutes :P
patent pivot
which meeting was it
green oriole
Long ago
thorny obsidian
Jan 17 I think, hard to look on my phone
sleek steppe
Would the regex work for just repo#num ?
patent pivot
17th january yeah
thorny obsidian
People just felt odd about it being automatic for all repos, which I still feel like it's weird. It makes sense in our dev categories for our projects
green oriole
I am guessing it is to avoid silly backtracking if you post chunks of #
Although that can be easily solved
I am not sure why the regex is generated tbh
patent pivot
kutiekat i swear to god the american spelling of organization in your minutes will be the death of me
thorny obsidian
:3c
green oriole
Hahaha
vale ibex
fixed
thorny obsidian
I change it everytime I make the meeting minutes which is always
vale ibex
Which one is one again
patent pivot
no you change it AFTER the fact as well
thorny obsidian
LMAO I do if I go back to review discussion
brazen charm
People just felt odd about it being automatic for all repos, which I still feel ...
I think if someone did use it, it'd have to be intentional with the format for different orgs/users, so not exactly surprising
vale ibex
let me see if I can find the text chat for this meeting
around here #staff-voice message
thorny obsidian
People were hung up on the automatic part when opened to all repos
patent pivot
yeah
patent pivot
seems that the primary thing was abuse combined with ratelimits
vale ibex
Maybe we just bring it up again on Sunday?
patent pivot
Yeah, sounds like a plan
vale ibex
I'm not heavily against the idea, but we should lookup rate limits
thorny obsidian
Oh that was the meeting for the help channel names
green oriole
Sounds good
patent pivot
5k/hr
green oriole
Don't mind me
patent pivot
but we use them elsewhere for other projects as well
patent pivot
don't think the feature is worth it for staff only
I think providing we lock it to a tight regex it should be fairly difficult to accidentally invoke
brazen charm
I really doubt that amount of requests would be reached even if the whole server was aware of the functionality (w/o malicious uses of it)
green oriole
Something like [a-zA-Z0-9_-]+\/[a-zA-Z0-9_-]+#\d+ should be fine-ish
vocal prairie
I think a simple regex that looks for sirl---bot#--for example works for accidental lancebots.
green oriole
Probably a shorthand for this, but heh
vocal prairie
-- being anything
brazen charm
the boundaries and lancebot being matched first should take care of it
fallen patrol
around here https://canary.discord.com/channels/267624335836053506/5416387620071...
duck
patent pivot
lol
tawdry vapor
Yes, [\w-] would be shorthand
green oriole
Oh nice
tawdry vapor
Also a bonus that it works with Unicode rather than ASCII only
Actually it doesn't seem like GitHub would allow Unicode in the URLs anyway
fallen patrol
We use authenticated requests everywhere, so I think around 50k/h? I don't fully...
i just realized, does this mean that sir-lancebot is authenticated into the python discord private repos
green oriole
Nope
tawdry vapor
In that case \w could be set to ASCII with a regex flag
green oriole
The token doesn't have any scope AFAIK
patent pivot
so
lancebot can read private repos
but the code used to fetch all pydis repos is unprivileged
so it doesn't actually see repos like organisation, admin-tasks, etc.
and therefore they don't go in the regex
green oriole
Oh really
fallen patrol
makes sense
green oriole
My new code make it be authenticated
@patent pivot can we drop the perms for the token?
patent pivot
I don't think we can, no
hmmmmmm
which token do we use on lancebot
hold on let me delve into k8s
vocal prairie
Is k8s short for kubernetes?
fallen patrol
which token do we use on lancebot
just make a request to https://api.github.com/user
oh wait no int e yet
patent pivot
Is k8s short for kubernetes?
yep
ah
looks like that token can't read private repos
x-oauth-scopes: gist, public_repo, read:discussion, read:enterprise, read:org, read:packages
green oriole
Cool
patent pivot
we should get someone on that eval implementation
fallen patrol
fwiw
@green oriole if you're into it, i'd add implementation to check if its public or not
patent pivot
do we need that
thorny obsidian
we should get someone on that eval implementation
Is it just ripping it from python/black knight?
green oriole
we should get someone on that eval implementation
Well if @hardy gorge has a minute that'd be perfect
fallen patrol
that way its proofed against security
patent pivot
the current implementation works since we just fetch it unprivileged
we can check when we add the org/repo syntax that fetched private repos can only be used in staff channels
but there are use cases for this feature for using it to fetch organisation issues
fallen patrol
why did that bot never become public 
patent pivot
security
fallen patrol
security by obscurity
thorny obsidian
Cool I'll see if I can rip it over since I wanted to dive into eval internald anyway
patent pivot
if we make our anti-abuse features public, people anti the anti-abuse
thorny obsidian
That's... Not security by obscurity
fallen patrol
but aren't all of those features on @stable mountain with anti abuse anyhow
vocal prairie
It's security by security not obscurity.
patent pivot
but aren't all of those features on <@!409107086526644234> with anti abuse anyho...
no, they're not
fallen patrol
unless there's some cogs in @stable mountain that are .gitignored
fallen patrol
no, they're not
then where are they?
patent pivot
certain components that we had in mind for BK such as improved defcon were ported over to defcon on @stable mountain
can't remember off the top of my head, but not sure we need to detail too much on that
thorny obsidian
Discord has gotten better with raid stuff so we didn't feel the need for a whole separate bot. We're making sure the anti pattern knowledge stays with us so if we need to protect against the raid we have it
vocal prairie
And worst case scenario, have an admin turn off send messages for the everyone role. Simple.
fallen patrol
And worst case scenario, have an admin turn off send messages for the everyone r...
i would be surprised if there wasn't a !hush entire server
thorny obsidian
I think the server silence feature got ported over? That or it's on zig's to do
fallen patrol
i also wouldn't be surprised if defcon was actually enabled today
patent pivot
thorny obsidian
Nice, the defcon upgrades are so nice
fallen patrol
….but that description is technically wrong :/
patent pivot
huh?
vocal prairie
What's steakbot? Is it a new project?
stable mountainBOT
thorny obsidian
Steakbot is joe's glorious slashbot child
patent pivot
What's steakbot? Is it a new project?
me playing around with slash commands a few months back
vocal prairie
Testing slash commands? Nice
patent pivot
when the support is there we can migrate core projects to use them
but until then it'll remain playful things like steakbot lol
patent pivot
huh?
fallen patrol
if the server was being raided wouldn't you want to shutdown voice as well?
patent pivot
in previous raids
the approach there has just been uhh
delete the voice channels
lol
fallen patrol
lock_vc()
for u in vc:
u.disconnect()
Literally just turn off view channels.
fallen patrol
no, voice channels are special
vocal prairie
And set Rules to override.
patent pivot
feel free to PR
patent pivot
done
patent pivot
why
fallen patrol
i can still talk
patent pivot
yeah, we handle that in voice ban
vocal prairie
But how do they get in in the first place?
patent pivot
just #voice-verification lol
vocal prairie
That's not an issue.
patent pivot
But how do they get in in the first place?
it's if role is removed during a voice session
which we account for in voice ban by removing from voice session
fallen patrol
just <#764802555427029012> lol
i opted out of zoot
thorny obsidian
Raid accounts generally won't be voice verified, so /shrug
patent pivot
i'd opt in to zoot then in that case
fallen patrol
vocal prairie
it's if role is removed during a voice session
Oops.
vocal prairie
I'm so confused, since Dave only has a couple files, yet it generates so much data. It's cool.
patent pivot
I guess I did remove it, so I'll grant it back lol
fallen patrol
if not, expect a dm in about 30 minutes
vocal prairie
i opted out of zoot
*Dave
patent pivot
I should add some notes to the privacy policy saying there are downsides of opting out
fallen patrol
i mean
patent pivot
we've considered message gating functionality before
fallen patrol
if you do opt out
after getting voice verifed
then you got the role
and should be fine
patent pivot
yeah, but if a) you then request clearing or b) we change thresholds then you may be ineligible
but yeah, opting out doesn't reset the countner
cold island
if the server was being raided wouldn't you want to shutdown voice as well?
uhhhh hmmmmm yeeeeah. That sounds like a thing
patent pivot
so if you have 50 messages and opt out then you are still eligible
since we still have that data
fallen patrol
then c) opt back in and requalify and then opt out again and dm joe to clear it
fwiw
joe is the one who will be more inconvenienced than me
vocal prairie
Because you'll spam his dms? /j
patent pivot
i have 30 days to handle a deletion request lol
fallen patrol
Because you'll spam his dms? /j
not entirely /j, he has a legal obligation to clear it.
but 30 days to do so
patent pivot
opting out is less legal compliance and more a generosity move
we don't store any information that requires opt out functionality to be present
patent pivot
user data is still in metricity
fallen patrol
if you didn't have the opt-out*
fallen patrol
uhhhh hmmmmm yeeeeah. That sounds like a thing
it isn't actually handled in the code. all voice channels can still be joined
fallen patrol
a portion of the solution would just be to remove connect perms from everyone as well
like
one line code
I can make a pr
patent pivot
if you didn't have the opt-out*
yeah, but at that point we may consider metricity data essential to the operation of the community
vocal prairie
How does metricity get so much data? I cannot find a place in the repo where it gets so many different types?
patent pivot
How does metricity get so much data? I cannot find a place in the repo where it ...
which data?
fallen patrol
yeah, but at that point we may consider metricity data essential to the operatio...
then me and the other 12 who have opted out will leave the server.
cold island
isn't that just an on_message event?
vocal prairie
which data?
Isn't that what gets the data for https://stats.pythondiscord.com?
patent pivot
the data there comes from bot
hmm
is it worth leaking something that provides info on this
vocal prairie
Ohhhh. So then what kind of data does metricity get?
patent pivot
Ohhhh. So then what kind of data does metricity get?
rich details on message counts per channel
vocal prairie
is it worth leaking something that provides info on this
Yes. PyDis operations are amazing.
I see.
fallen patrol
medium smh
patent pivot
it's not medium lol
thorny obsidian
Wait can you see my post through that?
patent pivot
lol i unpublished your thing kutie don't worry
thorny obsidian
Oh thank god
patent pivot
hahahahahah
fallen patrol
@cold island can you clarify which role has the connect permissions
i assume its the default_role
aka the (dangerous) everyone role
patent pivot
everyone has connect, voice verified has speak
fallen patrol
interesting
entirely irrelevant to any pr, why does the Muted role then block from connecting to all vcs?
seems like it should just block from speaking?
patent pivot
you can still be a nuisance if you have connect privileges
whole forge
with it blocking connect, people cant spam connect and leave
fallen patrol
ah
patent pivot
voice banned users generally don't fuck around after being voice banned
and if they do, it's nice to completely bar entry
patent pivot
No, voice ban just prevents you from verifying
whole forge
this has become #community-meta
patent pivot
lol
fallen patrol
its 0.001% relevant to a pr
patent pivot
it's semi related to implementing this extension to defcon so we're good
fallen patrol
/bot yeah
patent pivot
yeah, don't think so
fallen patrol
what about /sirbot
patent pivot
there is on lancebot
fallen patrol
how can i view that 
patent pivot
uhh
best to look at the raw version, it has comments https://raw.githubusercontent.com/python-discord/sir-lancebot/main/.github/pull_request_template.md
fallen patrol
it.… redirects.
so i saw the raw version and only the raw version
wait
huh
why
how
it somehow redirected but normally doesn't
ok
pr created
removes the connect permission when the server goes sour
vale ibex
?
fallen patrol
nvm
hardy gorge
@green oriole What do you need?
sleek steppe
For bot#1511 , I think we should set speak=False
dusky shoreBOT
fallen patrol
<@!407110650217627658> What do you need?
Int e command from black knight was the conversation context
fallen patrol
For bot#1511 , I think we should set `speak=False`
It's already false for the everyone role
It's granted with the voice verify role
And changing the permission while in a vc affects nothing because of when discord checks permissions
sleek steppe
It's already false for the everyone role
So?
fallen patrol
Read the next part
Changing permissions while in a vc affects nothing because of when discord checks permissions
Permissions are reflected when joining the vc and never again.
Until you leave and rejoin the vc
So if one was to turn that off, it wouldn't affect anything whatsoever
Only if for some reason there is a channel which has the everyone role with connect overrides to True
fallen patrol
@sleek steppe ^ that make sense?
vocal prairie
Why not hide channels?
fallen patrol
that would not be a simple single role change
that would affect text channels as well
vocal prairie
that would affect text channels as well
No, I mean why not hide the vc channels?
fallen patrol
because it would need to be an edit on every single vc channel
vocal prairie
True.
fallen patrol
or it would affect text channel as well if turning on view channels
this fix disables voice for everyone as well if you aren't already in a channel
patent pivot
that was me, yeah
patent pivot
I restarted it to reset some mitigations
fallen patrol
what mitigations
patent pivot
anti-abuse things
fallen patrol
like this thingy?
patent pivot
nope, that was just a token in our mod alerts
fallen patrol
so anti-abuse is closed source?
patent pivot
it's not closed source, it's just that we program certain snippets as and when
like there is no source code
fallen patrol
o
fallen patrol
we can add things in production through eval
like add a whole cog lol
patent pivot
we have a policy for not adding commands in production, so not really
fallen patrol
ah
fervent sage
smh you and your intelligent policies that stop stuff breaking
patent pivot
lul
fervent sage
its more fun when you get called at 4am because everything broke /s
fallen patrol
smh you and your intelligent policies that stop stuff breaking
ikr my bot i actually changed the bot to a duck instance
just for the lulz yesterday
patent pivot
one such mitigation from today did have adverse side effects lol
fervent sage
lol
fallen patrol
oh just 2004 events match this spam regex
nothing important
we just banned spaces its fine
not_much_happened_dont_worry_everyone
fallen patrol
ikr my bot i actually changed the bot to a duck instance
fervent sage
my craving for ci/cd hasnt gone away im scared lol
patent pivot
lol
I am working on this today https://grafana.com/blog/2019/08/21/how-grafana-labs-effectively-pairs-loki-and-kubernetes-events/
clever wraith
CI/CD makes me go 🥴
fallen patrol
@patent pivot I got conflicting info on this
^ no token used
should i make an issue before working on a pr for that?
patent pivot
hmm
fallen patrol
fwiw im not so inclined to make the pr for this one
patent pivot
yeah if you don't want to make the PR then issue 100%
fallen patrol
but the ratelimits go boom
fallen patrol
i mean
just copy the code from the other cog lol
or
perhaps make it request from github in a util function
¯_(ツ)_/¯
okay definte issue either way since there's multiple ways to approach this
and i might try and make the fix myself but that would require me setting up the bot on my servers
…
fallen patrol
so I'd open an issue
is this technically a bug
fallen patrol
also uh
vocal prairie
I mean, it doesn't matter that much. The template.
fallen patrol
that's why i was a bit curious on the checking if its a public repo thing because if we use the token on this cog and then sometime give it public support this is a major security hole for being able to see code of closed source repos in the python-discord organisation on github @patent pivot
patent pivot
for the .github command we can definitely sanity check the repos are public
fallen patrol
I will type organisation so sue me kutiekat
patent pivot
when we rework the org/repo#123 syntax then we'll handle public and private safely
fallen patrol
im thinking like
the request function for both of these could possibly be moved into a util file
and do all the checks in one place
fervent sage
when we rework the org/repo#123 syntax then we'll handle public and private safe...
wait we can do any org not just pydis? or am i reading that totally wrong lol
fallen patrol
yes
fallen patrol
sir-lancebot#642 sir-lancebot#630 there's an issue and pr on that exact thing
dusky shoreBOT
fallen patrol
but like
vocal prairie
No it isn't
fallen patrol
there's three seperate things here
vocal prairie
That's completely different, isn't it?
patent pivot
wait we can do any org not just pydis? or am i reading that totally wrong lol
we will be able to eventually
fervent sage
ah nice
patent pivot
well, we'll probably discuss it on sunday
fallen patrol
No it isn't
its in the pr
fervent sage
i'll try be awake 
vocal prairie
its in the pr
Oh, I see.
short snow
`npm/npm#1`
no, it is just specific to python discord
it uses tasks, to get the list of repos in every 30 minutes
patent pivot
right now it is
short snow
and then makes a regex out of that list
vocal prairie
That would need a very tight regEx, or it'd get ratelimited, right?
fervent sage
i think you overestimate how often people actually type that and underestimate the size of github's ratelimits lol
clever wraith
@patent pivot it'd be kinda cool if everyone could view the meeting minutes but that's just me
thorny obsidian
There are things we discuss and put up there that we don't want shared with the community early.
clever wraith
Ahh
vocal prairie
i think you overestimate how often people actually type that and underestimate t...
True, I guess people don't type #s much.
thorny obsidian
A lot of our internal updates happen there including the events we plan
fervent sage
even if people type a # it still wouldnt be in the right format most of the time
vocal prairie
Makes sense.
clever wraith
A lot of our internal updates happen there including the events we plan
Fair
fervent sage
(\w+/)?\w+#\d+ with a bit of refinement should be fine
fallen patrol
A lot of our internal updates happen there including the events we plan
and the debates over organisation vs organization.
clever wraith
and the debates over organisation vs organization.
Yeah they wanna keep that under wraps
thorny obsidian
Lolol that's not even a debate. That's me waging a one woman war against the brits
fallen patrol
Lolol that's not even a debate. That's me waging a one woman war against the bri...
im american and
fervent sage
im not happy that my precious quackstack has been invaded by 'color'
thorny obsidian
Organisation on notion yells at me, so I change it to organization :3c
fallen patrol
@patent pivot put a constraint on the meeting notes that doesn't allow organization
patent pivot
lol
fallen patrol
just like the db doesn't allow you to be infracted
patent pivot
brb getting job
vocal prairie
`(\w+/)?\w+#\d+` with a bit of refinement should be fine
But ignoring ratelimits, what if multple things are named the same. eg) ondiscord/bot#1 and pythondiscord/bot#1. That might be an hard if we want it to be really accurate.
dusky shoreBOT
vocal prairie
That wasn't supposed to happen.
vocal prairie
We should only count it if it has a space before.
patent pivot
we should go bot#1
dusky shoreBOT
fervent sage
fine if we're gonne be pedantic about including the whole thing, \b(\w+/)?\w+#\d+\b
patent pivot
notlikebot#1hahahaha
dusky shoreBOT
dusky shoreBOT
clever wraith
Hahabot#22
dusky shoreBOT
clever wraith
sir-lanceldjejsjejribot#69
dusky shoreBOT
fallen patrol
sir-lancebot#670 can someone change it to enhancement and not feature
dusky shoreBOT
fallen patrol
it gives me literally no option when creating so please don't tell me i could.
fervent sage
done
fallen patrol
danke
vocal prairie
Why'd you check both?
fallen patrol
Why'd you check both?
because i'd perfer to implement it but it's going to be like two weeks before i bother to set up a sir lancebot clone
so if anyone wants to do it before then they can go for it and i won't mind
lol i just realized i created my first prs in the pydis org today
vocal prairie
because i'd perfer to implement it but it's going to be like two weeks before i ...
Just join the test server Jason made.
He has premade configs
vocal prairie
What's the problem with that?
fallen patrol
i don't feel like cloning the repo rn
not until im a little more competent with git
fallen patrol
¯_(ツ)_/¯
short snow
This is my config.yml's urls section, which worked when you did:
$ docker-compose up -d
$ docker-compose up bot
But it gives couldn't connect to api when you do:
$ docker-compose up -d
$ pipenv run start
And, all the images are up
I can also go to 127.0.0.1:8000 and open the site
fallen patrol
This is my `config.yml`'s `urls` section, which worked when you did:
```bash
$ d...
random unrelated q, are you running a snekbox
short snow
yes
fallen patrol
you're running everything lolz
vocal wolf
@tawdry vapor had a missing message caused a 400 for you in the past? If so, do you remember what specifically created that error?
Context: https://github.com/python-discord/bot/pull/1414#discussion_r594641273
tawdry vapor
Not necessarily
But there are some endpoints that do that
The API is inconsistent like that
vocal wolf
ah
tawdry vapor
I don't remember what it was for
patent pivot
because it pulls from the invite, not the actual server
fallen patrol
I'll be leaving now sorry for the ping
patent pivot
we don't want presences intent enabled, but we want presences
fallen patrol
I thought it was some kind of error of sorts
thorny obsidian
Hard coded in what sense? Our Python bot is intended to only really work with our server
patent pivot
patent pivot
Ideally the server invite wouldn't be hard coded lol
just checked, it isn't hard coded
it's a config option
here is the code in !server that fetches the counts from that configured invite https://github.com/python-discord/bot/blob/main/bot/exts/info/information.py#L169-L176
fallen patrol
Hard coded in what sense? Our Python bot is intended to only really work with ou...
Well yeah, but if something happens to the pydis invite of python it'd be nice to be able to not have it hard coded, which it isn't, so the point is moot
vocal prairie
@fallen patrol Partnered servers are warned 4 weeks in advance if they're on the way to losing partnership.
That definitely long enough to change things.
patent pivot
!remind 10h metricity#3
dusky shoreBOT
stable mountainBOT
fallen patrol
!remind 10h metricity#3
why
why are you doing this to the 10 of us who opted out
patent pivot
because i put my privacy hat on
and then i put my data hat on
and came to the conclusion that the opt out functionality is more harmful than it is good
fallen patrol
😔
patent pivot
It's not any personal data
it's data that one can already accumulate through discord search or our existing message logs
patent pivot
no
that's stated in the privacy policy
metricity messages have an is_deleted property
vocal prairie
what about deleted messages
But mods can see that too.
And they're the only ones who see metricity metrics.
So it's not really that much of a change.
fallen patrol
so metricity logs more than python in a sense
vocal prairie
Not really.
patent pivot
not really, metricity never has contents, ever
fallen patrol
api_messagedeletioncontext & api_deletedmessage
There were 285 message deletion contexts in the database, within these there were 1,300 stored deleted messages.
These have been deleted since they are stored only for review and are deleted after no longer than 30 days.
patent pivot
that's Python, yeah
magic arch
just dont rename anything, joe ;) lolll
patent pivot
those are the anti-spam held messages including context
fallen patrol
so metricity has more info on when i sent a message than python does.
magic arch
so metricity has more info on when i sent a message than python does.
if it was spam then yes, otherwise prob not
fallen patrol
which means you could figure out every time i've deleted or edited a message, no?
patent pivot
deleted, yes, edited, no
magic arch
they log deleted and edited stuff with @stable mountain i think
fallen patrol
you don't store edits?
patent pivot
not with metricity
python handles edits
and yeah, to an extent python can see deletions and edits
vocal prairie
which means you could figure out every time i've deleted or edited a message, no...
You're not losing any privacy then.
thorny obsidian
The logging of the typical deletion and edits is kept within discord though
vocal prairie
python handles edits
Crap, then you see all my embarrassing spelling mistakes.
patent pivot
yeah, that's true, messages get an edit timestamp at the discord api side
patent pivot
Crap, then you see all my embarrassing spelling mistakes.
hahahahaha
magic arch
Crap, then you see all my embarrassing spelling mistakes.
if it makes you feel any better, you're not alone there lol
patent pivot
but yeah, the use of metricity data is pretty vast, and this opt out isn't worth the loss of clarity
thorny obsidian
It's okay, I flood that log with my terrible spelling when I use my phons
fallen patrol
but yeah, the use of metricity data is pretty vast, and this opt out isn't worth...
are the uses detailed in depth anywhere?
patent pivot
I detailed some of them on the issue
fallen patrol
metricity#3
dusky shoreBOT
patent pivot
nominations use metricity data now, voice verification, channel usage metrics
a few uses inside moderation as well
fallen patrol
a few uses inside moderation as well
what cogs in python
fallen patrol
like?
patent pivot
metabase, grafana, the postgres shell
patent pivot
yeah, exactly that
fallen patrol
metabase, grafana, the postgres shell
sources?
patent pivot
I'm not talking like systems here, this is just ways that we query the data, we have uses inside moderation
thorny obsidian
The grafana stuff is obnoxiously useful for talentpool stuff and general quick data trends
magic arch
I'm curious, how many folks have actually opted out?
vocal prairie
Can we see a limited version of the grafana?
Or is that exactly what the stats page is?
patent pivot
I'm curious, how many folks have actually opted out?
only ever a handful
fallen patrol
only ever a handful
in that i see the reason to remove it but also to not remove it since its only a few people
patent pivot
lol trying to find a dash without internal info
magic arch
You could possibly reach out to those few if it's not that many to give a heads up
vocal prairie
You could possibly reach out to those few if it's not that many to give a heads ...
It'll be changelogged
patent pivot
that data comes from the metricity table
metabase is where we tend to crunch message data more often
vocal prairie
What are those categories at the top? I can't read it on my phone
patent pivot
What are those categories at the top? I can't read it on my phone
the categories on the top graph? not sure what you are referring to
vocal prairie
Yeah
patent pivot
red = not passed native gate, green = passed
vocal prairie
The title
vocal prairie
red = not passed native gate, green = passed
Oh ok. Ty
patent pivot
Users pending native gate
vocal prairie
That's pretty cool
magic arch
It'll be changelogged
I mean like a warning ahead of time of some kind, never know if people will want to abandon the server or something haha
Probably not a super big concern since its not that many though
fallen patrol
Probably not a super big concern since its not that many though
its like 6 of us ffs
vocal prairie
I mean like a warning ahead of time of some kind, never know if people will want...
Will one day of metrics hurt them?
fallen patrol
Will one day of metrics hurt them?
"you opted out! We just opted you back in!" is a pretty bad announcement
patent pivot
vocal prairie
True
patent pivot
patent pivot
Is data removed after a user leaves?
it's anonymised, yeah
fallen patrol
i think they'd have to give a date to remove it by and changelog it before its actually removed. I think.
thorny obsidian
I still want to make a gif of the peak active times changing from Oct -> Feb
vocal prairie
Ooh, those are some nice insights.
thorny obsidian
The AoC activity shift is pretty amusing
patent pivot
i think they'd have to give a date to remove it by and changelog it before its a...
for this PR?
it's not a compliance thing, this isn't personal data
vocal prairie
AoC?
thorny obsidian
Advent of Code
vocal prairie
Got it
patent pivot
I'll just amend all relevant documentation at once when I merge the PR
I agree, and that's why we don't word it like that 😛
thorny obsidian
AOC != AoC
fallen patrol
really hope i spelled that correctly
fallen patrol
AOC != AoC
*''
patent pivot
I think the body of the PR explains it fairly well, it's a fairly misleading opt out policy since it only actually opts you out from the part that is fully anonymous
fallen patrol
….
fallen patrol
so you mean to tell me i've opted out of being anonymous this entire time
brb dming the data controller
patent pivot
no, that's not what I'm saying
magic arch
Would it hurt to share that like a day or two early though? Still seems like a weird thing to do without public notice imo.
patent pivot
I'm saying that the component that you opted out of, the message counters, is the part that is anonymous
fallen patrol
brb dming the data controller
guys who's the data controller again
patent pivot
Would it *hurt* to share that like a day or two early though? Still seems like a...
oh yeah for sure I'll give it a couple of days
fallen patrol
Would it *hurt* to share that like a day or two early though? Still seems like a...
i have nothing but bad vibes if its not early, its legit shady business practice to change stuff with no time before going into effect
magic arch
oh yeah for sure I'll give it a couple of days
Yeah that's all I'm saying, to just make sure the folks it affects are aware if possible ahead of time
But then again I have no say in this so... carry on 😂
vocal prairie
I wouldn't be surprised if you at least have some. They consider a lot of community ideas. 🤷♂️
patent pivot
this is the current structure of the held data
so the former two are handled as part of the data review and removals, amendments, etc. are carried out there
and metricity message data is handled in the opt out functionality of metricity
vocal prairie
Nice diagram
vocal prairie
so the former two are handled as part of the data review and removals, amendment...
Makes sense
So you're not opting out of anything specific to you?
patent pivot
but the misleading factor there is that opting out of messages is in no way an opt out of other data collection, and I think that the best way to make that clear is not to have misleading stuff there and instead to integrate that into the data review process
and as I mentioned, the message metrics harvested by metricity are proving very useful in many areas of the organisation, so it makes sense for them to be treated as such
vocal prairie
True. When I first read the opt out thing, I assumed you opted out of everything. It's much clearer when you remove that whole idea
Metricity#3
patent pivot
but I do agree that forewarning on things like this is important, so making the privacy policy amendments beforehand and changelogging it is a good move to make, so I'll give it a good week between publishing that and these changes going live
vocal prairie
metricity#3
dusky shoreBOT
vocal prairie
Nice talking with you all, but now I must sleep. Good night!
vocal wolf
https://github.com/python-discord/sir-lancebot/pull/666 needs one last review
ty @cold moon
cold moon
I have idea for @dusky shore : command that fetches information about compound from PubChem API.
cold moon
Here is API docs https://pubchemdocs.ncbi.nlm.nih.gov/pug-rest-tutorial
brazen charm
Would that be useful here? I feel like lancebot has a few too many api wrappers which don't really improve the project for beginners, and I believe WolframAlpha can get the information too
stable mountainBOT
@green oriole
It has arrived!
green oriole
Chris got it
placid ermine
would love a review on lancebot#668
dusky shoreBOT
dusky shoreBOT
short snow
!remind 5H review sir lancebot-668
stable mountainBOT
vale ibex
would love a review on lancebot#668
How long does this take the render?
EG if I gave it some long, complex, input, could it take a few seconds?
vale ibex
I'd say if its >1 second, with any input, then we should run it in an executor
@placid ermine Do you have experience with running code in executors?
short snow
btw, I am talking about the max case, otherwise it is instant rendering.
placid ermine
<@!787351231332483102> Do you have experience with running code in executors?
yeah, somewhat
just run it in a ProcessPool right
timid sentinel
yeah would probably be worth it, timing the _render it seems mostly fine but takes a bit longer on very complex inputs lol
short snow
👀 that input
timid sentinel
oh no no i just added the timing to test it
timid sentinel
lol I just did r"\frac{0}{0}".replace('0', r'\frac{0+0}{0}').replace('0', r'\frac{0+0}{0+0}').replace('0', r'\frac{0+0}{0+0}').replace('0', r'\frac{0+0}{0+0}').replace('0', '1'), that's as much as discord's text limit allows 😔
short snow
lol I just did `r"\frac{0}{0}".replace('0', r'\frac{0+0}{0}').replace('0', r'\fr...
✅ Your eval eval output is posted here: https://paste.pythondiscord.com/qibocukoyu.txt?noredirect
placid ermine
Take a look at https://github.com/python-discord/sir-lancebot/pull/597/files#dif...
hmm
that is pretty much what i had done in https://github.com/gurkult/gurkbot/blob/b28d4349986e6a078f4306eccf0790388a90a7fc/bot/exts/fun/bonker.py#L108-L109
short snow
I have a question regarding the test server, since everyone is a super user or a admin on the test server, anyone can run fork bombs on the user's bot, so do you guys prevent that in some way, or just trust everyone 🕊️
patent pivot
trust, that's why it's staff only
short snow
ah, are contributors also in it?
fallen patrol
ah, are contributors also in it?
no, they're not staff
i mean
you could make an issue to possibly make it a little more friendly for building
patent pivot
yeah it's helpers+
patent pivot
you could make an issue to possibly make it a little more friendly for building
huh?
as in setting it up?
patent pivot
that is a project of ours
vocal prairie
You should make a bot that has all the emojis
patent pivot
cc @green oriole
vocal prairie
And it can put them all in a server
patent pivot
shouldn't smartconfig resolve some pain here Ak?
short snow
smart config
fallen patrol
as in setting it up?
an additional config in @stable mountain to like disable most of the stuff except for from bot.owner
green oriole
Henlo
vocal prairie
Hi
patent pivot
an additional config in <@!409107086526644234> to like disable most of the stuff...
Why?
short snow
i could possibly make a script for server setup
patent pivot
That isn't functionality we need
green oriole
Yeah, Smartconfig will come with an helper command to generate the whole server and config with one command
patent pivot
We need multi-user access during our testing
vocal prairie
Yeah, Smartconfig will come with an helper command to generate the whole server ...
Ooh, that's cool.
fallen patrol
no but
you could do it in like
4 lines of code for the benefit of most contributors
green oriole
Well, not really
patent pivot
I don't think it is for the benefit of most contributors
It's for the benefit of server staff maybe
in the test server
but even then like this isn't a protection we need
because a) people don't run dev bots 24/7 and b) if we have trust issues with staff then that's a wider problem, this shouldn't be a problem that needs a solution
as for contributors, just run it on a bot account that is in a server with only yourself
vocal prairie
To make sure I'm understanding correctly, smartconfig will allow you to set up test servers with one command?
green oriole
Yep
green oriole
The end goal is that you create a server, invite the bot, run a command grab a coffee because of ratelimits, restart the bot and you are good to go
It should also be able to sync with the configuration when changes are made
vocal prairie
Ratelimits smh
fallen patrol
The end goal is that you create a server, invite the bot, run a command *grab a ...
so the bot will be a hosted bot from pydis?
patent pivot
No, it's a package
green oriole
Nah, it is your development bot
vocal prairie
It should also be able to sync with the configuration when changes are made
Oh, that's useful
fallen patrol
then how tf can it sync to this server if its not in it??
vocal prairie
No need to copy and paste
green oriole
It will be inside the source code of @stable mountain, only loaded when debug mode is enabled
patent pivot
the server configuration is defined in the repo yeah
green oriole
The sync command will also be inside the repo
patent pivot
because we don't want to exact mirror of this server
it's bloated for development server setups
users don't need to synchronise modmail threads, archived channels and other internal things with their dev servers
short snow
till then, i willl just come up with a setup script for this server
patent pivot
(Hence why the template we provided doesn't have all the channels)
green oriole
(We will have to think on a way to merge some channels in one inside the configuration but it should be fairly simple)
vocal prairie
@crude gyro In regards to the second part of site#453, I was wondering if I should place anything where the timeline used to be on the home page?
dusky shoreBOT
vocal prairie
Also, is there a specific place in the "more" section of the navbar you want the link? I have it under Contributing at the moment.
crude gyro
<@!95872159741644800> In regards to the second part of site#453, I was wondering...
uhh. I wasn't aware we removed the timeline.
to me it doesn't really make sense to remove it until we find something to replace it with.
@green oriole any idea when this component disappeared?
oh it didn't, it's still there?
crude gyro
<@!95872159741644800> In regards to the second part of site#453, I was wondering...
hmm I still see that component.
green oriole
it is just not on the top row anymore
short snow
it was earlier nah my bad, the 100k banner was there
green oriole
I don't think we should remove the interactive timeline undernearth
vocal prairie
Oh, I though the issue mentioned it.
Oh, I misread, my bad.
But I'll still open the pr to move the timeline to the navbar.
Any thoughts about placement for that?
green oriole
Yep, it would be nice to have it up there
vocal prairie
Also, is there a specific place in the "more" section of the navbar you want the...
.
vocal prairie
isn't timeline already on the navbar, in the more section?
Oh, that image is my copy
short snow
ah ok
green oriole
Above FAQ is probably good
vocal prairie
I'll go ahead an open a pr.
PR has been made. Also, is there a plan to add anything where the 100k banner was? I
crude gyro
But I'll still open the pr to move the timeline to the navbar.
move seems to indicate you're removing the widget from the landing page
vocal prairie
Oh, I mean "add". My bad.
vocal prairie
Do we want to remove the mobile banner?
crude gyro
?
vocal prairie
On mobile, the 100k banner still appears.
vocal prairie
You just have to remove this section:
It's this portion of the code.
gritty wind
It’s not wrong, but it definitely conveys the message that we just hit 100k
When we’re almost at 200
vocal prairie
In my opinion, we shouldn't remove it, but rephrase it for the timeline.
stable mountainBOT
@patent pivot
It has arrived!
stable mountainBOT
@short snow
It has arrived!
clever wraith
sirlancebot#668
dusky shoreBOT
dusky shoreBOT
clever wraith
o
green oriole
Just need @exotic ember's review on sir-lancebot#642 to merge it 🥺
green oriole
I guess if kosa isn't available someone can review for them and dismiss
short snow
<@736234578745884682>
Looks like few good ppl already reviewed it and I have nothing to say, but well I am gonna do some bikeshedding on the output message 
vale ibex
and to be clear, this change won't make it output properly, it just won't output anything 😄
short snow
Yeah
green oriole
I'd have a couple of nitpicking to do but I'll refrain myself haha
thorny obsidian
For lance, why are some hardcoded and some not?
short snow
Lol, I would like to add timing to it
green oriole
For lance, why are some hardcoded and some not?
Who knows
short snow
Some are not hard coded, as they are not needed for normal testing ig
green oriole
This will be obliterated by smartconfig which should come further down the road
If you need any for testing feel free to introduce an env var
gritty wind
How long have we been talking about smartconfig coming soon™️
green oriole
But I don't think it is worth switching everywhere
gritty wind
You really don't need most of the roles in development
green oriole
~~How long have we been talking about smartconfig coming soon™️~~
I said "further down the road" haha
gritty wind
the only real use is for bypassing checks
short snow
Lol, I was gonna just ask that scal, but AK’s pr needs review
gritty wind
and admins=mods in sir-lance for the most part
green oriole
Hopefully we should merge the main PR soon-ish
gritty wind
and admin=owner in both
clever wraith
hows the progress on quackstack?
short snow
We don’t really need postgres on quackstack
fervent sage
We don’t really need postgres on quackstack
progress not postgres :P
green oriole
I wouldn't care if it isn't completed this year, compared to other projects
short snow
Autocorrect, iPad isn’t that smart