#dev-contrib | Python | Page 87

gritty wind Feb 27, 2021, 5:11 AM

#

I mean it's not a full answer, but yeah technically the bot only uses the API for DB ineraction

#

We also use the API for the site, at least for now

fallen patrol Feb 27, 2021, 5:12 AM

#

is there a UI on the site for database interaction?

#

since ik there's a sign in on the site for admins 👀

gritty wind Feb 27, 2021, 5:12 AM

#

So it's a bit complex

#

Yes there is some interaction with the API from the site

#

but most of it is done with the bot

#

For example, one of our most crucial DBs is the infractions one

#

The one with all the warns, notes, bans, etc

#

Now you could technically view that from the site

#

but we mostly use the Python bot

#

Anyways, questions about the new API would best be directed to Sebastiaan whenever he's online

fallen patrol Feb 27, 2021, 5:14 AM

#

ah

short snow Feb 27, 2021, 5:26 AM

#

do we have a db on sir-lance?

gritty wind Feb 27, 2021, 5:26 AM

#

sort of

#

we have redis

#

but part of the sir-lancebot ethos is that it's easy to setup and contribute to

#

so no DB

short snow Feb 27, 2021, 5:27 AM

#

ah yes redis.

gritty wind Feb 27, 2021, 5:27 AM

#

Redis is technically a DB, but it's more volatile, so we can't store really important stuff on it

short snow Feb 27, 2021, 5:27 AM

#

how do u think i should make the wtf python command, just using regex?

gritty wind Feb 27, 2021, 5:28 AM

#

I'm not sure what you have in mind with regex?

short snow Feb 27, 2021, 5:28 AM

#

matching the example pattern

#

go to raw link of the readme.md

#

or maybe just using split, first ssplit for ### :eyes: Example and then so on ...

gritty wind Feb 27, 2021, 5:31 AM

#

So the comment claims all sections are h3s

#

and each of them has a section ID

short snow Feb 27, 2021, 5:31 AM

#

yes

gritty wind Feb 27, 2021, 5:32 AM

#

imo what would be best is to store a mapping of raw text version of the names to IDs

#

then fuzzy match

#

the raw names, and the inputs would be stripped of any formatting, and we'll just match A-Z

#

that should make it easy on the algorithm, and the user

#

all that would leave is fetching the content by IDs

#

I don't know if the document is structured in a way to make that possible?

#

Ah actually

#

we can avoid all that, and just steal it from the table of contents

#

that has a mapping of anchors to names

short snow Feb 27, 2021, 5:38 AM

#

gritty wind imo what would be best is to store a mapping of raw text version of the names to...

manually or through tasks and parsing?

gritty wind Feb 27, 2021, 5:38 AM

#

parsing on startup/when the data is missing from the cache

short snow Feb 27, 2021, 5:39 AM

#

gritty wind we can avoid all that, and just steal it from the table of contents

oohh yes

#

no need of ids

#

just the table

#

and spliting on the table and the next ###

gritty wind Feb 27, 2021, 5:39 AM

#

yesh, it has all the anchors, so you can just append to the base URL, and boom, you've got the subsection

short snow Feb 27, 2021, 5:40 AM

#

that would get us the link, and then we can use regex/splitting to get the content of that section

hardy gorge Feb 27, 2021, 8:39 AM

#

@short snow instead of building one monolithic application that does everything, the API provides a layer of abstraction, which means that the bot does not also has to handle the database (models, migrations, database connections). Instead, that's split up into a different application that exposes a straightforward API.

short snow Feb 27, 2021, 9:38 AM

#

hardy gorge <@736234578745884682> instead of building one monolithic application that does e...

Oh thanks, but i think that was meant for @fallen patrol

vocal wolf Feb 27, 2021, 9:46 AM

#

This needs reviewing: https://github.com/python-discord/site/pull/447

fallen patrol Feb 27, 2021, 3:51 PM

#

hardy gorge <@736234578745884682> instead of building one monolithic application that does e...

Right, but what advantage does that bring to the bot?

hardy gorge Feb 27, 2021, 4:40 PM

#

@fallen patrol It keeps the complexity of the bot lower and abstract database communication: You don't have to worry about managing a database from with the bot; that's handled by a dedicated application that offers an API to the bot. It's a pretty common design pattern these days: Instead of trying to do all things within one bot, you split up the services into multiple microservices.

fallen patrol Feb 27, 2021, 4:40 PM

#

Ah that makes sense.

plucky maple Feb 27, 2021, 8:41 PM

#

how does one request a feature? It might be a feature for the bot, or it might require a change in discord itself; dunno which

brazen charm Feb 27, 2021, 8:44 PM

#

You can open an issue on the bot repo https://github.com/python-discord/bot/issues (assuming it's for @stable mountain). Regarding on whether it's possible to implement in the first place maybe just ask here beforehand

plucky maple Feb 27, 2021, 8:46 PM

#

ya ok

#

so here's my "workflow" around here: I sit around and wait for a help channel to become occupied. I look in at it; and 95% of the time I can't help the guy, or they're already being helped, or whatever, so I mute the channel for an hour. The other 5% of the time, I start responding. I am rarely active in more than three such channels at once.

#

This works pretty well (and I suspect a lot of other helpers are doing something like this)

#

but "one hour" is totally arbitrary; what I really want is: "Mute until someone else claims this channel".

#

Oh and I want to be able to do this from the keyboard, too 🙂

brazen charm Feb 27, 2021, 8:53 PM

#

What channels are muted can't really be controlled outside of your client (I believe it doesn't even transfer between different clients on the same account), I think the best the bot could do is notify you on new channels but you can just see that from a new channel being added to the bottom

plucky maple Feb 27, 2021, 8:54 PM

#

yeah I figured it'd be something like that

#

probably requires a change in discord itself

clever wraith Feb 27, 2021, 8:54 PM

#

wow joe chatting

patent pivot Feb 27, 2021, 8:54 PM

#

Yeah, not feasible at Discord's side. They most likely won't add APIs for integrations to alter client things like that

patent pivot Feb 27, 2021, 8:55 PM

#

clever wraith wow joe chatting

clever wraith Feb 27, 2021, 8:56 PM

#

Are you the founder of python

#

@patent pivot

patent pivot Feb 27, 2021, 8:56 PM

#

No, I'm just an owner of this server.

clever wraith Feb 27, 2021, 8:56 PM

#

this is not the official discord server of python

patent pivot Feb 27, 2021, 8:57 PM

#

We're not official in the sense that we're run by the Python Software Foundation but we are listed at https://python.org/community/ and work closely with the PSF.

Python.org

Our Community

The official home of the Python Programming Language

clever wraith Feb 27, 2021, 8:59 PM

#

hmm okey

gritty wind Feb 28, 2021, 4:36 AM

#

plucky maple but "one hour" is totally arbitrary; what I *really* want is: "Mute until someon...

So I’ve wanted something like this for a while

#

We have an internal issue with possible solutions

#

I could look into reviving it

short snow Feb 28, 2021, 10:29 AM

#

#dev-log message Uhh shouldn't this type of PRs be directly pushed to master

#

also scal, there is some problem with the wtf python readme, it doesn't seem to follow the same pattern. I can send you the WIP cog file in DMs if u want it

gritty wind Feb 28, 2021, 10:31 AM

#

short snow https://discord.com/channels/267624335836053506/622895325144940554/8155182975334...

It doesn't really matter, this is just a formality

gritty wind Feb 28, 2021, 10:31 AM

#

short snow also scal, there is some problem with the wtf python readme, it doesn't seem to ...

Which ones don't follow the pattern?

short snow Feb 28, 2021, 10:32 AM

#

uhh, i can send you the code, it would be better to understand, since i have numbered them

#

like it logs Sending: {number}, when you do .wtf

gritty wind Feb 28, 2021, 10:34 AM

#

I don't really understand, but I'd much prefer you put it in the paste bin or push the changes

short snow Feb 28, 2021, 10:35 AM

#

alright, i will use a pastebin

short snow Feb 28, 2021, 10:36 AM

#

gritty wind I don't really understand, but I'd much prefer you put it in the paste bin or ...

https://paste.pythondiscord.com/sifumiqiga.py

#

there u go

#

lemme send the ones which are sent

#

Sending sr.no: 0
Sending sr.no: 1
Sending sr.no: 16
Sending sr.no: 25
Sending sr.no: 26
Sending sr.no: 40
Sending sr.no: 43
Sending sr.no: 55

gritty wind Feb 28, 2021, 10:38 AM

#

i don't understand what this means

#

What can I do to reproduce the bug?

short snow Feb 28, 2021, 10:40 AM

#

gritty wind What can I do to reproduce the bug?

just add the cog to bot.exts.evergreen and then do .wtf in your bot commands

#

and watch your terminal

#

first it sends the data as:

- regex
- findall match

gritty wind Feb 28, 2021, 11:00 AM

#

Why are you using the previous header in the search for the next header?

short snow Feb 28, 2021, 11:15 AM

#

am using the next header and the current header

#

to get the content between them

short snow Feb 28, 2021, 11:45 AM

#

( @gritty wind incase u missed it)

gritty wind Feb 28, 2021, 11:45 AM

#

I don't know

#

this seems like a very strange solution

#

how would it even work on the last entry?

#

I think you should rethink the approach, but I don't have time to look at it right now

#

If it's impossible, then just links are fine

short snow Feb 28, 2021, 11:47 AM

#

Alright i will try if i can find any other solution

patent pivot Feb 28, 2021, 2:04 PM

#

Anyone on Windows with Docker?

brazen charm Feb 28, 2021, 2:06 PM

#

Yep

patent pivot Feb 28, 2021, 2:07 PM

#

looking at your contrib guides issue on meta, trying to find the best solution

#

host.docker.internal works, but then people still need to add subdomains

gritty wind Feb 28, 2021, 2:09 PM

#

what are you doing?

patent pivot Feb 28, 2021, 2:11 PM

#

https://github.com/python-discord/meta/issues/60

GitHub

Update the contributing guides · Issue #60 · python-discord/meta

On windows, docker is now handled through docker desktop which already comes with compose installed making the docker compose requirement for the projects extraneous on the platform. In https://pyt...

#

we refer to a command that doesn't exist

#

On MacOS it seems that localhost works (or at least it does for me)

