#dev-contrib | Python | Page 99

fallen patrol Apr 11, 2021, 8:58 PM

#

i do it there in my bot

#

no wait

#

lines 159-160 and 170

gritty wind Apr 11, 2021, 8:58 PM

#

isn;t that uploaded seperately as well

patent pivot Apr 11, 2021, 8:58 PM

#

yeah, this is probably worth going upstream

fallen patrol Apr 11, 2021, 8:58 PM

#

yeah but it becomes the embed

gritty wind Apr 11, 2021, 8:58 PM

#

as in, uploaded to the thumb

fallen patrol Apr 11, 2021, 8:58 PM

#

gritty wind isn;t that uploaded seperately as well

yeah imma go there

patent pivot Apr 11, 2021, 8:59 PM

#

but I think like

fallen patrol Apr 11, 2021, 8:59 PM

#

but at this point i just wanted to explain it to scale

patent pivot Apr 11, 2021, 8:59 PM

#

pulling down and uploading the server icon for every thread is a bit of a weird implementation

fallen patrol Apr 11, 2021, 8:59 PM

#

yes ik its not a big deal this is like the only server that updates it often

gritty wind Apr 11, 2021, 8:59 PM

#

lol I'd need to look into dpy's embeds, probably easier if I tinker with it haha

patent pivot Apr 11, 2021, 9:00 PM

#

I'd open an issue with them and see what they say before writing a fix

#

or ask in the server

fallen patrol Apr 11, 2021, 9:00 PM

#

patent pivot pulling down and uploading the server icon for every thread is a bit of a weird ...

imo a different solution would be to actually add it as a config feature

gritty wind Apr 11, 2021, 9:00 PM

#

Anyways, you'd better open that issue soon, ~~from my experience issues don't get addressed for weeks~~

fallen patrol Apr 11, 2021, 9:00 PM

#

so a server can set a specific icon to show in the footer

#

and it will always show that icon

#

in that case this server could also make it a static pydis image

#

and a bonus is if it were an icon hosted on the site then 😄 it could actually be updated and not break embeds

#

fwiw, its a minor enough issue and won't affect 99% of servers

#

its just the server that changes their icon 3/week

#

which means that the last thread has broken embeds and even then its just a transparent spot

#

~~but enough to annoy me~~

patent pivot Apr 11, 2021, 9:03 PM

#

drop an issue on the tracker or in https://discord.gg/F34cRU8

fallen patrol Apr 11, 2021, 10:19 PM

#

I'll make an issue for the below if even one helper+ says yes

#

.colo[u]r command for @dusky shore to show info about a color

patent pivot Apr 11, 2021, 10:20 PM

#

didn't we specify that

#

or was that just discussion here

fallen patrol Apr 11, 2021, 10:20 PM

#

Probably just here

patent pivot Apr 11, 2021, 10:20 PM

#

alright

#

and uhhh

#

#dev-contrib message

fallen patrol Apr 11, 2021, 10:21 PM

#

Ah

#

I wasn't thinking of an api

#

I was thinking generating the color on our own and everything, yeah

patent pivot Apr 11, 2021, 10:22 PM

#

then yeah, I think that's an issue we could approve

fallen patrol Apr 11, 2021, 10:22 PM

#

It'd be a fun first pr for me

#

Cool

patent pivot Apr 11, 2021, 10:23 PM

#

I do like the idea of being able to fetch the nearest identifiable colour thinkingcrew

fallen patrol Apr 11, 2021, 10:23 PM

#

Sir lancebot is standalone, yeah? Aka no other pydis projects needed

#

Unlike python

patent pivot Apr 11, 2021, 10:23 PM

#

https://chir.ag/projects/name-that-color/#FF0000

Name that Color - Chirag Mehta : chir.ag

Chirag Mehta - Personal Web Log - 'blog - Chime Softwares - Indian Blogger live from Saint Petersburg, Florida, St. Pete, FL

#

uhhh

#

yeah

fallen patrol Apr 11, 2021, 10:24 PM

#

patent pivot I do like the idea of being able to fetch the nearest identifiable colour <:thin...

... Would that be an api?

patent pivot Apr 11, 2021, 10:24 PM

#

it has APIs and stuff and optionally redis (you can use fakeredis locally)

#

no, if it is API only then it's a no go

#

but that site has openly available JavaScript

fallen patrol Apr 11, 2021, 10:24 PM

#

As of now I can see a way to do everything except get a near named color

fervent sage Apr 11, 2021, 10:24 PM

#

i have redis and it errored when not using fakeredis Thonk

fallen patrol Apr 11, 2021, 10:24 PM

#

Also uh

#

does sir Lance use PIL or pillow?

patent pivot Apr 11, 2021, 10:25 PM

#

pillow iirc

glass pecan Apr 11, 2021, 10:25 PM

#

fallen patrol As of now I can see a way to do everything except get a near named color

near named can be done by distance from closest value, that doesn't really need an api, just some static data

fervent sage Apr 11, 2021, 10:25 PM

#

PIL is the module pillow provides is it not?

fallen patrol Apr 11, 2021, 10:25 PM

#

glass pecan near named can be done by distance from closest value, that doesn't really need ...

zoomEyes guess I uh need to figure out how to represent colors lol

#

Distance from closest value of what

patent pivot Apr 11, 2021, 10:26 PM

#

RGB

glass pecan Apr 11, 2021, 10:26 PM

#

hex and rgb both are the same values, so yeah i don't see much of a difference on what you pick

#

it's just whatever it's stored as, really

patent pivot Apr 11, 2021, 10:26 PM

#

https://github.com/ryanzec/name-that-color/blob/master/lib/ntc.js has a set of them

GitHub

ryanzec/name-that-color

