#dev-contrib | Python | Page 161

molten bough Dec 21, 2021, 10:12 PM

#

oh that's a fork too, okay

#

I was wondering

fallen patrol Dec 21, 2021, 10:13 PM

#

I've been working with disnake myself

clever wraith Dec 21, 2021, 10:13 PM

#

everything is a fork

#

even me lmao

molten bough Dec 21, 2021, 10:13 PM

#

clearly we all should have been using disco this whole time

#

haha

gritty wind Dec 21, 2021, 10:13 PM

#

We're not switching right now, I don't think we even have enough active members to get a PR through, or all the inevitable fire-putting-out PRs

molten bough Dec 21, 2021, 10:14 PM

#

metricity is small enough that you likely don't need to put out many fires

#

though tbf this isn't a huge issue

fallen patrol Dec 21, 2021, 10:14 PM

#

gritty wind We're not switching right now, I don't think we even have enough active members ...

hey I'll have you know I have converted two public bots so far to forks and there were no fires!

gritty wind Dec 21, 2021, 10:14 PM

#

It's fine with metricity honestly, worst case scenario if metricity goes down is we lose a bit of data

#

That's already happened

#

Most people didn't notice

clever wraith Dec 21, 2021, 10:14 PM

#

they are all drop-in replacements tbf

fallen patrol Dec 21, 2021, 10:14 PM

#

okay that's not accurate, one of the hosting datacenters had a fire but that wasn't related.

gritty wind Dec 21, 2021, 10:14 PM

#

Bot goes down and this server goes poof

#

We're in no rush to upgrade right now, I'll leave it at that

molten bough Dec 21, 2021, 10:15 PM

#

we have users that rely on our stats to figure out which channels are worth looking at in the morning

#

which is amusing

#

but this doesn't break that

fallen patrol Dec 21, 2021, 10:15 PM

#

clever wraith they are all drop-in replacements tbf

ye, some more drop in than others

gritty wind Dec 21, 2021, 10:15 PM

#

I'm trying to debug this right now, but honestly the command may be harder than having the bot just print out any calls to terminal

#

It ran once, then the command stopped working

#

I don't get it

fallen patrol Dec 21, 2021, 10:16 PM

#

gritty wind I'm trying to debug this right now, but honestly the command may be harder than ...

I told you to use the thingy smh

molten bough Dec 21, 2021, 10:16 PM

#

debugger when

vale ibex Dec 21, 2021, 10:16 PM

#

Where did you add the process commands?

gritty wind Dec 21, 2021, 10:16 PM

#

Debugger is active, but nothing is even being called

molten bough Dec 21, 2021, 10:16 PM

#

hmm

gritty wind Dec 21, 2021, 10:16 PM

#

bottom of on_message

fallen patrol Dec 21, 2021, 10:16 PM

#

use the thingy event to get the thingy debug in the thingy

vale ibex Dec 21, 2021, 10:16 PM

#

There are a bunch of quit early returns in on_message

gritty wind Dec 21, 2021, 10:16 PM

#

fallen patrol use the thingy event to get the thingy debug in the thingy

I am, I'm just using the command to get it in a nice format for review

vale ibex Dec 21, 2021, 10:16 PM

#

look at my PR and add it to the same place

fallen patrol Dec 21, 2021, 10:16 PM

#

vale ibex There are a bunch of quit early returns in on_message

this is why i said to add a new listener event smh

vale ibex Dec 21, 2021, 10:17 PM

#

We don't have any cogs

#

so no way to add listeners...

fallen patrol Dec 21, 2021, 10:17 PM

#

do you need one?

gritty wind Dec 21, 2021, 10:17 PM

#

Shouldn't we check for bot

molten bough Dec 21, 2021, 10:17 PM

#

kinda bikesheddy

fallen patrol Dec 21, 2021, 10:17 PM

#

!d discord.ext.commands.Bot.listen

gritty wind Dec 21, 2021, 10:17 PM

#

(in your PR)

stable mountainBOT Dec 21, 2021, 10:17 PM

#

discord.ext.commands.Bot.listen


@listen(name=None)```
A decorator that registers another function as an external event listener. Basically this allows you to listen to multiple events from different places e.g. such as [`on_ready()`](https://discordpy.readthedocs.io/en/master/api.html#discord.on_ready "discord.on_ready")

The functions being listened to must be a [coroutine](https://docs.python.org/3/library/asyncio-task.html#coroutine "(in Python v3.9)").

Example...

fallen patrol Dec 21, 2021, 10:17 PM

#

yeah you don't need a cog for that

vale ibex Dec 21, 2021, 10:18 PM

#

Ah fair enough

vale ibex Dec 21, 2021, 10:18 PM

#

gritty wind Shouldn't we check for bot

wdym?

clever wraith Dec 21, 2021, 10:18 PM

#

molten bough kinda bikesheddy

plays welcome to python discord video

gritty wind Dec 21, 2021, 10:18 PM

#

if not message.author.bot: process_commands

molten bough Dec 21, 2021, 10:18 PM

#

welcom to huudraulic.. wait

vale ibex Dec 21, 2021, 10:18 PM

#

gritty wind if not message.author.bot: process_commands

I mean, you could lol

gritty wind Dec 21, 2021, 10:19 PM

#

Yeah still no lock with the sock command

vale ibex Dec 21, 2021, 10:19 PM

#

but the bot isn't really going to be invoking it's own commands

fallen patrol Dec 21, 2021, 10:19 PM

#

literally move the current listener to this ```py
@bot.listen("on_message")
async def process_messages():

gritty wind Dec 21, 2021, 10:19 PM

#

Am I just not adding the cog or something

fallen patrol Dec 21, 2021, 10:19 PM

#

and then remove process commands from it

gritty wind Dec 21, 2021, 10:19 PM

#

vale ibex but the bot isn't really going to be invoking it's own commands

I'm more thinking the other bots

#

Craft a nice eval, ping the bot, tell it do whatever

vale ibex Dec 21, 2021, 10:19 PM

#

Ahh

fallen patrol Dec 21, 2021, 10:19 PM

#

sigh

vale ibex Dec 21, 2021, 10:19 PM

#

Where are you loading the extension?

gritty wind Dec 21, 2021, 10:20 PM

#

Last line of main

#

bot.add_cog(Internal(bot))

#

I know it's working because

#

#

It's just not working anymore

fallen patrol Dec 21, 2021, 10:20 PM

#

gritty wind Last line of main

last?

#

so after bot.run()?

vale ibex Dec 21, 2021, 10:20 PM

#

bot.run isn't in main

gritty wind Dec 21, 2021, 10:20 PM

#

There is no bot.run here

#

that's in __main__

molten bough Dec 21, 2021, 10:20 PM

#

I keep forgetting d.py doesn't use a type heirarchy for that

#

would be a lot easier to listen

fallen patrol Dec 21, 2021, 10:20 PM

#

!paste

molten bough Dec 21, 2021, 10:20 PM

#

spoiled by kord lmao

#

does d.py not have trace logging you can use to see the gateway events?

#

if so, seems like a simpler solution

gritty wind Dec 21, 2021, 10:21 PM

#

not afaik

fallen patrol Dec 21, 2021, 10:22 PM

#

molten bough does d.py not have trace logging you can use to see the gateway events?

it does, and I suggested it above

gritty wind Dec 21, 2021, 10:22 PM

#

I don't think they have any logging

#

I've already patched the event

#

If that's what you're suggesting

fallen patrol Dec 21, 2021, 10:22 PM

#

@molten bough it has this

#

which will log the type of event received

gritty wind Dec 21, 2021, 10:22 PM

#

Also I know that everything is loaded properly because:

fallen patrol Dec 21, 2021, 10:22 PM

#

sigh i'm on pc again

gritty wind Dec 21, 2021, 10:22 PM

#

#

those messages are seconds apart

molten bough Dec 21, 2021, 10:23 PM

#

you seem to be going to a lot of effort to debug a debugging tool, is my point

#

so maybe it's worth taking a simpler route

gritty wind Dec 21, 2021, 10:23 PM

#

The simpler route here is collecting all these metrics by hand

#

D.py exposes them in an event, but not in a readable way

#

The function that gets the output is not being called

molten bough Dec 21, 2021, 10:24 PM

#

so then all you should have to do is work up the potential call tree until you find a breakpoint that does get hit

gritty wind Dec 21, 2021, 10:25 PM

#

That means diving into d.py's source, and I was hoping to avoid that :P

molten bough Dec 21, 2021, 10:25 PM

#

ah right, but d.py does a lot of dynamic dispatch

gritty wind Dec 21, 2021, 10:26 PM

#

I've got it working now btw

#

Had to change the prefix /shrug

molten bough Dec 21, 2021, 10:26 PM

#

I'm gonna get a paracetamol. /hj

gritty wind Dec 21, 2021, 10:27 PM

#

Alright, so update

#

the event does fire

#

(bottom right)

#

but it doesn't call the handler for some reason

molten bough Dec 21, 2021, 10:28 PM

#

Well discord sends the event, it doesn't fire clearly

clever wraith Dec 21, 2021, 10:28 PM

#

gritty wind That means diving into d.py's source, and I was hoping to avoid that \:P

and to this day, he has yet to be seen again

fallen patrol Dec 21, 2021, 10:28 PM

#

gritty wind the event does fire

odd

gritty wind Dec 21, 2021, 10:29 PM

#

I can get it to fire now if I don't restart the bot

#

Which means the only thing I have left to do

molten bough Dec 21, 2021, 10:29 PM

#

I was gonna say d.py's source isn't that bad but it is pretty spaghetti tbf

gritty wind Dec 21, 2021, 10:29 PM

#

Is to dive into the d.py source again

molten bough Dec 21, 2021, 10:29 PM

#

I'm still salty about the caching race condition I found that they didn't want to fix in case it broke the entire stack lmao

gritty wind Dec 21, 2021, 10:30 PM

#

Yeah sounds about right

fallen patrol Dec 21, 2021, 10:30 PM

#

yeah it fires

gritty wind Dec 21, 2021, 10:30 PM

#

Ok, breakpoints loaded, next up is seeing what happens

fallen patrol Dec 21, 2021, 10:30 PM

#

gritty wind the event does fire

wow you debugged everything and added a whole command and I just added a print listener 😂

#

and yes my bot is running in docker because I couldn't figure out how to run metricity outside of docker rn

#

!d discord.Guild.threads

stable mountainBOT Dec 21, 2021, 10:32 PM

#

discord.Guild.threads