gritty wind Feb 28, 2021, 2:12 PM

#

Ah I see

brazen charm Feb 28, 2021, 2:12 PM

#

Yeah I'm using localhost now, not sure if it didn't work before or I didn't try it

gritty wind Feb 28, 2021, 2:12 PM

#

it's also localhost on windows

patent pivot Feb 28, 2021, 2:12 PM

#

huh

#

well

#

that simplifies things lol

#

this look correct then? https://pythondiscord.com/pages/contributing/hosts-file/

Python Discord | Preparing Your Hosts File

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.

gritty wind Feb 28, 2021, 2:14 PM

#

why don't we just have people access it through localhost? CORS?

patent pivot Feb 28, 2021, 2:15 PM

#

subdomainss

#

(which we will be deprecating soon, so eventually yeah we will host on localhost)

gritty wind Feb 28, 2021, 2:16 PM

#

hmm

#

everything else looks fine

green oriole Feb 28, 2021, 2:17 PM

#

Couldn't we slap a middleware in the config to get rid of subdomains inside docker? I guess that would be quite a bit of work

gritty wind Feb 28, 2021, 2:17 PM

#

too much work if we're gonna move away from the setup

brazen charm Feb 28, 2021, 2:18 PM

#

What issues are there with subdomains? seems to work fine for me

green oriole Feb 28, 2021, 2:18 PM

#

gritty wind too much work if we're gonna move away from the setup

Yeah true

gritty wind Feb 28, 2021, 2:19 PM

#

shit you're right

#

lmao the SO answer is wrong

brazen charm Feb 28, 2021, 2:19 PM

#

Ah, browser works but the bot gets a domain name error

patent pivot Feb 28, 2021, 2:24 PM

#

I've made most of the changes listed by people on that issue

#

i also captchad myself while submitting an article smh

gritty wind Feb 28, 2021, 2:25 PM

#

Sweet

#

while you're at it

#

could you just

patent pivot Feb 28, 2021, 2:25 PM

#

lol, for XSS

gritty wind Feb 28, 2021, 2:25 PM

#

yeet Contributing.MD

#

or make it not an essay

patent pivot Feb 28, 2021, 2:26 PM

#

lol

#

probably worth discussing with folks like

#

what we need in there

#

@green oriole hello thoughts

brazen charm Feb 28, 2021, 2:26 PM

#

Yeah its scope is maybe a bit too wide, and doesn't really look like new contribs read it

gritty wind Feb 28, 2021, 2:27 PM

#

there are also some rules in there that we don't follow at all, or in very limited contexts

brazen charm Feb 28, 2021, 2:27 PM

#

Broke the first "rule" quite a few times

gritty wind Feb 28, 2021, 2:28 PM

#

Yeah that one in particular is very over bearing

green oriole Feb 28, 2021, 2:28 PM

#

It is on the core dev meeting agenda

gritty wind Feb 28, 2021, 2:28 PM

#

the core dev meeting is a bit of a unicorn atm

green oriole Feb 28, 2021, 2:28 PM

#

Let me just..

gritty wind Feb 28, 2021, 2:30 PM

#

2 is very explicit, but most people feel more comfortable not following it, and 5 is meaningless. 7 feels very redundant, when we can simply tell people to comment on issues they want to be assigned to

patent pivot Feb 28, 2021, 2:31 PM

#

anyway I think that's the meta issue mostly resolved

#

so I'll close that and we'll look at the guides on GH in the meeting

gritty wind Feb 28, 2021, 2:37 PM

#

One more thing actually

#

https://pythondiscord.com/pages/contributing/bot/#using-docker

#

docker-compose requirement is only needed on linux

#

it comes included on windows and mac

#

same goes for site

#

it has already been changed by Chris on sir-lance

patent pivot Feb 28, 2021, 2:40 PM

#

updated

short snow Feb 28, 2021, 3:40 PM

#

Well @gritty wind I couldn't think of any other way, should I just use links? Or if someone has anything in mind they can just let me know

trim cradle Feb 28, 2021, 3:40 PM

#

@green oriole (as our glorious open source lead) this doesn't say how to run the unit tests: https://pythondiscord.com/pages/contributing/bot/

Python Discord | Contributing to Bot

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.

green oriole Feb 28, 2021, 3:41 PM

#

That is true

trim cradle Feb 28, 2021, 3:41 PM

#

how to do

gritty wind Feb 28, 2021, 3:41 PM

#

Pipenv run test

#

Or is it report

green oriole Feb 28, 2021, 3:41 PM

#

that

gritty wind Feb 28, 2021, 3:41 PM

#

One of those things

short snow Feb 28, 2021, 3:41 PM

#

Can anyone share a article or link on writing unit tests, never worked with them. Thanks

green oriole Feb 28, 2021, 3:41 PM

#

but it all described on the README in the tests folder

gritty wind Feb 28, 2021, 3:41 PM

#

Actually if you’re on pycharm they have a much stronger test suite you could use

green oriole Feb 28, 2021, 3:41 PM

#

https://github.com/python-discord/bot/blob/master/tests/README.md

GitHub

python-discord/bot

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

#

@short snow ^ this is a very good resource actually, and they are some additional resources at the bottom of the page

short snow Feb 28, 2021, 3:43 PM

#

green oriole <@!736234578745884682> ^ this is a very good resource actually, and they are som...

Tyvm

gritty wind Feb 28, 2021, 3:43 PM

#

Tl;dr for writing tests: input goes into modular component, correct output comes out

green oriole Feb 28, 2021, 3:43 PM

#

np

gritty wind Feb 28, 2021, 3:43 PM

#

Unit testing is then finding the input that is likely to go in, and cause problems

short snow Feb 28, 2021, 3:43 PM

#

And on what type of features do u write tests?

gritty wind Feb 28, 2021, 3:43 PM

#

If it’s bot

#

All of them

#

Though it isn’t enforced

green oriole Feb 28, 2021, 3:44 PM

#

Eeehhh

gritty wind Feb 28, 2021, 3:44 PM

#

We aim for 100% coverage, though that’s like a negative priority at the moment

green oriole Feb 28, 2021, 3:44 PM

#

We did say that 100% was stupid

short snow Feb 28, 2021, 3:44 PM

#

Ah alright 👌

gritty wind Feb 28, 2021, 3:45 PM

#

green oriole We did say that 100% was stupid

I think if we’ll have tests somewhere, it’ll be on features

#

Otherwise it’s mostly infrastructure to support the bot, which we probably won’t test as much

green oriole Feb 28, 2021, 3:45 PM

#

Hmm

#

I added a section linking to this document at the bottom of the guide

trim cradle Feb 28, 2021, 3:49 PM

#

for some reason I get permissions issues if I run pipenv run test but I can't run it with sudo

#

I'll figure it out, I guess

green oriole Feb 28, 2021, 3:50 PM

#

What do you get?

trim cradle Feb 28, 2021, 3:50 PM

#

PermissionError: [Errno 13] Permission denied: '/home/steele/projects/pydis/bot/logs/bot.log' this gets raised in every test

vale ibex Feb 28, 2021, 3:51 PM

#

Did you run pipenv run start under sudo?

#

That could cause permission errors on the log file

#

maybe a ~~chmod~~ chown could solve it

trim cradle Feb 28, 2021, 3:51 PM

#

vale ibex Did you run pipenv run start under sudo?

sudo can't see pipenv, so I'm just running pipenv run test exactly

#

(bot) steele@steele-mint:~/projects/pydis/bot$ sudo pipenv run start          
sudo: pipenv: command not found

short snow Feb 28, 2021, 3:53 PM

#

short snow Well <@386599446571384843> I couldn't think of any other way, should I just use ...

In case u misses it @gritty wind

#

Uhh try checking the perms of bot.log

green oriole Feb 28, 2021, 3:56 PM

#