A command line utility that uses the library that the name that color project (http://chir.ag/projects/ntc/) uses that also provides additional functionality - ryanzec/name-that-color

fallen patrol Apr 11, 2021, 10:26 PM

#

Oh

patent pivot Apr 11, 2021, 10:26 PM

#

1,566 to be precise

glass pecan Apr 11, 2021, 10:27 PM

#

xkcd have color names too that i find somewhat entertaining to go through

#

https://xkcd.com/color/rgb.txt

patent pivot Apr 11, 2021, 10:27 PM

#

949 on that list

glass pecan Apr 11, 2021, 10:27 PM

#

on xkcd? should be 954

#

https://xkcd.com/color/rgb/

patent pivot Apr 11, 2021, 10:28 PM

#

glass pecan Apr 11, 2021, 10:28 PM

#

odd hm

#

oh well

#

still plenty enough

patent pivot Apr 11, 2021, 10:28 PM

#

yeah

#

we could combine the two and deduplicate if we really wanted

fallen patrol Apr 11, 2021, 10:29 PM

#

patent pivot we could combine the two and deduplicate if we really wanted

~~or if i want to~~

#

im uh

#

gonna put myself as wanting to write this

glass pecan Apr 11, 2021, 10:29 PM

#

there's no stress regardless

fallen patrol Apr 11, 2021, 10:30 PM

#

yeah fwiw don't expect this done any time soon 😂

patent pivot Apr 11, 2021, 10:30 PM

#

good place to start is specifying out the issue and tagging on what needs to be done

fallen patrol Apr 11, 2021, 10:30 PM

#

i've never worked with pillow, never worked with colo[u]rs, never worked with etcetra

glass pecan Apr 11, 2021, 10:30 PM

#

there's no rush as it's not a priority. just keep up communication/updates as you go so it doesn't look abandoned

fallen patrol Apr 11, 2021, 10:31 PM

#

ok

#

also uh

#

i want to set the base right here

#

color or colour.
I would perfer color

#

unless pydis specs are british/eu

glass pecan Apr 11, 2021, 10:31 PM

#

lol

#

color with an alias of colour

#

both are allowed

sleek steppe Apr 11, 2021, 10:31 PM

#

colour

patent pivot Apr 11, 2021, 10:31 PM

#

lol

fallen patrol Apr 11, 2021, 10:31 PM

#

i will make both work

patent pivot Apr 11, 2021, 10:32 PM

#

most internal spelling is british for pydis

glass pecan Apr 11, 2021, 10:32 PM

#

true joe

fallen patrol Apr 11, 2021, 10:32 PM

#

but its which one im gonna use as the base var

patent pivot Apr 11, 2021, 10:32 PM

#

but I don't think we've set a standard

glass pecan Apr 11, 2021, 10:32 PM

#

probably because of contrib distrubution

#

not a standard, yeah

fervent sage Apr 11, 2021, 10:32 PM

#

colour because colour is correct mmLol

fallen patrol Apr 11, 2021, 10:32 PM

#

~~how about i write it as color and then when its done i'll just replace everything with colour~~

patent pivot Apr 11, 2021, 10:32 PM

#

but we never use all the z's you americans keep putting in words

fallen patrol Apr 11, 2021, 10:32 PM

#

its amerizans

glass pecan Apr 11, 2021, 10:33 PM

#

patent pivot but we never use all the z's you americans keep putting in words

and that's zeds, not zees

fallen patrol Apr 11, 2021, 10:33 PM

#

fwiw my current discord language setting is

glass pecan Apr 11, 2021, 10:33 PM

#

doubt you'd notice much of a difference on discord ui

fallen patrol Apr 11, 2021, 10:33 PM

#

DECOUPLE TIMESTAMPS FROM LANGUAGE

glass pecan Apr 11, 2021, 10:33 PM

#

only with their weird spellcheck thing that i have disabled

patent pivot Apr 11, 2021, 10:33 PM

#

going by the previous discussion, some ideas are:
– random rgb or hsv color
– color conversions
– color to name
– color generation with pillow

complimentary/analogous/triadic/tetradic colors

fallen patrol Apr 11, 2021, 10:33 PM

#

glass pecan doubt you'd notice much of a difference on discord ui

#

only timestamps unfortunately

glass pecan Apr 11, 2021, 10:34 PM

#

no that's a system setting

fallen patrol Apr 11, 2021, 10:34 PM

#

i like 24 h so i set to UK

glass pecan Apr 11, 2021, 10:34 PM

#

oh you mean the format

#

lol i thought you meant the time itself

fallen patrol Apr 11, 2021, 10:34 PM

#

yeah..

#

~~actually i once tried changing system time and discord was correct to my timezone so~~

fallen patrol Apr 11, 2021, 10:34 PM

#

patent pivot going by the previous discussion, some ideas are: – random rgb or hsv color – co...

yeah this

glass pecan Apr 11, 2021, 10:35 PM

#

it doesn't have to implement everything in one hit though

#

just put out something basic to begin with if you want and progressively update

patent pivot Apr 11, 2021, 10:35 PM

#

yeah, we can always leave the issue open and mark things off as checkboxes

fallen patrol Apr 11, 2021, 10:35 PM

#

glass pecan it doesn't have to implement everything in one hit though

no it must be perfect lolol

glass pecan Apr 11, 2021, 10:36 PM

#

all the more reason to keep it simple lol

fallen patrol Apr 11, 2021, 10:36 PM

#

how i typically roll with my own development is i implement one feature and then do other stuff for a week, and then come back to it and implement something else and then repeat

#

i uh, haven't touched my bot code in around 3 days

#

which sucks because it has like 3 total commands

#

fallen patrol Apr 11, 2021, 10:41 PM

#

patent pivot going by the previous discussion, some ideas are: – random rgb or hsv color – co...

what are colo[u]r conversions?

glass pecan Apr 11, 2021, 10:41 PM

#

hex to rgb values to blah

fallen patrol Apr 11, 2021, 10:41 PM

#

patent pivot Apr 11, 2021, 10:41 PM

#

RGB to HSV/HSL/CMYK/etc

fallen patrol Apr 11, 2021, 10:41 PM

#

well that wasn't obvious at all /s

#

i use power toys color pickers smh

patent pivot Apr 11, 2021, 10:42 PM

#

and vice versa

fallen patrol Apr 11, 2021, 10:42 PM

#

#

i like

#

should know what conversions are

glass pecan Apr 11, 2021, 10:42 PM

#

there's some values that honestly don't need to be included because they wouldn't be used anyway

patent pivot Apr 11, 2021, 10:43 PM

#

I think it'd be interesting to have some of the more abstract ones as a demo of how they actually convert

glass pecan Apr 11, 2021, 10:43 PM

#

cmyk and rgb are common. hsl is weird

patent pivot Apr 11, 2021, 10:43 PM

#

HSV probably

#

HSL is weird

#

hsv prettier

#

though colorsys has both doesn't it

#

>>> colorsys.rgb_to_hls(255, 255, 255)
(0.0, 255.0, 0.0)
>>> colorsys.rgb_to_hls(255, 255, 0)
(0.16666666666666666, 127.5, -1.007905138339921)

fallen patrol Apr 11, 2021, 10:46 PM

#

wut

sleek steppe Apr 11, 2021, 10:46 PM

#

stdlib

fallen patrol Apr 11, 2021, 10:47 PM

#

oh

#

i knew that 😰

sleek steppe Apr 11, 2021, 10:47 PM

#

!d colorsys

stable mountainBOT Apr 11, 2021, 10:47 PM

#

`colorsys`

Source code: Lib/colorsys.py

The colorsys module defines bidirectional conversions of color values between colors expressed in the RGB (Red Green Blue) color space used in computer monitors and three other coordinate systems: YIQ, HLS (Hue Lightness Saturation) and HSV (Hue Saturation Value). Coordinates in all of these color spaces are floating point values. In the YIQ space, the Y coordinate is between 0 and 1, but the I and Q coordinates can be positive or negative. In all other spaces, the coordinates are all between 0 and 1.

short snow Apr 11, 2021, 10:58 PM

#

one more core dev approval needed on quackstack#31 if anyone has got some time

#

https://github.com/python-discord/quackstack/pull/31

GitHub

Switch random generation to use HSV over RGB by Shivansh-007 · Pull...

Closes #26
Uses implementation of the staff PyWeek submission

Adds colors.py which contains all util functions i.e. make_color, make_duck_colors, make_man_duck_colors.

Get rid of random RGB ge...

fallen patrol Apr 11, 2021, 10:59 PM

#

@ mods that says colors not colours

#

really had to be an account with the name mod smh

vocal prairie Apr 11, 2021, 11:18 PM

#

Would it be worth it to allow !clean to skip certain messages, and only clean ones from a certain time? For example, if a mod wants to purge 10 messages and skip the last two that were posted, they can't do that right now. Is that something that could help, or would that be pointless?

fallen patrol Apr 11, 2021, 11:24 PM

#

!src clean

stable mountainBOT Apr 11, 2021, 11:24 PM

#

Command: clean

Commands for cleaning messages in channels.

Source Code

Go to GitHub

#

Bad argument

Unable to convert '!purge' to valid command, tag, or Cog.

fallen patrol Apr 11, 2021, 11:25 PM

#

!src purge

stable mountainBOT Apr 11, 2021, 11:25 PM

#

Command: clean

Commands for cleaning messages in channels.

Source Code

Go to GitHub

fallen patrol Apr 11, 2021, 11:26 PM

#

are you

#

fucking

#

…

#

TIL if you click a link, even by accident, on github while writing an issue it just follows the link

#

and if you go back, you've lost all of your progress

#

@patent pivot i'll make that color issue soon i was almost finished too

#

but github was like nope

#

sad_gugu

vocal prairie Apr 11, 2021, 11:28 PM

#

~~That's why you constantly keep saving to your clipboard…~~

short snow Apr 11, 2021, 11:28 PM

#

go to open issue back, your stuff would be there

fallen patrol Apr 11, 2021, 11:28 PM

#

short snow go to open issue back, your stuff would be there

where

short snow Apr 11, 2021, 11:28 PM

#

on the new issue editor

fallen patrol Apr 11, 2021, 11:28 PM

#

ah, that's why its not, I was on preview

#

i clicked it when on preview and not when on create

#

sleek steppe Apr 11, 2021, 11:29 PM

#

colo[u]r

fallen patrol Apr 11, 2021, 11:29 PM

#

w/e i'll remake it later smh

vocal prairie Apr 11, 2021, 11:30 PM

#

vocal prairie Would it be worth it to allow `!clean` to skip certain messages, and only clean ...

By this I mean sorta like slice(), but in this case, you can choose how many messages back you want to start purging.

short snow Apr 11, 2021, 11:30 PM

#

sleek steppe colo[u]r

~~yeah am fixing that~~

fallen patrol Apr 11, 2021, 11:30 PM

#

short snow ~~yeah am fixing that~~

wat

#

that was my issue to sir-lancebot

short snow Apr 11, 2021, 11:31 PM

#

oh my bad, i thought about the quackstack pr

fallen patrol Apr 11, 2021, 11:31 PM

#

fallen patrol

in this screenshot

fallen patrol Apr 11, 2021, 11:47 PM

#

bot#1498

dusky shoreBOT Apr 11, 2021, 11:47 PM

#

GitHub

IssueOpen [bot] #1498 Show the result of !eval in #bot-commands when it is used in a channel where it is not allowed.

fallen patrol Apr 11, 2021, 11:48 PM

#

is it possible to implement this throughout the entire bot?

#

for all commands

vocal prairie Apr 11, 2021, 11:48 PM

#

Why not comment on the issue?

#

It might get lost otherwise.

short snow Apr 11, 2021, 11:49 PM

#

It would be great if someone could reply on this, https://github.com/python-discord/bot/issues/1154#issuecomment-785466997

GitHub

Add projects subcommands to source command · Issue #1154 · python-d...

Add subcommands for different PyDis projects to !source command. For example, SeasonalBot: !source seasonalbot. These subcommands should give short description about project and link to GitHub. I t...

thorny obsidian Apr 11, 2021, 11:49 PM

#

I think we specifically don't want to do this for eval, especially with py-gen. Because all it does is have people run !eval on code that could take up quite a bit of room and then go to bot-commands to get the result.

fallen patrol Apr 11, 2021, 11:50 PM

#

vocal prairie Why not comment on the issue?

done

thorny obsidian Apr 11, 2021, 11:50 PM

#

But I'll to find previous discussion on it to make sure I understood the consensus correctly

fallen patrol Apr 11, 2021, 11:50 PM

#

thorny obsidian I think we specifically don't want to do this for eval, especially with py-gen. ...

its been there for 7 days if you wanna comment on it

vocal prairie Apr 12, 2021, 1:04 AM

#

Uh, does anyone know how long on average it takes to docker-compose the public-stats repo? It's been about 10 minutes, and I'm still waiting.

patent pivot Apr 12, 2021, 1:07 AM

#

uhhh

#

I didn't set that up

#

any output logs?

vocal prairie Apr 12, 2021, 1:08 AM

#

Just constantly loading public-stats-graphite stuff.

#

patent pivot Apr 12, 2021, 1:11 AM

#

that's fine, that's just the output of graphite

vocal prairie Apr 12, 2021, 1:11 AM

#

Oh, okay.

patent pivot Apr 12, 2021, 1:12 AM

#

you should be able to browse to localhost:8898 I thikn

vocal prairie Apr 12, 2021, 1:13 AM

#

That localhost worked, although the actual graphs are blank. Strange. Probably data still loading.

fallen patrol Apr 12, 2021, 1:13 AM

#

https://xkcd.com/color/rgb/
https://github.com/ryanzec/name-that-color/blob/master/lib/ntc.js joe
pillow

don't mind me

patent pivot Apr 12, 2021, 1:13 AM

#

vocal prairie That localhost worked, although the actual graphs are blank. Strange. Probably d...

there isn't any data in the graphite is probably why

vocal prairie Apr 12, 2021, 1:14 AM

#

Oh, that would make sense.

patent pivot Apr 12, 2021, 1:14 AM

#

graphite is empty after initial startup, you can fill it by running an instance of the bot or with some dummy scripts

vocal prairie Apr 12, 2021, 1:14 AM

#

I probably have to set up the bot and site api.

patent pivot Apr 12, 2021, 1:14 AM

#

I'm not sure of the cleanest way to seed such data

#

are you planning on amending the stats portal?

vocal prairie Apr 12, 2021, 1:16 AM

#

I was thinking about it. Possibly. I also wanted to see how the graphs are generated so nicely.

patent pivot Apr 12, 2021, 1:16 AM

#

Ah, there is a nice library I use

#

one sec

vocal prairie Apr 12, 2021, 1:17 AM

#

The UI is really nice.

patent pivot Apr 12, 2021, 1:17 AM

#

https://www.npmjs.com/package/react-chartjs-2

npm

react-chartjs-2

vocal prairie Apr 12, 2021, 1:17 AM

#

Cool, thanks for the link!

#

What do you think of a stat for most used topical channels?

#

Like messages per topical channel.

patent pivot Apr 12, 2021, 1:18 AM

#

sounds pretty interesting, although we don't track channel categories on the graphite circuit so such a chart is more suited for metricity data, which could probably come with a later iteration of the stats portal

vocal prairie Apr 12, 2021, 1:20 AM

#

That makes sense. I'll look forward to it!

fallen patrol Apr 12, 2021, 1:42 AM

#

metricity#3

dusky shoreBOT Apr 12, 2021, 1:42 AM

#

GitHub

PROpen [metricity] #3 Remove opt-out functionality

fallen patrol Apr 12, 2021, 1:42 AM

#

still not merged

#

☺️

patent pivot Apr 12, 2021, 1:44 AM

#

i should uhhh

vocal prairie Apr 12, 2021, 1:44 AM

#

patent pivot sounds pretty interesting, although we don't track channel categories on the gra...

Should I open an issue so that the idea isn't lost?

fallen patrol Apr 12, 2021, 1:45 AM

#

patent pivot i should uhhh

ffs why did i bring it up

patent pivot Apr 12, 2021, 1:45 AM

#

it's definitely one on my list for if we ever migrate

vocal prairie Apr 12, 2021, 1:45 AM

#

Okie.

patent pivot Apr 12, 2021, 1:45 AM

#

fallen patrol ffs *why did i bring it up*

I was going to put something out today anyway and then the new policy goes live next Saturday

fallen patrol Apr 12, 2021, 1:45 AM

#

sunday tho i thought you were gonna give a weeks notice

#

and today is sunday almost monday

patent pivot Apr 12, 2021, 1:46 AM

#

alright, sunday then

fallen patrol Apr 12, 2021, 1:46 AM

#

but technically monday because this server goes offa utc

#

#dev-contrib message

#

vocal prairie Apr 12, 2021, 1:46 AM

#

Your goal is to delay this as much as possible, isn't it

fallen patrol Apr 12, 2021, 1:46 AM

#

a good week

fallen patrol Apr 12, 2021, 1:46 AM

#

vocal prairie Your goal is to delay this as much as possible, isn't it

yeah pretty much

patent pivot Apr 12, 2021, 1:47 AM

#

Sunday at 12 UTC is the date

clever wraith Apr 12, 2021, 1:48 AM

#

hello

patent pivot Apr 12, 2021, 1:52 AM

#

announcement made

#

!remind 6d10h6M metricity PR 3 can be merged

stable mountainBOT Apr 12, 2021, 1:53 AM

#

Sure thing!

Your reminder will arrive in 6 days, 10 hours and 6 minutes!

patent pivot Apr 12, 2021, 1:55 AM

#

good game

fallen patrol Apr 12, 2021, 1:55 AM

#

https://pydis.com/privacy
double redirect smh

Python Discord | Privacy

We're a large, friendly community focused around the Python programming language. Our community is open to those who wish to learn the language, as well as those looking to help others.

#

wait no

#

https://pydis.com/privacy
https://pythondiscord.com/pages/privacy/
https://www.notion.so/Privacy-Policy-4ceb50b84d164541bd5b0b7046aa7e6a
https://www.notion.so/Python-Discord-Privacy-ee2581fea4854ddcb1ebc06c1dbb9fbd

#

it redirects me through all of that smh

fallen patrol Apr 12, 2021, 1:57 AM

#

patent pivot good game

…well played

tawdry vapor Apr 12, 2021, 1:58 AM

#

What is this change for

fallen patrol Apr 12, 2021, 1:58 AM

#

no more ability to opt out of metricity

patent pivot Apr 12, 2021, 1:58 AM

#

remove opt out functionality from metricity

fallen patrol Apr 12, 2021, 1:58 AM

#

hm

#

for merging it

#

don't people who opt out have to consent to opting back in?

#

yeah

#

the announcement doesn't say that part

patent pivot Apr 12, 2021, 2:00 AM

#

not really, because it's not an opt out as a measure of compliance, it's just disabling a feature

thorny obsidian Apr 12, 2021, 2:00 AM

#

Why would they have to consent to opting back in? The announcement is letting people know there won't be opt out functionality and while they remain in server they'll have the metricity data tracked

fallen patrol Apr 12, 2021, 2:01 AM

#

right

#

it would be as simple to say that's how you agree

#

by sending messages

#

or even remaining in the server

#

that would be considered consent, no?

patent pivot Apr 12, 2021, 2:02 AM

#

it's not even a matter of "by doing X you consent" really

#

think of it more as a feature toggle

#

people disabled the feature toggle but now we're removing that toggle entirely

#

it's not something people need to consent to

fallen patrol Apr 12, 2021, 2:10 AM

#

how does one link to a header within markdown

#

its [text](#Proposed%20Implementation) right?

#

I'm trying to link to a header within my issue yes

vocal prairie Apr 12, 2021, 2:11 AM

#

[text](#Proposed-Implementation) if I remember correctly.

fallen patrol Apr 12, 2021, 2:12 AM

#

welp it doesn't work anyways oh well

#

sir-lancebot#677

dusky shoreBOT Apr 12, 2021, 2:15 AM

#

GitHub

IssueOpen [sir-lancebot] #677 Create a colo[u]r command

fallen patrol Apr 12, 2021, 2:15 AM

#

someone please assign me

#

😛

magic arch Apr 12, 2021, 2:32 AM

#

fallen patrol welp it doesn't work anyways oh well

Yeah you can't do that within issues/PRs unfortunately

fallen patrol Apr 12, 2021, 4:26 AM

#

fallen patrol sir-lancebot#677

Someone please still assign mel

#

This was talked about in server with Joe and scragley

short snow Apr 12, 2021, 4:36 AM

#

you could link to the context, in reasoning section

fallen patrol Apr 12, 2021, 4:55 AM

#

Can't/won't add it to the issue rn because I'm laying in bed at 1am but

#

#dev-contrib message

#

Will add it tomorrow if I remember

dusky shoreBOT Apr 12, 2021, 4:57 AM

#

GitHub

IssueOpen [sir-lancebot] #677 Create a colo[u]r command

stable mountainBOT Apr 12, 2021, 4:57 AM

#

Yep.

Your reminder will arrive in 5 hours!

stable mountainBOT Apr 12, 2021, 9:57 AM

#

@fallen patrol

It has arrived!

Here's your reminder: add [this context]([#dev-contrib message](/guild/267624335836053506/channel/635950537262759947/)) to [sir-lancebot#677](https://github.com/python-discord/sir-lancebot/issues/677).
[Jump back to when you created the reminder](#dev-contrib message)

placid ermine Apr 12, 2021, 11:29 AM

#

bump sir-lancebot#668

dusky shoreBOT Apr 12, 2021, 11:29 AM

#

GitHub

PROpen [sir-lancebot] #668 Add latex cog

fallen patrol Apr 12, 2021, 12:25 PM

#

stable mountain <@717983911824588862>

….why

#

why did you ruin my embeds

short snow Apr 12, 2021, 12:29 PM

#

fallen patrol why did you ruin my embeds

#bot-commands message boom

fallen patrol Apr 12, 2021, 12:30 PM

#

Pepe_clap

fervent sage Apr 12, 2021, 2:15 PM

#

hey @ashen isle dont wanna step on your toes, mind if i upload a screenshot of what the tag for bot#1517 looks like for people :P

dusky shoreBOT Apr 12, 2021, 2:15 PM

#

GitHub

PROpen [bot] #1517 Create identity.md

ashen isle Apr 12, 2021, 2:15 PM

#

eh, can't people use the preview thing

fervent sage Apr 12, 2021, 2:15 PM

#

preview thing?

ashen isle Apr 12, 2021, 2:15 PM

#

oh that's not a thing

#

yeah pls do that

#

oh it is a thing

#

vale ibex Apr 12, 2021, 2:16 PM

#

Yea, there's the md previewer in github, but it'll look slightly different as an embed

fervent sage Apr 12, 2021, 2:16 PM

#

i mean like

#

ashen isle Apr 12, 2021, 2:16 PM

#

oh, yeah sure

fervent sage Apr 12, 2021, 2:17 PM

#

random side note i really appreciate how github handles images

vocal prairie Apr 12, 2021, 2:17 PM

#

Yeah.

#

It's nice how they upload it too.

fallen patrol Apr 12, 2021, 2:21 PM

#

fervent sage random side note i really appreciate how github handles images

wdym

sleek steppe Apr 12, 2021, 2:22 PM

#

pasting an image in comments in github uploads it and formats it to be an image automatically

fallen patrol Apr 12, 2021, 2:24 PM

#

ah

#

yeah

#

even better

#

![image](put any link here to an image)

#

I made this pr https://github.com/Run1e/AceBot/pull/32 that used 4 images because i went crazy

fallen patrol Apr 12, 2021, 2:51 PM

#

uh

#

where is the token code

sleek steppe Apr 12, 2021, 2:55 PM

#

?

#

the filter?

fallen patrol Apr 12, 2021, 2:55 PM

#

https://github.com/python-discord/bot/blob/main/bot/exts/filters/token_remover.py

#

that

#

….

#

help_channels is a loaded cog, ya?

#

or maybe its HelpChannels

sleek steppe Apr 12, 2021, 2:59 PM

#

yes

fallen patrol Apr 12, 2021, 3:01 PM

#

well then

#

guess im opening an issue for a bug

#

i-

patent pivot Apr 12, 2021, 3:03 PM

#

Issue for what?

fallen patrol Apr 12, 2021, 3:03 PM

#

!src HelpChannels

stable mountainBOT Apr 12, 2021, 3:03 PM

#

Help Command

An interactive instance for the bot help command.

Source Code

Go to GitHub

fallen patrol Apr 12, 2021, 3:03 PM

#

should be the help channel cog, ya?

#

because

#

https://github.com/python-discord/bot/blob/51af1369e0d9c2ad185f0c0920b599b7187ed077/bot/exts/info/source.py#L19

GitHub

python-discord/bot

The community bot for the Python Discord community - python-discord/bot

#

        if argument.lower().startswith("help"):
            return ctx.bot.help_command

#

should be

#

is / ==

#

instead

#

!src helpnomatterwhatiputhere

stable mountainBOT Apr 12, 2021, 3:05 PM

#

Help Command

An interactive instance for the bot help command.

Source Code

Go to GitHub

patent pivot Apr 12, 2021, 3:06 PM

#

ah, yeah

fallen patrol Apr 12, 2021, 3:07 PM

#

even though the fix is like

#

one line

sleek steppe Apr 12, 2021, 3:07 PM

#

fallen patrol `is` / `==`

==

fallen patrol Apr 12, 2021, 3:07 PM

#

im not gonna be the one to commit it

#

if someone wants to just make the fix right now i'll skip the issue

#

i mean

#

if someone is present and wants to do it

sleek steppe Apr 12, 2021, 3:08 PM

#

I mean I can do it

fallen patrol Apr 12, 2021, 3:08 PM

#

if you have a bot instance that runs lol

sleek steppe Apr 12, 2021, 3:11 PM

#

all my commits on bot are so small lol

fallen patrol Apr 12, 2021, 3:12 PM

#

lol

placid ermine Apr 12, 2021, 3:13 PM

#

(i am once again bumping sir-lancebot#668)

dusky shoreBOT Apr 12, 2021, 3:13 PM

#

GitHub

PROpen [sir-lancebot] #668 Add latex cog

vale ibex Apr 12, 2021, 3:13 PM

#

sleek steppe all my commits on bot are so small lol

Hah my first few PRs were tiny too, just look at bot#1392 bot#1390 and bot#1388

dusky shoreBOT Apr 12, 2021, 3:13 PM

#

GitHub

PRMerged [bot] #1392 Don't fuzzy search for tags when tag is on cooldown
[bot] #1390 Protect against overflows caused by large expirations
[bot] #1388 Gracefully handle non-existing infractions

fallen patrol Apr 12, 2021, 3:13 PM

#

so that's why i like the first one more

#

btw could someone assign me to sir-lancebot#677

dusky shoreBOT Apr 12, 2021, 3:13 PM

#

GitHub

IssueOpen [sir-lancebot] #677 Create a colo[u]r command

vale ibex Apr 12, 2021, 3:14 PM

#

fallen patrol btw could someone assign me to sir-lancebot#677

You'll get assigned once it's approved 😄

fallen patrol Apr 12, 2021, 3:14 PM

#

well

#

joe did approve it just not in github

#

#dev-contrib message

vale ibex Apr 12, 2021, 3:15 PM

#

Yea, sure, but as joe said we still need to think about your proposed solution

vale ibex Apr 12, 2021, 3:16 PM

#

placid ermine (i am once again bumping sir-lancebot#668)

We're all volunteers, so please don't expect that all of your PRs will be looked at straight away

#

The fact that it's in the list of open PRs means that we will take a look at it at some point

placid ermine Apr 12, 2021, 3:17 PM

#

sure thing

fallen patrol Apr 12, 2021, 3:17 PM

#

vale ibex Yea, sure, but as joe said we still need to think about your proposed solution

😦 i mean 😛

placid ermine Apr 12, 2021, 3:17 PM

#

sorry if i appeared a bit pushy

vale ibex Apr 12, 2021, 3:17 PM

#

placid ermine sorry if i appeared a bit pushy

No it's fine, just wanted to set your expectations 😄

fallen patrol Apr 12, 2021, 3:19 PM

#

bot#1519

dusky shoreBOT Apr 12, 2021, 3:19 PM

#

GitHub

IssueOpen [bot] #1519 Logic in Source Command's Help Command Check

fallen patrol Apr 12, 2021, 3:19 PM

#

good first issue 😛

sleek steppe Apr 12, 2021, 3:19 PM

#

not sure if we needed the issue lol

fallen patrol Apr 12, 2021, 3:20 PM

#

since i'm not the one who's going to submit a fix, i made an issue

#

if i was going to make the fix, i wouldn't have made the issue

#

but

#

i don't/won't set up @stable mountain

ashen isle Apr 12, 2021, 3:21 PM

#

@fervent sage can you do it again 🥺 i don't have the bot setup

sleek steppe Apr 12, 2021, 3:22 PM

#

How do you install packages on docker? I keep on getting ModelNotFound errors lol

vocal prairie Apr 12, 2021, 3:22 PM

#

sleek steppe How do you install packages on docker? I keep on getting ModelNotFound errors lo...

pipenv sync --dev

vale ibex Apr 12, 2021, 3:23 PM

#

docker-compose up --build

sleek steppe Apr 12, 2021, 3:23 PM

#

ah

vocal prairie Apr 12, 2021, 3:23 PM

#

Oh, i thought it was pipenv

sleek steppe Apr 12, 2021, 3:23 PM

#

I feel like there's no need to test this change tho

fallen patrol Apr 12, 2021, 3:24 PM

#

there is

#

because

#

there's a default help command and that's why that code exists

#

in the issue i noted that i think it could be removed entirely

#

because that code isn't on sir-lancebot

#

.src src

#

i hate u

sleek steppe Apr 12, 2021, 3:25 PM

#

fallen patrol in the issue i noted that i think it could be removed entirely

I don't think that's a good idea

fallen patrol Apr 12, 2021, 3:25 PM

#

https://github.com/python-discord/sir-lancebot/blob/main/bot/exts/evergreen/source.py#L37-L48
https://github.com/python-discord/bot/blob/main/bot/exts/info/source.py#L51-L62

GitHub

python-discord/sir-lancebot

A Discord bot started as a community project for Hacktoberfest 2018, later evolved to an introductory project for aspiring new developers starting out with open source development. - python-discord...

GitHub

python-discord/bot

The community bot for the Python Discord community - python-discord/bot

sleek steppe Apr 12, 2021, 3:26 PM

#

Because sir-lancebot doesn't use commands.HelpCommand

#

The type of the command itself is discord.ext.commands.help._HelpCommandImpl

#

which of course isn't on the repository

green oriole Apr 12, 2021, 3:29 PM

#

It is a custom command, yes

#

Well, I think?

sleek steppe Apr 12, 2021, 3:30 PM

#

green oriole It is a custom command, yes

Yep. However, it's registered as a command as a _HelpCommandImpl object.

green oriole Apr 12, 2021, 3:31 PM

#

hmm interesting

fallen patrol Apr 12, 2021, 3:32 PM

#

#dev-log message
@ashen isle that's how i make mistakes

ashen isle Apr 12, 2021, 3:32 PM

#

lul

fallen patrol Apr 12, 2021, 3:32 PM

#

y'know

#

before i used git

sleek steppe Apr 12, 2021, 3:32 PM

#

green oriole hmm interesting

https://github.com/Rapptz/discord.py/blob/master/discord/ext/commands/help.py#L331-L334 here's how it adds the command

GitHub

Rapptz/discord.py

An API wrapper for Discord written in Python. Contribute to Rapptz/discord.py development by creating an account on GitHub.

fallen patrol Apr 12, 2021, 3:32 PM

#

commiting was scarier than writing the code

sleek steppe Apr 12, 2021, 3:33 PM

#

Ehh I'm just gonna push because docker is taking too long

fallen patrol Apr 12, 2021, 3:33 PM

#

ashen isle lul

I used to commit by copying the file, pasting my changes into it, running it, and then copying the file back to github and pasting it

ashen isle Apr 12, 2021, 3:33 PM

#

bruh

vocal prairie Apr 12, 2021, 3:34 PM

#

fallen patrol I used to commit by copying the file, pasting my changes into it, running it, an...

That's what I used to do.

#

Until I learned version control.

fallen patrol Apr 12, 2021, 3:34 PM

#

vocal prairie Until I learned version control.

lol i fucked it up yesterday

vocal prairie Apr 12, 2021, 3:34 PM

#

So I had 7 commits instead of 1.

green oriole Apr 12, 2021, 3:34 PM

#

@ashen isle I added a screenshot to yar Pr

fallen patrol Apr 12, 2021, 3:35 PM

#

#

~~note: they didn't squash merge~~

sleek steppe Apr 12, 2021, 3:36 PM

#

yay docker started after 8 minutes

fallen patrol Apr 12, 2021, 3:36 PM

#

this should've been 3 commits but i uh

#

(https://github.com/Run1e/AceBot/pull/31)

sleek steppe Apr 12, 2021, 3:36 PM

#

I'm gonna make convert a staticmethod too VeryCool

fallen patrol Apr 12, 2021, 3:37 PM

#

hey you should wait until the issue is approved zoomeyes

fallen patrol Apr 12, 2021, 3:37 PM

#

patent pivot ah, yeah

bot#1519

dusky shoreBOT Apr 12, 2021, 3:37 PM

#

GitHub

IssueOpen [bot] #1519 Logic in Source Command's Help Command Check

sleek steppe Apr 12, 2021, 3:38 PM

#

fallen patrol Apr 12, 2021, 3:38 PM

#

holy shit

sleek steppe Apr 12, 2021, 3:38 PM

#

wait it got merged?

fallen patrol Apr 12, 2021, 3:39 PM

#

no idea

patent pivot Apr 12, 2021, 3:39 PM

#

yeah

#

it did

fallen patrol Apr 12, 2021, 3:39 PM

#

256 commits got merged

#

what got merged

patent pivot Apr 12, 2021, 3:39 PM

#

it's in dev log

vale ibex Apr 12, 2021, 3:39 PM

#

!d str.join

stable mountainBOT Apr 12, 2021, 3:39 PM

#

`str.join`

str.join(iterable)```
Return a string which is the concatenation of the strings in *iterable*. A [`TypeError`](exceptions.html#TypeError "TypeError") will be raised if there are any non-string values in *iterable*, including [`bytes`](#bytes "bytes") objects. The separator between elements is the string providing this method.

patent pivot Apr 12, 2021, 3:39 PM

#

bot#1014

dusky shoreBOT Apr 12, 2021, 3:39 PM

#

GitHub

PRMerged [bot] #1014 Doc improvements

sleek steppe Apr 12, 2021, 3:39 PM

#

nice

fallen patrol Apr 12, 2021, 3:39 PM

#

but what is it

patent pivot Apr 12, 2021, 3:39 PM

#

read the PR body

sleek steppe Apr 12, 2021, 3:39 PM

#

fallen patrol but what is it

Doc Improvements

patent pivot Apr 12, 2021, 3:39 PM

#

I can't summarise all changes on mobile

vale ibex Apr 12, 2021, 3:39 PM

#

It's an improvement to the docs command, read the PR 😄

green oriole Apr 12, 2021, 3:40 PM

#

🎉 🎉 🎉 🎉 🎉 🎉 🎉 Thaaaannnnkksss @brazen charm, the doc PR is merged! 🎉 🎉 🎉 🎉 🎉 🎉 🎉

patent pivot Apr 12, 2021, 3:40 PM

#

yeah, huge shout out to Numerlor

fallen patrol Apr 12, 2021, 3:40 PM

#

patent pivot I can't summarise all changes on mobile

…who's writing the #changelog then

green oriole Apr 12, 2021, 3:40 PM

#

was about to ask that haha

patent pivot Apr 12, 2021, 3:40 PM

#

we don't have a requirement to changelog within 5 minutes of PR merge lol

green oriole Apr 12, 2021, 3:41 PM

#

I guess the person who merged it 👀

patent pivot Apr 12, 2021, 3:41 PM

#

I'll happily write a changelog later today

fallen patrol Apr 12, 2021, 3:41 PM

#

green oriole was about to ask that haha

~~thanks for volunteering~~ /s

green oriole Apr 12, 2021, 3:41 PM

#

I don't mind writing one either, but I don't know much about the PR

thorny obsidian Apr 12, 2021, 3:41 PM

#

Hot damn that's a beefy PR

patent pivot Apr 12, 2021, 3:42 PM

#

ya

vale ibex Apr 12, 2021, 3:42 PM

#

Hah yea, hit the limit of github for review bot too 😅

patent pivot Apr 12, 2021, 3:42 PM

#

lul

fallen patrol Apr 12, 2021, 3:42 PM

#

green oriole I don't mind writing one either, but I don't know much about the PR

largest pr to the bot yet

#

and it does stuff

green oriole Apr 12, 2021, 3:43 PM

#

That may be one of the largest, yeah

sleek steppe Apr 12, 2021, 3:43 PM

#

numelor's third in commits now 😄

fallen patrol Apr 12, 2021, 3:43 PM

#

waiting for it to deploy

#

!d str.join

stable mountainBOT Apr 12, 2021, 3:43 PM

#

str.join


str.join(iterable)```
Return a string which is the concatenation of the strings in *iterable*. A [`TypeError`](https://docs.python.org/3/library/exceptions.html#TypeError "TypeError") will be raised if there are any non-string values in *iterable*, including [`bytes`](https://docs.python.org/3/library/stdtypes.html#bytes "bytes") objects. The separator between elements is the string providing this method.

patent pivot Apr 12, 2021, 3:43 PM

#

!d aiohttp.ClientSession

stable mountainBOT Apr 12, 2021, 3:43 PM

#

aiohttp.ClientSession

class aiohttp.ClientSession(*, connector=None, loop=None, cookies=None, headers=None, skip_auto_headers=None, auth=None, json_serialize=json.dumps, version=aiohttp.HttpVersion11, ...)```
The class for creating client sessions and making requests.

patent pivot Apr 12, 2021, 3:43 PM

#

nice

sleek steppe Apr 12, 2021, 3:43 PM

#

Wait you can link to discord now

fallen patrol Apr 12, 2021, 3:43 PM

#

first one was the new one already lol

sleek steppe Apr 12, 2021, 3:43 PM

#

!d discord

fallen patrol Apr 12, 2021, 3:43 PM

#

AYYYYY THAT'S NEW

patent pivot Apr 12, 2021, 3:44 PM

#

!remind 5h changelog

stable mountainBOT Apr 12, 2021, 3:44 PM

#

Yeah okay.

Your reminder will arrive in 5 hours!

fallen patrol Apr 12, 2021, 3:44 PM

#

not that you can link but that

#

!d discord works

stable mountainBOT Apr 12, 2021, 3:44 PM

#

discord

In order to work with the library and the Discord API in general, we must first create a Discord Bot account.

Creating a Bot account is a pretty straightforward process.

fallen patrol Apr 12, 2021, 3:44 PM

#

it used to say generic page n shit

#

acongablob

sleek steppe Apr 12, 2021, 3:44 PM

#

It used to not find it 😔

fallen patrol Apr 12, 2021, 3:45 PM

#

ikr

#

!d aiohttp

stable mountainBOT Apr 12, 2021, 3:45 PM

#

aiohttp

Common data structures used by aiohttp internally.

fallen patrol Apr 12, 2021, 3:45 PM

#

~~ok done spamming~~

sleek steppe Apr 12, 2021, 3:45 PM

#

I'll write my pr now

vocal prairie Apr 12, 2021, 3:45 PM

#

@brazen charm Wonderful job! 🎉

sleek steppe Apr 12, 2021, 3:46 PM

#

I can fix this issue too...

#

Or should I just leave it?

#

Yeah I don't think the help extension will be unloaded

fallen patrol Apr 12, 2021, 3:53 PM

#

fuzzy matching didn't catch this?

#

#

cc @tranquil topaz @vale ibex

vale ibex Apr 12, 2021, 3:54 PM

#

nah, they're different enough that difflib didn't pick it up

fallen patrol Apr 12, 2021, 3:54 PM

#

wait

#

is that a lib

#

dope

vale ibex Apr 12, 2021, 3:54 PM

#

It's a built-in

#

!d difflib.get_close_matches

stable mountainBOT Apr 12, 2021, 3:55 PM

#

difflib.get\_close\_matches


difflib.get_close_matches(word, possibilities, n=3, cutoff=0.6)```
Return a list of the best “good enough” matches. *word* is a sequence for which close matches are desired (typically a string), and *possibilities* is a list of sequences against which to match *word* (typically a list of strings).

Optional argument *n* (default `3`) is the maximum number of close matches to return; *n* must be greater than `0`.

Optional argument *cutoff* (default `0.6`) is a float in the range [0, 1]. Possibilities that don’t score at least that similar to *word* are ignored.

The best (no more than *n*) matches among the possibilities are returned in a list, sorted by similarity score, most similar first.

cold moon Apr 12, 2021, 3:56 PM

#

bot#836 and bot#954 need review

dusky shoreBOT Apr 12, 2021, 3:56 PM

#

GitHub

PROpen [bot] #836 Tags cog unit tests and tags cog fixes
PROpen [bot] #954 Error Handler tests

fallen patrol Apr 12, 2021, 3:56 PM

#

i uh

cold moon Apr 12, 2021, 3:56 PM

#

One of them have been open over a year, another little bit less

fallen patrol Apr 12, 2021, 3:58 PM

#

why did sir-lance allow this riddle?

#

messages between the dialogs have been removed for clarity

vale ibex Apr 12, 2021, 3:58 PM

#

Yea, we should remove that

fallen patrol Apr 12, 2021, 3:59 PM

#

also

#

its a bad riddle anyhow

sleek steppe Apr 12, 2021, 3:59 PM

#

Nice the bot removes those random trashcan reactions now

fallen patrol Apr 12, 2021, 3:59 PM

#

fallen patrol why did sir-lance allow this riddle?

#ot0-psvm’s-eternal-disapproval message

vale ibex Apr 12, 2021, 4:01 PM

#

sir-lancebot#678

dusky shoreBOT Apr 12, 2021, 4:01 PM

#

GitHub

PROpen [sir-lancebot] #678 Remove inappropriate riddle

fallen patrol Apr 12, 2021, 4:01 PM

#

uh

#

@vale ibex these riddles are just

#

https://github.com/python-discord/sir-lancebot/blob/375c247ad0098db12b40fc2c1ad36e417c95f9e9/bot/resources/easter/easter_riddle.json#L43-L48

#

lemme perhaps find a better list and jsonify it

#

Why was the little girl sad after the race?

Because an egg beater

timid sentinel Apr 12, 2021, 4:02 PM

#

That's.... no.... it was a race between the little girl and an egg, and the egg won the race

fallen patrol Apr 12, 2021, 4:03 PM

#

oh

#

the other one tho was bad

sleek steppe Apr 12, 2021, 4:06 PM

#

bot#1521

dusky shoreBOT Apr 12, 2021, 4:06 PM

#

GitHub

PROpen [bot] #1521 Use == instead of str.startswith when getting the source for a Command

sleek steppe Apr 12, 2021, 4:07 PM

#

I made it a staticmethod because I didn't like the unused variable

fallen patrol Apr 12, 2021, 4:26 PM

#

shipit

placid ermine Apr 12, 2021, 4:31 PM

#

fallen patrol why did sir-lance allow this riddle?

lmfao

fallen patrol Apr 12, 2021, 4:33 PM

#

chris made a pr to remove it

placid ermine Apr 12, 2021, 4:47 PM

#

lesgoooooooo

cold moon Apr 12, 2021, 4:50 PM

#

And latex is here!

vocal wolf Apr 12, 2021, 4:52 PM

#

While y'all are here, please give a review to sir-lancebot#642

dusky shoreBOT Apr 12, 2021, 4:52 PM

#

GitHub

PROpen [sir-lancebot] #642 Allow automatic linking of issues everywhere

patent pivot Apr 12, 2021, 4:56 PM

#

.latex j^{oe}_{mama}

#

uhh

vale ibex Apr 12, 2021, 4:57 PM

#

.latex j^{oe}_{mama}

dusky shoreBOT Apr 12, 2021, 4:57 PM

#

$latex.png$

vale ibex Apr 12, 2021, 4:58 PM

#

shrugR

patent pivot Apr 12, 2021, 4:58 PM

#

.latex $j^{oe}_{mama}$

dusky shoreBOT Apr 12, 2021, 4:58 PM

#

vale ibex Apr 12, 2021, 4:58 PM

#

Nice

placid ermine Apr 12, 2021, 4:58 PM

#

interesting

vale ibex Apr 12, 2021, 4:58 PM

#

$ is for math things, like this

vocal wolf Apr 12, 2021, 5:01 PM

#

.latex $\lnot (p \land q)$

dusky shoreBOT Apr 12, 2021, 5:01 PM

#

You have to stop.

Your input was invalid:
\lnot (p \land q)
^
Unknown symbol: \land, found '' (at char 9), (line:1, col:10)

Usage:
.latex <text>

vocal wolf Apr 12, 2021, 5:02 PM

#

wat

placid ermine Apr 12, 2021, 5:02 PM

#

interesting

vocal wolf Apr 12, 2021, 5:02 PM

#

this means I can't do truth tables

#

literally unusable /s

#

@green oriole automatic linking has been merged. Thank you, @cold moon!

sleek steppe Apr 12, 2021, 5:04 PM

#

bot#1521 needs reviews 😀

dusky shoreBOT Apr 12, 2021, 5:04 PM

#

GitHub

PRMerged Add "Now playing" status

sleek steppe Apr 12, 2021, 5:04 PM

#

Huh

fallen patrol Apr 12, 2021, 5:04 PM

#

….wait

fallen patrol Apr 12, 2021, 5:04 PM

#

vocal wolf <@!407110650217627658> automatic linking has been merged. Thank you, <@!44631059...

i-

#

why does it not show the repo and name anymore 😔

#

sir-lancebot#1 bot#1

dusky shoreBOT Apr 12, 2021, 5:04 PM

#

GitHub

IssueClosed Set up some simple Discord bot boilerplate.

fallen patrol Apr 12, 2021, 5:04 PM

#

uh

#

~~seems it might be buggy~~

fervent sage Apr 12, 2021, 5:04 PM

#

quackstack#12

dusky shoreBOT Apr 12, 2021, 5:04 PM

#

GitHub

IssueClosed API: Retrieve a random ducky with optional seed.

fallen patrol Apr 12, 2021, 5:05 PM

#

bot#1
sir-lancebot#2

fervent sage Apr 12, 2021, 5:05 PM

#

beautiful

dusky shoreBOT Apr 12, 2021, 5:05 PM

#

GitHub

PRMerged Set up some simple Discord bot boilerplate
Add "Now playing" status

sleek steppe Apr 12, 2021, 5:05 PM

#

dusky shore

We're not gonna talk about how this linked #1?

fallen patrol Apr 12, 2021, 5:05 PM

#

wait

#

what

vocal wolf Apr 12, 2021, 5:05 PM

#

hmmmmmmmmm

fallen patrol Apr 12, 2021, 5:05 PM

#

its buggy lol

#

~~blunt af~~

sleek steppe Apr 12, 2021, 5:05 PM

#

bot#1521

dusky shoreBOT Apr 12, 2021, 5:05 PM

#

GitHub

PRMerged Add "Now playing" status

fallen patrol Apr 12, 2021, 5:05 PM

#

sir-lancebot#200

sleek steppe Apr 12, 2021, 5:05 PM

#

yert

dusky shoreBOT Apr 12, 2021, 5:05 PM

#

GitHub

PRMerged Set up some simple Discord bot boilerplate

fallen patrol Apr 12, 2021, 5:05 PM

#

ohhhhh

vocal wolf Apr 12, 2021, 5:05 PM

#

https://github.com/python-discord/sir-lancebot/pull/642#pullrequestreview-622731534

#

something to do with this

fallen patrol Apr 12, 2021, 5:05 PM

#

without looking at the code, the regex is fucky

#

its not getting anything after the first number

vocal wolf Apr 12, 2021, 5:06 PM

#

(maybe)

fallen patrol Apr 12, 2021, 5:06 PM

#

sir-lancebot#200 is not 2

dusky shoreBOT Apr 12, 2021, 5:06 PM

#

GitHub

PRMerged Set up some simple Discord bot boilerplate

fallen patrol Apr 12, 2021, 5:06 PM

#

but it is showing 2

#

sir-lancebot#123

dusky shoreBOT Apr 12, 2021, 5:06 PM

#

GitHub

IssueClosed Set up some simple Discord bot boilerplate.

fallen patrol Apr 12, 2021, 5:06 PM

#

then that's issue 1 for sir-lancebot

#

@green oriole this was your commit, sir-lancebot#642

vocal wolf Apr 12, 2021, 5:07 PM

#

that ain't good

fallen patrol Apr 12, 2021, 5:07 PM

#

oh and now it doesn't even link at all

#

sir-lancebot#642

dusky shoreBOT Apr 12, 2021, 5:08 PM

#

GitHub

IssueClosed [Feature Request] October PR User Stats Command

fallen patrol Apr 12, 2021, 5:08 PM

#

ahhh, pr 6, exactly what i wanted

surreal venture Apr 12, 2021, 5:08 PM

#

its just using the first digit of the number

sleek steppe Apr 12, 2021, 5:08 PM

#

Why was the number .+ anyways

fallen patrol Apr 12, 2021, 5:09 PM

#

surreal venture its just using the first digit of the number

yeah

sleek steppe Apr 12, 2021, 5:09 PM

#

Ahh I see

#

Wait nvm

vocal wolf Apr 12, 2021, 5:10 PM

#

it's PR time

fallen patrol Apr 12, 2021, 5:10 PM

#

there's more to the regex than that

fallen patrol Apr 12, 2021, 5:10 PM

#

vocal wolf it's PR time

do get this not being caught either bot#2

#

because its matching the entire message

vocal wolf Apr 12, 2021, 5:11 PM

#

I'm a little rusty on regex but I'll see what I can do

surreal venture Apr 12, 2021, 5:11 PM

#

How would I add the reddit emojis to my server?

#

I need them for the stackoverflow command

sleek steppe Apr 12, 2021, 5:12 PM

#

Copy image addresses & download VeryCool

fallen patrol Apr 12, 2021, 5:12 PM

#

for those who wanna help with the regex for linking

#

AUTOMATIC_REGEX = re.compile(r"((?P<org>.+?)\/)?(?P<repo>.+?)#(?P<number>.+?)")

#

that's the regex that matches issues

sleek steppe Apr 12, 2021, 5:12 PM

#

Ahh

#

It's matching the entire mesaage

#

Right?

surreal venture Apr 12, 2021, 5:13 PM

#

sleek steppe Copy image addresses & download <:VeryCool:585580174750253057>

As in, how would I put it in my code? If i just put the id of the emoji as a constant, wont it be unique to my server? so it wont work for the python server right?

fallen patrol Apr 12, 2021, 5:13 PM

#

sleek steppe It's matching the entire mesaage

it matches everything before the # on the same line and only one character after the #

whole forge Apr 12, 2021, 5:13 PM

#

oh hey

#

i just found the regex bug

#

already have a fix too @vocal wolf

vocal wolf Apr 12, 2021, 5:14 PM

#

oh sick

sleek steppe Apr 12, 2021, 5:14 PM

#

#123

fallen patrol Apr 12, 2021, 5:14 PM

#

paste it so we can test it 🥺

green oriole Apr 12, 2021, 5:14 PM

#

fallen patrol <@!407110650217627658> this was your commit, sir-lancebot#642

What did I do

vocal wolf Apr 12, 2021, 5:14 PM

#

@green oriole your regex broke

green oriole Apr 12, 2021, 5:14 PM

#

Wait, the PR is merged?

fallen patrol Apr 12, 2021, 5:14 PM

#

green oriole What did I do

the regex went buh bye

fallen patrol Apr 12, 2021, 5:14 PM

#

green oriole Wait, the PR is merged?

yeah

sleek steppe Apr 12, 2021, 5:14 PM

#

green oriole Wait, the PR is merged?

Yep and it broke

whole forge Apr 12, 2021, 5:14 PM

#

the regex is actually quite bad on the first look

green oriole Apr 12, 2021, 5:15 PM

#

Oh cool

whole forge Apr 12, 2021, 5:15 PM

#

im gonna extend my fix to exactly represent gihub naming rules/limits

fallen patrol Apr 12, 2021, 5:15 PM

#

was it not supposed to be merged? ~~say yes~~

green oriole Apr 12, 2021, 5:15 PM

#

I forgot the word boundaries didn't I

green oriole Apr 12, 2021, 5:15 PM

#

fallen patrol was it not supposed to be merged? ~~say yes~~

Well, not really haha, but that's my fault

sleek steppe Apr 12, 2021, 5:15 PM

#

.+ is never usually good lol thonk

fallen patrol Apr 12, 2021, 5:15 PM

#

green oriole I forgot the word boundaries didn't I

it matches everything before the # on the same line and only one character after the #

green oriole Apr 12, 2021, 5:15 PM

#

I was still making sure that everything worked correctly

vocal wolf Apr 12, 2021, 5:15 PM

#

@whole forge yeah please do it, my regex experience isn't very good

whole forge Apr 12, 2021, 5:15 PM

#

np

fallen patrol Apr 12, 2021, 5:16 PM

#

green oriole Well, not really haha, but that's my fault

u should've converted it to a draft pr smh

whole forge Apr 12, 2021, 5:16 PM

#

gonna base it off of the limits listed here because there are no official ones
https://github.com/isiahmeadows/github-limits

GitHub

isiahmeadows/github-limits

A collection of various non-quota limits with GitHub - isiahmeadows/github-limits

fallen patrol Apr 12, 2021, 5:16 PM

#

preventing it from being merged at all

sharp timber Apr 12, 2021, 5:16 PM

#

\w+? seems more appropriate

green oriole Apr 12, 2021, 5:16 PM

#

. should be [\w-] most probably

fallen patrol Apr 12, 2021, 5:16 PM

#

sharp timber \w+? seems more appropriate

hello cat

sleek steppe Apr 12, 2021, 5:16 PM

#

~~I mean how many people approved lol~~

fallen patrol Apr 12, 2021, 5:16 PM

#

sleek steppe ~~I mean how many people approved lol~~

two

vocal wolf Apr 12, 2021, 5:17 PM

#

fallen patrol two

4

sleek steppe Apr 12, 2021, 5:17 PM

#

More than two

vocal wolf Apr 12, 2021, 5:17 PM

#

it's fiiiiiiiine

#

we'll fix it up

sharp timber Apr 12, 2021, 5:17 PM

#

anyway, I'll get something working in regexr

sleek steppe Apr 12, 2021, 5:17 PM

#

+? is the same thing as * right

green oriole Apr 12, 2021, 5:17 PM

#

Nah

sharp timber Apr 12, 2021, 5:17 PM

#

((?P<org>[\w\-]+?)\/)?(?P<repo>[\w\-]+?)#(?P<number>[\w-]+?)

green oriole Apr 12, 2021, 5:18 PM

#

It is between 1 and infinite but lazy

whole forge Apr 12, 2021, 5:18 PM

#

bast youre missing numbers and +? is bad practice

#

https://regex101.com/r/qhAaAV/1

regex101

regex101: build, test, and debug regex

Regular expression tester with syntax highlighting, explanation, cheat sheet for PHP/PCRE, Python, GO, JavaScript, Java. Features a regex quiz & library.

#

heres mine

sharp timber Apr 12, 2021, 5:18 PM

#

\w does numbers

whole forge Apr 12, 2021, 5:18 PM

#

wait does it

#

oh it does

sharp timber Apr 12, 2021, 5:18 PM

#

!e ```py
import re
print(re.match("\w+", "abcd123").group(0))

stable mountainBOT Apr 12, 2021, 5:18 PM

#

@sharp timber :white_check_mark: Your eval job has completed with return code 0.

abcd123

vocal wolf Apr 12, 2021, 5:18 PM

#

@surreal venture replying to your comment on the stackoverflow command, I hope you're able to finish it off this week :D

sharp timber Apr 12, 2021, 5:19 PM

#

Why is +? bad practice?

whole forge Apr 12, 2021, 5:19 PM

#

becasue that matches really long thins like shfisdahfkjdsafhnsadjfsakdfhkdasjhahskdf\askdfjvasdjvasdjfvsjdafvjsdavfjasdvfjsdfvj#1 too which arent even valid repo names

sharp timber Apr 12, 2021, 5:19 PM

#

I mean, if we have a length limit we should use that

whole forge Apr 12, 2021, 5:19 PM

#

should always try to match as exactly as posinle

sharp timber Apr 12, 2021, 5:19 PM

#

I just didn't think there was one

whole forge Apr 12, 2021, 5:19 PM

#

length limit is 39 and 39

#

wait

#

no

#

100 for repo

#

39 for user

sleek steppe Apr 12, 2021, 5:20 PM

#

Wow 100

sharp timber Apr 12, 2021, 5:20 PM

#

Yeah that could be.. interesting

vocal prairie Apr 12, 2021, 5:20 PM

#

Yeah, it's 100.

#

Just tested it.

whole forge Apr 12, 2021, 5:20 PM

#

^

vale ibex Apr 12, 2021, 5:20 PM

#

So ((?P<org>[a-zA-Z0-9]{1,39}?)\/)?(?P<repo>[a-zA-Z0-9\-\.]{1,100}?)#(?P<number>[0-9]+)?

whole forge Apr 12, 2021, 5:21 PM

#

https://regex101.com/r/V2ji8M/2

sharp timber Apr 12, 2021, 5:21 PM

#

I don't think we need the ? lazy specifier at all

#

Since we always have hard match edges, there's no reason to "match as little" because we should always match a given amount

whole forge Apr 12, 2021, 5:21 PM

#

yea nope

#

no need

#

also @vale ibex my regex101 link has \w since apparently as bast proved that does numbers

#

tho i wonder

sharp timber Apr 12, 2021, 5:22 PM

#

The only place it makes a difference is when the terminating characters can be within the original match sequence, and that's not the case here, / is not valid in usernames or repo names I believe

whole forge Apr 12, 2021, 5:23 PM

#

nope, alphanumeric and - for user, alphanumeric and - and . for repo

tawdry vapor Apr 12, 2021, 5:23 PM

#

If you want a very accurate regex for github org/repo then this is it:

(?P<org>(?!-)(?:[^\W_]|-(?!-))+(?<!-))/(?P<repo>[\w.-]+)

#

It has some strange rules about where hyphens can go for org names

whole forge Apr 12, 2021, 5:23 PM

#

tawdry vapor If you want a very accurate regex for github org/repo then this is it: ``` (?P<o...

why the double negative?

sharp timber Apr 12, 2021, 5:24 PM

#

Are _ not valid for org names?

whole forge Apr 12, 2021, 5:24 PM

#

tawdry vapor Apr 12, 2021, 5:24 PM

#

No apparently. I researched this a while ago so feel free to confirm

vocal prairie Apr 12, 2021, 5:24 PM

#

It isn't for usernames either.

tawdry vapor Apr 12, 2021, 5:24 PM

#

It's from here https://github.com/python-discord/bot/pull/1028

GitHub

Cog for code snippet links by dolphingarlic · Pull Request #1028 · ...

I thought that it would be pretty cool if Discord could show the contents of a snippet link instead of just the link to the snippet, so I made a bot for that.
However, since adding third-party bots...

vocal prairie Apr 12, 2021, 5:24 PM

#

sharp timber Apr 12, 2021, 5:24 PM

#

It'd be clearer to use [A-Za-z0-9\-] I think, rather than the double negative

tawdry vapor Apr 12, 2021, 5:25 PM

#

The regex is complicated to account for the hyphen rules

whole forge Apr 12, 2021, 5:25 PM

#

updated mine https://regex101.com/r/V2ji8M/3

tawdry vapor Apr 12, 2021, 5:25 PM

#

That being said, I don't think you really need to obey the hyphen rules

#

It's good enough without them

vale ibex Apr 12, 2021, 5:26 PM

#

Well, we have the regex to deal with them now anyway, so no harm in obeying them

tawdry vapor Apr 12, 2021, 5:26 PM

#

If you can come up with a better regex for that then go for it

fallen patrol Apr 12, 2021, 5:26 PM

#

tawdry vapor That being said, I don't think you really need to obey the hyphen rules

just .strip('-') after the regex hehe

sharp timber Apr 12, 2021, 5:27 PM

#

lol

tawdry vapor Apr 12, 2021, 5:27 PM

#

No not quite

#

It doesn't allow consecutive hyphens too

whole forge Apr 12, 2021, 5:27 PM

#

actualyl @tawdry vapor yours doesnt obey the - rules eitehr

tawdry vapor Apr 12, 2021, 5:27 PM

#

Even though the error message they give doesn't say that

vocal prairie Apr 12, 2021, 5:27 PM

#

Yeah it does.

#

It says single hyphens only.

sharp timber Apr 12, 2021, 5:27 PM

#

can't start with one though

tawdry vapor Apr 12, 2021, 5:27 PM

#

Oh sorry

sharp timber Apr 12, 2021, 5:28 PM

#

honestly this is regex torture

fallen patrol Apr 12, 2021, 5:28 PM

#

whole forge actualyl <@!137073073168973824> yours doesnt obey the - rules eitehr

his matches the org / repo
he never said anything about matching the issues

tawdry vapor Apr 12, 2021, 5:28 PM

#

whole forge actualyl <@!137073073168973824> yours doesnt obey the - rules eitehr

I think I may have only done it for org names and not for repo names, but I guess you can copy it over

vocal prairie Apr 12, 2021, 5:28 PM

#

Can't start or end with one either with org/usernames

sharp timber Apr 12, 2021, 5:28 PM

#

If we are going to enforce these reqs, we should enforce them in the surrounding python

vale ibex Apr 12, 2021, 5:28 PM

#

whole forge actualyl <@!137073073168973824> yours doesnt obey the - rules eitehr

fallen patrol Apr 12, 2021, 5:28 PM

#

vocal prairie Can't start or end with one either with org/usernames

*can't

sharp timber Apr 12, 2021, 5:28 PM

#

if match and not org.startswith("-") or org.endswith("-") and "--" not in org

#

or something like that

green oriole Apr 12, 2021, 5:28 PM

#

I don't think it is worth enforcing edge cases here as the API will just return 404 of the user doesn't exist

fallen patrol Apr 12, 2021, 5:28 PM

#

sharp timber If we are going to enforce these reqs, we should enforce them in the surrounding...

this is a legit question, can you slowmode this channel for now 😂

green oriole Apr 12, 2021, 5:29 PM

#

We can afford a failed request

fallen patrol Apr 12, 2021, 5:29 PM

#

green oriole I don't think it is worth enforcing edge cases here as the API will just return ...

having a better regex does mean less requests to github

#

so if someone is spamming stuff it won't end up making a bunch of requests

tawdry vapor Apr 12, 2021, 5:29 PM

#

green oriole I don't think it is worth enforcing edge cases here as the API will just return ...

Yeah and that's why I didn't actually ask for them to use that regex in the PR. I just wanted to mention it for correctness sake.

fallen patrol Apr 12, 2021, 5:30 PM

#

like

whole forge Apr 12, 2021, 5:31 PM

#

i think we should just ignore leading dashes

#

that way if someone writes
-python/cpython#123 it still works

fallen patrol Apr 12, 2021, 5:31 PM

#

whole forge i think we should just ignore leading dashes

in that case, just strip('-') after the regex

whole forge Apr 12, 2021, 5:31 PM

#

nah

#

just

#

https://regex101.com/r/V2ji8M/4

#

this one does it

vale ibex Apr 12, 2021, 5:32 PM

#

Cannot include consecutive hyphens

fallen patrol Apr 12, 2021, 5:32 PM

#

why is it unreadable

whole forge Apr 12, 2021, 5:32 PM

#

because you have a custom dark theme?

vale ibex Apr 12, 2021, 5:32 PM

#

fallen patrol why is it unreadable

Do you have a dark mode addon?

fallen patrol Apr 12, 2021, 5:32 PM

#

uhm, yes

vale ibex Apr 12, 2021, 5:33 PM

#

that's why 😄

fallen patrol Apr 12, 2021, 5:33 PM

#

and the website has a darkmode too

tawdry vapor Apr 12, 2021, 5:33 PM

#

whole forge <https://regex101.com/r/V2ji8M/4>

underscores are allowed in repo names so [\w.-] is fine to use.

whole forge Apr 12, 2021, 5:33 PM

#

wait are they, lemme check

tawdry vapor Apr 12, 2021, 5:33 PM

#

fallen patrol Apr 12, 2021, 5:33 PM

#

vale ibex `Cannot include consecutive hyphens`

this is @whole forge's but as you can see it doesn't work https://regex101.com/r/HmSS7Z/1

#

it matches

#

wut/sd--f#1

tawdry vapor Apr 12, 2021, 5:35 PM

#

the hyphens are only enforced for org names

#

repo names dont care about hyphens

gritty wind Apr 12, 2021, 5:35 PM

#

We can have multiple regexes for different parts

fallen patrol Apr 12, 2021, 5:35 PM

#

wait so repos can have double --?

#

well then

#

#

okay then ye

#

((?P<org>[a-zA-Z0-9][a-zA-Z0-9\-]{1,39})\/)?(?P<repo>[a-zA-Z0-9\-\.]{1,100})#(?P<number>[0-9]+) works, its what @whole forge sent

whole forge Apr 12, 2021, 5:37 PM

#

not quite

#

its missing the underscore

#

in repo name

#

one sec, let me try to get the double dash thing figured out

green oriole Apr 12, 2021, 5:37 PM

#

fallen patrol so if someone is spamming stuff it won't end up making a bunch of requests

We have 5k requests/h, who cares

whole forge Apr 12, 2021, 5:38 PM

#

in that case lets ignore the double dash

green oriole Apr 12, 2021, 5:38 PM

#

They'd get automuted before they can do anything meaningful

whole forge Apr 12, 2021, 5:38 PM

#

https://regex101.com/r/V2ji8M/5

tawdry vapor Apr 12, 2021, 5:38 PM

#

I can't think of a better way in regex than [^\W_]|-(?!-)

sharp timber Apr 12, 2021, 5:39 PM

#

enforce the non-double dash in python

fallen patrol Apr 12, 2021, 5:39 PM

#

GOT IT

sharp timber Apr 12, 2021, 5:39 PM

#

the reason to use regex here is for processing speed, so we don't need to interact with it through python exactly. A for loop over every message is unclear, and so is parsing out every little part

fallen patrol Apr 12, 2021, 5:39 PM

#

((?P<org>(?!-)(?:[^\W_]|-(?!-))+(?<!-){1,39})\/)?(?P<repo>[a-zA-Z0-9\-\.\_]{1,100})#(?P<number>[0-9]+)

#

https://regex101.com/r/HmSS7Z/2

whole forge Apr 12, 2021, 5:39 PM

#

fallen patrol Apr 12, 2021, 5:40 PM

#

whole forge

because the -- isn't valid

whole forge Apr 12, 2021, 5:40 PM

#

i think at this point its better to just let the API error out

sharp timber Apr 12, 2021, 5:40 PM

#

but applying some validation in python on top of that doesn't sound like a problem and sounds like the right way to do it if there's some difficulty doing it in regex

whole forge Apr 12, 2021, 5:40 PM

#

fallen patrol because the -- isn't valid

right but then the whole regex needs to fail

#

wait hm

#

do we assume that someone with a double dash made a typo, or that a double dash is part of the rest of the message

#

i would assume typo, then it shouldn't match anything

fallen patrol Apr 12, 2021, 5:41 PM

#

part of the rest of the message

whole forge Apr 12, 2021, 5:41 PM

#

right but whats more likely, that someone has a message that goes--repo/name#1 or that someone typos hello--world\name#1

tawdry vapor Apr 12, 2021, 5:41 PM

#

You're matching on a word boundary anyway, right?

sharp timber Apr 12, 2021, 5:42 PM

#

I don't think we are at the moment

tawdry vapor Apr 12, 2021, 5:43 PM

#

You probably should since this could very well match part of some arbitrary URL

whole forge Apr 12, 2021, 5:43 PM

#

good point

vocal prairie Apr 12, 2021, 5:43 PM

#

Wait, you can have #s in urls?

whole forge Apr 12, 2021, 5:43 PM

#

yea

tawdry vapor Apr 12, 2021, 5:43 PM

#

Yes they're called fragments

vocal prairie Apr 12, 2021, 5:43 PM

#

I thought that could only be after a /

whole forge Apr 12, 2021, 5:44 PM

#

~~https://github.com/python-discord/bot#1~~

GitHub

python-discord/bot

The community bot for the Python Discord community - python-discord/bot

#

lol

fervent sage Apr 12, 2021, 5:44 PM

#

for example: https://vcokltfre.dev#extras

vocal prairie Apr 12, 2021, 5:44 PM

#

whole forge ~~https://github.com/python-discord/bot#1~~

Oh, oops

tawdry vapor Apr 12, 2021, 5:44 PM

#

vocal prairie I thought that could only be after a /

Not necessarily e.g. https://example.com/index.html#12

sharp timber Apr 12, 2021, 5:45 PM

#

Yeah, that's a problem

whole forge Apr 12, 2021, 5:45 PM

#

wait no im dumb

#

issues are slash separated

tawdry vapor Apr 12, 2021, 5:45 PM

#

If you match by a word boundary then it'd be better to have the entire match fail when a hyphen issue is detected. That seems easier to do in Python rather than in regex.

#

Having the regex partially match to skip the hyphen issues would be a bad idea

sharp timber Apr 12, 2021, 5:46 PM

#

!e ```py
import re
print(re.search(r"((?P<org>[\w-]+?)/)?(?P<repo>[\w.-]+)#(?P<number>\d+)", "https://example.com/index.html#12"))

#

Eventually I'll figure out why this isn't matching

fallen patrol Apr 12, 2021, 5:47 PM

#

it shouldn't be

tawdry vapor Apr 12, 2021, 5:47 PM

#

Cause you're using match rather than search

whole forge Apr 12, 2021, 5:47 PM

#

could you not jsut use sth like regex101 isntead of evaling?

sharp timber Apr 12, 2021, 5:47 PM

#

I could

#

I wanted it to be a clear example though. That ship has sailed.

#

matches on the site, so why doesn't it match here..

tawdry vapor Apr 12, 2021, 5:48 PM

#

Why are you using [\w.-]+ for the number

#

\d+

sharp timber Apr 12, 2021, 5:48 PM

#

matches com/test.html#15

tawdry vapor Apr 12, 2021, 5:48 PM

#

Also you aren't excluding the underscore from org name

sharp timber Apr 12, 2021, 5:49 PM

#

Yeah, this was an earlier one I grabbed because the ones after I couldn't make work

#

I think we should match cases with -- preceding because people will want to do that sometimes

tawdry vapor Apr 12, 2021, 5:49 PM

#

What do you mean

sharp timber Apr 12, 2021, 5:50 PM

#

You might be running into an issue with regex--python-discord/bot#15--or it might be something else

#

if we want to put in the effort to match that, imo, we should. It doesn't match at the moment for some reason

tawdry vapor Apr 12, 2021, 5:51 PM

#

I think there'd be no reason to type it in any way other than exactly like python-discord/bot#15

#

Anything else should fail

vocal prairie Apr 12, 2021, 5:51 PM

#

What about in parentheses?

whole forge Apr 12, 2021, 5:51 PM

#

at this point regex is not sufficient as a parser

tawdry vapor Apr 12, 2021, 5:51 PM

#

Well the hyphen checks can easily be done in Python after the regex has a match

tawdry vapor Apr 12, 2021, 5:52 PM

#

vocal prairie What about in parentheses?

Yeah, good point

whole forge Apr 12, 2021, 5:53 PM

#

vocal prairie What about in parentheses?

wdym?

vocal prairie Apr 12, 2021, 5:53 PM

#

I was talking about what Mark said.

#

That anything but it exactly should be ignored.

whole forge Apr 12, 2021, 5:54 PM

#

well i definitely not anything, precedding dot, slash or the word being too long should be ignored imo

#

because urls

tawdry vapor Apr 12, 2021, 5:55 PM

#

It may be easier to make exceptions for certain characters like brackets rather than trying to blacklist

vocal prairie Apr 12, 2021, 5:55 PM

#

That would probably be best.

whole forge Apr 12, 2021, 5:56 PM

#

well if you want to do that, knock yourself out

tawdry vapor Apr 12, 2021, 5:56 PM

#

Like (?:$|\s+)[([{]?

#

Anyway, I'm done tinkering with it

fallen patrol Apr 12, 2021, 6:01 PM

#

hm

#

another thing the command needs is

#

a check, if there is a command invoked in the message

#

oh my god

vocal prairie Apr 12, 2021, 6:02 PM

#

fallen patrol a check, if there is a command invoked in the message

Why?

fallen patrol Apr 12, 2021, 6:02 PM

#

~~why was this okayed~~

#

.issue sir-lancebot#55

#

#

https://github.com/python-discord/sir-lancebot#55

patent pivot Apr 12, 2021, 6:02 PM

#

lol

fallen patrol Apr 12, 2021, 6:02 PM

#

~~why was this authorized~~

patent pivot Apr 12, 2021, 6:02 PM

#

?

#

I don't think we need the snarky attitude.

fallen patrol Apr 12, 2021, 6:03 PM

#

i mean

fervent sage Apr 12, 2021, 6:03 PM

#

mistakes happen

fallen patrol Apr 12, 2021, 6:03 PM

#

we just talked about the regex for a while

#

but yeah

fervent sage Apr 12, 2021, 6:03 PM

#

thats just the way things are

fallen patrol Apr 12, 2021, 6:03 PM

#

sorry

vocal wolf Apr 12, 2021, 6:23 PM

#

@vale ibex ty for pr

vale ibex Apr 12, 2021, 6:24 PM

#

NP 😄

#

Do our containers support simd instructions?

#

I'm just wondering if it's worth swapping lancebot to pillow-simd in this image PR

patent pivot Apr 12, 2021, 6:27 PM

#

you raise a good question

#

I think so

sleek steppe Apr 12, 2021, 6:28 PM

#

bot#1521 needs one more review from a staff/contributor.

dusky shoreBOT Apr 12, 2021, 6:28 PM

#

GitHub

PRMerged Add "Now playing" status

sleek steppe Apr 12, 2021, 6:28 PM

#

ahhh lol

#

https://github.com/python-discord/bot/pull/1521

patent pivot Apr 12, 2021, 6:28 PM

#

lmfao

vale ibex Apr 12, 2021, 6:30 PM

#

patent pivot I think so

I might experiment with our high resolution repo to see

patent pivot Apr 12, 2021, 6:30 PM

#

we have uhh

#

12 AMD EPYC 7601 32-Core

#

no, we don't

#

wait

#

what do we have lol

#

uhh maybe we do? trying to interpret /proc/cpuinfo

vale ibex Apr 12, 2021, 6:31 PM

#

Could you throw cat /proc/cpuinfo in here?

#

oh lol

patent pivot Apr 12, 2021, 6:31 PM

#

either way, /proc/cpuinfo suggests we have several of the required instruction sets for SIMD

#

yeah one sec

#

https://paste.pythondiscord.com/fibutohuvu.apache?noredirect

#

I initially thought 4 cores but the core ID on each processor doesn't increment

vale ibex Apr 12, 2021, 6:32 PM

#

yea, avx is in the flags

#

we should be good

patent pivot Apr 12, 2021, 6:32 PM

#

nice

vocal wolf Apr 12, 2021, 6:33 PM

#

@fervent sage Context: https://github.com/python-discord/sir-lancebot/pull/676

"google" will be the only key that is ever fetched. Would you mind including the fix in your PR?

Problem: https://github.com/python-discord/sir-lancebot/blob/main/bot/exts/easter/april_fools_vids.py#L17

fervent sage Apr 12, 2021, 6:35 PM

#

oop, not my code originally but sure i'll fix that

vocal wolf Apr 12, 2021, 6:35 PM

#

Thanks!

fallen patrol Apr 12, 2021, 6:36 PM

#

vocal wolf <@!297045071457681409> Context: <https://github.com/python-discord/sir-lancebot/...

additional problem: the current logic means that google has an equal chance of every other creator to be pulled and isn't related to the video count at all

fervent sage Apr 12, 2021, 6:37 PM

#

hm that's a good point

#

initially i was going to do this

#

but hmmm

sleek steppe Apr 12, 2021, 6:37 PM

#

😩

fallen patrol Apr 12, 2021, 6:37 PM

#

imo i would reformat the entire json

sleek steppe Apr 12, 2021, 6:37 PM

#

list(self.yt_vids) please

fervent sage Apr 12, 2021, 6:37 PM

#

i think i will

fallen patrol Apr 12, 2021, 6:38 PM

#

sleek steppe `list(self.yt_vids)` please

wait

#

that would make almost more sense if the json was formatted better

fervent sage Apr 12, 2021, 6:38 PM

#

imo it'd be better to have a list of

{
  "url": "",
  "channel": ""
}
``` since video name is never used anyway

fallen patrol Apr 12, 2021, 6:38 PM

#

one way to do it would be

#

weight each channel

fervent sage Apr 12, 2021, 6:39 PM

#

that's just needlessly complex imo

fallen patrol Apr 12, 2021, 6:39 PM

#

in order to not reformat the json

#

well, sorting the json by who, makes it a bit nicer to read

#

👏

#

@sleek steppe vn

#

!source HelpChannels

stable mountainBOT Apr 12, 2021, 6:40 PM

#

Cog: HelpChannels

Manage the help channel system of the guild.

Source Code

Go to GitHub

sleek steppe Apr 12, 2021, 6:40 PM

#

yes

fervent sage Apr 12, 2021, 6:46 PM

#

wow RNG doesn't like me today

#

i tested the command almost 100 times before it chose one other than google

#

was beginning to think it was somehow broken, but alas randomness sucks lol

sleek steppe Apr 12, 2021, 6:47 PM

#

are you just picking randomly from all of the videos?

fervent sage Apr 12, 2021, 6:48 PM

#

yes

#

see latest commit

#

hm while im here i'm tempted to alias it as fools too

sleek steppe Apr 12, 2021, 6:48 PM

#

ah

#

not sure about that one

fervent sage Apr 12, 2021, 6:49 PM

#

which one

#

picking randomly or fools alias?

sleek steppe Apr 12, 2021, 6:49 PM

#

fools

fervent sage Apr 12, 2021, 6:50 PM

#

also is this a better name for the PR by now?

vale ibex Apr 12, 2021, 6:50 PM

#

ugggh fighting pipenv to install pillowsimd

#

can't find the right version to install which has wheels available

vocal wolf Apr 12, 2021, 6:50 PM

#

diff --git a/bot/exts/easter/april_fools_vids.py b/bot/exts/easter/april_fools_vids.py
index efe7e67..1882fd7 100644
--- a/bot/exts/easter/april_fools_vids.py
+++ b/bot/exts/easter/april_fools_vids.py
@@ -8,28 +8,22 @@ from discord.ext import commands
 log = logging.getLogger(__name__)
 
 
+with open("bot/resources/easter/april_fools_vids.json", encoding="utf-8") as f:
+    ALL_VIDS = load(f)
+
+
 class AprilFoolVideos(commands.Cog):
     """A cog for April Fools' that gets a random April Fools' video from Youtube."""
 
     def __init__(self, bot: commands.Bot):
         self.bot = bot
-        self.yt_vids = self.load_json()
         self.youtubers = ['google']  # will add more in future
 
-    @staticmethod
-    def load_json() -> dict:
-        """A function to load JSON data."""
-        p = Path('bot/resources/easter/april_fools_vids.json')
-        with p.open(encoding="utf-8") as json_file:
-            all_vids = load(json_file)
-        return all_vids
-
     @commands.command(name='fool')
     async def april_fools(self, ctx: commands.Context) -> None:
         """Get a random April Fools' video from Youtube."""
-        random_youtuber = random.choice(self.youtubers)
-        category = self.yt_vids[random_youtuber]
-        random_vid = random.choice(category)
+        random_youtuber = random.choice(ALL_VIDS)
+        random_vid = random.choice(ALL_VIDS[random_youtuber])
         await ctx.send(f"Check out this April Fools' video by {random_youtuber}.\n\n{random_vid['link']}")

#

static func ehhhh

fervent sage Apr 12, 2021, 6:51 PM

#

i can yeet the static funtion if you want

#

it seems a bit... overcomplicated

vocal wolf Apr 12, 2021, 6:51 PM

#

static func is just unnecessary

fervent sage Apr 12, 2021, 6:51 PM

#

yea

vale ibex Apr 12, 2021, 6:52 PM

#

!paste

stable mountainBOT Apr 12, 2021, 6:52 PM

#

Pasting large amounts of code

If your code is too long to fit in a codeblock in discord, you can paste your code here:
https://paste.pydis.com/

After pasting your code, save it by clicking the floppy disk icon in the top right, or by typing ctrl + S. After doing that, the URL should change. Copy the URL and post it here so others can see it.

fervent sage Apr 12, 2021, 6:52 PM

#

tbh i can simplify the whole thing to basically https://paste.pythondiscord.com/muxonogake.py

#

which removes a ton of needless bloat

vale ibex Apr 12, 2021, 6:53 PM

#

https://paste.pythondiscord.com/wexedapete.properties?noredirect

#

Anyone fancy helping out here, been stuck for like 10 mins

#

upgrading and downgrading my pip and pipenv

#

trying different versions of pillow-simd

fallen patrol Apr 12, 2021, 6:54 PM

#

!pypi pillow-simd

stable mountainBOT Apr 12, 2021, 6:54 PM

#

Pillow-SIMD v7.0.0.post3

Python Imaging Library (Fork)

fallen patrol Apr 12, 2021, 6:54 PM

#

what

#

why not use Pillow

#

what is the simd

sleek steppe Apr 12, 2021, 6:57 PM

#

channel, url = video["channel"], video["url"]

Do we allow this here lol

vale ibex Apr 12, 2021, 6:58 PM

#

Pillow simd uses simd instructions, among other speed improvements, which makes it considerably faster than regular pillow

fallen patrol Apr 12, 2021, 6:58 PM

#

should i make an issue to
add https://pillow.readthedocs.io/en/stable/ to the docs command
inv: (https://pillow.readthedocs.io/en/stable/objects.inv)

#

nvm ^

patent pivot Apr 12, 2021, 6:58 PM

#

answered in meta this

#

(answer was we have it)

fallen patrol Apr 12, 2021, 6:59 PM

#

wait is it issued?

#

!src d

stable mountainBOT Apr 12, 2021, 6:59 PM

#

Command: docs

Look up documentation for Python symbols.

Source Code

Go to GitHub

patent pivot Apr 12, 2021, 6:59 PM

#

issued?

fallen patrol Apr 12, 2021, 6:59 PM

#

how do you add them?

patent pivot Apr 12, 2021, 6:59 PM

#

there is a bot command

#

or

#

fallen patrol Apr 12, 2021, 6:59 PM

#

dang

patent pivot Apr 12, 2021, 6:59 PM

#

fallen patrol Apr 12, 2021, 6:59 PM

#

so never an issue

sleek steppe Apr 12, 2021, 7:00 PM

#

!d PIL

stable mountainBOT Apr 12, 2021, 7:00 PM

#

PIL

patent pivot Apr 12, 2021, 7:00 PM

#

takes an integer like 0.3234234 and truncates it to a set length

#

!e ```py
integer = 0.1234123

print(f"{integer:.3f}")

stable mountainBOT Apr 12, 2021, 7:01 PM

#

@patent pivot :white_check_mark: Your eval job has completed with return code 0.

0.123

patent pivot Apr 12, 2021, 7:01 PM

#

yep, so the outputs in !ping are to 3 decimal places

fallen patrol Apr 12, 2021, 7:03 PM

#

python can do anything

#

except run fast

vale ibex Apr 12, 2021, 7:04 PM

#

!remind 2h try again <#dev-contrib message>

stable mountainBOT Apr 12, 2021, 7:04 PM

#

You're the boss!

Your reminder will arrive in 2 hours!

fallen patrol Apr 12, 2021, 7:04 PM

#

~~although if you use cython you can run faster than cpython~~

sleek steppe Apr 12, 2021, 7:05 PM

#

pypy

fervent sage Apr 12, 2021, 7:10 PM

#

on the subject of this sorta thing, something else that might interest you is this

sleek steppe Apr 12, 2021, 7:16 PM

#

You can combine them together too

>>> format(123123123.123123, ',.3f') # Does the same thing as f"{123123123.123123:,.3f}"
'123,123,123.123'

patent pivot Apr 12, 2021, 8:21 PM

#

alright what is ooming lol

fervent sage Apr 12, 2021, 8:22 PM

#

lmao

patent pivot Apr 12, 2021, 8:22 PM

#

lol is it the latex command

fervent sage Apr 12, 2021, 8:22 PM

#

so thats why lancebot is dying repeatedly

patent pivot Apr 12, 2021, 8:22 PM

#

yeah

#

lol

gritty wind Apr 12, 2021, 8:22 PM

#

do we not

patent pivot Apr 12, 2021, 8:22 PM

#

gritty wind Apr 12, 2021, 8:22 PM

#

have like

#

ouch

fervent sage Apr 12, 2021, 8:22 PM

#

just give it more ram and pretend everything is fine :)

patent pivot Apr 12, 2021, 8:23 PM

#

200Mi limit on lancebot

#

so

#

yeah

#

i could just give it more

#

lol

#

probably the latex command

gritty wind Apr 12, 2021, 8:23 PM

#

Can you

fervent sage Apr 12, 2021, 8:23 PM

#

is it a problem with constantly gaining ram or just that something (likely latex) needs more than it can give during one op

gritty wind Apr 12, 2021, 8:23 PM

#

pull eexact crash timestamps

patent pivot Apr 12, 2021, 8:24 PM

#

uhhh

#

yea

#

2021-04-12T19:44:19Z
2021-04-12T20:16:01Z
2021-04-12T224Z

#

great

#

brilliant emote

gritty wind Apr 12, 2021, 8:24 PM

#

lol

#

let's see if I know how to logs

patent pivot Apr 12, 2021, 8:25 PM

#

2021-04-12T19:44:19Z
2021-04-12T20:16:01Z
2021-04-12T20:20:24Z

gritty wind Apr 12, 2021, 8:26 PM

#

is the int e ~~patched~~ added yet

fervent sage Apr 12, 2021, 8:26 PM

#

im curious why for int e jishaku isnt used? is it just because the bot is meant to be do-it-yourself-y or is there another reason idk about?