property threads: List[discord.threads.Thread]```
A list of threads that you have permission to view.

New in version 2.0.

fallen patrol Dec 21, 2021, 10:32 PM

#

this is why

molten bough Dec 21, 2021, 10:32 PM

#

What's why?

fallen patrol Dec 21, 2021, 10:33 PM

#

that's not all threads

#

that's only active threads

gritty wind Dec 21, 2021, 10:33 PM

#

So... ?

molten bough Dec 21, 2021, 10:33 PM

#

Yeah that's fine

fallen patrol Dec 21, 2021, 10:33 PM

#

wait i might be too tired

#

ignore me

gritty wind Dec 21, 2021, 10:33 PM

#

It's not getting to the coro await step which calls our code

#

But all the steps before that are the spaghetti dispatching from d.py internals

fallen patrol Dec 21, 2021, 10:34 PM

#

gritty wind It's not getting to the coro await step which calls our code

sync_channels?

gritty wind Dec 21, 2021, 10:34 PM

#

Which means the function one level up is called about 40 times for this one message

gritty wind Dec 21, 2021, 10:34 PM

#

fallen patrol sync_channels?

on_thread_update

#

I think I'll call that route a dead end, but why is it dispatching for one event listener but not the other

fallen patrol Dec 21, 2021, 10:35 PM

#

odd

#

calls mine just fine..

#

what's the repro steps?

molten bough Dec 21, 2021, 10:35 PM

#

archive thread

#

start bot

#

unarchive thread

#

archive it again

gritty wind Dec 21, 2021, 10:35 PM

#

What, your on_thread_update gets called?

fallen patrol Dec 21, 2021, 10:35 PM

#

i didn't do the repo steps

gritty wind Dec 21, 2021, 10:35 PM

#

The second archive in that chain does get called for me

#

It's just the first update after starting the bot does not fire if it's an unarchive

molten bough Dec 21, 2021, 10:36 PM

#

No, that fires fine

#

I checked that

vale ibex Dec 21, 2021, 10:36 PM

#

I am going to guess its similar to message deletes, where d.py doesn't fire that event if the thread isn't cached

molten bough Dec 21, 2021, 10:36 PM

#

that's why it's in the database to begin with

gritty wind Dec 21, 2021, 10:36 PM

#

Really?

#

Hm, strange

fallen patrol Dec 21, 2021, 10:37 PM

#

vale ibex I am going to guess its similar to message deletes, where d.py doesn't fire that...

sounds accurate

#

can repro

gritty wind Dec 21, 2021, 10:37 PM

#

It's not firing at all for me with this

fallen patrol Dec 21, 2021, 10:37 PM

#

same ^

molten bough Dec 21, 2021, 10:37 PM

#

I got the sync logging when I did that

#

so it must have fired

gritty wind Dec 21, 2021, 10:38 PM

#

Is that the one from the startup process maybe?

molten bough Dec 21, 2021, 10:38 PM

#

vale ibex I am going to guess its similar to message deletes, where d.py doesn't fire that...

this is one of my favourite reasons to shit on d.py by the way haha

gritty wind Dec 21, 2021, 10:38 PM

#

Do you see two sync logs up to step 3

molten bough Dec 21, 2021, 10:38 PM

#

gritty wind Is that the one from the startup process maybe?

nope, it was after the user sync

fallen patrol Dec 21, 2021, 10:39 PM

#

tries with a fork

gritty wind Dec 21, 2021, 10:39 PM

#

Strange

molten bough Dec 21, 2021, 10:39 PM

#

#

you can see it at the bottom

gritty wind Dec 21, 2021, 10:39 PM

#

molten bough archive it again

Is that using my repo, or your repro

fallen patrol Dec 21, 2021, 10:39 PM

#

fallen patrol *tries with a fork*

lol they don't have it patched either

molten bough Dec 21, 2021, 10:40 PM

#

that's what happened when I reported it

#

I rebuilt and restarted, waited for user sync, unarchived, archived

gritty wind Dec 21, 2021, 10:40 PM

#

The second unarchiving and rearchiving would've meant you'd see that log again

vale ibex Dec 21, 2021, 10:40 PM

#

Out of interest, could you change guild.threads to await guild.active_threads() in the channel syncer scale?

gritty wind Dec 21, 2021, 10:40 PM

#

So I think it's only firing once like me and arl are experiencing

vale ibex Dec 21, 2021, 10:40 PM

#

On mobile now so can't do it myself

molten bough Dec 21, 2021, 10:40 PM

#

I mean yeah it is firing once

gritty wind Dec 21, 2021, 10:41 PM

#

sure yeah

molten bough Dec 21, 2021, 10:41 PM

#

we were already at that equilibrium haha

vale ibex Dec 21, 2021, 10:41 PM

#

It might be the cache hasn't updated by that point

fallen patrol Dec 21, 2021, 10:41 PM

#

vale ibex It might be the cache hasn't updated by that point

nah, archived threads aren't cached period

vale ibex Dec 21, 2021, 10:41 PM

#

I am aware

molten bough Dec 21, 2021, 10:41 PM

#

are you telling me this is probably the same race condition I mentioned earlier

vale ibex Dec 21, 2021, 10:41 PM

#

Possibly

gritty wind Dec 21, 2021, 10:41 PM

#

Nope, no change

vale ibex Dec 21, 2021, 10:41 PM

#

Alright

#

I'm gonna leave this to you guys since I don't have any access to a pc right now lol

gritty wind Dec 21, 2021, 10:42 PM

#

The syncer isn't being called at all, so that might be expected

molten bough Dec 21, 2021, 10:42 PM

#

I'll have to go to bed soon anyway haha

fallen patrol Dec 21, 2021, 10:42 PM

#

hmmm

#

https://discord.com/developers/docs/topics/threads#unarchiving-a-thread

Discord Developer Portal

Discord Developer Portal — API Docs for Bots and Developers

Integrate your service with Discord — whether it's a bot or a game or whatever your wildest imagination can come up with.

#

given this, if there's a thread_member update event we could use that....

vale ibex Dec 21, 2021, 10:42 PM

#

Well it's not crash looping now at least, so it's not an urgent fix

molten bough Dec 21, 2021, 10:43 PM

#

yeah this doesn't break anything for us aside from data integrity

vale ibex Dec 21, 2021, 10:43 PM

#

Yea, my command that's up for pr will solve that too

molten bough Dec 21, 2021, 10:43 PM

#

which would be a pretty big break if this wasn't the initial behaviour I guess haha

fallen patrol Dec 21, 2021, 10:43 PM

#

!d discord.on_thread_member_join

stable mountainBOT Dec 21, 2021, 10:43 PM

#

discord.on\_thread\_member\_join


discord.on_thread_member_join(member)``````py

discord.on_thread_member_remove(member)```
Called when a [`ThreadMember`](https://discordpy.readthedocs.io/en/master/api.html#discord.ThreadMember "discord.ThreadMember") leaves or joins a [`Thread`](https://discordpy.readthedocs.io/en/master/api.html#discord.Thread "discord.Thread").

You can get the thread a member belongs in by accessing [`ThreadMember.thread`](https://discordpy.readthedocs.io/en/master/api.html#discord.ThreadMember.thread "discord.ThreadMember.thread").

This requires [`Intents.members`](https://discordpy.readthedocs.io/en/master/api.html#discord.Intents.members "discord.Intents.members") to be enabled.

New in version 2.0.

vale ibex Dec 21, 2021, 10:43 PM

#

You'll be able to give it two epochs and it'll sync all messages between them

molten bough Dec 21, 2021, 10:43 PM

#

what does your public stats dashboard look like from all this, by the way?

#

yeah I saw that haha

vale ibex Dec 21, 2021, 10:44 PM

#

Our usage of threads is minimal so probably no change

fallen patrol Dec 21, 2021, 10:45 PM

#

vale ibex Our usage of threads is minimal so probably no change

will it ever change, and we use threads more here?

molten bough Dec 21, 2021, 10:45 PM

#

I mean I already don't know what it looks like

gritty wind Dec 21, 2021, 10:46 PM

#

fallen patrol !d discord.on_thread_member_join

Not being called either

molten bough Dec 21, 2021, 10:46 PM

#

was wondering for comparison haha

gritty wind Dec 21, 2021, 10:46 PM

#

But also making it to socket command

#

nvm, it isn't

#

Unless it's GUILD_MEMBERS_CHUNK

vale ibex Dec 21, 2021, 10:46 PM

#

Oh, when you say public stats, do you mean stats.pythondiscord.com or soemthing else?

fallen patrol Dec 21, 2021, 10:46 PM

#

yeahh, i should have realised that since I have streaming logs for that

molten bough Dec 21, 2021, 10:47 PM

#

ah yeah that's what I was wondering

#

huh that's a lot more basic than ours

#

guess this isn't the right place to ask why go basic though haha

clever wraith Dec 21, 2021, 10:48 PM

#

that page could be replaced with a metabase dashboard tbh

molten bough Dec 21, 2021, 10:48 PM

#

oh that's not a metabase dashboard

gritty wind Dec 21, 2021, 10:48 PM

#

You'd most likely need Joe's DMs for that

#

It's uhh

#

I can get you the thing

molten bough Dec 21, 2021, 10:48 PM

#

nah it's fine

#

we're using metabase already

#

https://stats-quilt.gserv.me/public/dashboard/9b181b97-bd7f-4ab0-87ed-3239f9149932

fallen patrol Dec 21, 2021, 10:48 PM

#

btw could someone take a look at bot#2022? I've looked at that and need some more opinions

dusky shoreBOT Dec 21, 2021, 10:48 PM

#

GitHub

IssueOpen Incorrect handling of tags by the !source command

gritty wind Dec 21, 2021, 10:48 PM

#

I think we considered metabase, but the resource overhead might've been too high

#

These are just generated on the server occasionally and served statically

molten bough Dec 21, 2021, 10:49 PM

#

yeah that's probs more sensible at this scale tbf

#

okay, I've got a date with youtube and some headache pills, but thanks for trying to look into this promptly

fallen patrol Dec 21, 2021, 10:50 PM

#

molten bough https://stats-quilt.gserv.me/public/dashboard/9b181b97-bd7f-4ab0-87ed-3239f91499...

suggestion: needs something to fill this whitespace

molten bough Dec 21, 2021, 10:50 PM

#

that's just what metabase does at the top of a dashboard

fallen patrol Dec 21, 2021, 10:50 PM

#

ah ;-;

stable mountainBOT Dec 21, 2021, 11:05 PM

#

It has arrived!

Here's your reminder: @fallen patrol Have you seen this? #dev-log message
[Jump back to when you created the reminder](#dev-contrib message)

patent pivot Dec 21, 2021, 11:06 PM

#

non-zero chance we caused the linode block storage outage in frankfurt today mmLul

#

good job team, keep up the good work

fallen patrol Dec 21, 2021, 11:09 PM

#

patent pivot non-zero chance we caused the linode block storage outage in frankfurt today <:m...

what happened

patent pivot Dec 21, 2021, 11:09 PM

#

nothing we directly did

#

but we opened a few tickets about our block storage problems the other day

#

today a fix was deployed

fallen patrol Dec 21, 2021, 11:09 PM

#

lmao

patent pivot Dec 21, 2021, 11:09 PM

#

#

rmsthink

fallen patrol Dec 21, 2021, 11:12 PM

#

fun.

#

more like a "fix"

patent pivot Dec 21, 2021, 11:14 PM

#

either way we saw some improved volume handling after the skirmish earlier today

#

so that's +++++

#

ideally we have no problems in first place, please linode kubernetes engine MmmHmm

#

had a look at the upgrade we did that moved us to deb 11

#

there was no changelog lol

#

it was just an upgrade offered from kubernetes 1.21 to 1.22

#

https://kubernetes.io/blog/2021/08/04/kubernetes-1-22-release-announcement/

Kubernetes

Kubernetes 1.22: Reaching New Peaks

Authors: Kubernetes 1.22 Release Team
We’re pleased to announce the release of Kubernetes 1.22, the second release of 2021!
This release consists of 53 enhancements: 13 enhancements have graduated to stable, 24 enhancements are moving to beta, and 16 enhancements are entering alpha. Also, three features have been deprecated.
In April of this yea...

#

which wasn't a release that contained anything major

#

added support for swap memory which was nice

#

but we don't use that lol

molten bough Dec 21, 2021, 11:29 PM

#

Wonder how annoying I'd be if I asked for voice channels and minutes in metricity as well yeef

#

Also we.. Have a discord experiment that might mean metricity is inaccurate right now, depending on how d.py works

#

Yeah I can see this server doesn't have it

thorny obsidian Dec 21, 2021, 11:35 PM

#

@summer garden question! When I give epoch something like .epoch 2 weeks, this is part of the output: "Could not parse the following token(s): "

What exactly does that mean? I'm not sure what the ignored_tokens section of code is doing

patent pivot Dec 21, 2021, 11:37 PM

#

wait

#

lol

#

we'll probably add support once it's GA

molten bough Dec 21, 2021, 11:37 PM

#

I just sent a DM to Scaleios because discord doesn't want us talking about it outside of the servers it's on

#

Haha

patent pivot Dec 21, 2021, 11:38 PM

#

need to either patch it in or switch to a fork that supports it

#

oh lol

molten bough Dec 21, 2021, 11:38 PM

#

Smh Joe lmao

patent pivot Dec 21, 2021, 11:38 PM

#

I didn't even realise they'd said anything, I heard it through unofficial sources lol

molten bough Dec 21, 2021, 11:38 PM

#

But yeah we are testing that

vale ibex Dec 21, 2021, 11:38 PM

#

I mean it's already data mined lol

patent pivot Dec 21, 2021, 11:38 PM

#

yea

vale ibex Dec 21, 2021, 11:38 PM

#

I found out about it via reddit

patent pivot Dec 21, 2021, 11:39 PM

#

either way we'll support it when it's GA

#

and we have a way to support it

molten bough Dec 21, 2021, 11:39 PM

#

It doesn't seem to actually be breaking d.py

patent pivot Dec 21, 2021, 11:39 PM

#

it won't

molten bough Dec 21, 2021, 11:39 PM

#

But it's literally just.. You get message events for voice channels now

#

That's it

patent pivot Dec 21, 2021, 11:39 PM

#

yeah

patent pivot Dec 21, 2021, 11:40 PM

#

molten bough Wonder how annoying I'd be if I asked for voice channels and minutes in metricit...

as for this i can't think of a super nice way to track this

#

i guess track start and end of every session a user spends in voice

#

as a row

molten bough Dec 21, 2021, 11:40 PM

#

That's about all I can think of too

#

But then I'm not sure how you'd calculate based on join/leave on metabase

patent pivot Dec 21, 2021, 11:40 PM

#

would have to be native sql

molten bough Dec 21, 2021, 11:40 PM

#

Sadge

fallen patrol Dec 21, 2021, 11:41 PM

#

gonna leave this here

#

bot#2025 needs one review

dusky shoreBOT Dec 21, 2021, 11:41 PM

#

GitHub

PROpen fix: pass required argument to docs cache delete

fallen patrol Dec 21, 2021, 11:42 PM

#

it's a tiny pr

#

it's now merged, thanks

summer garden Dec 22, 2021, 12:18 AM

#

thorny obsidian <@!348828427333140482> question! When I give epoch something like `.epoch 2 week...

Sorry I would have answered sooner but I was baking pizza. A token would just be like a word. They have that phrasing in the dateutil docs so I kept it

Here's some more info I found in the docs just now

... breaks the time string into lexical units (tokens), which
        can be parsed by the parser. Lexical units are demarcated by changes in
        the character set, so any continuous string of letters is considered
        one unit, any continuous string of numbers is considered one unit.

#

thanks for the review, i'll look over it now

#

How do you feel about the wording of The resulting date and time is: ...? I feel there's a better word in place of "resulting"

patent pivot Dec 22, 2021, 12:22 AM

#

let's see if snekbox works in podman huh

thorny obsidian Dec 22, 2021, 12:23 AM

#

summer garden How do you feel about the wording of `The resulting date and time is: ...`? I fe...

I think that's fine, it could maybe be "Date and time parsed as: ..."

summer garden Dec 22, 2021, 12:23 AM

#

ah I think that's much better

austere hornet Dec 22, 2021, 1:11 AM

#

stable mountain

!remind 15H remind arl about this

stable mountainBOT Dec 22, 2021, 1:11 AM

#

You got it!

Your reminder will arrive on <t:1640189502:F>!

clever wraith Dec 22, 2021, 8:57 AM

#

patent pivot non-zero chance we caused the linode block storage outage in frankfurt today <:m...

I will send a message to work saying "if anything goes down today, blame python discord"

clever wraith Dec 22, 2021, 9:25 AM

#

Sadge

short snow Dec 22, 2021, 1:47 PM

#

Would appreciate reviews on sir-lancebot#930 and sir-lancebot#745

dusky shoreBOT Dec 22, 2021, 1:47 PM

#

GitHub

PROpen Global Leaderboard
PROpen Add logic for preserving unloaded Cogs.

fallen patrol Dec 22, 2021, 2:04 PM

#

hmmm not sure i agree this is better

#

my biggest concern is that they aren't fully aligned

austere hornet Dec 22, 2021, 2:22 PM

#

short snow Would appreciate reviews on sir-lancebot#930 and sir-lancebot#745

Also sir-lancebot#901 please (needs a core dev review!!)

dusky shoreBOT Dec 22, 2021, 2:22 PM

#

GitHub

PROpen Madlibs

short snow Dec 22, 2021, 2:23 PM

#

fallen patrol my biggest concern is that they aren't fully aligned

that's the reason i am not really liking it, me and chris iterated over it many times but we can't really get it alligned

green oriole Dec 22, 2021, 3:05 PM

#

You physically can't align them

#

The solution would be to wrap the number in an inline codeblock

#

That way each number takes the same width

solemn kayak Dec 22, 2021, 3:09 PM

#

.wa seems to be broken for certain inputs

#

.wa 1-tanh^2(x)

#

I’m guessing it’s the ones which take too long to compute

dusky shoreBOT Dec 22, 2021, 3:10 PM

#

Wolfram Alpha

Wolfram API key is invalid or missing.

green oriole Dec 22, 2021, 3:10 PM

#

.wa what is wolfram

#

... Thanks

cold island Dec 22, 2021, 3:10 PM

#

.wa short 1+1

dusky shoreBOT Dec 22, 2021, 3:10 PM

#

Wolfram Alpha

2

green oriole Dec 22, 2021, 3:10 PM

#

How weird

#

.source wa

#

...

cold island Dec 22, 2021, 3:11 PM

#

.wa short 1-tanh^2(x)

dusky shoreBOT Dec 22, 2021, 3:11 PM

#

Wolfram Alpha

1 - tanh^2(x)

green oriole Dec 22, 2021, 3:11 PM

#

I really wonder why it would be 403-ing

dim pelican Dec 22, 2021, 3:11 PM

#

The wolfram response for the floor is made of floor

green oriole Dec 22, 2021, 3:11 PM

#

Oooooorr we hit the rate limit

#

Which seems quite likely I guess

cold island Dec 22, 2021, 3:11 PM

#

.wa x^2

dusky shoreBOT Dec 22, 2021, 3:12 PM

#

Wolfram Alpha

cold island Dec 22, 2021, 3:12 PM

#

.wa 1-tanh^2(x)

dusky shoreBOT Dec 22, 2021, 3:12 PM

#

Wolfram Alpha

Wolfram API key is invalid or missing.

fallen patrol Dec 22, 2021, 3:13 PM

#

green oriole Oooooorr we hit the rate limit

yeah, there's also a pretty strict ratelimit on non-staff, plus a per guild ratelimit

cold island Dec 22, 2021, 3:13 PM

#

.wa 1-(tanh(x))^2

fallen patrol Dec 22, 2021, 3:13 PM

#

also staff aren't subject to that ratelimit ;-;

#

so like, every time zig runs, he's running it regardless of the ratelimit

cold island Dec 22, 2021, 3:13 PM

#

It's still failing so that's not it

dusky shoreBOT Dec 22, 2021, 3:13 PM

#

Wolfram Alpha

Wolfram API key is invalid or missing.

cold island Dec 22, 2021, 3:14 PM

#

I wonder if there are types of API keys or something

green oriole Dec 22, 2021, 3:16 PM

#

They don't document what rate limits will return

#

So for all we know 403 may be a rate limited exceeded error

#

We should add logging there

cursive relic Dec 22, 2021, 3:17 PM

#

what is the member_arg in https://github.com/python-discord/bot/blob/1302632c39469a124e85e6b43c5278526874ce5e/bot/decorators.py#L177
I did not understand the explanation in docstring

stable mountainBOT Dec 22, 2021, 3:17 PM

#

bot/decorators.py line 177

def respect_role_hierarchy(member_arg: function.Argument) -> t.Callable:```

green oriole Dec 22, 2021, 3:17 PM

#

So if you have a ban command with the first argument being the user, then it should be 1

#

Or target if it is named target for example, which is a better option tbf

cursive relic Dec 22, 2021, 3:18 PM

#

all arguments counted self,ctx?

green oriole Dec 22, 2021, 3:18 PM

#

Good question

#

You should use the name to be honest

cursive relic Dec 22, 2021, 3:19 PM

#

name of argument instead of number?

green oriole Dec 22, 2021, 3:19 PM

#

Yes

cursive relic Dec 22, 2021, 3:20 PM

#

seems like self is not counted, but ctx and so on are

#

and good to know that it works with both, name and position. Thx 😄

stable mountainBOT Dec 22, 2021, 4:11 PM

#

austere hornet !remind 15H remind arl about this

It has arrived!

Here's your reminder: remind arl about this
[Jump back to when you created the reminder](#dev-contrib message)

ember tree Dec 22, 2021, 4:36 PM

#

where i can get code of react-nft-challenge

fallen patrol Dec 22, 2021, 4:41 PM

#

@summer garden why do your commits not link to your github profile?

austere hornet Dec 22, 2021, 4:42 PM

#

stable mountain

@fallen patrol Bump

fallen patrol Dec 22, 2021, 4:46 PM

#

tldr not sure if there's other flaws with that input style, so implement the original concept in a method so it's easy to change how the input is gathered

#

@gritty wind i have Debian 10 on a machine and it likely has v1 cgroup so I think I can test snekbox there

#

wait how did you test on several different oses?

gritty wind Dec 22, 2021, 4:53 PM

#

Span up a ton of VPSes and installed snekbox on each

#

The problem I ran into with deb 10 is the docker install script installed a version of compose that’s refusing to work with snekbox

#

Between testing another OS and fixing compose for this one

#

I made a decision

#

I have no doubt it’ll work on whatever we use

patent pivot Dec 22, 2021, 4:56 PM

#

gritty wind The problem I ran into with deb 10 is the docker install script installed a vers...

what error out of interest

gritty wind Dec 22, 2021, 4:58 PM

#

I think it said it couldn’t recognize the version

#

Something about 1.1 and 1.2 as available options

#

I tried updating the package, but it didn’t do anything

patent pivot Dec 22, 2021, 4:59 PM

#

interesting

gritty wind Dec 22, 2021, 5:03 PM

#

If I get more time, I’ll spin up another VPS and check, but we’ll see

gritty wind Dec 22, 2021, 5:18 PM

#

No more time 🤡

patent pivot Dec 22, 2021, 5:26 PM

#

u r such a troll omg

fallen patrol Dec 22, 2021, 5:54 PM

#

gritty wind No more time 🤡

imagine having time

summer garden Dec 22, 2021, 6:12 PM

#

fallen patrol <@348828427333140482> why do your commits not link to your github profile?

I do commits from the pycharm ui. I guess whatever way they verify accounts with github doesn't count as it being my account

fallen patrol Dec 22, 2021, 6:13 PM

#

my assumption is your configured git email doesn't match what's on your github account

summer garden Dec 22, 2021, 7:03 PM

#

@fallen patrolthanks for your comment on the pr. I don't see how your suggestion would work though

fallen patrol Dec 22, 2021, 7:03 PM

#

which pr?

summer garden Dec 22, 2021, 7:04 PM

#

sir-lancebot#983

dusky shoreBOT Dec 22, 2021, 7:04 PM

#

GitHub

PROpen Epoch command

summer garden Dec 22, 2021, 7:05 PM

#

regarding manually throwing in the ...ago or in...

summer garden Dec 22, 2021, 7:05 PM

#

summer garden <@!717983911824588862>thanks for your comment on the pr. I don't see how your su...

haha sorry this was very vague

#

The problem is that the dateutil parser will accept something like 6 hours as absolute time (take the current time and set the hour to 6), even though it might've been meant as relative

#

There's no practical way to differentiate between the two beforehand

fallen patrol Dec 22, 2021, 7:09 PM

#

ahhh....

summer garden Dec 22, 2021, 7:10 PM

#

I'm not sure I'm making any sense here

fallen patrol Dec 22, 2021, 7:10 PM

#

you are

#

although I don't like that behavior as absolute time, and think that we should do something to get it... not absolute

#

fwiw I have a particular interest in however we solve this problem, as I'm working on a command for bot which may benefit from what we discover here

summer garden Dec 22, 2021, 7:12 PM

#

the only thing I can think of is making sure the date string is "full", as in it includes (year), month, day, (time?)

fallen patrol Dec 22, 2021, 7:12 PM

#

fallen patrol fwiw I have a particular interest in however we solve this problem, as I'm worki...

(bot#1924 for context)

dusky shoreBOT Dec 22, 2021, 7:12 PM

#

GitHub

IssueOpen Allow passing end time to !remind

fallen patrol Dec 22, 2021, 7:12 PM

#

lemme look at the implementation in a little bit

austere hornet Dec 22, 2021, 8:43 PM

#

Quick question: is there a specific rule about games on Sir Lancebot that says you can't make games for it that are going to take ___ amount of time or longer?

patent pivot Dec 22, 2021, 8:48 PM

#

uhhh

#

no specific rule

#

but games that last like

#

upwards of 5-10 minutes are kinda eh

#

we restart the bot enough that it might become quite annoying

austere hornet Dec 22, 2021, 8:48 PM

#

Ah gotcha, thanks!

fallen patrol Dec 22, 2021, 9:00 PM

#

patent pivot upwards of 5-10 minutes are kinda eh

this is why we have redis 🤡

patent pivot Dec 22, 2021, 9:00 PM

#

we are not storing game state in redis

#

please and thank you

#

love, devops

fallen patrol Dec 22, 2021, 9:00 PM

#

that reminds me

#

there's a ttt history command

#

and I was thinking, maybe it'd just make sense to remove that?

#

austere hornet Dec 22, 2021, 9:02 PM

#

fallen patrol

Ohh yeah seems useless

#

For a second I was thinking there is a history of ttt command and was gonna ask why you wanted to remove that lol

fallen patrol Dec 22, 2021, 9:04 PM

#

fallen patrol and I was thinking, maybe it'd just make sense to remove that?

I've already an open pull to move ttt to interactions so it wouldn't be that hard to lump into the same pull, actually would make the current implementation easier

fallen patrol Dec 23, 2021, 12:07 AM

#

@hoary haven fwiw I think the patron pr is waiting on the author, it was mostly complete when I reviewed

#

bot#1917

dusky shoreBOT Dec 23, 2021, 12:08 AM

#

GitHub

PROpen Patreon

hoary haven Dec 23, 2021, 12:08 AM

#

yeah i saw

fallen patrol Dec 23, 2021, 12:09 AM

#

smh I wish there was a trashcan on those embeds because I want my typos gone..

austere hornet Dec 23, 2021, 12:10 AM

#

fallen patrol smh I wish there was a trashcan on those embeds because I want my typos gone..

Yeah same

#

That would be useful

fallen patrol Dec 23, 2021, 12:11 AM

#

it's part of github command enhancements, which I can't remember the number rn

#

but it's waiting on sir-lancebot#778

dusky shoreBOT Dec 23, 2021, 12:12 AM

#

GitHub

PROpen Merge automatic issue linking into the github cog

austere hornet Dec 23, 2021, 12:13 AM

#

dusky shore

Oh wow that PR is old

#

June 29

fallen patrol Dec 23, 2021, 12:55 AM

#

@thorny obsidian so what are your thoughts on lance's error handler vs @stable mountain ? saw your comment and you were talking about having a different behavior for a specific command rather than going to the error handler. They behave differently on the two bots, so should we equalise their behaviour and port the best features of each to the other?

thorny obsidian Dec 23, 2021, 12:57 AM

#

Indifferent? I'm not a core dev.

I don't think the two bots need to match 1:1 for how they implement things. Anything that we do want standardized will be so in bot-core and then it will just be handled. Unless it's a critical bug or something that needs to be fixed, I'm hesitant to port things over from python bot

fallen patrol Dec 23, 2021, 12:57 AM

#

then why display the help for the command vs an error?

#

the error message for lance does show how to use the command

thorny obsidian Dec 23, 2021, 12:58 AM

#

My comment was that what bones has implemented already works fine and I don't see a strong need to change it.

fallen patrol Dec 23, 2021, 1:17 AM

#

ah

fallen patrol Dec 23, 2021, 2:43 AM

#

do lance and bot share the same redis ?

#

hmmm

#

yeah i think they do

fallen patrol Dec 23, 2021, 3:28 AM

#

🥲

#

finally metricity is running

#

ohhhhhhhhhh that's why i had so much trouble

#

...

#

site's docker compose exposes the db on 7777

#

metricity's docker compose puts the db on 5432

obsidian patio Dec 23, 2021, 9:55 AM

#

If anyone’s available, I’d love to get some reviews on sir-lancebot#988. Shouldn’t be too big of a PR

dusky shoreBOT Dec 23, 2021, 9:55 AM

#

GitHub

PROpen Add Twemoji utility command

short snow Dec 23, 2021, 9:57 AM

#

no unicode format?

obsidian patio Dec 23, 2021, 10:00 AM

#

What do you mean?

#

Like \UF29E0? That’s supported

#

The regex I made for parsing it is meant to be pretty generous with what it supports

short snow Dec 23, 2021, 10:25 AM

#

Yeah, I was talking about unicode format in the resultant embed, which isn't present on it now pithink

#

https://user-images.githubusercontent.com/65498475/147164315-786dc91c-554d-4f78-8cc8-967580c79aa6.png

#

also we already have emojis on lance, if its possible to use that instead of emoji

#

!pip emojis

stable mountainBOT Dec 23, 2021, 10:31 AM

#

emojis v0.6.0

Emojis for Python

obsidian patio Dec 23, 2021, 10:32 AM

#

short snow also we already have `emojis` on lance, if its possible to use that instead of `...

I tried it. But it didn’t have too many good features for my use cases

obsidian patio Dec 23, 2021, 10:33 AM

#

short snow Yeah, I was talking about unicode format in the resultant embed, which isn't pre...

The format for the codepoint on the embed is just the one I found the cleanest and simplest

short snow Dec 23, 2021, 10:33 AM

#

!pip emoji

stable mountainBOT Dec 23, 2021, 10:33 AM

#

emoji v1.6.1

Emoji for Python

short snow Dec 23, 2021, 10:33 AM

#

one sec

#

ok cool, but if someone wants the format of 👪 in unicode, that won't be possible right?

static canyon Dec 23, 2021, 10:39 AM

#

obsidian patio If anyone’s available, I’d love to get some reviews on sir-lancebot#988. Shouldn...

Fyi since a spring-cleanup-like PR on lancebot we no longer use typing.List

#

It's list[xxx] now

#

Same for tuple, dict, etc.

obsidian patio Dec 23, 2021, 10:49 AM

#

short snow ok cool, but if someone wants the format of 👪 in unicode, that won't be possibl...

The Unicode codepoint is right there? I’ve never really used the \uXXXXX format in design—only codepoints—so I don’t think that format would make sense

obsidian patio Dec 23, 2021, 10:50 AM

#

static canyon It's `list[xxx]` now

Ah, neat! Could you leave a review comment and I’ll fix it

short snow Dec 23, 2021, 10:52 AM

#

I saw unicode mentioned in the original issue hence pointed it out, 👍

obsidian patio Dec 23, 2021, 10:53 AM

#

For input, yes

static canyon Dec 23, 2021, 11:06 AM

#

obsidian patio Ah, neat! Could you leave a review comment and I’ll fix it

Done 👍

#

I may be able to do a full review at a latter date, not sure

obsidian patio Dec 23, 2021, 11:06 AM

#

Thanks, Tizzy

austere hornet Dec 23, 2021, 12:11 PM

#

static canyon Fyi since a spring-cleanup-like PR on lancebot we no longer use typing.List

That was one opened by Kat (restructuring Sir Lancebot), right?

static canyon Dec 23, 2021, 12:13 PM

#

austere hornet That was one opened by Kat (restructuring Sir Lancebot), right?

Nope

#

It was something else

austere hornet Dec 23, 2021, 12:13 PM

#

static canyon It was something else

Oh ok

static canyon Dec 23, 2021, 12:14 PM

#

It was sir-lancebot#802 @austere hornet @obsidian patio

dusky shoreBOT Dec 23, 2021, 12:14 PM

#

GitHub

PRMerged Fix type annotations

austere hornet Dec 23, 2021, 12:14 PM

#

Ah ok thanks!

#

Who is ventaquil on Github btw? Like what Discord account corresponds to that Github user?

static canyon Dec 23, 2021, 12:16 PM

#

I might know but not sure so won't say in case it's wrong lemon_sweat

austere hornet Dec 23, 2021, 12:18 PM

#

Oh ok no worries

gritty wind Dec 23, 2021, 12:23 PM

#

MorowyKomandos

austere hornet Dec 23, 2021, 12:30 PM

#

Ahh thanks!

cold island Dec 23, 2021, 2:38 PM

#

@obtuse arrow I left a review on bot#1940, not sure if you saw

dusky shoreBOT Dec 23, 2021, 2:38 PM

#

GitHub

PROpen Sync Help Cooldown role with help channel claimants

frigid latch Dec 23, 2021, 4:53 PM

#

sorry i wasn't clear i wanted to know if there is smth i can contribute to the site

#

ping me next time when replying please

gritty wind Dec 23, 2021, 4:53 PM

#

frigid latch ping me next time when replying please

Have a look at our issue tracker, it’s where we put all tasks

frigid latch Dec 23, 2021, 4:54 PM

#

gritty wind Have a look at our issue tracker, it’s where we put all tasks

may i know how to see that?

gritty wind Dec 23, 2021, 4:54 PM

#

The link is in the embed

#

https://github.com/python-discord/site/issues

#

Make sure to see the contributing guide too, which is on our website

https://www.pythondiscord.com/pages/guides/pydis-guides/contributing/site/

frigid latch Dec 23, 2021, 4:55 PM

#

thanks

hoary haven Dec 23, 2021, 6:38 PM

#

is there a way to get my editor to render these images? working on markdown files for the site

#

#

![Available help channels](/static/images/content/help_channels/available_channels.png)

fallen patrol Dec 23, 2021, 6:39 PM

#

hoary haven is there a way to get my editor to render these images? working on markdown file...

which editor?

hoary haven Dec 23, 2021, 6:39 PM

#

vs code

brazen charm Dec 23, 2021, 6:39 PM

#

could temporarily add https://www.pythondiscord.com before the static

hoary haven Dec 23, 2021, 6:40 PM

#

sure i'll do that as a last resort lol

fallen patrol Dec 23, 2021, 6:40 PM

#

I have this extension for the previews:
Name: Markdown All in One
Id: yzhang.markdown-all-in-one
Description: All you need to write Markdown (keyboard shortcuts, table of contents, auto preview and more)
Version: 3.4.0
Publisher: Yu Zhang
VS Marketplace Link: https://marketplace.visualstudio.com/items?itemName=yzhang.markdown-all-in-one

Markdown All in One - Visual Studio Marketplace

Extension for Visual Studio Code - All you need to write Markdown (keyboard shortcuts, table of contents, auto preview and more)

hoary haven Dec 23, 2021, 6:40 PM

#

i've not done any setup btw, just cloned the site repo and started editing

#

yeah i installed that extension

fallen patrol Dec 23, 2021, 6:41 PM

#

lemme open my site env

hoary haven Dec 23, 2021, 6:41 PM

#

hoary haven

this is the preview provided by ctrl shift v

brazen charm Dec 23, 2021, 6:41 PM

#

or I guess you could try making it a file URL, don't think anything will be able to resolve it without changes though

fallen patrol Dec 23, 2021, 6:42 PM

#

which file is that?

hoary haven Dec 23, 2021, 6:42 PM

#

pydis_site/apps/content/resources/guides/pydis-guides/help-channel-guide.md

fallen patrol Dec 23, 2021, 6:42 PM

#

trying to find it in site's structure but not familiar with it

#

oh my god that's so long lmfao

#

oh.

#

odd.... the preview does not show up on my end either

hoary haven Dec 23, 2021, 6:44 PM

#

yeah i'll go with numerlor's option haha. easier to "read" when all the images show up in preview

fallen patrol Dec 23, 2021, 6:45 PM

#

sorry there's not a better solution

#

wait--

#

you could run the site, and look at the live preview in browser

#

docker-compose up & then open 127.0.0.1:8000

hoary haven Dec 23, 2021, 6:46 PM

#

i have no experience with docker and won't be able to install it on this work machine anyway. one day though. thanks BlushFluent

gritty wind Dec 23, 2021, 7:40 PM

#

If you have access to pip and python

#

You can get the base requirements for docs by hand

#

And generate a static preview

#

Or if your PR you’ll automatically get a preview from actions

hoary haven Dec 23, 2021, 7:47 PM

#

i do have pip and python

gritty wind Dec 23, 2021, 7:52 PM

#

Try “pip install poetry && poetry install” in the site repo

#

That’ll give you everything you need

#

If you run “poetry run task static”, it’ll do all you need

hoary haven Dec 23, 2021, 7:57 PM

#

i am so afraid of the word poetry. will let you know how it goes

gritty wind Dec 23, 2021, 8:07 PM

#

You can install things by hand too

#

Without poetry

hoary haven Dec 23, 2021, 8:07 PM

#

i don't have the latest pwsh so i replaced && with ; and it worked it seems

#

made the virtualenv in my appdata/local dir, can't make it in the project dir i think i've heard with poetry.. is that right?

gritty wind Dec 23, 2021, 8:08 PM

#

It should be fine actually

#

Though poetry can create the env for you

#

If you want to use your own, make sure to activate it before running poetry install

hoary haven Dec 23, 2021, 8:10 PM

#

ok think i got it

outer oasis Dec 23, 2021, 8:10 PM

#

hoary haven made the virtualenv in my `appdata/local` dir, can't make it in the project dir ...

Poetry creates virtual environments in appdata by default

There's a config option to change it

If you create your virtual environment first, Poetry can detect it

brazen charm Dec 23, 2021, 8:11 PM

#

doesn't it use %userprofile%/.virtualenvs by default?

gritty wind Dec 23, 2021, 8:12 PM

#

App data for some reason /shrug

fallen patrol Dec 23, 2021, 8:33 PM

#

hoary haven i am so afraid of the word `poetry`. will let you know how it goes

you aren't an English teacher, you shouldn't be afraid of it smh /s

Edit: I actually have no idea if you are an English teacher or not so ignore this if you are

austere hornet Dec 23, 2021, 8:49 PM

#

fallen patrol you aren't an English teacher, you shouldn't be afraid of it smh /s Edit: I act...

Haha

cold island Dec 23, 2021, 10:31 PM

#

!otn a mina the english teacher

stable mountainBOT Dec 23, 2021, 10:31 PM

#

:ok_hand: Added mina-the-english-teacher to the names list.

austere hornet Dec 23, 2021, 10:41 PM

#

Good one

hoary haven Dec 23, 2021, 10:42 PM

#

yuck english dv_pandaSadOwO

stable mountainBOT Dec 23, 2021, 10:59 PM

#

It has arrived!

Here's your reminder: here we go again smh
[Jump back to when you created the reminder](#dev-contrib message)

austere hornet Dec 23, 2021, 11:57 PM

#

@fallen patrol @summer garden Have you seen my latest comments on sir-lancebot#985?

dusky shoreBOT Dec 23, 2021, 11:57 PM

#

GitHub

IssueOpen Sudoku command

summer garden Dec 23, 2021, 11:58 PM

#

I don't think the issue with a 9x9 board is taking up the channel. My main concern was how long the game would last and also the tediousness of inputting numbers by row and column

#

I just don't think it's well suited to be played in discord

#

Sorry I should have replied sooner. It slipped my mind after I read the pr comment

#

And I don't necessarily agree with immediately telling the player if the number they inputted is right or wrong. I feel like that spoils it to some extent

#

It would make people more likely to guess, rather than exercise logic

austere hornet Dec 24, 2021, 12:03 AM

#

summer garden I don't think the issue with a 9x9 board is taking up the channel. My main conce...

Oh I see. I mean, I get that, but as I proposed here: https://github.com/python-discord/sir-lancebot/issues/985#issuecomment-998423458, I could do that to make it less tedious. But tbh I don't see how it's extremely tedious to input 3 numbers. Is it because you have to do it many times throughout the game? Is that what you're saying?

austere hornet Dec 24, 2021, 12:04 AM

#

summer garden And I don't necessarily agree with immediately telling the player if the number ...

I don't necessarily agree with that because just saying the input is right or wrong doesn't actually give it away. At least, I don't really see how it does.

fallen patrol Dec 24, 2021, 12:04 AM

#

summer garden I just don't think it's well suited to be played in discord

(this has been addressed) #dev-contrib message

austere hornet Dec 24, 2021, 12:04 AM

#

fallen patrol (this has been addressed) https://canary.discord.com/channels/267624335836053506...

Oh yeah forgot about that, thanks

summer garden Dec 24, 2021, 12:07 AM

#

austere hornet I don't necessarily agree with that because just saying the input is right or wr...

Often in sudoku there are two possible spots for a number in a given row or column or whatever. The goal is to deduct which one of those squares is correct. If you verified if the person's input is right or wrong it shifts the game towards guessing rather than logical reasoning

austere hornet Dec 24, 2021, 12:08 AM

#

summer garden Often in sudoku there are two possible spots for a number in a given row or colu...

Really? I thought you can't repeat any numbers in a given row or column

#

Unless I'm misunderstanding

summer garden Dec 24, 2021, 12:08 AM

#

*two or more possible spots, of which only one is correct

hoary haven Dec 24, 2021, 12:08 AM

#

you can't, but it spoils it if you're told whether your move is correct or not each time you make a move

tawdry vapor Dec 24, 2021, 12:08 AM

#

There could be multiple valid options, but later when more of the puzzle it solved it turns out that your choice was incorrect

austere hornet Dec 24, 2021, 12:10 AM

#

hoary haven you can't, but it spoils it if you're told whether your move is correct or not e...

Hmm. I'm still not quite understanding why.

hoary haven Dec 24, 2021, 12:11 AM

#

it's the whole part of the puzzle and the fun! especially on a reduced board size, you shouldn't spoil it imo

tawdry vapor Dec 24, 2021, 12:11 AM

#

It's part of the challenge to know ahead of time which choice is correct, without having the rest of the cells filled out already to show you.

fallen patrol Dec 24, 2021, 12:11 AM

#

^

#

originally, sudoku (and still now) was/is played on paper, which never had checking lol

hoary haven Dec 24, 2021, 12:12 AM

#

btw, have you thought of how you're going to allow for users to edit their entries? (change or remove)

austere hornet Dec 24, 2021, 12:13 AM

#

hoary haven btw, have you thought of how you're going to allow for users to edit their entri...

Oooooh. I didn't think about that. That's a great idea. If I do end up doing this, I'll definitely need that

fallen patrol Dec 24, 2021, 12:13 AM

#

presumably just use the same input method as the initial input

#

oh wait

#

hmmm

austere hornet Dec 24, 2021, 12:14 AM

#

fallen patrol presumably just use the same input method as the initial input

But wouldn't that just be like entering a new guess?

summer garden Dec 24, 2021, 12:15 AM

#

I propose that either columns be named alphabetically and rows numerically or vice versa. This would remove some confusion in placing your number in the wrong spot. For example .sudoku B6 8. Removing a square might be something like .sudoku B6 X

#

I don't know if other people do this, but I always get my axes mixed up 🙄

austere hornet Dec 24, 2021, 12:15 AM

#

summer garden I propose that either columns be named alphabetically and rows numerically or vi...

Ooooh yeah that sounds a lot better

austere hornet Dec 24, 2021, 12:16 AM

#

summer garden I don't know if other people do this, but I always get my axes mixed up 🙄

Heh, I don't think it matters what axes is labeled with letters and which one with numbers afaik

#

I think Y axes should be labeled with letters and X with numbers (it's like chess if you know how to play that)

summer garden Dec 24, 2021, 12:18 AM

#

I still think 9x9 is rather large. 4x4 is admittedly very easy. If we were to meet in the middle, that would be 6x6. The only thing about that is that the grids are rectangular not square

austere hornet Dec 24, 2021, 12:20 AM

#

summer garden I still think 9x9 is rather large. 4x4 is admittedly very easy. If we were to me...

I mean, I think 6x6 could still work

08cdd9a5cfd3532c6be9280c77939772--sudoku-puzzles-critical-thinking.png

austere hornet Dec 24, 2021, 12:21 AM

#

summer garden I propose that either columns be named alphabetically and rows numerically or vi...

Btw, you don't want to use .sudoku, you just run that once, then just enter B6 8 or B6 X. I think it's better that way. What do you think?

#

Btw, the 8 was just an example of a number for the board, right? Just checking

summer garden Dec 24, 2021, 12:24 AM

#

yeah that's right

hoary haven Dec 24, 2021, 12:28 AM

#

also, this makes sense to do as a text based game imo.. not something with pillow

summer garden Dec 24, 2021, 12:28 AM

#

I know we're quite deep into the sudoku rabbit hole, but another alternative is kenken - a similar "grid" game. The rules are obviously different, but it would allow us to play on a smaller board, (as small as 3x3) without it becoming too easy. The rules are here if you want to give it a quick read http://www.kenkenpuzzle.com/howto/solve

austere hornet Dec 24, 2021, 12:28 AM

#

hoary haven also, this makes sense to do as a text based game imo.. not something with pillo...

Yeah I don't really understand how Pillow works anyway so I wasn't planning to use it

summer garden Dec 24, 2021, 12:29 AM

#

if not, I'm still down for a 6x6 sudoku

austere hornet Dec 24, 2021, 12:30 AM

#

summer garden I know we're quite deep into the sudoku rabbit hole, but another alternative is ...

Oh my I know that game I've played in school before lol. It's pretty cool. I could probably do both (not at the same time ofc) but yeah kenkens are awesome. I'll write up an issue

summer garden Dec 24, 2021, 12:31 AM

#

I was trying to find the name of this back when this sudoku idea first came up. Apparently it's huge in japan - the new yorker has a nice mini documentary about it

austere hornet Dec 24, 2021, 12:32 AM

#

summer garden I was trying to find the name of this back when this sudoku idea first came up. ...

Ah I see yeah I think I knew that when I first learned about kenkens in school. Regardless if no objections I could start with the 6x6 sudoku and then do the KenKen

austere hornet Dec 24, 2021, 12:33 AM

#

summer garden I know we're quite deep into the sudoku rabbit hole, but another alternative is ...

.bm

summer garden Dec 24, 2021, 12:33 AM

#

The only issue would be the formatting into text, as kenken requires the little subscript "objective" in the corner of each box

austere hornet Dec 24, 2021, 12:34 AM

#

summer garden The only issue would be the formatting into text, as kenken requires the little ...

Oh yeah. Hmm. I'll think about that more and if you come up with anything feel free to ping.

austere hornet Dec 24, 2021, 1:29 AM

#

Quick question: is this the code for the eval command on @stable mountain? https://github.com/python-discord/bot/blob/main/bot/exts/utils/snekbox.py

brisk brook Dec 24, 2021, 1:29 AM

#

Yeah that's the code that interacts with Snekbox

austere hornet Dec 24, 2021, 1:53 AM

#

Does anyone know where I could find the code on Snekbox that adds a trashcan reaction to the eval message from the bot that allows the command invoker to delete that message?

#

I'm referring to this:

brazen charm Dec 24, 2021, 1:58 AM

#

https://github.com/python-discord/bot/blob/94b4ad61c203e5b8b33ace2ef639a9fb2fdea4e4/bot/exts/utils/snekbox.py#L222

stable mountainBOT Dec 24, 2021, 1:58 AM

#

bot/exts/utils/snekbox.py line 222

scheduling.create_task(wait_for_deletion(response, (ctx.author.id,)), event_loop=self.bot.loop)```

austere hornet Dec 24, 2021, 2:00 AM

#

brazen charm https://github.com/python-discord/bot/blob/94b4ad61c203e5b8b33ace2ef639a9fb2fdea...

Ah thanks

austere hornet Dec 24, 2021, 2:34 AM

#

austere hornet Does anyone know where I could find the code on Snekbox that adds a trashcan rea...

Question: is this entire block of code relevant to this? https://github.com/python-discord/bot/blob/94b4ad61c203e5b8b33ace2ef639a9fb2fdea4e4/bot/exts/utils/snekbox.py#L214-L222

stable mountainBOT Dec 24, 2021, 2:34 AM

#

bot/exts/utils/snekbox.py lines 214 to 222

filter_cog = self.bot.get_cog("Filtering")
filter_triggered = False
if filter_cog:
    filter_triggered = await filter_cog.filter_eval(msg, ctx.message)
if filter_triggered:
    response = await ctx.send("Attempt to circumvent filter detected. Moderator team has been alerted.")
else:
    response = await ctx.send(msg)
scheduling.create_task(wait_for_deletion(response, (ctx.author.id,)), event_loop=self.bot.loop)```

fallen patrol Dec 24, 2021, 2:39 AM

#

austere hornet Question: is this entire block of code relevant to this? https://github.com/pyth...

no

austere hornet Dec 24, 2021, 2:40 AM

#

This isn't the right channel for that question, I think that question goes in #career-advice

fallen patrol Dec 24, 2021, 2:40 AM

#

the important thing is actually the source of wait_for_deletion

#

what are you trying to do?

austere hornet Dec 24, 2021, 2:42 AM

#

fallen patrol what are you trying to do?

So if you recall a few days ago, you made a typo when you were trying to use the GitHub Issue/PR Link command and you kept typing the wrong number or something I think and that gave me the idea of having a trashcan feature just like the eval command. I'm in the process of writing an issue now

#

I was just trying to see how I should implement it

fallen patrol Dec 24, 2021, 2:42 AM

#

sir-lancebot#968

dusky shoreBOT Dec 24, 2021, 2:42 AM

#

GitHub

IssueOpen github command enhancements

austere hornet Dec 24, 2021, 2:42 AM

#

Ohh

fallen patrol Dec 24, 2021, 2:42 AM

#

i have a slew of enhancements to make to that command once sir-lancebot#778 is merged

dusky shoreBOT Dec 24, 2021, 2:42 AM

#

GitHub

PROpen Merge automatic issue linking into the github cog

austere hornet Dec 24, 2021, 2:43 AM

#

Ahh gotcha

#

Ok nvm then

fallen patrol Dec 24, 2021, 2:43 AM

#

but yeah, you'd just copy https://github.com/python-discord/bot/blob/94b4ad61c203e5b8b33ace2ef639a9fb2fdea4e4/bot/utils/messages.py#L60-L105 to the corresponding file in lance

austere hornet Dec 24, 2021, 2:44 AM

#

fallen patrol but yeah, you'd just copy https://github.com/python-discord/bot/blob/94b4ad61c20...

.bm just in case

austere hornet Dec 24, 2021, 2:44 AM

#

fallen patrol but yeah, you'd just copy https://github.com/python-discord/bot/blob/94b4ad61c20...

I see, thanks

fallen patrol Dec 24, 2021, 2:44 AM

#

austere hornet .bm just in case

I mean, if 778 never gets merged....

austere hornet Dec 24, 2021, 2:44 AM

#

fallen patrol *I mean, if 778 never gets merged....*

Heh

fallen patrol Dec 24, 2021, 3:29 AM

#

@austere hornet now that we've bikeshedded that much...

To avoid spam of the "Hint" button, I'm going to record the last time the button is pressed using the built-in time module in Python and count 3 minutes from that time and after that time elapses, allow the user to click that button again. Then the process repeats.

you can edit the message and view to enable/disable the buttons btw

#

!d discord.ui.Button.disabled

stable mountainBOT Dec 24, 2021, 3:30 AM

#

discord.ui.Button.disabled


property disabled: bool```
Whether the button is disabled or not.

austere hornet Dec 24, 2021, 4:30 AM

#

fallen patrol <@853309207922212965> now that we've bikeshedded that much... > To avoid spam of...

.bm

austere hornet Dec 24, 2021, 4:31 AM

#

fallen patrol <@853309207922212965> now that we've bikeshedded that much... > To avoid spam of...

!remind 12H

stable mountainBOT Dec 24, 2021, 4:31 AM

#

Alright.

Your reminder will arrive on <t:1640363494:F>!

gray hound Dec 24, 2021, 5:05 AM

#

Hello!
I found a small typo in the tools section of the website (pythondiscord.com)
I think it is more of a font issue than a typo.
The link says: our met a repo instead of our meta repo.
Screenshot attached.

fallen patrol Dec 24, 2021, 5:06 AM

#

gray hound Hello! I found a small typo in the tools section of the website (pythondiscord.c...

could you link the page?

gray hound Dec 24, 2021, 5:07 AM

#

yeah sure

#

https://www.pythondiscord.com/resources/tools/

tawdry vapor Dec 24, 2021, 5:09 AM

#

Which browser are you using?

fallen patrol Dec 24, 2021, 5:10 AM

#

gray hound Hello! I found a small typo in the tools section of the website (pythondiscord.c...

it is font, since it doesn't happen for me

gray hound Dec 24, 2021, 5:10 AM

#

Alright.
Thanks for looking into it though.

fallen patrol Dec 24, 2021, 5:12 AM

#

gray hound Alright. Thanks for looking into it though.

which browser are you using?

gray hound Dec 24, 2021, 5:12 AM

#

Brave

fallen patrol Dec 24, 2021, 5:13 AM

#

try looking the shields, perhaps they're blocking the font we use from loading

gray hound Dec 24, 2021, 5:13 AM

#

No. Still the same result.

tawdry vapor Dec 24, 2021, 5:13 AM

#

Can you right click on the link and select inspect, then go to the "computed" tab on the right?

#

At the bottom it should then show the name of the rendered font

gray hound Dec 24, 2021, 5:14 AM

#

It's Roboto

#

For me

tawdry vapor Dec 24, 2021, 5:15 AM

#

Alright. That is actually one of the listed fonts in the CSS that it attempts to use. I don't have it on my system though so I can't see what it'd look like for me.

#

One would assume that if it's explicitly requesting that font then it should look okay.

gray hound Dec 24, 2021, 5:19 AM

#

Guess it's alright then.

#

Here's the page's preview I'm getting.

tawdry vapor Dec 24, 2021, 5:28 AM

#

The kerning is all messed up but I don't know why

#

Unsure if it's an issue on your end or on the site's end.

#

It's especially bad for the f and t characters

fallen patrol Dec 24, 2021, 5:32 AM

#

tawdry vapor The kerning is all messed up but I don't know why

keming

gray hound Dec 24, 2021, 5:45 AM

#

If it's my side, how can I know? How to fix?

#

Sorry for late response.

tawdry vapor Dec 24, 2021, 5:50 AM

#

I'm unsure, sorry

gray hound Dec 24, 2021, 6:01 AM

#

It's okay.
Thanks for letting me know about the actual problem, though. I really appreciate it.

frigid latch Dec 24, 2021, 6:05 AM

#

the site requires postgres, or is there a docker setup?

#

coz one without docker looks complex

tawdry vapor Dec 24, 2021, 6:10 AM

#

Yes there is a Docker set up documented in the setup guide.

#

https://www.pythondiscord.com/pages/guides/pydis-guides/contributing/

Python Discord - Contributing

A guide to contributing to our open source projects.

fallen patrol Dec 24, 2021, 6:12 AM

#

frigid latch the site requires postgres, or is there a docker setup?

Tldr docker compose up

frigid latch Dec 24, 2021, 6:18 AM

#

fallen patrol Tldr `docker compose up`

ok thanks

static canyon Dec 24, 2021, 9:23 AM

#

@obsidian patio have left a slightly more in-depth review on sir-lancebot#988

dusky shoreBOT Dec 24, 2021, 9:23 AM

#

GitHub

PROpen Add Twemoji utility command

obsidian patio Dec 24, 2021, 9:50 AM

#

@static canyon thanks!

clever wraith Dec 24, 2021, 1:49 PM

#

Uhm in the form you guys gave, I see infinite loading. is that a bug or something?

#

#

oh i guess that's it.
Anyways, it would be better to have client side error i think?
Steps to regenerate would be just entering the country name and hit enter.

#

also UI is a bit confusing which does not represent that we NEED to enter discord info.
I mean I found out that I NEED to log in after I've seen the error.

patent pivot Dec 24, 2021, 2:18 PM

#

yeah, that's something that we will improve

#

discord data is a flag on the form like "collect data" or "don't collect data", it's not a field in the sense that shipping is and so it's handled a bit differently.

#

the frontend should reject submission until the data is collected, and the backend is already doing that validation (hence why frontend is getting an error from backend)

cedar frigate Dec 24, 2021, 2:57 PM

#

is opening gh issues on the bot repo the preferred way to request something?

magic arch Dec 24, 2021, 2:58 PM

#

cedar frigate is opening gh issues on the `bot` repo the preferred way to request something?

Depends on what you're requesting

cold island Dec 24, 2021, 2:59 PM

#

Yes you can just open an issue. You can also discuss it here first if you want

cedar frigate Dec 24, 2021, 3:11 PM

#

thanks, it isn't fully formed yet lemon_swag

stable mountainBOT Dec 24, 2021, 4:31 PM

#

austere hornet !remind 12H

It has arrived!

Here's your reminder: @austere hornet now that we've bikeshedded that much...

To avoid spam of the "Hint" button, I'm going to record the last time the button is pressed using the built-in time module in Python and count 3 minutes from that time and after that time elapses, allow the user to click that button again. Then the process repeats.

you can edit the message and view to enable/disable the buttons btw
[Jump back to when you created the reminder](#dev-contrib message)

austere hornet Dec 24, 2021, 4:40 PM

#

fallen patrol <@853309207922212965> now that we've bikeshedded that much... > To avoid spam of...

Sorry, I'm a bit confused. Didn't I already say that I would add a feature to the Sudoku game that would allow the player to edit their guess? And yes I figured out the part about enabling/disabling the buttons, thanks

fallen patrol Dec 24, 2021, 4:40 PM

#

hm?

austere hornet Dec 24, 2021, 4:41 PM

#

Well you said "you can edit the message", not sure what you mean by that

austere hornet Dec 24, 2021, 4:42 PM

#

austere hornet Oooooh. I didn't think about that. That's a great idea. If I do end up doing thi...

Remember this?

#

@fallen patrol ^

fallen patrol Dec 24, 2021, 4:53 PM

#

!d discord.Message.edit

stable mountainBOT Dec 24, 2021, 4:53 PM

#

discord.Message.edit


await edit(content=..., embed=..., embeds=..., attachments=..., suppress=..., delete_after=None, allowed_mentions=..., view=...)```
This function is a [*coroutine*](https://docs.python.org/3/library/asyncio-task.html#coroutine).

Edits the message.

The content must be able to be transformed into a string via `str(content)`.

Changed in version 1.3: The `suppress` keyword-only parameter was added.

fallen patrol Dec 24, 2021, 4:53 PM

#

you can edit the message and disable the buttons, that was all

austere hornet Dec 24, 2021, 4:53 PM

#

fallen patrol you can edit the message and disable the buttons, that was all

Oh ok, yeah I know that 🙂 Thanks!

fallen patrol Dec 24, 2021, 4:54 PM

#

cool 👍

#

I'm (mostly) afk from now until the 27th or so

#

@short snow yes you will get your review smh

patent pivot Dec 24, 2021, 5:14 PM

#

clever wraith oh i guess that's it. Anyways, it would be better to have client side error i th...

#

just fixed this

#

now enter won't submit unless you've authorised with discord, if you haven't it'll scroll down until the "discord login" button is visible

#

https://github.com/python-discord/forms-frontend/pull/380

GitHub

Block submission when missing OAuth2 scopes by jb3 · Pull Request #...

This PR blocks form submission until authenticated with the required scopes for the form. If the Enter key shortcut is used to submit early without filling in auth the form is automatically scrolle...

austere hornet Dec 24, 2021, 6:56 PM

#

Hey @patent pivot, question for you. I'm in the process of writing an issue on Lance for another game called Kenken. (This was suggested by bones, see #dev-contrib message). I was DM'ing him just now and he thinks I should make Sudoku and Kenken in one cog. However, I don't think that is a good idea since they are 2 separate games. What do you think?

patent pivot Dec 24, 2021, 7:04 PM

#

should be a separate cog probably, not sure

austere hornet Dec 24, 2021, 7:06 PM

#

patent pivot should be a separate cog probably, not sure

Yeah that's what I'm thinking

hoary haven Dec 24, 2021, 7:17 PM

#

oki i'm home now and would like to try running the site locally
https://www.pythondiscord.com/pages/guides/pydis-guides/contributing/site/ is this comprehensive enough/updated enough to get me going?

Python Discord - Contributing to Site

A guide to setting up and configuring Site.

#

and i'm now on mac, and have pyenv installed

vale ibex Dec 24, 2021, 7:21 PM

#

yup, that should cover it

#

Running on docker is still recommended though

hoary haven Dec 24, 2021, 7:50 PM

#

do you guys just install all pydis dependencies on 1 virtual environment? 1 virtual environment for each repo?

fallen patrol Dec 24, 2021, 7:55 PM

#

hoary haven do you guys just install all pydis dependencies on 1 virtual environment? 1 virt...

1 venv for each repo

#

aside from that's how prod works, it's just good practice to not share venvs between projects

hoary haven Dec 24, 2021, 7:59 PM

#

and poetry is a dependency manager, and doesn't create environments.. correct? so i should use something like pipenv in conjunction with it

fallen patrol Dec 24, 2021, 7:59 PM

#

it does create venvs, actually

vale ibex Dec 24, 2021, 7:59 PM

#

poetry creates virtual environments, yea

fallen patrol Dec 24, 2021, 7:59 PM

#

however I prefer to create my own venv and it automatically uses that

#

python3.9 -m virtualenv .venv

#

poetry install

hoary haven Dec 24, 2021, 8:01 PM

#

mk

fallen patrol Dec 24, 2021, 8:01 PM

#

then I set the local venv in my editor config and never have to think about it again

fallen patrol Dec 24, 2021, 8:01 PM

#

vale ibex poetry creates virtual environments, yea

I've had problems with that so I don't let it do that anymore, personally

vale ibex Dec 24, 2021, 8:01 PM

#

interesting, I haven't ran into any issues with that specifically

#

You seem to hit all the issues 😅

fallen patrol Dec 24, 2021, 8:02 PM

#

yeah pretty much

#

if poetry has a bug I will find it whether I want to or not

#

and I have

#

in other news I really only use poetry for dependency management and pep 517.

hoary haven Dec 24, 2021, 8:03 PM

#

ok i think i installed poetry in multiple places unintentionally but i think i got it lol

fallen patrol Dec 24, 2021, 8:03 PM

#

lol I use pipx

#

so one copy of poetry and that's it :)

#

much better than their installer

brazen charm Dec 24, 2021, 8:04 PM

#

I just did a normal pip install, never had any issues

#

apart from when they broke their cache lookups

fallen patrol Dec 24, 2021, 8:05 PM

#

but that [using pip] is installed to site packages, so by using pipx it's in its own little private corner, er venv

brazen charm Dec 24, 2021, 8:06 PM

#

not like I use the system python for anything beyond poetry and pipenv

hoary haven Dec 24, 2021, 8:07 PM

#

it failed to install Installing psycopg2-binary (2.8.6): Failed

#

https://paste.pythondiscord.com/urutugasem.sql

#

everything else seems to be have installed fine

fallen patrol Dec 24, 2021, 8:09 PM

#

what os

gritty wind Dec 24, 2021, 8:10 PM

#

If you have Postgres locally, that’ll possibly help you work past it, but ideally it should work

vale ibex Dec 24, 2021, 8:10 PM

#

seems like an m1 mac issue

fallen patrol Dec 24, 2021, 8:11 PM

#

ahhh, yeah

#

sigh

#

well

#

given

#

you don't need that

#

okay so first try poetry update and then poetry install again

gritty wind Dec 24, 2021, 8:12 PM

#

Poetry update will fuck the lock file

fallen patrol Dec 24, 2021, 8:12 PM

#

yeah that's the goal

#

if there's an updated version then we can switch to that

gritty wind Dec 24, 2021, 8:12 PM

#

There are much newer versions

#

But given that this is not appearing in the issue tracker

#

I don’t think it’s version specific

#

Have you tired the provided troubleshooting steps in the error

fallen patrol Dec 24, 2021, 8:13 PM

#

!pip psycopg2-binary

stable mountainBOT Dec 24, 2021, 8:13 PM

#

psycopg2-binary v2.9.2

psycopg2 - Python-PostgreSQL Database Adapter

gritty wind Dec 24, 2021, 8:13 PM

#

Actually you know what

#

It doesn’t have m1 images at all

#

You need to build from source

#

And the instructions say building from source requires Pg

outer oasis Dec 24, 2021, 8:14 PM

#

Yep, it will probably be able to build from source if PG is installed

fallen patrol Dec 24, 2021, 8:15 PM

#

so the next best thing is don't install it

gritty wind Dec 24, 2021, 8:15 PM

#

It really isn’t, we haven’t tried to install it

fallen patrol Dec 24, 2021, 8:15 PM

#

IIRC Mina was only trying to view static content

hoary haven Dec 24, 2021, 8:15 PM

#

gritty wind Have you tired the provided troubleshooting steps in the error

no not yet

fallen patrol Dec 24, 2021, 8:15 PM

#

fallen patrol IIRC Mina was only trying to view static content

@hoary haven this still true?

gritty wind Dec 24, 2021, 8:15 PM

#

The thing is arl you still need all imports to succeed

#

Even if doing static builds

#

We still import everything

hoary haven Dec 24, 2021, 8:16 PM

#

erm while working on editing site markdown files i wanted the static images to render basically

fallen patrol Dec 24, 2021, 8:18 PM

#

gritty wind Even if doing static builds

I kinda want to work on this to make it not true lol

gritty wind Dec 24, 2021, 8:19 PM

#

Tbh, I don’t think it’s worth it

#

Saying you need to install project deps to use the project is not really

#

Much of a stretch

vale ibex Dec 24, 2021, 8:20 PM

#

I mean you could just install docker

#

and run it via that

#

you just won't get pre-commit stuff then

fallen patrol Dec 24, 2021, 8:21 PM

#

Mina can't install docker on work machine

gritty wind Dec 24, 2021, 8:22 PM

#

I think she said she’s home

#

Yeah

fallen patrol Dec 24, 2021, 8:22 PM

#

oh

#

lol

#

@hoary haven just docker compose up it then :)

hoary haven Dec 24, 2021, 8:23 PM

#

yeah at home now

#

is the pre-commit stuff the linting stuff though?

vale ibex Dec 24, 2021, 8:23 PM

#

That --platform=linux/amd64 arg I added coming in use lol

fallen patrol Dec 24, 2021, 8:23 PM

#

and if you do want the dev dependencies for pre commit you can export what poetry installs, then delete the postgres dependencies

hoary haven Dec 24, 2021, 8:24 PM

#

bc that's like the one thing i'd like lol

fallen patrol Dec 24, 2021, 8:24 PM

#

poetry export --without-hashes > requirements.txt

gritty wind Dec 24, 2021, 8:24 PM

#

Yeah that’s the linting stuff

fallen patrol Dec 24, 2021, 8:24 PM

#

fallen patrol ```sh poetry export --without-hashes > requirements.txt ```

then delete the postgres dependencies from here

#

finally while in the venv, pip install those dependencies and finally pre-commit install --install-hooks

#

will get you the linting deps :)

fallen patrol Dec 24, 2021, 8:26 PM

#

vale ibex That `--platform=linux/amd64` arg I added coming in use lol

& my issues with poetry too lol

#

I have enough experience with poetry to basically not use it for everything it does 🥲

gritty wind Dec 24, 2021, 8:28 PM

#

Honestly just get windows

vale ibex Dec 24, 2021, 8:28 PM

#

@hoary haven if you follow this answer, it should get pg_config installed https://stackoverflow.com/a/21080707

Stack Overflow

How to find pg_config path

Complete newbie here, trying to set up Django to work with PostgreSQL.

I'm using mac osx 10.6.8. I have also installed PostgreSQL 9.3

When I run pip install psycopg2 in terminal I get the follow...

gritty wind Dec 24, 2021, 8:28 PM

#

Despite what everyone says

#

It works better than everything else

hoary haven Dec 24, 2021, 8:28 PM

#

lolol really?

fallen patrol Dec 24, 2021, 8:28 PM

#

gritty wind It works better than everything else

can confirm, bought windows pro to use docker on windows

vale ibex Dec 24, 2021, 8:28 PM

#

yup, windows is really nice

hoary haven Dec 24, 2021, 8:29 PM

#

gritty wind It works better than everything else

for what, specifically

gritty wind Dec 24, 2021, 8:29 PM

#

Programming

#

I’ve got wsl, and I can’t think of anything I can’t do

#

Nothing breaks either

#

That Microsoft quality

hoary haven Dec 24, 2021, 8:29 PM

#

haha i hope to be proficient at using both since i use both regularly

#

oh i haven't tried wsl though

fallen patrol Dec 24, 2021, 8:29 PM

#

gritty wind I’ve got wsl, and I can’t think of anything I can’t do

sudo poweroff

#

check and mate

gritty wind Dec 24, 2021, 8:30 PM

#

Who turns off their PC wtf

fallen patrol Dec 24, 2021, 8:30 PM

#

afk

gritty wind Dec 24, 2021, 8:30 PM

#

Actually you don’t even need to turn windows off, it’ll turn itself off to update

vale ibex Dec 24, 2021, 8:30 PM

#

not me 🙂

gritty wind Dec 24, 2021, 8:31 PM

#

Perfect lol

hoary haven Dec 24, 2021, 9:01 PM

#

alright. i don't really know what i did, but i basically ignored the psycopg2 stuff and just did docker-compose up and 127.0.0.1:8000 works shrug

#

it told me to use 0.0.0.0:8000 though, which doesn't work

gritty wind Dec 24, 2021, 9:02 PM

#

Docker installs it’s own dependencies, which is why it’s usually the goto option

hoary haven Dec 24, 2021, 9:05 PM

#

so i could have skipped poetry altogether?

vale ibex Dec 24, 2021, 9:06 PM

#

Docker allows you to run the project without running anything

#

however, in-editor code completion and pre-commit linting requires you to have dependancies installed locally

fallen patrol Dec 24, 2021, 9:06 PM

#

hoary haven so i could have skipped poetry altogether?

yesno

#

if you use docker it uses poetry to build the image

hoary haven Dec 24, 2021, 9:07 PM

#

ok

fallen patrol Dec 24, 2021, 9:08 PM

#

but unless you use vscode or can set up your editor to code in a docker container... you would need to install the dependencies in a local venv anyways

#

did you ever get the pre commit dependencies installed?

hoary haven Dec 24, 2021, 9:09 PM

#

poetry run task precommit i did this earlier and it seems to have worked

fallen patrol Dec 24, 2021, 9:09 PM

#

did you install the dependencies?

hoary haven Dec 24, 2021, 9:10 PM

#

poetry install ? yeah i did that first and besides the pysocpg2 everything installed fine

fallen patrol Dec 24, 2021, 9:10 PM

#

hm, did everything else get installed?

#

I kinda doubt it...

hoary haven Dec 24, 2021, 9:10 PM

#

it told me it did lol. how do i check?

fallen patrol Dec 24, 2021, 9:11 PM

#

fallen patrol ```sh poetry export --without-hashes > requirements.txt ```

do this, and compare it to the output of pip freeze

#

any differences, weren't installed

#

your best bet is to just take the output of that poetry export, remove the postgres deps, and pip install that

hoary haven Dec 24, 2021, 9:13 PM

#

https://paste.pythondiscord.com/molokaqaye.yaml this is my poetry show

#

so that's what i'm supposed to have? not what i actually have installed?

#

yeah looks like it

fallen patrol Dec 24, 2021, 9:15 PM

#

yeah, poetry show doesn't show you what is installed AFAIK

hoary haven Dec 24, 2021, 9:18 PM

#

mk yeah not everything was installed. none of the flake8 stuff was

fallen patrol Dec 24, 2021, 9:18 PM

#

the important part ;-;

#

legit you could get away with a global pre-commit and all of the flake8 stuff for linting in the poetry env, nothing else needed

hoary haven Dec 24, 2021, 9:23 PM

#

fallen patrol ```sh poetry export --without-hashes > requirements.txt ```

wait is this supposed to show me what poetry wants to install?
it shows psycopg2-binary but doesn't show flake8.

fallen patrol Dec 24, 2021, 9:24 PM

#

oh f I forgot

fallen patrol Dec 24, 2021, 9:24 PM

#

hoary haven wait is this supposed to show me what poetry *wants* to install? it shows psycop...

add the --dev flag

hoary haven Dec 24, 2021, 9:24 PM

#

when i pip list tbh it looks like i have everything, flake8 stuff and pre-commit

hoary haven Dec 24, 2021, 9:26 PM

#

fallen patrol add the `--dev` flag

ok i see flake8 stuff and pre-commit when i do that

tawdry vapor Dec 24, 2021, 9:28 PM

#

Poetry installs development dependencies by default, unlike pipenv.

#

Not sure why the dev flag made a difference

sleek steppe Dec 24, 2021, 9:37 PM

#

tawdry vapor Not sure why the dev flag made a difference

Not for poetry export appearantly

fallen patrol Dec 24, 2021, 9:38 PM

#

^

fallen patrol Dec 24, 2021, 9:38 PM

#

tawdry vapor Poetry installs development dependencies by default, unlike pipenv.

poetry export by default exports only main dependencies and with hashes

fallen patrol Dec 24, 2021, 11:24 PM

#

@brisk brook you occasionally use the integrated vscode extension to review and leave comments on github, don't you lol

brisk brook Dec 24, 2021, 11:26 PM

#

I tried it but hated it, what makes you think that?

#

Is it something I do that you can do in that?

fallen patrol Dec 24, 2021, 11:28 PM

#

this #dev-log message

it escaped the worker which filters all of these events. I've only known it to escape when I submit a comment on a pull from the extension

#

comment on github? gets caught no problem
comment on the extension? it escapes

brisk brook Dec 24, 2021, 11:29 PM

#

Caught refers to it not being sent yeah?

fallen patrol Dec 24, 2021, 11:30 PM

#

yeah

brisk brook Dec 24, 2021, 11:30 PM

#

What I do is that I go through the whole diff and make comments (sometimes only one, like here). Then in the top right I select "Review changes" and pick "Comment"

fallen patrol Dec 24, 2021, 11:36 PM

#

hmm I wonder if that contributes

brisk brook Dec 25, 2021, 12:11 AM

#

How do I authenticate with the site API?

brazen charm Dec 25, 2021, 12:13 AM

#

authorization token header

brisk brook Dec 25, 2021, 12:14 AM

#

Even when setting Authorization to suitable-for-development-only (copied from the site page after being set up in an .env file) I get: ```json
{
"detail": "Authentication credentials were not provided."
}

brazen charm Dec 25, 2021, 12:16 AM

#

the api key should be badbot something

brisk brook Dec 25, 2021, 12:16 AM

#

brazen charm Dec 25, 2021, 12:16 AM

#

so Authorization: Token badbotsomething

#

if you have the bot working then it's in the .env or env vars

brisk brook Dec 25, 2021, 12:17 AM

#

Ah, no I am just running site standalone

brazen charm Dec 25, 2021, 12:18 AM

#

Just to get the key; it should also be somewhere in the guides

brisk brook Dec 25, 2021, 12:18 AM

#

I am looking through this: https://www.pythondiscord.com/pages/guides/pydis-guides/contributing/site/

Python Discord - Contributing to Site

A guide to setting up and configuring Site.

cold island Dec 25, 2021, 12:18 AM

#

https://www.pythondiscord.com/pages/guides/pydis-guides/contributing/bot/#with-the-bot-running-locally

Python Discord - Contributing to Bot

A guide to setting up and configuring Bot.

#

First bullet

brazen charm Dec 25, 2021, 12:19 AM

#

the suitable-for-development-only is for something different

#

You're also missing the Token auth type

brisk brook Dec 25, 2021, 12:21 AM

#

cold island https://www.pythondiscord.com/pages/guides/pydis-guides/contributing/bot/#with-t...

Ah there you go!

brisk brook Dec 25, 2021, 12:21 AM

#

brazen charm You're also missing the Token auth type

Yes I started using that after you mentioned it a bit above and got "invalid token" instead.

short snow Dec 25, 2021, 2:55 AM

#

fallen patrol <@736234578745884682> yes you will get your review smh

the eyes weren't for that, the eyes were for mostly afk where mostly belongs to some different dictionary

#

also merry Christmas ducky_santa guys

austere hornet Dec 25, 2021, 3:06 AM

#

short snow also merry Christmas <:ducky_santa:655360331002019870> guys

Thanks, you too! (btw I don't really celebrate and also off topic for this channel)

fallen patrol Dec 25, 2021, 4:09 AM

#

~~nextcord may be going zap~~

patent pivot Dec 25, 2021, 9:19 AM

#

?

#

that's not really relevant to this channel

cold island Dec 25, 2021, 8:37 PM

#

Is there a way to get the bot instance without passing it around?

gritty wind Dec 25, 2021, 8:37 PM

#

Import it from bot

#

What are you doing anyways

cold island Dec 25, 2021, 8:38 PM

#

filters

#

Writing the infraction logic

tawdry vapor Dec 25, 2021, 8:39 PM

#

bot.instance

#

@cold island speaking of filters, what do you think about stripping spaces from text before applying the URL regex? Would've prevent the raid the other day, probably.

cold island Dec 25, 2021, 8:43 PM

#

I wasn't around for that raid, what was in it?

austere hornet Dec 25, 2021, 8:45 PM

#

cold island I wasn't around for that raid, what was in it?

I think Mark is talking about the reaction raid

tawdry vapor Dec 25, 2021, 8:45 PM

#

A discord invite link that had random amount of spaces between characters

cold island Dec 25, 2021, 8:46 PM

#

Oh strip in that sense

tawdry vapor Dec 25, 2021, 8:47 PM

#

Of coruse there'd still be other circumventions but might as well prevent this easy one

cold island Dec 25, 2021, 8:47 PM

#

Yeah I'm trying to think if it can cause falses

tawdry vapor Dec 25, 2021, 8:47 PM

#

I think URLs are generally pretty distinct, especially if the scheme is included

#

But if we have a filter just for bad.com then it could be a false positive for imnotbad.com

#

Wait that came out wrong

#

I think it would be like we have a filter for imreallybad.com and it would pick up im really bad.com

#

but less likely to be a false positive if it were written as http:// im really bad.com

#

Don't remember if the raid included a scheme in the links

cold island Dec 25, 2021, 8:50 PM

#

Rather than URL it could be done for the invites filter

#

Would be much safer

tawdry vapor Dec 25, 2021, 8:51 PM

#

Yeah, probably

gritty wind Dec 25, 2021, 8:52 PM

#

Hm don’t we validate scheme too anyway

#

Also mark wow

#

Alright, good to know

cold island Dec 25, 2021, 8:52 PM

#

For general URLs there's a PR (or should be) that moves away from looking for schemes

tawdry vapor Dec 25, 2021, 8:52 PM

#

gritty wind Also mark wow

What?

gritty wind Dec 25, 2021, 8:53 PM

#

I didn’t know you were bad smh

tawdry vapor Dec 25, 2021, 8:53 PM

#

🙄

fallen patrol Dec 25, 2021, 8:56 PM

#

tawdry vapor <@!512354988157173763> speaking of filters, what do you think about stripping sp...

that could work? but it'd need to know when to do it and when it's already been done.

discord.gg/python discord would evaluate to what?
or discord.gg/dpy d f g h would go to what?

cold island Dec 25, 2021, 8:56 PM

#

lol please stop

gritty wind Dec 25, 2021, 8:56 PM

#

What are you posting

static canyon Dec 25, 2021, 8:56 PM

#

Arl pls stop

gritty wind Dec 25, 2021, 8:56 PM

#

I have a feeling you’re pinging the mods

fallen patrol Dec 25, 2021, 8:57 PM

#

can someone repost what I'm saying ;-;

static canyon Dec 25, 2021, 8:57 PM

#

gritty wind I have a feeling you’re pinging the mods

Yep

static canyon Dec 25, 2021, 8:57 PM

#

fallen patrol can someone repost what I'm saying ;-;


since both of those first urls are whitelisted but discord.gg/pythondiscord isn't```

fallen patrol Dec 25, 2021, 8:58 PM

#

also sorry I didn't realise invites mentioned mods ;-; since the bot catches them lol

fallen patrol Dec 25, 2021, 8:58 PM

#

gritty wind I have a feeling you’re pinging the mods

just a feeling, huh?

static canyon Dec 25, 2021, 8:59 PM

#

fallen patrol also sorry I didn't realise invites mentioned mods ;-; since the bot catches the...

Anything the bot "catches" notifies the mods

gritty wind Dec 25, 2021, 8:59 PM

#

I don’t have the mod role if you’ll notice

#

Most things do lol, mostly the point

#

Until zig is done at least

#

One day

fallen patrol Dec 25, 2021, 9:00 PM

#

gritty wind Most things do lol, mostly the point

ik that one filter I was working on didn't ping mods sooo

static canyon Dec 25, 2021, 9:01 PM

#

fallen patrol ik that one filter I was working on didn't ping mods sooo

Whether or not it pings the mod shouldn't matter. If your message keeps getting deleted then don't keep resending it

#

That's probably a good rule to go by

#

(sorry if that comes off overly harsh, can't really tell)

fallen patrol Dec 25, 2021, 9:01 PM

#

ye

#

afk

tawdry vapor Dec 25, 2021, 9:08 PM

#

fallen patrol that could work? but it'd need to know when to do it and when it's already been ...

True. I was thinking about it for URLs in general. Invites are problematic since the domain is fine but the part after is what matters

#

I think maybe what could work for invites is if the text is first searched as-is, and if nothing is found, strip spaces and search again. On this second try, if something that looks like an invite is found, but can't be parsed further, err on the side of caution and remove it.

fallen patrol Dec 25, 2021, 9:25 PM

#

however, what about with the above vs the above?

#

(the ones i can't send)

tawdry vapor Dec 25, 2021, 9:37 PM

#

It would evaluate the .gg/python discord as .gg/python, see that it's a valid invite, and stop.

#

If it evaluated .gg/python discord and .gg/python was not allowed/valid, then it would strip spaces, check again, detect .gg/pythondiscord, see that's not whitelisted, and remove it.

hoary haven Dec 25, 2021, 9:39 PM

#

if you want a sample of one of the raid messages from yesterday: d is cor d. g g/h D2 Cm Jh tzj JOIN NOW! | erk7d

fallen patrol Dec 25, 2021, 9:50 PM

#

hoary haven if you want a sample of one of the raid messages from yesterday: `d is cor d. g ...

what's this part?

hoary haven Dec 25, 2021, 9:51 PM

#

that part differed in each message, to avoid filters looking for duplicate messages i presume

fallen patrol Dec 25, 2021, 9:51 PM

#

ahhh

cold island Dec 25, 2021, 9:59 PM

#

tawdry vapor If it evaluated `.gg/python discord` and `.gg/python` was not allowed/valid, the...

If .gg/python is already not allowed, then I don't think it's necessary to look further. The main improvement here seems to be joining the parts before the slash

tawdry vapor Dec 25, 2021, 10:01 PM

#

You're right, i was overthinking it

fallen patrol Dec 25, 2021, 10:04 PM

#

cold island If `.gg/python` is already not allowed, then I don't think it's necessary to loo...

I think it was an example

#

perhaps a better one would be dpy or ~3 characters

cold island Dec 25, 2021, 10:27 PM

#

fallen patrol I think it was an example

yes, it's an example

#

Okay, 6 hours of writing filters is enough, time to sleep

vale ibex Dec 25, 2021, 10:35 PM

#

So I realised I never got around to writing this for whatever reason, so I've spent the last two hours or so getting a dev env setup on my laptop lol

#

sir-lancebot#991

dusky shoreBOT Dec 25, 2021, 10:35 PM

#

GitHub

PROpen Aoc completer role

austere hornet Dec 25, 2021, 10:37 PM

#

Quick question about the Github Issue/PR Link feature on Lance. Was there any reason @dusky shore was chosen over @stable mountain for this feature? I feel like this feature would be better on @stable mountain .

gritty wind Dec 25, 2021, 10:38 PM

#

These sort of fun feature usually go on lance, while the more crucial features go on python

austere hornet Dec 25, 2021, 10:38 PM

#

gritty wind These sort of fun feature usually go on lance, while the more crucial features g...

Ah I see

gritty wind Dec 25, 2021, 10:39 PM

#

It helps keep python more stable while reducing the review requirement on non-critical stuff

#

But there are exceptions sometimes

austere hornet Dec 25, 2021, 10:39 PM

#

gritty wind But there are exceptions sometimes

I see, thanks

thorny obsidian Dec 25, 2021, 10:41 PM

#

vale ibex So I realised I never got around to writing this for whatever reason, so I've sp...

Is it ready for review?

vale ibex Dec 25, 2021, 10:42 PM

#

yup 😄

thorny obsidian Dec 25, 2021, 10:42 PM

#

Sweet, I'll take a look now then

vale ibex Dec 25, 2021, 10:42 PM

#

I've also had a lot of baileys today so excuse simple mistakes lol

thorny obsidian Dec 25, 2021, 10:42 PM

#

Also Merry Christmas!

vale ibex Dec 25, 2021, 10:43 PM

#

and to you! Everyone is sleeping so I thought I'd be productive lol

thorny obsidian Dec 25, 2021, 10:43 PM

#

Haha, I'm 2 wine glasses deep trying to deal with a racist and misogynistic uncle, so I'm probably same level

vale ibex Dec 25, 2021, 10:43 PM

#

yikes

#

glad to provide some escape then lol

thorny obsidian Dec 25, 2021, 10:45 PM

#

oh! Is the role actually ready yet?

vale ibex Dec 25, 2021, 10:45 PM

#

Yea

#

Given it to myself

#

Just need to update the tree to the pink version

thorny obsidian Dec 25, 2021, 10:45 PM

#

Can I bug you for the sleigh that Fronto posted?

vale ibex Dec 25, 2021, 10:46 PM

#

want that to be the icon?

thorny obsidian Dec 25, 2021, 10:46 PM

#

to at least try it out, it's very relevant to AoC and cute

vale ibex Dec 25, 2021, 10:46 PM

#

#dev-branding message

#

this?

thorny obsidian Dec 25, 2021, 10:46 PM

#

yeah!

vale ibex Dec 25, 2021, 10:47 PM

#

how's it look?

thorny obsidian Dec 25, 2021, 10:47 PM

#

oh that's actually so cute (cc: @trail pilot)

trail pilot Dec 25, 2021, 10:47 PM

#

Ooh, it looks awesome

thorny obsidian Dec 25, 2021, 10:48 PM

#

I like it, but unsure if someone who didn't know what the icon was would recognize it. Although I'm willing to take that risk

trail pilot Dec 25, 2021, 10:48 PM

#

the details are a bit fuzzy around the edges but otherwise I think it looks like a sleigh

#

but tbh, I could see people mistaking it to be the Docker logo

#

like some weird, modified version of the Docker logo

vale ibex Dec 25, 2021, 10:48 PM

#

hah

thorny obsidian Dec 25, 2021, 10:49 PM

#

I'm also fine with the tree + star inside of it

clever wraith Dec 25, 2021, 10:49 PM

#

It is very small on mobile

#

It could be a bit hard to guess what it is

trail pilot Dec 25, 2021, 10:50 PM

#

the tree looks better tbh

#

it's more distinct

vale ibex Dec 25, 2021, 10:50 PM

#

This is the tree with star

clever wraith Dec 25, 2021, 10:51 PM

#

I also prefer the tree

vale ibex Dec 25, 2021, 10:51 PM

#

star with snowflake

#

this one isn't transparent, but that can be fixed

clever wraith Dec 25, 2021, 10:52 PM

#

Nah, can hardly see the snowflake

thorny obsidian Dec 25, 2021, 10:52 PM

#

That snowflake is hard to see. I say let's stick with the tree + star

vale ibex Dec 25, 2021, 10:52 PM

#

cool

trail pilot Dec 25, 2021, 10:52 PM

#

Also, the role is being given out in Jan. right?

#

or will it be given out by today

vale ibex Dec 25, 2021, 10:52 PM

#

it's being given out as soon as the PR is merged 😄

trail pilot Dec 25, 2021, 10:52 PM

#

Got it

thorny obsidian Dec 25, 2021, 10:53 PM

#

Probably whenever it gets merged. I'll make an announcement when it happens to let folks know

fallen patrol Dec 25, 2021, 10:53 PM

#

gritty wind These sort of fun feature usually go on lance, while the more crucial features g...

given that the other github autolinker is on @stable mountain i really don't understand how this feature is divvyed up

vale ibex Dec 25, 2021, 10:53 PM

#

I'm going to join the rest and head to bed, I'll take a look at PR comments tomorrow 🙂

gritty wind Dec 25, 2021, 10:53 PM

#

The answer is: randomly

thorny obsidian Dec 25, 2021, 10:54 PM

#

vale ibex I'm going to join the rest and head to bed, I'll take a look at PR comments tomo...

night!

fallen patrol Dec 25, 2021, 10:54 PM

#

although I've asked before and got an answer about why not to merge them, just forgot it, so nvm

fallen patrol Dec 25, 2021, 10:54 PM

#

gritty wind The answer is: randomly

ah, we use heads/tails to determine where a feature goes 🤡

#

if it lands on its side somehow we put it on @radiant merlin

clever wraith Dec 25, 2021, 10:57 PM

#

I am a bit too lazy to do an actual review as it would require me to setup a test server

#

Code wise it looks good

#

Except those two comments

fallen patrol Dec 25, 2021, 11:00 PM

#

clever wraith I am a bit too lazy to do an actual review as it would require me to setup a tes...

lol I have one that I can invite you to

clever wraith Dec 25, 2021, 11:00 PM

#

lol thanks, I'll have to see about that tomorrow

#

Going to slep rn

fallen patrol Dec 25, 2021, 11:00 PM

#

cc @short snow ^

patent pivot Dec 25, 2021, 11:39 PM

#

good evening gamers

#

anyone want a review for christmas

thorny obsidian Dec 25, 2021, 11:40 PM

#

lmao thank you @patent pivot

patent pivot Dec 25, 2021, 11:40 PM

#

ur so welcome

static canyon Dec 25, 2021, 11:50 PM

#

patent pivot anyone want a review for christmas

bot#1951 & bot#2013 both need one more review for merging 👀

dusky shoreBOT Dec 25, 2021, 11:50 PM

#

GitHub

PROpen Display whether DM was sent to user when listing infraction(s).
PROpen Add text indicating when user fetched by message

static canyon Dec 25, 2021, 11:52 PM

#

Hmm apparently I never got around to fixing tests on 2013, but you can still review lemon_sweat

#

In fact, I'll do that now