trim cradle `PermissionError: [Errno 13] Permission denied: '/home/steele/projects/pydis/bot...

Obliterate the logs folder and make sure that you own the bot folder and that permissions are well set

trim cradle Feb 28, 2021, 3:57 PM

#

sudo rm -r logs/ && mkdir logs && touch logs/bot.log seems to have fixed it

#

but why didn't I have permissions for that folder...?

gritty wind Feb 28, 2021, 3:58 PM

#

Because you betrayed windows

green oriole Feb 28, 2021, 3:58 PM

#

It has been generated inside the container, so it was owned by root with aggressive permissions

trim cradle Feb 28, 2021, 4:08 PM

#

gritty wind Because you betrayed windows

!otn a wipe windows install linux the ultimate betrayal

#

let's see if I can just run the tests in the infraction directory

gritty wind Feb 28, 2021, 4:09 PM

#

If only it could fit

#

I think you can pass the module’s name to run it

#

Which might be something like bot.ext.infractions

#

Idk I let pycharm handle it all

green oriole Feb 28, 2021, 4:10 PM

#

You need slashes, maybe?

trim cradle Feb 28, 2021, 4:14 PM

#

gritty wind Idk I let pycharm handle it all

I don't think I've integrated the pipenv situation into the pycharm project

gritty wind Feb 28, 2021, 4:15 PM

#

Yeah it should still be pretty similar

#

You type out the module after the command and it’ll run that

fallen patrol Feb 28, 2021, 4:15 PM

#

trim cradle `!otn a wipe windows install linux the ultimate betrayal`

Do it 👀

trim cradle Feb 28, 2021, 4:21 PM

#

fallen patrol Do it 👀

It seems that Microsoft wants to improve their support for Python so I don't want to shit on their efforts

green oriole Feb 28, 2021, 4:22 PM

#

...lol

#

Can I though?

trim cradle Feb 28, 2021, 4:22 PM

#

I won't stop you

gritty wind Feb 28, 2021, 4:22 PM

#

I think it's too long anyways

trim cradle Feb 28, 2021, 4:22 PM

#

though I added an object is an instance of a class and I have no regrets about that one

fallen patrol Feb 28, 2021, 4:23 PM

#

gritty wind I think it's too long anyways

100 character limit for channel names :D

gritty wind Feb 28, 2021, 4:24 PM

#

> isinstance(object, class)

Syntaxerror

fallen patrol Feb 28, 2021, 4:25 PM

#

gritty wind \> isinstance(object, class) > Syntaxerror

> if object is class:

False

green oriole Feb 28, 2021, 4:25 PM

#

Well, MS adding more Python support is just them pointing their user to use broken installs just for the sake of using their services

gritty wind Feb 28, 2021, 4:25 PM

#

Ehh it's not worth the conflict lol

fallen patrol Feb 28, 2021, 4:25 PM

#

Mm, python3 in console opens windows store to install python....

#

Even if you already installed it.

green oriole Feb 28, 2021, 4:25 PM

#

If they really wanted to add more support they'd vendor CPython and call it a day

gritty wind Feb 28, 2021, 4:25 PM

#

fallen patrol Even if you already installed it.

that's untrue

#

python installs as python not python3

#

so python3 still opens the store

fallen patrol Feb 28, 2021, 4:26 PM

#

Even if you install python via the store tho

#

Which I did thinking it might stop it when I type python3

#

And it still opens the store 😔

gritty wind Feb 28, 2021, 4:26 PM

#

The command is just an alias for open store pretty much lol

fallen patrol Feb 28, 2021, 4:26 PM

#

green oriole Can I though?

Pleeeease add it.

gritty wind Feb 28, 2021, 4:26 PM

#

we're not going to add it

#

anyways I'm more used to python on linux too lol

trim cradle Feb 28, 2021, 4:27 PM

#

It looks like for the tests for the infraction system, it tests to make sure that the message that would get sent to the user for note infractions looks correct, and doesn't get sent

#

and that's quite a superfluous test.

green oriole Feb 28, 2021, 4:27 PM

#

fallen patrol Pleeeease add it.

Oh, I meant shitting on MS, this name is way too long haha

fallen patrol Feb 28, 2021, 4:27 PM

#

Afaik the limit for channel names is 100 characters, so it is possible

fallen patrol Feb 28, 2021, 4:28 PM

#

gritty wind anyways I'm more used to python on linux too lol

👀 I'll put it this way

#

I've used windows subsystem for Linux with vim installed and mounted my c drive instead of using notepad.

fallen patrol Feb 28, 2021, 4:29 PM

#

green oriole Oh, I meant shitting on MS, this name is way too long haha

Think I meant to reply to this lol

green oriole Feb 28, 2021, 4:30 PM

#

That's still really long, even if it is under the limit

fallen patrol Feb 28, 2021, 4:30 PM

#

!ot anyways, I'm now extremely offtopic, if someone wants to go there, I'll be there

#

On topic--

#

!src ot

stable mountainBOT Feb 28, 2021, 4:32 PM

#

Bad argument

Unable to convert 'ot' to valid command, tag, or Cog.

#

Command Help

!source [source_item]
Can also use: src

Display information and a GitHub link to the source code of a command, tag, or cog.

fallen patrol Feb 28, 2021, 4:32 PM

#

Why does that not bring up the tag?

green oriole Feb 28, 2021, 4:32 PM

#

Because ot is fuzzymatching the tag offtopic

fallen patrol Feb 28, 2021, 4:33 PM

#

It's off-topic

#

Which makes it hard to match

green oriole Feb 28, 2021, 4:33 PM

#

aah

fallen patrol Feb 28, 2021, 4:33 PM

#

green oriole Because `ot` is fuzzymatching the tag `offtopic`

You mean when I do !ot it's fuzzy matched and not coded as an alias?

green oriole Feb 28, 2021, 4:34 PM

#

yes

fallen patrol Feb 28, 2021, 4:34 PM

#

!ofto

#

Oh..... Okay

green oriole Feb 28, 2021, 4:34 PM

#

It is on cooldown

fallen patrol Feb 28, 2021, 4:34 PM

#

~~No i just deleted it~~

#

With the trashcan

fallen patrol Feb 28, 2021, 4:36 PM

#

green oriole Because `ot` is fuzzymatching the tag `offtopic`

!ofto

stable mountainBOT Feb 28, 2021, 4:36 PM

#

Off-topic channels

There are three off-topic channels:
• #ot0-psvm’s-eternal-disapproval
• #ot1-perplexing-regexing
• #ot2-never-nester’s-nightmare

Their names change randomly every 24 hours, but you can always find them under the OFF-TOPIC/GENERAL category in the channel list.

Please read our off-topic etiquette before participating in conversations.

fallen patrol Feb 28, 2021, 4:36 PM

#

Ah

vale ibex Feb 28, 2021, 7:49 PM

#

https://github.com/python-discord/sir-lancebot/pull/604 👀

#

From the staff meeting

gritty wind Feb 28, 2021, 7:49 PM

#

you're too quick

#

let me uhh

#

take a look

#

@vale ibex I'm looking at it

#

it'll work

#

but I think it may be a bit too unscoped

#

the channels and categories are used in a bunch of places

vale ibex Feb 28, 2021, 7:54 PM

#

Yea, I purposefully didn't add staff-voice to the global whitelist, in case it becomes too spammy

gritty wind Feb 28, 2021, 7:54 PM

#

if it's just the issues that we want to whitelist, we can do that pretty easily on its own

vale ibex Feb 28, 2021, 7:55 PM

#

But it's easy enough to do so

gritty wind Feb 28, 2021, 7:55 PM

#

I'll let more people chime in, but I don't mind it either way

#

My preference is locking it to the issues

vale ibex Feb 28, 2021, 7:56 PM

#

Are the channel and category constants used as whitelists in the entirety?

#

I saw the global whitelist just references certain channels/categories, rather than them all

gritty wind Feb 28, 2021, 7:57 PM

#

The categories are used in certain overrides

#

Not in too many places which is why I don't mind it

#

but also I don't see a harm in limiting the scope

#

Which is why I want others to chime in

vale ibex Feb 28, 2021, 7:58 PM

#

i tried to look for that, I couldn't see anywhere they were used in their entirety

#

IE cheatsheet.py uses Categories.help_in_use

gritty wind Feb 28, 2021, 7:59 PM

#

fair enough, I think I initially misunderstood this change

#

yeah this lgtm

#

I thought the whitelist changes were made directly to the global whitelist

#

but your solution is more what I had in mind

vocal wolf Feb 28, 2021, 7:59 PM

#

I did review thing

vale ibex Feb 28, 2021, 8:00 PM

#

Ahhh the global whitelist is line 303 in constants.py afaik

#

Should I just alphabetise them? @vocal wolf

#

I tried to follow the order that the channels appear in the server

vocal wolf Feb 28, 2021, 8:01 PM

#

well I don't think the rest of the constants file is alphabetized iirc

#

@gritty wind didn't you do a PR a very long time ago doing said alphabetization?

gritty wind Feb 28, 2021, 8:02 PM

#

python

vocal wolf Feb 28, 2021, 8:02 PM

#

right

gritty wind Feb 28, 2021, 8:02 PM

#

nightmare it was lol

vocal wolf Feb 28, 2021, 8:03 PM

#

@vale ibex ye it's fine for now, we'll probably make a PR at some point to attempt to alphabetize @dusky shore

#

just make the order the same

vale ibex Feb 28, 2021, 8:03 PM

#

Sure

#

Do you want me to change the issues cog to be in the order of the constants? or the other way around?

vocal wolf Feb 28, 2021, 8:15 PM

#

vale ibex Do you want me to change the issues cog to be in the order of the constants? or ...

issues in order of constants will do

#

@obsidian patio poke https://github.com/python-discord/bot/pull/1439

obsidian patio Feb 28, 2021, 8:15 PM

#

Hi mr. poker

vale ibex Feb 28, 2021, 8:15 PM

#

vocal wolf issues in order of constants will do

Heh, that's what I had guessed, done it already 😄

vocal wolf Feb 28, 2021, 8:15 PM

#

vale ibex Heh, that's what I had guessed, done it already 😄

Very cool

obsidian patio Feb 28, 2021, 8:58 PM

#

@vocal wolf I've added a final request to it. Are you fine with if I commit that suggestion directly and we can merge it?

vocal wolf Feb 28, 2021, 8:58 PM

#

yes

obsidian patio Feb 28, 2021, 9:00 PM

#

Perfect—done!

vocal wolf Feb 28, 2021, 9:01 PM

#

I'll merge when all checks pass

obsidian patio Feb 28, 2021, 9:01 PM

#

Nice nice

vocal wolf Feb 28, 2021, 9:01 PM

#

@obsidian patio ah god dammit we probably should've capitalized "the"

#

in the print statement

obsidian patio Feb 28, 2021, 9:02 PM

#

Oh yes. That was something I was gonna comment earlier today, but I forgot it now. Want me to commit it and we'll call it a day?

vocal wolf Feb 28, 2021, 9:03 PM

#

ok here's what we'll do

#

commit that and then I squash and force push then merge

obsidian patio Feb 28, 2021, 9:03 PM

#

Was just thinking the same. Lovely

vocal wolf Feb 28, 2021, 9:03 PM

#

very cool

#

tell me when it's done

#

alright

#

I see it, doing the thing now.

obsidian patio Feb 28, 2021, 9:05 PM

#

Nice, the thing

vocal wolf Feb 28, 2021, 9:11 PM

#

@obsidian patio force push success, I will now merge when checks have passed.

obsidian patio Feb 28, 2021, 9:11 PM

#

Nicee

vocal wolf Feb 28, 2021, 9:16 PM

#

@ashen isle thanks for the contribution, the tag has been merged!

ashen isle Feb 28, 2021, 9:16 PM

#

:3

obsidian patio Feb 28, 2021, 9:17 PM

#

Touchdown confirmed

vocal wolf Feb 28, 2021, 9:17 PM

#

Once the bot reconnects, we can then see the change

obsidian patio Feb 28, 2021, 9:17 PM

#

I've been waiting to say that for 15 minutes

vocal wolf Feb 28, 2021, 9:23 PM

#

!comparison

stable mountainBOT Feb 28, 2021, 9:23 PM

#

Assignment vs. Comparison

The assignment operator (=) is used to assign variables.

x = 5
print(x)  # Prints 5

The equality operator (==) is used to compare values.

if x == 5:
    print("The value of x is 5")

vocal wolf Feb 28, 2021, 9:23 PM

#

@ashen isle it works well

ashen isle Feb 28, 2021, 9:24 PM

#

yay

#

ugh, wait, that capital P is terrible

obsidian patio Feb 28, 2021, 9:24 PM

#

Hahah, that's pep 8

vocal wolf Feb 28, 2021, 9:25 PM

#

yep

#

It had to be done

vale ibex Feb 28, 2021, 10:11 PM

#

@gritty wind I have a commit in there to match the ordering 😄

gritty wind Feb 28, 2021, 10:12 PM

#

it doesn't though 🤔

#

devprojects would switch with the other dev one

#

also that's the categories, didn't the channels need to be rearranged too?

vale ibex Feb 28, 2021, 10:13 PM

#

Oh no, I moved development, but put it in the wrong place facepalm

gritty wind Feb 28, 2021, 10:14 PM

#

haha consider my approval permanent

vale ibex Feb 28, 2021, 10:15 PM

#

gritty wind also that's the categories, didn't the channels need to be rearranged too?

I couldn't spot what @vocal wolf meant by that one, both the constants and the cog go organisation...mod_meta...mod_tools...staff_voice

gritty wind Feb 28, 2021, 10:15 PM

#

I think the confusion was probably because it isn't in alphabetical order

#

it looks fine yeah

vale ibex Feb 28, 2021, 10:16 PM

#

want me to commit the fix and squash ?

#

so its just 1 reorder fix

gritty wind Feb 28, 2021, 10:16 PM

#

can we squash merge on sir-lance

vale ibex Feb 28, 2021, 10:16 PM

#

Sure

gritty wind Feb 28, 2021, 10:16 PM

#

nope

#

please force push then

vale ibex Feb 28, 2021, 10:16 PM

#

I mean we can squash before merging, but not on merge

vocal wolf Feb 28, 2021, 10:16 PM

#

disabled on repos, you have to do cli stuff

gritty wind Feb 28, 2021, 10:16 PM

#

ye

vocal wolf Feb 28, 2021, 10:17 PM

#

If it was ordered, then nevermind my comment lol

gritty wind Feb 28, 2021, 10:23 PM

#

eh I thought you'll merge the whole PR into one commit

#

but this is fine

vale ibex Feb 28, 2021, 10:23 PM

#

Mistakes have been forgotten

gritty wind Feb 28, 2021, 10:23 PM

#

xith apprrrrrove

vale ibex Feb 28, 2021, 10:23 PM

#

Yea, could have done that 😂

#

I guess the ordering issue was there before me, so having a separate commit to fix it isn't so bad

gritty wind Feb 28, 2021, 10:23 PM

#

(or request changes, I'm not your boss)

vocal wolf Feb 28, 2021, 10:56 PM

#

gritty wind xith apprrrrrove

omg ping me next time

#

@vale ibex @gritty wind merged

#

@obsidian patio congrats!

gritty wind Feb 28, 2021, 10:58 PM

#

Imagine not having push notifications

obsidian patio Feb 28, 2021, 11:00 PM

#

Thanks Xith

short snow Mar 1, 2021, 1:41 AM

#

@obsidian patio Congrats for Project Lead, which projwct lead are you btw?

#

Even scal and fire got it

#

and xith and lemon got new domain lead nice

obsidian patio Mar 1, 2021, 1:45 AM

#

Thanks

#

It's called "Hidden Gems," which is a YouTube series where we'll be showcasing different features of the python standard library, that most people don't know about already

short snow Mar 1, 2021, 1:50 AM

#

Oh cool logo_hidden_gems gem_red gem_red

deft patrol Mar 1, 2021, 2:48 AM

#

obsidian patio It's called "Hidden Gems," which is a YouTube series where we'll be showcasing d...

AWESOME 😄

#

In future you'll the mod one day 😄

#

@obsidian patio

fallen patrol Mar 1, 2021, 3:01 AM

#

Afaict, he's the only non mod or admin lead.

thorny obsidian Mar 1, 2021, 3:16 AM

#

I was just the Event Lead before I was asked to join the Admin team. So it's not unprecedented.

fallen patrol Mar 1, 2021, 3:18 AM

#

O

clever wraith Mar 1, 2021, 8:13 AM

#

whats a preferred way to learn sir lanbot?
directly fall into code or there are some good .md files?

gritty wind Mar 1, 2021, 8:14 AM

#

Hmm so

#

You should check out the contributing.md to set up the bot

#

but after that, its mostly just normal d.py cogs with your code

#

what issue are you looking to work on?

clever wraith Mar 1, 2021, 8:15 AM

#

i'm not exactly looking for any specific right now as i've got exams but i'll look for something simple once, and once i got an accepted pull, i can work for game like issues.

#

there's hangman one i think.

gritty wind Mar 1, 2021, 8:16 AM

#

So there may or may not be a recommended implementation on certain issues, which could help you when starting out

#

but if you've worked with discord.py cogs, you'll be fine

#

a good reference game could be the connect 4 game

clever wraith Mar 1, 2021, 8:16 AM

#

its in our repo?

gritty wind Mar 1, 2021, 8:17 AM

#

it should be yeah

#

.source connect4

dusky shoreBOT Mar 1, 2021, 8:17 AM

#

Command: connect_four

Play the classic game of Connect Four with someone!

Source Code

Go to GitHub

clever wraith Mar 1, 2021, 8:18 AM

#

makes sense. okay i'll play around with game to see how stuff works, and will check first-issue label for a simple start.

gritty wind Mar 1, 2021, 8:18 AM

#

sounds good, when you're ready to start working, just leave a comment on an issue

clever wraith Mar 1, 2021, 8:19 AM

#

like before starting?

gritty wind Mar 1, 2021, 8:19 AM

#

yes

#

that way you can get assinged to it

#

so we don't have multiple people working on one issue

clever wraith Mar 1, 2021, 8:19 AM

#

okay great. will do once i get my schedule clear.

#

thanks for help!

obsidian patio Mar 1, 2021, 11:52 AM

#

Many of you probably know about https://carbon.now.sh already, which lets you generate these nice images of your code

#

One idea that hit me is to make a .carbon command for @dusky shore, where you can post your code and get back the formatted result (as an image). Do you have any thoughts on this?

#

.carbon ```py
for i in range(5):
print(f"Hello number {i}")

#

^ This is an example of how it would be used

gritty wind Mar 1, 2021, 11:56 AM

#

yeah one major concern

#

does it have any form of API

obsidian patio Mar 1, 2021, 11:56 AM

#

I did some research, but couldn't find a very conclusive answer

gritty wind Mar 1, 2021, 11:56 AM

#

one thing I see is that you can load in github gists by appending to the URl

obsidian patio Mar 1, 2021, 11:57 AM

#

Yeah

gritty wind Mar 1, 2021, 11:57 AM

#

but the output seems much more difficult

#

and looks like it requires scrapping of some sort

obsidian patio Mar 1, 2021, 11:57 AM

#

Did you see Ben's VSCode tinder thing? That used carbon iirc

#

Reading through an issue on the repo, it also seems like a few people have made APIs for it and that they might be working on an official one

gritty wind Mar 1, 2021, 11:59 AM

#

Yup, was looking at that now

#

But that would be an additional service we have to host

#

which is something we try and avoid on sir-lance

obsidian patio Mar 1, 2021, 12:00 PM

#

Hmm, yeah

#

Do you think scraping the image would be too bad? A lot of people want to learn scraping, so this could be a legitimate use case for it

gritty wind Mar 1, 2021, 12:01 PM

#

I honestly don't even know if it's allowed

#

this is their robots

#

User-agent: *
Disallow: /*?*
Allow: /*

#

Ah yeah the ToS forbids it

#

https://www.notion.so/TERMS-OF-USE-ff2ce22a7e9848c89c6be46b44297583

obsidian patio Mar 1, 2021, 12:02 PM

#

Ah, alright

gritty wind Mar 1, 2021, 12:04 PM

#

hmm carbon-api does provide an online api it seems

#

From the readme:
https://carbonnowsh.herokuapp.com/?code=Made+with+Love+in+India+by+Sumanjay&theme=darcula&backgroundColor=rgba(144, 19, 254, 100)

#

I don't know why that link broke so hard

obsidian patio Mar 1, 2021, 12:05 PM

#

Spaces, I think

#

That's nice though. Means we might have a chance to do this

#

https://carbonnowsh.herokuapp.com/?code=print("hello world!")&theme=darcula&backgroundColor=rgb(55, 57, 62)

#

What do you think of the actual idea of a command like this?

gritty wind Mar 1, 2021, 12:11 PM

#

It seems cool, so thumbs up from me.

#

The implementation needs some thinking

obsidian patio Mar 1, 2021, 12:11 PM

#

Yeah, definitely. I personally don't have time to implement it, but I'm fairly confident that someone will want to do it

#

I'll open an issue for it

gritty wind Mar 1, 2021, 12:12 PM

#

hmm let me see if anyone has ideas about the implementation first

#

The API is fine, but I don't want to strain the service if it can be avoided

obsidian patio Mar 1, 2021, 12:13 PM

#

Yeah, understandable

#

!remind 48h Progress?

stable mountainBOT Mar 1, 2021, 12:14 PM

#

Yep.

Your reminder will arrive in 2 days!

short snow Mar 1, 2021, 12:17 PM

#

wait, pycharm and vscode has a plugin for this

#

and if they have a plugin, means they are somehow interacting with the website

#

that means even we can do it

gritty wind Mar 1, 2021, 12:17 PM

#

I think its probably generated in VSCode

short snow Mar 1, 2021, 12:17 PM

#

I will see if i can find anything on the net

gritty wind Mar 1, 2021, 12:17 PM

#

the ToS is explicitly against scrapping

#

and automation

short snow Mar 1, 2021, 12:18 PM

#

hmmm, and it is pretty fast, so maybe even we can implement it

#

Uhh, maybe after tomorrow's exam I can look into it

patent pivot Mar 1, 2021, 12:18 PM

#

The API above is also a ToS violation fwiw this

#

It uses Pyppeteer which is just headless chrome

#

all the APIs I see listed are just headless browser instances

#

the carbon devs seem open to adding an API, but no progress updates in a while

gritty wind Mar 1, 2021, 12:21 PM

#

ok so

#

I'm looking at the readme

#

they have a CLI

#

I wonder if that would violate ToS

#

I assume not because they list it in their readme

obsidian patio Mar 1, 2021, 12:22 PM

#

An official CLI?

gritty wind Mar 1, 2021, 12:23 PM

#

not official

#

but it's listed there

#

so its endorsed

#

https://github.com/mixn/carbon-now-cli

patent pivot Mar 1, 2021, 12:23 PM

#

interesting

#

all of these solutions use a headless browser

#

which i am very eh on

#

I'd rather not run an instance of chrome just to have lancebot run

short snow Mar 1, 2021, 12:24 PM

#

https://github.com/mixn/carbon-now-cli/blob/master/src/headless-visit.js

#

oh scal already posted it

patent pivot Mar 1, 2021, 12:25 PM

#

yeah

#

i'm going to say

#

the conditions for this to be implemented

short snow Mar 1, 2021, 12:25 PM

#

hmmm, what if we install the cli on the machine and call bash commands?

patent pivot Mar 1, 2021, 12:26 PM

#

that's still a headless chrome instance

gritty wind Mar 1, 2021, 12:26 PM

#

if you can't easily install it from code, no

short snow Mar 1, 2021, 12:26 PM

#

store the image locally, and delete it after every run

patent pivot Mar 1, 2021, 12:26 PM

#

nah

#

it's not about that

#

it's about the fact that the bot has to start up a browser

gritty wind Mar 1, 2021, 12:27 PM

#

An alternative

#

is to shelf this idea until an API is available

patent pivot Mar 1, 2021, 12:27 PM

#

My conditions:
– We can't run an instance of chrome locally for Lancebot just to generate code snippets, I have no idea if that would even work in Docker
– We can't violate the ToS, any solution must be made by/officially endorsed by the Carbon team
– We use an API, we can't expect users to install a suite of terminal commands for certain features, it should be implemented in Python

#

Yeah, that's my thoughts

gritty wind Mar 1, 2021, 12:27 PM

#

Or to host a pydis instance, but that's really ehhhhhh

patent pivot Mar 1, 2021, 12:27 PM

#

nah

green oriole Mar 1, 2021, 12:27 PM

#

Resources go brrr

short snow Mar 1, 2021, 12:28 PM

#

short snow hmmm, what if we install the cli on the machine and call bash commands?

what about this?

green oriole Mar 1, 2021, 12:28 PM

#

That doesn't sound like a wise use of our limited resources

#

Sounds like a lot of security concerns

patent pivot Mar 1, 2021, 12:28 PM

#

short snow what about this?

see my points

#

the CLI is just a browser instance

#

it starts up a browser and opens the carbon page

short snow Mar 1, 2021, 12:29 PM

#

oh yeah

#

my bad

patent pivot Mar 1, 2021, 12:29 PM

#

and also see the third point, we shouldn't use a command for this

short snow Mar 1, 2021, 12:29 PM

#

right

patent pivot Mar 1, 2021, 12:30 PM

#

I'm with Scale here that we should keep the idea at the back of our heads until there is some official API available

obsidian patio Mar 1, 2021, 12:30 PM

#

Unless they release the API, it doesn't sound too positive

short snow Mar 1, 2021, 12:30 PM

#

if we implement it on our like we do for image commands?

#

but only python code

patent pivot Mar 1, 2021, 12:30 PM

#

yes, we probably could write an implementation of it fairly easily

#

pygments would probably help with the highlightnig

gritty wind Mar 1, 2021, 12:31 PM

#

It would have to be very stripped down feature wise

patent pivot Mar 1, 2021, 12:31 PM

#

yeah

short snow Mar 1, 2021, 12:31 PM

#

alright, i will see if i can implement it. Maybe next week, once this school stuff gets over

#

or anyone else can

short snow Mar 1, 2021, 12:51 PM

#

sir-lancebot#600 and sir-lancebot#595 can anyone review these two?

dusky shoreBOT Mar 1, 2021, 12:51 PM

#

GitHub

PROpen [sir-lancebot] #600 Improve Wikipedia Command
PROpen [sir-lancebot] #595 Update Tic Tac Toe Command

fallen patrol Mar 1, 2021, 12:51 PM

#

@patent pivot that update 👀

#

#

I just don't see it

patent pivot Mar 1, 2021, 12:52 PM

#

uhhhhh

gritty wind Mar 1, 2021, 12:52 PM

#

hmm why is forms not signed

#

ssl error on my end

fallen patrol Mar 1, 2021, 12:52 PM

#

Ye, when I refresh, that happens

gritty wind Mar 1, 2021, 12:52 PM

#

but damn thats a cool screen lmao

patent pivot Mar 1, 2021, 12:52 PM

#

lol

#

i have an idea

gritty wind Mar 1, 2021, 12:53 PM

#

too many redirects

#

wat

#

oh yeah it's spamming requests

patent pivot Mar 1, 2021, 12:54 PM

#

hmmmmmm

fallen patrol Mar 1, 2021, 12:55 PM

#

Mmm yes I like permanent loading

#

#

~~Reminds me of everything I've coded~~

gritty wind Mar 1, 2021, 12:55 PM

#

hey the app works, ~~blame joe~~

patent pivot Mar 1, 2021, 12:55 PM

#

why is it only our forms subdomains

gritty wind Mar 1, 2021, 12:56 PM

#

are we serving anything else from a webpack bundle?

fallen patrol Mar 1, 2021, 12:56 PM

#

There's an app??

gritty wind Mar 1, 2021, 12:56 PM

#

app as in

#

web app

fallen patrol Mar 1, 2021, 12:57 PM

#

O

#

Also, not sure if this is supposed to happen

patent pivot Mar 1, 2021, 12:57 PM

#

that's intentional lol

#

it's not a real domain

fallen patrol Mar 1, 2021, 12:57 PM

#

Oh okay

patent pivot Mar 1, 2021, 12:57 PM

#

but uhhhhh yeah seems forms is returning a bad certificate

#

oh wait no it's not that was just me forgetting to send my client cert ignore lol

#

well it might be i'll need to debug something

gritty wind Mar 1, 2021, 12:58 PM

#

would that explain the too many redirects?

cold moon Mar 1, 2021, 12:59 PM

#

I get Invalid SSL certificate

#

If I click refresh

patent pivot Mar 1, 2021, 12:59 PM

#

gritty wind would that explain the too many redirects?

nah

#

yeah the cert is invalid

#

becauseeeeee uhhh

#

ah netlify is refusing to generate one

#

one second

gritty wind Mar 1, 2021, 12:59 PM

#

I get the ssl error too, but the too many redirects if I have console loaded

cold moon Mar 1, 2021, 1:00 PM

#

Sometimes with SSL error there is such redirect errors

gritty wind Mar 1, 2021, 1:00 PM

#

actually

#

I don't think this is netlify

#

forms-api is down too

#

unless that's netlify as well?

cold moon Mar 1, 2021, 1:01 PM

#

No, this is Kubernetes

gritty wind Mar 1, 2021, 1:01 PM

#

well the backend itself is, but there is a frontend component to it

#

with the swagger stuff and what not

patent pivot Mar 1, 2021, 1:02 PM

#

okay so

#

netlify cert gen for forms.pythondiscord.com isn't workingn

#

that should be fixed now but will take some time to propagate

cold moon Mar 1, 2021, 1:03 PM

#

Is this reason for backend SSL error also?

patent pivot Mar 1, 2021, 1:03 PM

#

as for the forms-api i'm looking into it

#

no

fallen patrol Mar 1, 2021, 1:03 PM

#

Now forms is just entirely dead

patent pivot Mar 1, 2021, 1:03 PM

#

lol

#

nah that's from netlify

fallen patrol Mar 1, 2021, 1:04 PM

#

patent pivot nah that's from netlify

patent pivot Mar 1, 2021, 1:05 PM

#

yeah

#

lol

#

I know why

#

backend is out

#

gritty wind Mar 1, 2021, 1:05 PM

#

hmmmmmm

#

why isn't it that lol

#

it should be

patent pivot Mar 1, 2021, 1:05 PM

#

uhhh

#

can only have one level of TLS with CF on the free plan iirc

gritty wind Mar 1, 2021, 1:06 PM

#

aww

#

oh well

fallen patrol Mar 1, 2021, 1:06 PM

#

Smh its down because you are on a free plan but TIL there's a free plan??

patent pivot Mar 1, 2021, 1:06 PM

#

it's not down because of that

gritty wind Mar 1, 2021, 1:06 PM

#

no its down because joe is bad at typing

patent pivot Mar 1, 2021, 1:06 PM

#

it's down because i forgot to change a parameter lul

#

okay backend is back up

#

https://forms-api.pythondiscord.com/

fallen patrol Mar 1, 2021, 1:07 PM

#

👁️

gritty wind Mar 1, 2021, 1:09 PM

#

so the frontend is no longer unsigned but

#

still spamming requests

patent pivot Mar 1, 2021, 1:09 PM

#

lol

#

can't reproduce

#

frontend is back here

#

no spam

gritty wind Mar 1, 2021, 1:10 PM

#

Time to uninstall chrome ig

#

Fine on my phone too, I’ll check back in a few

patent pivot Mar 1, 2021, 1:10 PM

#

lol

fallen patrol Mar 1, 2021, 1:11 PM

#

Same issue for forms here tho

patent pivot Mar 1, 2021, 1:12 PM

#

certs all fine here

#

you might have a cached DNS entry

cold moon Mar 1, 2021, 1:13 PM

#

I had to force reload

fallen patrol Mar 1, 2021, 1:13 PM

#

Okay it loaded

#

I take it forms still don't work tho

#

#

Still submitting

gritty wind Mar 1, 2021, 1:14 PM

#

No that’s just a broke part of the app

#

Still waiting on a review to fix it 👀

fallen patrol Mar 1, 2021, 1:15 PM

#

Is the broken part it didn't make me sign in first?

gritty wind Mar 1, 2021, 1:15 PM

#

Yesh

patent pivot Mar 1, 2021, 1:21 PM

#

okay things all look healthy now

fallen patrol Mar 1, 2021, 1:22 PM

#

Aside from the app just being broken 👀

#

It's new, yeah?

gritty wind Mar 1, 2021, 1:22 PM

#

Well it’s unfinished not really broken

#

But yeah

patent pivot Mar 1, 2021, 1:24 PM

#

yeah it's not in prod by any means

short snow Mar 1, 2021, 1:25 PM

#

is forms backend open for contribution?

patent pivot Mar 1, 2021, 1:26 PM

#

as of now we're keeping it staff-only & the issue tracker is internal, when we go into production we may revisit that though

short snow Mar 1, 2021, 1:33 PM

#

(are contributors allowed?)

gritty wind Mar 1, 2021, 1:33 PM

#

it's only staff at the moment

#

partially because of the internal tracker

fallen patrol Mar 1, 2021, 1:34 PM

#

is the internal tracker internal code?

gritty wind Mar 1, 2021, 1:34 PM

#

internal issue tracker, where we put all the tasks that need to be done

cold moon Mar 1, 2021, 1:36 PM

#

We can't open this page to public because this contain private information.

patent pivot Mar 1, 2021, 1:40 PM

#

Not at the moment, anyway

short snow Mar 1, 2021, 1:41 PM

#

ah alright

fallen patrol Mar 1, 2021, 1:58 PM

#

gritty wind internal issue tracker, where we put all the tasks that need to be done

right, but what runs it?

gritty wind Mar 1, 2021, 1:58 PM

#

Notion.so

fallen patrol Mar 1, 2021, 1:58 PM

#

closed or open source code, basically

gritty wind Mar 1, 2021, 1:59 PM

#

The issue tracker is hosted on notion, so I guess closed source?

#

We don't run the tracker

#

we just use it lol

fallen patrol Mar 1, 2021, 1:59 PM

#

ah

obsidian patio Mar 1, 2021, 2:04 PM

#

How would I set the site to debug mode?

short snow Mar 1, 2021, 2:05 PM

#

in config.yml just put DEBUG=True ig, lemme have a look

#

you are running the bot or the site?

obsidian patio Mar 1, 2021, 2:06 PM

#

Site

short snow Mar 1, 2021, 2:06 PM

#

alright

obsidian patio Mar 1, 2021, 2:07 PM

#

It seems like environment variables would be the way to go, but I can't seem to get it working correctly

gritty wind Mar 1, 2021, 2:07 PM

#

Are you using the docker-compose, and if not, what are you putting in your env vars?

obsidian patio Mar 1, 2021, 2:08 PM

#

I'm using docker-dompose, yeah

gritty wind Mar 1, 2021, 2:08 PM

#

It should be on by default

obsidian patio Mar 1, 2021, 2:08 PM

#

I'm trying to disable it

gritty wind Mar 1, 2021, 2:08 PM

#

Ah right

#

set it to 0

obsidian patio Mar 1, 2021, 2:08 PM

#

The question is where

gritty wind Mar 1, 2021, 2:08 PM

#

Docker compose

#

or if you can

short snow Mar 1, 2021, 2:08 PM

#

open docker-compose.yml

gritty wind Mar 1, 2021, 2:08 PM

#

your terminal's environment

#

it won't load an .env though

obsidian patio Mar 1, 2021, 2:09 PM

#

Aah, alright

short snow Mar 1, 2021, 2:09 PM

#

      DEBUG: 0 # set this to ), it is 1 as default

#

in web/environment:

obsidian patio Mar 1, 2021, 2:09 PM

#

Should I also remove the debug flag?

short snow Mar 1, 2021, 2:09 PM

#

and while using pipenv, just in .env in your root

short snow Mar 1, 2021, 2:10 PM

#

obsidian patio Should I also remove the debug flag?

yes remove it

obsidian patio Mar 1, 2021, 2:12 PM

#

Yep

gritty wind Mar 1, 2021, 2:12 PM

#

You shouldn't have to 🤷‍♂️

#

It's running without debug when I just change the 0 for me

short snow Mar 1, 2021, 2:13 PM

#

we have to, have a look at the manage.py

#

but it does normally too ig

#

it is over written

gritty wind Mar 1, 2021, 2:13 PM

#

or you can just test it and see that it'll literally work 😛

obsidian patio Mar 1, 2021, 2:13 PM

#

With the flag, it runs debug in the terminal

#

Without it, it runs it like production

#

But the site is the same, it seems

short snow Mar 1, 2021, 2:14 PM

#

debug only manages the errors

#

it will show you the errors with debug true otherwise it will show the error page

#

are u trying to test the pydis wallpaper for errors?

obsidian patio Mar 1, 2021, 2:16 PM

#

@gritty wind The Django documentation says that I should be able to just put a 404.html file in the root of templates/ and that it should then be used automatically as the 404 page. That's not what I'm experiencing though.

This is what I currently have

#

But this is what I get when requesting a page that doesn't exist

#

Oh wait... that's not a 404

gritty wind Mar 1, 2021, 2:17 PM

#

What page are you trying to visit

obsidian patio Mar 1, 2021, 2:17 PM

#

http://pythondiscord.local:8000/life

#

Just something random that shouldn't exist. I've tried other URLs as well

gritty wind Mar 1, 2021, 2:18 PM

#

http://pythondiscord.local:8000/asdsa

#

hmm that should 404

#

it does for me

obsidian patio Mar 1, 2021, 2:18 PM

#

Same for that one. It gives me 400

gritty wind Mar 1, 2021, 2:19 PM

#

I wonder if it's unique to the no debug

#

one sec

obsidian patio Mar 1, 2021, 2:19 PM

#

Oh, you know what. I just checked the logs (should've done that earlier tbh)

#

gritty wind Mar 1, 2021, 2:21 PM

#

yeaaaah that's why we need debug lol

short snow Mar 1, 2021, 2:21 PM

#

lol

obsidian patio Mar 1, 2021, 2:23 PM

#

Yes!

gritty wind Mar 1, 2021, 2:23 PM

#

shweet

short snow Mar 1, 2021, 2:23 PM

#

niceee

#

u are writing the error pages?

obsidian patio Mar 1, 2021, 2:24 PM

#

Making a 404 page

#

Possibly 500 as well

patent pivot Mar 1, 2021, 2:24 PM

#

related to #changelog heh

short snow Mar 1, 2021, 2:25 PM

#

look up on code pen for some sweet designs, and you can then use those designs for the error pages, but with your own taste

patent pivot Mar 1, 2021, 2:25 PM

#

it'll be something along the lines of https://PythonDiscord.com/.env

#

We've already rolled those designs out upstream at cloudflare but want to tie them into Django as well

short snow Mar 1, 2021, 2:26 PM

#

arghh, postegres started late than site, and site existed we need to fix this too ig

patent pivot Mar 1, 2021, 2:26 PM

#

Interesting

gritty wind Mar 1, 2021, 2:26 PM

#

site does actually wait for db

#

and retries like a million times

#

so if it failed all retries

#

well your DB isn't coming up

short snow Mar 1, 2021, 2:27 PM

#

#

it came up

patent pivot Mar 1, 2021, 2:28 PM

#

It waits for Postgres to start

#

But maybe not actually be ready

fallen patrol Mar 1, 2021, 2:28 PM

#

http://pythondiscord.local:8000/life
how do you set up stuff like this 👀

gritty wind Mar 1, 2021, 2:28 PM

#

patent pivot But maybe not actually be ready

can we do a postgres readiness check?

patent pivot Mar 1, 2021, 2:28 PM

#

hmmmmm

short snow Mar 1, 2021, 2:28 PM

#

i need to run docker-compose up web again

patent pivot Mar 1, 2021, 2:28 PM

#

we can check that the TCP port is open

obsidian patio Mar 1, 2021, 2:29 PM

#

Also, I need to confess something

patent pivot Mar 1, 2021, 2:29 PM

#

lol

vale ibex Mar 1, 2021, 2:29 PM

#

pg_isready exists, not sure if its present on the docker image though

obsidian patio Mar 1, 2021, 2:29 PM

#

This is the fine art of bodging

patent pivot Mar 1, 2021, 2:29 PM

#

hmm that's true chris

short snow Mar 1, 2021, 2:29 PM

#

how many migrations do we have lol

gritty wind Mar 1, 2021, 2:29 PM

#

all of them

obsidian patio Mar 1, 2021, 2:29 PM

#

Is there a better way to add it as an allowed host, without modifying the defaults?

patent pivot Mar 1, 2021, 2:30 PM

#

!remind 20M is Postgres alive?????????????!?!?? Is it????? Are you sure?????? Does donker know!??????

stable mountainBOT Mar 1, 2021, 2:30 PM

#

Okay.

Your reminder will arrive in 20 minutes!

gritty wind Mar 1, 2021, 2:30 PM

#

staying in debug lol

patent pivot Mar 1, 2021, 2:30 PM

#

obsidian patio Is there a better way to add it as an allowed host, without modifying the defaul...

there is an env var iirc

fallen patrol Mar 1, 2021, 2:30 PM

#

~~really thought that was 20 months~~

thorny obsidian Mar 1, 2021, 2:30 PM

#

donker

short snow Mar 1, 2021, 2:30 PM

#

lol

patent pivot Mar 1, 2021, 2:30 PM

#

donker and koobernoots

thorny obsidian Mar 1, 2021, 2:31 PM

#

(that's my helpful contribution of the day, you're welcome)

obsidian patio Mar 1, 2021, 2:31 PM

#

I tried env vars, but with little luck here. Eh, I'll just bodge it since it's just for my local testing

short snow Mar 1, 2021, 2:31 PM

#

!otn koobernoots-powered-pydis

thorny obsidian Mar 1, 2021, 2:33 PM

#

!otn a koobernoots

stable mountainBOT Mar 1, 2021, 2:33 PM

#

:ok_hand: Added koobernoots to the names list.

short snow Mar 1, 2021, 2:34 PM

#

on running docker-compose up the site is running on http://pythondiscord.local:8000 right?

gritty wind Mar 1, 2021, 2:35 PM

#

it's on localhost

#

You have to set the pythondiscord.local in your hosts

cold moon Mar 1, 2021, 2:36 PM

#

This is used because you can't do api.localhost:8000

obsidian patio Mar 1, 2021, 2:39 PM

#

#

Hmmmmm

#

A truly terrible Monty Python pun

fallen patrol Mar 1, 2021, 2:40 PM

#

👁️

#

please, I beg you, don't commit that

#

👀 also why is there not a robots.txt file? 🤔

green oriole Mar 1, 2021, 2:43 PM

#

Ah

#

Wasn't Scott working on this?

#

Like, ages ago

obsidian patio Mar 1, 2021, 2:44 PM

#

The 404 page?

brazen charm Mar 1, 2021, 2:44 PM

#

robots

short snow Mar 1, 2021, 2:44 PM

#

scott meaning ela?

obsidian patio Mar 1, 2021, 2:44 PM

#

Ah

green oriole Mar 1, 2021, 2:45 PM

#

Nah, that was another staffer who left

#

Alright, we definitely should have one

short snow Mar 1, 2021, 2:46 PM

#

always open a issue on it before you work
or is already opened vester

stable mountainBOT Mar 1, 2021, 2:50 PM

#

@patent pivot

It has arrived!

Here's your reminder: is Postgres alive?????????????!?!?? Is it????? Are you sure?????? Does donker know!??????.
[Jump back to when you created the reminder](#dev-contrib message)

thorny obsidian Mar 1, 2021, 2:51 PM

#

!otn a does-donker-know

stable mountainBOT Mar 1, 2021, 2:51 PM

#

:ok_hand: Added does-donker-know to the names list.

obsidian patio Mar 1, 2021, 2:58 PM

#

Is there a way to get the server invite in a Django template, instead of hardcoding the value?

gritty wind Mar 1, 2021, 3:01 PM

#

discord.gg/python?

obsidian patio Mar 1, 2021, 3:01 PM

#

I meant more if we had a constant for it, so it's not hardcoded

#

Looking through the source code, it seems like it's hardcoded in other places as well though

gritty wind Mar 1, 2021, 3:02 PM

#

yeah that's probably fine

#

it will be painful if we lose our vanity url, but its fine for now

obsidian patio Mar 1, 2021, 3:02 PM

#

Yeah

fallen patrol Mar 1, 2021, 3:45 PM

#

who runs @tropic flowerbot and where is the source?

gritty wind Mar 1, 2021, 3:45 PM

#

Joeee made it

#

Source in pins for #discord-bots

fallen patrol Mar 1, 2021, 3:45 PM

#

clearly the right user

#

@lusty lantern

patent pivot Mar 1, 2021, 3:46 PM

#

wait the source is pinned? Lol

gritty wind Mar 1, 2021, 3:46 PM

#

Maybe

#

There is one that specifically mentions steakbot

fallen patrol Mar 1, 2021, 3:47 PM

#

yeah but no source

patent pivot Mar 1, 2021, 3:47 PM

#

good lol

#

source is horrible

fallen patrol Mar 1, 2021, 3:47 PM

#

@patent pivot 👀

patent pivot Mar 1, 2021, 3:47 PM

#

lol

fallen patrol Mar 1, 2021, 3:48 PM

#

why is this bot so old when it uses late 2020 features?

#

what did it use to doooo

#

oh jeez I regret looking at joe's github

patent pivot Mar 1, 2021, 3:49 PM

#

https://gist.github.com/jb3/47c0496920dd61a50ed9c503d7d676e9

Gist

A simple implementation of Discord Slash Commands in Python with Mi...

A simple implementation of Discord Slash Commands in Python with Microsoft Function Apps. - LICENSE

#

why regret lol

patent pivot Mar 1, 2021, 3:49 PM

#

fallen patrol *why is this bot so old when it uses late 2020 features?*

had purpose before slash commands and now repurposed ez

obsidian patio Mar 1, 2021, 3:49 PM

#

Any idea what's up?

patent pivot Mar 1, 2021, 3:49 PM

#

that's an implementation of slash commands with azure

obsidian patio Mar 1, 2021, 3:49 PM

#

This is the thing I added the stylesheet to, instead of embedding it into the HTML

patent pivot Mar 1, 2021, 3:50 PM

#

hmmmm

#

not sure

#

Any other details in console?

#

that should be accompanied by an error

fallen patrol Mar 1, 2021, 3:50 PM

#

what server framework is it using?

patent pivot Mar 1, 2021, 3:50 PM

#

steakbot?

obsidian patio Mar 1, 2021, 3:51 PM

#

This is all I see. Where do you want me to look?

fallen patrol Mar 1, 2021, 3:51 PM

#

no, gurkan

patent pivot Mar 1, 2021, 3:51 PM

#

Django

patent pivot Mar 1, 2021, 3:51 PM

#

obsidian patio This is all I see. Where do you want me to look?

hmm not sure why errors aren't getting logged

fallen patrol Mar 1, 2021, 3:51 PM

#

patent pivot why regret lol

you have like 7 forked github bots 😂

patent pivot Mar 1, 2021, 3:51 PM

#

fallen patrol you have like 7 forked github bots 😂

do I? lol

fallen patrol Mar 1, 2021, 3:51 PM

#

@obsidian patio 👀 didn't you turn off debug mode

eager fable Mar 1, 2021, 3:51 PM

#

Hi Joe :)

obsidian patio Mar 1, 2021, 3:52 PM

#

Was actually just writing about that. The site isn't running in debug mode, but the terminal runs in debug mode, if that makes sense

patent pivot Mar 1, 2021, 3:52 PM

#

hm

fallen patrol Mar 1, 2021, 3:52 PM

#

patent pivot do I? lol

yeah, and another several discord related repos, including one that allows you to use the api from command line :sus:

obsidian patio Mar 1, 2021, 3:53 PM

#

This is where I have the css file, and I'm trying to access it via ```html
<link rel="stylesheet" href="{% static "css/error_pages.css" %}">

patent pivot Mar 1, 2021, 3:53 PM

#

fallen patrol yeah, and another several discord related repos, including one that allows you t...

that's not sus that's legit usage lol

#

it doesn't use any tokens

fallen patrol Mar 1, 2021, 3:53 PM

#

i mean, how do you authenicate it

#

ohhhhh

patent pivot Mar 1, 2021, 3:53 PM

#

it doesn't lol

fallen patrol Mar 1, 2021, 3:53 PM

#

wat the f*ck

patent pivot Mar 1, 2021, 3:53 PM

#

right now it just uses the invite API

fallen patrol Mar 1, 2021, 3:53 PM

#

:squints:

patent pivot Mar 1, 2021, 3:53 PM

#

lol

#

invite api is unauthenticated

#

so if you have an invite you can fetch new details

fallen patrol Mar 1, 2021, 3:55 PM

#

ah

#

yeah, @stable mountain uses that for invite whitelisting

#

surprised more bots don't do that

#

!shrug

patent pivot Mar 1, 2021, 3:55 PM

#

yeah

fallen patrol Mar 1, 2021, 3:55 PM

#

~~smh dead bots~~

#

anyways, time to attempt to git

obsidian patio Mar 1, 2021, 3:57 PM

#

Aah, got it

#

I needed to {% load static %}

harsh path Mar 1, 2021, 4:03 PM

#

ugh why does flake8 pass locally but not on the repo, is there something extra i need to do to check import order?

gritty wind Mar 1, 2021, 4:03 PM

#

It may not be using the plugins

#

how are you running it?

harsh path Mar 1, 2021, 4:04 PM

#

i'm just running pre-commit

gritty wind Mar 1, 2021, 4:04 PM

#

that should only install it I think?

#

Do you see passed in the output

#

pipenv run lint

harsh path Mar 1, 2021, 4:06 PM

#

it says everything passed

#

i figured it out

gritty wind Mar 1, 2021, 4:09 PM

#

It really shouldn't have passed

#

it may not be using the config

#

but that would highly depend on the way you're running things

short snow Mar 1, 2021, 4:22 PM

#

dusky shore

Bump.

#

https://github.com/python-discord/error-pages private repo?

obsidian patio Mar 1, 2021, 4:24 PM

#

Yep

gritty wind Mar 1, 2021, 4:25 PM

#

eee that looks awesome

#

It looks like wallpaper

#

pertty cool

short snow Mar 1, 2021, 4:25 PM

#

True

obsidian patio Mar 1, 2021, 4:25 PM

#

Hahah thanks scale

short snow Mar 1, 2021, 4:26 PM

#

Can we use constants for bg and discord invite?

subtle kraken Mar 1, 2021, 4:27 PM

#

@short snow I imagine you have tested wiki?

short snow Mar 1, 2021, 4:28 PM

#

Yes

#

check the screenshots

subtle kraken Mar 1, 2021, 4:28 PM

#

ah yes per usual I disregard the convo and just look at the code

#

sorry

short snow Mar 1, 2021, 4:29 PM

#

no problem, even i do that sometimes

short snow Mar 1, 2021, 4:31 PM

#

short snow Can we use constants for bg and discord invite?

@obsidian patio

subtle kraken Mar 1, 2021, 4:36 PM

#

@short snow 🎉

#

I sure hope I didn't miss some breaking thing that'll break the bot

short snow Mar 1, 2021, 4:36 PM

#

subtle kraken <@!736234578745884682> 🎉

Tyvm

#

U didn’t 😛

subtle kraken Mar 1, 2021, 4:38 PM

#

very nice improvements btw

#

looks really clean

short snow Mar 1, 2021, 4:38 PM

#

lemon_fingerguns_shades

#

#sir-lancebot-playground lemon_hearteyes

green oriole Mar 1, 2021, 6:01 PM

#

@obsidian patio it should be 2018-2021 IMO

#

Not just 2021

obsidian patio Mar 1, 2021, 6:01 PM

#

That was fast

green oriole Mar 1, 2021, 6:02 PM

#

I am always watching the dev log haha

obsidian patio Mar 1, 2021, 6:02 PM

#

Ahahah

#

But yeah, I’ll make it a span

gritty wind Mar 1, 2021, 6:04 PM

#

I don't think we do license bumps

#

Joe explained the legality behind it

#

but I'm too dumb to explain it

obsidian patio Mar 1, 2021, 6:04 PM

#

Hmmm, okay

gritty wind Mar 1, 2021, 6:05 PM

#

keep it up until he chimes in though

green oriole Mar 1, 2021, 6:05 PM

#

DEVOOOOPPSSS DAAAADDYYY

patent pivot Mar 1, 2021, 6:06 PM

#

@obsidian patio license bumps are not necessary

#

there is an implicit copyright on code from the minute it is conceived as an idea

obsidian patio Mar 1, 2021, 6:06 PM

#

Hmm

#

Yeah

patent pivot Mar 1, 2021, 6:07 PM

#

though i will do some research

#

and ask my lawyer

gritty wind Mar 1, 2021, 6:07 PM

#

lol so fancy

obsidian patio Mar 1, 2021, 6:07 PM

#

That is only a flex

green oriole Mar 1, 2021, 6:07 PM

#

"I'll ask my lawyer"

#

I am calling the flex police, sir

rapid swallow Mar 1, 2021, 6:07 PM

#

"I'll consult my lawyers"

patent pivot Mar 1, 2021, 6:08 PM

#

lol

obsidian patio Mar 1, 2021, 6:08 PM

#

But yeah, sure thing. I’ll let you close/merge the PR when you decide

patent pivot Mar 1, 2021, 6:08 PM

#

the top tip for doing your own legal research

#

is don't do your own legal research

#

okay lawyer is typing

gritty wind Mar 1, 2021, 6:08 PM

#

is that legal advice

patent pivot Mar 1, 2021, 6:10 PM

#

lol

#

okay so

#

– copyright registration is a US thing because US copyright law is "fucking abysmal", but it's useful elsewhere for reference
– changing the copyright date is actually pretty harmful in some cases, because if you update the date you are no longer claiming copyright for the time period before that, therefore if someone else publishes the work then you making a claim on it is difficult

if we really do want to update the date we should instead add a new date so like ©️ 2020, 2022 if we make substantial modification

thorny obsidian Mar 1, 2021, 6:16 PM

#

copyright and trademarks is a hell spawn of a legal field

patent pivot Mar 1, 2021, 6:16 PM

#

yes

#

that

#

I guess we like

#

could do 2020-2021

#

but even then that's stupid

#

because we have 95 years of copyright already under US copyright law

#

so why even bother capping it lol

#

though

#

hm

#

okay more legal research needed lol

#

there is some guidance from the FSF which says yes you should include the year as a comma separated item in the license file providing that you have made copyrightable changes in that year (so you can't just bump as the first change in the year)

#

okay, I don't think it's necessary

fallen patrol Mar 1, 2021, 6:36 PM

#

👀

fallen patrol Mar 1, 2021, 6:37 PM

#

patent pivot is don't do your own legal research

but how do you find the lawyer in the first place

#

asking for a friend, clearly

green oriole Mar 1, 2021, 6:40 PM

#

By being Joe, of course

#

Isn't your day a lawyer actually?

patent pivot Mar 1, 2021, 6:49 PM

#

green oriole Isn't your day a lawyer actually?

he's legally qualified, but not a lawyer

patent pivot Mar 1, 2021, 6:50 PM

#

fallen patrol *but how do you find the lawyer in the first place*

good networking yeah

fallen patrol Mar 1, 2021, 6:50 PM

#

👁️

green oriole Mar 1, 2021, 6:58 PM

#

Aha, that's the catch

fallen patrol Mar 1, 2021, 7:00 PM

#

^ i don't have that 😔

#

does anyone here aside from joe have it, lol

green oriole Mar 1, 2021, 7:01 PM

#

Nah

#

Joe is like an edge router

#

He has kind of little relations with everyone

clever wraith Mar 1, 2021, 10:29 PM

#

is the .pyfacts PR good? it's sir-lancebot#596

dusky shoreBOT Mar 1, 2021, 10:29 PM

#

GitHub

PROpen [sir-lancebot] #596 Add .pyfacts command

slender grove Mar 2, 2021, 1:37 AM

#

Wow github

short snow Mar 2, 2021, 1:39 AM

#

Are u trying to get voice verified?

ember cloud Mar 2, 2021, 3:05 AM

#

should there be a !environment-variables command?

#

it'd be really really useful

vocal prairie Mar 2, 2021, 3:21 AM

#

ember cloud should there be a !environment-variables command?

That's a suggestion that would probably be better off in #community-meta where it can thrive amongst its peers.

ember cloud Mar 2, 2021, 3:23 AM

#

either channel seems fine

fallen patrol Mar 2, 2021, 5:31 AM

#

add !resources to the cooldown like !ot and others are

#

reference #ot0-psvm’s-eternal-disapproval message

short snow Mar 2, 2021, 5:53 AM

#

@vocal wolf , the feature is already done

#

it is waiting for review

#

i did one approval.

vocal wolf Mar 2, 2021, 5:53 AM

#

oh really?

#

oops

#

comment removed

#

ah I didn't see the PR linked

#

rip

short snow Mar 2, 2021, 5:54 AM

#

lol

#

review it

vocal wolf Mar 2, 2021, 5:55 AM

#

you don't tell me what to do ~~I already am xd~~

short snow Mar 2, 2021, 5:55 AM

#

😛 nice

#

sir-lancebot#595 and this too if u have time, a very shoort pr

dusky shoreBOT Mar 2, 2021, 5:56 AM

#

GitHub

PROpen [sir-lancebot] #595 Update Tic Tac Toe Command

eternal owl Mar 2, 2021, 6:35 AM

#

Hey @vocal wolf , regarding the pride event calendar, can we just make an all year round calendar rather than for pride only

vocal wolf Mar 2, 2021, 6:36 AM

#

I suggest making an issue and seeing what other people think, I'm not too sure myself.

eternal owl Mar 2, 2021, 6:36 AM

#

Okay

vocal wolf Mar 2, 2021, 6:36 AM

#

Thanks!

#

❤️

eternal owl Mar 2, 2021, 6:36 AM

#

I will make one when I reach home, current in uni 😬

short snow Mar 2, 2021, 6:37 AM

#

Go study!

eternal owl Mar 2, 2021, 6:37 AM

#

vocal wolf ❤️

lemon_fingerguns_shades

eternal owl Mar 2, 2021, 6:37 AM

#

short snow Go study!

Lunch break

green oriole Mar 2, 2021, 6:37 AM

#

If it is a year around calendar I don't think we should restrict it to the pride season

eternal owl Mar 2, 2021, 6:37 AM

#

short snow Mar 2, 2021, 6:37 AM

#

Which uni?

eternal owl Mar 2, 2021, 6:38 AM

#

PESU, Bangalore

short snow Mar 2, 2021, 6:38 AM

#

Oh

eternal owl Mar 2, 2021, 6:38 AM

#

green oriole If it is a year around calendar I don't think we should restrict it to the pride...

Yes, there was a discussion before on using Google calendar

green oriole Mar 2, 2021, 6:38 AM

#

Interesting idea

eternal owl Mar 2, 2021, 6:38 AM

#

That discussion was 2 years ago

#

In the prev Dev contrib channel

green oriole Mar 2, 2021, 6:39 AM

#

Haha, man that's old

eternal owl Mar 2, 2021, 6:39 AM

#

Ye

green oriole Mar 2, 2021, 6:39 AM

#

I probably wasn't even here

eternal owl Mar 2, 2021, 6:40 AM

#

#dev-contrib message

#

@green oriole

green oriole Mar 2, 2021, 6:41 AM

#

Hmm, it sounds like a lot of work

#

I don't think we want to host any web page for now as we are trying to simplify the site as much as possible

eternal owl Mar 2, 2021, 6:42 AM

#

Alright

vocal wolf Mar 2, 2021, 6:42 AM

#

damn 2 years ago lol

eternal owl Mar 2, 2021, 6:42 AM

#

We can talk about this during staff meet I guess if we want a pydis calendar too

green oriole Mar 2, 2021, 6:44 AM

#

Yeah, that could work

deft patrol Mar 2, 2021, 6:54 AM

#

vocal wolf damn 2 years ago lol

haha ye

vocal wolf Mar 2, 2021, 7:04 AM

#

https://github.com/python-discord/sir-lancebot/pull/482

GitHub

Startup channel check by WillDaSilva · Pull Request #482 · python-d...

Relevant Issues
Closes #393
Description
At startup, before the bot sends the "Connected!" message in the dev_log channel, a list of the available channels is obtained, and the cha...

#

needs reviews

vocal wolf Mar 2, 2021, 7:29 AM

#

@obsidian patio how close are you to finishing sir-lancebot#436?? This is the best feature lol

dusky shoreBOT Mar 2, 2021, 7:29 AM

#

GitHub

IssueOpen [sir-lancebot] #436 Beardify — Lemoji beard structure text generator

vocal wolf Mar 2, 2021, 7:29 AM

#

Just make sure you have a cooldown and all so we don't break the CPU

obsidian patio Mar 2, 2021, 7:33 AM

#

Hahahah, that is one of my favorite features as well

#

I had a working implementation, but it wasn't pythonic enough and required way too much I/O (don't think it would work with the new setup)

#

Right now, it's a bit on hold

vocal wolf Mar 2, 2021, 7:37 AM

#

alright

#

thanks for the update!

short snow Mar 2, 2021, 7:39 AM

#

vocal wolf needs reviews

Alright maybe am free in the evening, will give it a go

vocal wolf Mar 2, 2021, 7:39 AM

#

short snow Alright maybe am free in the evening, will give it a go

already merged lol

short snow Mar 2, 2021, 7:39 AM

#

Uhh ok

#

lol

vocal wolf Mar 2, 2021, 7:39 AM

#

ak and ks went over it

timid sentinel Mar 2, 2021, 9:06 AM

#

Would allowing snekbox, say, a 10 PID limit instead of 1 still offer a similar level of security?

gritty wind Mar 2, 2021, 9:06 AM

#

well it would make certain things much more difficult to secure against I would imagine

#

right now you can't do bash commands, do network requests, etc, etc

#

why do you want to raise it?

timid sentinel Mar 2, 2021, 9:10 AM

#

It was more of a theoretical question than thinking of any use cases in particular, although my thinking was that theoretically you should be able to do anything you could do from creating another process, in the python process, so the PID limit is mainly to stop people spam-creating a bunch of processes? I'm not sure if that's right though

patent pivot Mar 2, 2021, 9:27 AM

#

hmmm

#

the PID limit is how we lock things to Python yeah

gritty wind Mar 2, 2021, 9:30 AM

#

Hmm here's something interesting

#

the http_connecter is breaking for me

#

I can't use snekbox at all from the bot

brazen charm Mar 2, 2021, 9:30 AM

#

Could the thread limit be raised? Or is the creation of those limited in some other way

patent pivot Mar 2, 2021, 9:41 AM

#

hmmm

#

I don't think we restrict threads at all

short snow Mar 2, 2021, 9:41 AM

#

How is this scal?