#dev-contrib | Python | Page 133

short snow Sep 6, 2021, 3:11 PM

#

i tried making a bot setup but it wasn't really possible in a 100% sure way (for future updates also), and only possible solution was hard coding

#

since the key vars and the channel names aren't 100% same, and not all all configured channels are contained in the template

gritty wind Sep 6, 2021, 3:13 PM

#

Currently inactive, not enough manpower/demand

green oriole Sep 6, 2021, 3:13 PM

#

lol, wait until it passes

#

okay, you can review now

short snow Sep 6, 2021, 3:15 PM

#

akarys is there a write up for the smart config solution, like how it would be working?

green oriole Sep 6, 2021, 3:15 PM

#

Uhh, maybe on the PR itself?

short snow Sep 6, 2021, 3:16 PM

#

nah the pr description isn't really descriptive

#

probably this, lemme go through it smartconfig#5

dusky shoreBOT Sep 6, 2021, 3:17 PM

#

GitHub

IssueOpen Plan for this project

green oriole Sep 6, 2021, 3:18 PM

#

I mean, if you are curious you can read through #5 and #10, but I would say so far smartconfig is an internal project, I'm afraid

short snow Sep 6, 2021, 3:20 PM

#

you probably can lol, ferrisPensive okay

#

git rebase upstream/main right? to get the new changes into my branch

clever wraith Sep 6, 2021, 3:24 PM

#

Time to go RTD diving

#

Holy christ

gritty wind Sep 6, 2021, 3:27 PM

#

#dev-log message

#

While true

#

It'll probably never be fixed because we won't open a PR specifically for this

#

I don't think anyone will mind if you kaizen it

#

cc FrenchSosig

green oriole Sep 6, 2021, 3:32 PM

#

Mh. I guess it doesn't hurt anyone as those lines are already changed. I can do that later on.

vale ibex Sep 6, 2021, 3:36 PM

#

I missed this on the bot too

#

smh

green oriole Sep 6, 2021, 3:36 PM

#

Honesty I would have requested the same changes if I saw that haha

vale ibex Sep 6, 2021, 3:36 PM

#

😂

short snow Sep 6, 2021, 3:38 PM

#

after 8 git rebase --continue, finally solved the conflicts 😄

#

next pR time

green oriole Sep 6, 2021, 3:38 PM

#

Thank you sir Chris

#

Sir JL dare I say

vale ibex Sep 6, 2021, 3:39 PM

#

lol

green oriole Sep 6, 2021, 3:45 PM

#

Did you download them one by one haha

short snow Sep 6, 2021, 3:45 PM

#

please upload them somewhere for me

green oriole Sep 6, 2021, 3:45 PM

#

Hahaha

short snow Sep 6, 2021, 3:46 PM

#

zip and dms

green oriole Sep 6, 2021, 3:46 PM

#

You know you could have just asked someone to make a zip with an int e?

short snow Sep 6, 2021, 3:46 PM

#

nor me until now

green oriole Sep 6, 2021, 3:46 PM

#

Haha

short snow Sep 6, 2021, 3:46 PM

#

wait can't you just join the test server now

#

rick

vocal prairie Sep 6, 2021, 3:47 PM

#

That's only staff iirc

green oriole Sep 6, 2021, 3:47 PM

#

Yep

#

There are some env vars

vocal prairie Sep 6, 2021, 3:47 PM

#

Nah, there are just a couple env vars

green oriole Sep 6, 2021, 3:48 PM

#

But for only a few constants, if you wish to add some feel feee

gritty wind Sep 6, 2021, 3:48 PM

#

All are documented in the page you linked, but you only need to set the ones in the first table

#

https://pythondiscord.com/pages/guides/pydis-guides/contributing/sir-lancebot/env-var-reference

short snow Sep 6, 2021, 3:48 PM

#

@thorny obsidian I have mentioned your review on sir-lancebot#618 and resolved the $u$king conflicts 🙂

dusky shoreBOT Sep 6, 2021, 3:48 PM

#

GitHub

PROpen Wikiguess Game

green oriole Sep 6, 2021, 3:48 PM

#

Sus-king, hmm

short snow Sep 6, 2021, 3:49 PM

#

BOT_TOKEN="Iain'tgivingthistoyoumyboy"
BOT_GUILD=814190307309781032

BOT_ADMIN_ROLE_ID=814190307330883591
ROLE_HELPERS=814190307330883585
ROLE_CORE_DEVELOPERS=814190307330883588

CHANNEL_ANNOUNCEMENTS=814311418127581225
CHANNEL_DEVLOG=814190307616358460
CHANNEL_COMMUNITY_BOT_COMMANDS=814190307980607493
CHANNEL_DEVALERTS=814190307980607495

TRASHCAN_EMOJI="![trashcan](https://cdn.discordapp.com/emojis/814274677723365426.webp?size=128 "trashcan")"

USE_FAKEREDIS=true

#

tldr

#

that has two extra for the 745 pr

#

core dev role and devalerts

gritty wind Sep 6, 2021, 3:49 PM

#

I don't think you need the quotes on any of these lol

#

It's pinned here

vocal prairie Sep 6, 2021, 3:50 PM

#

https://discord.new/zmHtscpYN9E3

gritty wind Sep 6, 2021, 3:50 PM

#

#dev-contrib message

short snow Sep 6, 2021, 3:50 PM

#

giving them quotes makes it easier for me to read

gritty wind Sep 6, 2021, 3:50 PM

#

There it is

vocal prairie Sep 6, 2021, 3:51 PM

#

It's just a lot of copy pasting ids and adding emojis

gritty wind Sep 6, 2021, 3:53 PM

#

Most of the setup is copying the IDs

#

Or rather, 99% of it

#

Here is fine

#

Aren't you setting up this bot? Here is fine too

dry folio Sep 6, 2021, 3:55 PM

#

this is so beautiful

#

precommit is finally working

vocal prairie Sep 6, 2021, 3:56 PM

#

Are you preparing a test server on mobile? You have my eternal respect.

gritty wind Sep 6, 2021, 3:57 PM

#

Not sure what you're wondering about the logs, but you'll need a postgres instance, and optionally a redis instance

#

You can use our docker compose though

#

Which includes both, as well as the rest of the dependencies (site, snekbox)

#

You should also be following the guides here, otherwise you'll struggle more than you should

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

#

When I get back to making them public, or you can become a helper :)

vocal prairie Sep 6, 2021, 3:59 PM

#

What's the blocker to making forms public? Or has it just not happened

gritty wind Sep 6, 2021, 3:59 PM

#

Sitting down and making all issues public, which involves re scoping them, and writing the guides (possibly on site)

#

Actually getting you dawn on board was a big step in the process

#

Helped figure out all the pain points in the setup

vocal prairie Sep 6, 2021, 4:00 PM

#

Ahh. I can see if I can update some of the Notion guides to be more friendly

gritty wind Sep 6, 2021, 4:01 PM

#

I did update them, and the process after you joined the team, but yeah, if you see something you wanna change, do go for it

#

Well, it isn't public yet

#

Docker is sort of like having virtual machines for your project, but not really. We use it for 2 main reasons:

It allows us to create the exact same environment for all users that try to run the bot. Everyone is running the exact same image with the exact same configuration.
It means you don't have to manually install a bunch of dependencies (like databases), or install them on your system, they'll just live in docker.

#

It's completely optional, for instance I only run the databases and site in docker, but run the bot itself through pycharm so I can use a debugger

brazen charm Sep 6, 2021, 4:05 PM

#

for lancebot it isn't really necessary as you can skip redis for most features, does make the site and redis a whole lot easier to run though when they're needed (e.g. bot)

vocal prairie Sep 6, 2021, 4:07 PM

#

gritty wind I did update them, and the process after you joined the team, but yeah, if you s...

Oh yeah, the "Getting Started" guide is pretty clear

#

I think they could go on site whenever it's ready, it seems pretty good

gritty wind Sep 6, 2021, 4:15 PM

#

Don't worry, you are always welcome to ask questions

#

That message was in reference to another project though

dry folio Sep 6, 2021, 4:25 PM

#

https://github.com/python-discord/sir-lancebot/pull/856
rejoice, .bm can now be used everywhere!

#

i messed up my original .bm PR (no one saw the error as well lol), but this one has been tested several times and expands upon the original

vocal prairie Sep 6, 2021, 4:31 PM

#

Cloning a fork allows you to push your changes without changing your remote

#

But just cloning the main repo would work fine, yes

thorny obsidian Sep 6, 2021, 4:32 PM

#

A clone is a local copy. But our projects only allow branches and direct pushed by those in the org. If you want to create a PR and a branch, then you need to fork

vocal prairie Sep 6, 2021, 4:35 PM

#

Yeah. If you're just setting up your own bot instance without making code changes, you don't need to fork

gritty wind Sep 6, 2021, 4:40 PM

#

It's a variable that may be used later on in the file

#

Definition: https://github.com/python-discord/bot/blob/main/config-default.yml#L198

Usage: https://github.com/python-discord/bot/blob/main/config-default.yml#L331

#

You can ignore most emojis, you'll most likely only need the trashcan to start with, and whatever other ones if you get other errors

#

But if you're setting the bot up for your own use, yeah you might want to set them all

#

They are prefilled, but will only work if your bot is on this server (or rather our emojis ones)

#

But that isn't really possible

brazen charm Sep 6, 2021, 4:46 PM

#

you don't need to configure most of the channels fwiw, only some basics like the log, bot commands etc. but otherwise I'd only configure what you need for the functionality of what you're working on

gritty wind Sep 6, 2021, 4:46 PM

#

Not really, since it's only the trashcan you need

#

If you want the actual images, you can copy them from here

#

Or from the CDN directly

brazen charm Sep 6, 2021, 4:50 PM

#

speaking of all the configuring, any status on bot#1595 ?

dusky shoreBOT Sep 6, 2021, 4:50 PM

#

GitHub

IssueOpen Cleaning up config-default.yml and constants.py

austere hornet Sep 6, 2021, 5:01 PM

#

@dry folio Thanks so much for reviewing sir-lancebot#843!

dusky shoreBOT Sep 6, 2021, 5:01 PM

#

GitHub

PROpen Hangman

dry folio Sep 6, 2021, 5:02 PM

#

austere hornet <@769221607888912487> Thanks so much for reviewing sir-lancebot#843!

you're very welcome :D

dry folio Sep 6, 2021, 5:05 PM

#

austere hornet <@769221607888912487> Thanks so much for reviewing sir-lancebot#843!

hey, little suggestion: wouldn't it be cool to use a random word/dictionary API instead of a fixed file of 1000 words? that way, you'd get new words every single time (disadvantage is that the random words might be very obscure)

#

actually, you could make that into a sub game mode - say, when one puts .hangman hard, then can use those random words

green oriole Sep 6, 2021, 5:12 PM

#

brazen charm speaking of all the configuring, any status on bot#1595 ?

So you'd like to clean up unused constants and use actual code constants for the less changed ones?

brisk brook Sep 6, 2021, 5:13 PM

#

dry folio hey, little suggestion: wouldn't it be cool to use a random word/dictionary API ...

Do you think that urban dictionary could be used to grab a random word?

dry folio Sep 6, 2021, 5:13 PM

#

brisk brook Do you think that urban dictionary could be used to grab a random word?

grabs one randomly and it's nsfw

#

ban

green oriole Sep 6, 2021, 5:13 PM

#

That is a big no haha, urban dictionary has way too much nsfw stuff

brazen charm Sep 6, 2021, 5:13 PM

#

green oriole So you'd like to clean up unused constants and use actual code constants for the...

That was mostly put there for consideration as they take up some space in the yaml file, but I also don't think it's a big issue. The main point of it is to make the layout easier to navigate and remove unused and duplicate values

green oriole Sep 6, 2021, 5:15 PM

#

Alright, that sounds like fair changes, I think you can go for it

#

If that's something you are interested in contributing

vale ibex Sep 6, 2021, 5:16 PM

#

Most of the constants don't match one to one in the template

#

For simplicity, we merged a lot of the channels

austere hornet Sep 6, 2021, 5:17 PM

#

dry folio actually, you could make that into a sub game mode - say, when one puts `.hangma...

I love that idea! I will bring it up to Frontogenesis, he was the one who created the PR - I was just working together with him

vale ibex Sep 6, 2021, 5:17 PM

#

So just copy a channel/category for whatever seems the best

#

You can have multiple constants point to the same channel or category

#

Don't exist in the config?

#

You should only be changing the IDs, rather than the names

trail pilot Sep 6, 2021, 5:19 PM

#

dry folio hey, little suggestion: wouldn't it be cool to use a random word/dictionary API ...

I was going to do that originally but most dictionary APIs require a key

#

And I tried to apply for one of them for my own personal usage and they never got back lol

#

I think we're better off just using a set of 1000 words because I doubt the user will ever get another word again since if I'm going to be honest, not too many will play hangman

vale ibex Sep 6, 2021, 5:21 PM

#

Just copy a different channel id

#

Eg for this, just use the admins-mods category for both

austere hornet Sep 6, 2021, 5:23 PM

#

Hey @vale ibex , would you mind leaving a review on sir-lancebot#843 when you get a chance? Would really appreciate it. Thanks so much!

dusky shoreBOT Sep 6, 2021, 5:23 PM

#

GitHub

PROpen Hangman

vale ibex Sep 6, 2021, 5:24 PM

#

Can take a look on the weekend most likely

austere hornet Sep 6, 2021, 5:24 PM

#

vale ibex Can take a look on the weekend most likely

Ok, that's fine.

clever wraith Sep 6, 2021, 5:41 PM

#

Hey @austere hornet , can I talk to you about hangman for a second?

trail pilot Sep 6, 2021, 5:45 PM

#

clever wraith Hey <@853309207922212965> , can I talk to you about hangman for a second?

I think he's offline, you can create a group DM with both of us if you wanted to, or just wait for him to come back offline

#

online*

#

(I'm Shom770 btw)

clever wraith Sep 6, 2021, 5:45 PM

#

Ty Fronto

#

Can't at the moment, mobiles a bitch

trail pilot Sep 6, 2021, 5:46 PM

#

Ah alright

#

You can just DM me then and I'll relay whatever you said to him too

clever wraith Sep 6, 2021, 5:46 PM

#

Anyways I've got you here

#

So

trail pilot Sep 6, 2021, 5:46 PM

#

Yeah

#

Or just say it here haha

#

anyways

clever wraith Sep 6, 2021, 5:47 PM

#

I wanna talk to you about the tries variable

#

It doesn't seem to be consistent with some of the word lengths

#

Is that intentional?

trail pilot Sep 6, 2021, 5:47 PM

#

That's intentional, since in most hangman games you usually have a set amount of tries, regardless of the length

#

Ah wait

#

I see your point

#

if there's more than 7 unique letters

#

Well you can filter for the minimum unique letters

#

and maximum

#

so it's easier or hharder

vocal wolf Sep 6, 2021, 5:53 PM

#

@thorny obsidian we should probably change the labels now in @dusky shore's repo

thorny obsidian Sep 6, 2021, 5:53 PM

#

:D I can help with suggestions!!

vocal wolf Sep 6, 2021, 5:53 PM

#

please do lol

#

should we have different colors for the categories?

brisk brook Sep 6, 2021, 5:53 PM

#

~~Wait why does the labels need to change?~~ I see, because they're seasonal

thorny obsidian Sep 6, 2021, 5:54 PM

#

hold on, need to deal with some rando tagging me and saying I should be domain leads???

vocal wolf Sep 6, 2021, 5:56 PM

#

wack

clever wraith Sep 6, 2021, 6:10 PM

#

thorny obsidian hold on, need to deal with some rando tagging me and saying I should be domain l...

Lol it was someone saying you shouldn't

fallen patrol Sep 6, 2021, 6:11 PM

#

Also discord-python/bot works too

#

But seasonalbot never existed on discord-python

vocal wolf Sep 6, 2021, 6:13 PM

#

@thorny obsidian where hath thou gone

thorny obsidian Sep 6, 2021, 6:14 PM

#

Hi yes!

#

I am here

vocal wolf Sep 6, 2021, 6:14 PM

#

yes

#

must obtain new labels

vale ibex Sep 6, 2021, 6:15 PM

#

#

Yup

dry folio Sep 6, 2021, 6:15 PM

#

vale ibex

MODMAIL is cool

austere hornet Sep 6, 2021, 6:17 PM

#

Hey @clever wraith what's up?

thorny obsidian Sep 6, 2021, 6:18 PM

#

okay, so I don't really know that the area: x labels are supposed to mean. Like... what's frontend for lancebot look like? What's the backend? Those labels seem ill-fitting for our bot.

For the Categories, delete all current ones and replace with:
category: holidays <-- catch all for all fun holiday stuff
category: core <-- affects the core runnings of the bot (i.e. int e, source, ping)
category: events <-- my events stuff! AoC, Hacktoberfest, etc
category: fun and utilities <-- absorbs both practical and fun commands since I don't think there's a good reason to differentiate between them.

Have we ever used the invalid, meta: duplicate, or meta:external labels? afaik we just comment on the issues themselves.

#

and maaaaybe a category: content for when it's not a code change but just adding more content to the bot. Like with topics and the like

austere hornet Sep 6, 2021, 6:19 PM

#

austere hornet Hey <@456226577798135808> what's up?

Feel free to DM me if you want to talk

vale ibex Sep 6, 2021, 6:20 PM

#

vocal wolf Sep 6, 2021, 6:20 PM

#

frontend is for items similar to hardcoded embeds, changing the appearance of something or formatting, but more specifically the way something looks and not the functionality.

#

backend is for anything that isn't frontend

#

frontend could also be stuff in the README

thorny obsidian Sep 6, 2021, 6:21 PM

#

hmmmm, does it make sense to have those labels? Isn't it just frontend and then if it doesn't have that tag it's assumed to be backend so we cooould dooooo area: formatting. Also README stuff would fall under area: docs no?

vocal wolf Sep 6, 2021, 6:21 PM

#

yeah that's true

#

(the second part)

#

we don't use frontend often, no

thorny obsidian Sep 6, 2021, 6:22 PM

#

but the area stuff is \shrug for me, so I'm fine with leaving that be. It's mostly the category stuff we can change

vocal wolf Sep 6, 2021, 6:24 PM

#

category: fun and utilities <-- absorbs both practical and fun commands since I don't think there's a good reason to differentiate between them.
I think we should split them since utilities can mean stuff like wolfram when fun can mean cogs like catify

thorny obsidian Sep 6, 2021, 6:25 PM

#

I'm fine with that too~

clever wraith Sep 6, 2021, 6:25 PM

#

austere hornet Hey <@456226577798135808> what's up?

Resolved it with fronto

vocal wolf Sep 6, 2021, 6:26 PM

#

So we'll have holidays, core, events, fun, and utilities. For invalid, meta: duplicate, and meta: external, we've used them very rarely.

#

@thorny obsidian We've used invalid 18 times, the last time being April 6th of this year. meta: duplicate has been used 3 times, the last time in October of 2019. meta: external has never been used.

dry folio Sep 6, 2021, 6:31 PM

#

thorny obsidian okay, so I don't really know that the `area: x` labels are supposed to mean. Lik...

amazing new labels!

vocal wolf Sep 6, 2021, 6:31 PM

#

Also, we're not doing hacktoberfest this year, right?

dry folio Sep 6, 2021, 6:32 PM

#

thorny obsidian and maaaaybe a `category: content` for when it's not a code change but just addi...

yep, like expanding the .quiz command database lol (is that issue approved yet? i shall check)

vocal wolf Sep 6, 2021, 6:32 PM

#

might as well yeet that label

thorny obsidian Sep 6, 2021, 6:33 PM

#

vocal wolf Also, we're not doing hacktoberfest this year, right?

correct, we're not participating in hacktoberfest this year

vocal wolf Sep 6, 2021, 6:33 PM

#

alright

#

also, for colors

#

I was thinking using evergreen for the categories

#

#38786d

#

I'll just go with that for now, we can always change it later

vale ibex Sep 6, 2021, 6:35 PM

#

JUSTDOIT_REH

trail pilot Sep 6, 2021, 6:35 PM

#

dry folio yep, like expanding the `.quiz` command database lol (is that issue approved yet...

I hope it gets approved tbh

#

at this point the python quiz is just memorization

#

and who can type the fastest

dry folio Sep 6, 2021, 6:35 PM

#

yeah

thorny obsidian Sep 6, 2021, 6:36 PM

#

the issue you're talking about is sir-lancebot#815?

dusky shoreBOT Sep 6, 2021, 6:36 PM

#

GitHub

IssueOpen Quiz command: 30 - 70 new questions to the Python category

trail pilot Sep 6, 2021, 6:36 PM

#

Yep

thorny obsidian Sep 6, 2021, 6:37 PM

#

oh yeah go for it. I'll overstep my bounds and say that's fine

vale ibex Sep 6, 2021, 6:37 PM

#

Approved lol

trail pilot Sep 6, 2021, 6:37 PM

#

Are we allowed to ask if other issues can be approved since everyone is here or should we just wait

thorny obsidian Sep 6, 2021, 6:37 PM

#

@vocal wolf this can be closed, right? sir-lancebot#803

dusky shoreBOT Sep 6, 2021, 6:37 PM

#

GitHub

IssueOpen Type annotation improvements

vocal wolf Sep 6, 2021, 6:38 PM

#

closed

#

how does one describe the fun category

trail pilot Sep 6, 2021, 6:40 PM

#

vocal wolf how does one describe the fun category

I would probably say
An agglomeration of fun games and quizzes with other PyDis members idk

vocal wolf Sep 6, 2021, 6:40 PM

#

needs to be a short description

trail pilot Sep 6, 2021, 6:40 PM

#

ah

#

fun games and quizzes

vale ibex Sep 6, 2021, 6:40 PM

#

PyDis mandated fun, you will enjoy.

vocal wolf Sep 6, 2021, 6:41 PM

#

I was just going to say Features that are fun

trail pilot Sep 6, 2021, 6:41 PM

#

hm, i mean that could work but i feel like people would be confused since it's a bit broad

#

it's also a bit too literal haha

thorny obsidian Sep 6, 2021, 6:41 PM

#

Games and other fun features

vocal wolf Sep 6, 2021, 6:42 PM

#

Features related to fun and games

trail pilot Sep 6, 2021, 6:42 PM

#

yeah that sounds good

green oriole Sep 6, 2021, 6:45 PM

#

I'd remove "features"

#

Because it can be bug fixes and all

vale ibex Sep 6, 2021, 6:45 PM

#

It's for the metabase cog, you can ignore those

#

The same folder as the config and dockerfile

green oriole Sep 6, 2021, 6:46 PM

#

You have to create it in the root of the project, the folder you are in by default when opening github

trail pilot Sep 6, 2021, 6:48 PM

#

you have to create an application for the bot yourself, and then get the token from the application you made iirc

vale ibex Sep 6, 2021, 6:48 PM

#

The one from the discord app portal

vocal wolf Sep 6, 2021, 6:48 PM

#

what events does sir-lance usually contain

#

AoC, Hacktoberfest, something else

green oriole Sep 6, 2021, 6:49 PM

#

CC @thorny obsidian

thorny obsidian Sep 6, 2021, 6:49 PM

#

Those two for now. We'll be adding two more in the coming months: trivia night and esoteric challenges

#

our code jam stuff is in Python

vocal wolf Sep 6, 2021, 6:49 PM

#

alright

#

I'll just have those two for examples for now

brazen charm Sep 6, 2021, 6:49 PM

#

the api key is what's used to authenticate with the site api, it should be somewhere in the guides

vocal wolf Sep 6, 2021, 6:49 PM

#

time to update like

#

all the issues and PRs

#

the labels are uploaded

#

https://github.com/python-discord/sir-lancebot/labels

thorny obsidian Sep 6, 2021, 6:50 PM

#

🎉 Xith best triage lead

fervent sage Sep 6, 2021, 6:50 PM

#

xith > not xith

brazen charm Sep 6, 2021, 6:50 PM

#

https://pythondiscord.com/pages/guides/pydis-guides/contributing/bot/#environment-variables the example file has the value you need

Python Discord - Contributing to Bot

A guide to setting up and configuring Bot.

green oriole Sep 6, 2021, 6:50 PM

#

badbot13m0n8f570f942013fc818f234916ca531

#

Xith best triage of the whole org

vocal wolf Sep 6, 2021, 6:51 PM

#

fervent sage xith > not xith

I have no idea what this is supposed to mean lmao

fervent sage Sep 6, 2021, 6:51 PM

#

nor me

vocal wolf Sep 6, 2021, 6:51 PM

#

oh wait

green oriole Sep 6, 2021, 6:51 PM

#

Nah, that's the default value

vocal wolf Sep 6, 2021, 6:51 PM

#

no I get it

#

took me a couple seconds

brazen charm Sep 6, 2021, 6:51 PM

#

the guide could be clearer there, as you can only get it from there, the compose file or the site

fervent sage Sep 6, 2021, 6:52 PM

#

fervent sage xith > not xith

vocal wolf Sep 6, 2021, 6:53 PM

#

lmao

brazen charm Sep 6, 2021, 6:53 PM

#

only if you plan on working with the reddit cog

molten perch Sep 6, 2021, 6:54 PM

#

Btw. May I ask what the consensus on implementing dpy 2.0 features (eg.: views) in either of the bots, in the future?

fervent sage Sep 6, 2021, 6:55 PM

#

iirc it's to use them if you need to, but existing features wont be updated to use them

#

at least rn

green oriole Sep 6, 2021, 6:55 PM

#

We will look at it for sure, we wrote a bit about our migration policy in #dev-announcements

#

Actually, that'd be a nice thing to fit in botcore

#

Who is working on that again, are you working on botcore by any chance @cold island?

thorny obsidian Sep 6, 2021, 6:57 PM

#

I'd be interested in adjusting lancebot features as we work on them to incorporate some d.py v2 features

fervent sage Sep 6, 2021, 6:57 PM

#

a common base for all bots

thorny obsidian Sep 6, 2021, 6:58 PM

#

Instead of replicating the same core features (i.e. int e) across all bots, it uses a common base. So if we make upgrades to it, we don't have to replicate it across all bots

vale ibex Sep 6, 2021, 6:58 PM

#

Yea, we found ourselves copying things from python and lance, like pagination, Int e, help command etc

#

No, buttons and select menus are tho

fervent sage Sep 6, 2021, 6:58 PM

#

we should just use jishaku for eval kek

vale ibex Sep 6, 2021, 6:59 PM

#

We do for Arthur lol

vocal wolf Sep 6, 2021, 6:59 PM

#

time to go through every single issue of sir lance

green oriole Sep 6, 2021, 7:00 PM

#

thorny obsidian I'd be interested in adjusting lancebot features as we work on them to incorpora...

Like kaizen them in as we have PRs targeting 'em?

thorny obsidian Sep 6, 2021, 7:00 PM

#

Like for instance, if we're working to update AoC features, if we find a good use case for it, I wouldn't mind increasing the scope of AoC PRs to revamp it slightly

green oriole Sep 6, 2021, 7:01 PM

#

That sounds like a good idea

molten perch Sep 6, 2021, 7:01 PM

#

The reason I asked about dpy. 2 is because I had an issue about it. <#dev-contrib message>

green oriole Sep 6, 2021, 7:02 PM

#

You need to have both, with the config overriding the default config

green oriole Sep 6, 2021, 7:02 PM

#

molten perch The reason I asked about dpy. 2 is because I had an issue about it. <https://dis...

That sounds like an excellent idea, and probably the kind of stuff we would accept a PR for, as it brings a large beneficial UI change

vale ibex Sep 6, 2021, 7:05 PM

#

Don't change config default. Those are the values we use in prod

molten perch Sep 6, 2021, 7:05 PM

#

green oriole That sounds like an excellent idea, and probably the kind of stuff we would acce...

Glad to hear, I’ll open an issue for it. I was thinking about a generic view that could be used all around the code.

vale ibex Sep 6, 2021, 7:05 PM

#

Any values in config.yaml will Overwrite the default ones

#

Sounds good

thorny obsidian Sep 6, 2021, 7:08 PM

#

What OS are you on?

#

So you want to use WSL2 if you can

#

Otherwise, you can enable HyperV

#

Check out these install instructions: https://docs.docker.com/desktop/windows/install/

#

Once you have docker up and running, just do docker-compose up

green oriole Sep 6, 2021, 7:09 PM

#

thorny obsidian Otherwise, you can enable HyperV

(for which you need a win pro edition, so that may not be an option for you)

thorny obsidian Sep 6, 2021, 7:20 PM

#

You shouldn't need to install a specific linux OS, just having WSL2 there with Docker is fine

#

mhm, make sure docker desktop is running, then docker-compose up

vocal wolf Sep 6, 2021, 7:28 PM

#

Alright I've updated the labels in all PRs and issues in @dusky shore.

fallen patrol Sep 6, 2021, 7:29 PM

#

vocal wolf Alright I've updated the labels in all PRs and issues in <@!528937022996611082>.

only a bunch more repos to go party_blob

thorny obsidian Sep 6, 2021, 7:29 PM

#

@cursive relic what is giving you this error?

#

The docker-compose up or starting up docker desktop?

vocal wolf Sep 6, 2021, 7:30 PM

#

fallen patrol only a bunch more repos to go <a:party_blob:818919990635266079>

no? we only updated the labels in one repo

fallen patrol Sep 6, 2021, 7:30 PM

#

oh, lol

dry folio Sep 6, 2021, 8:05 PM

#

lol, moderator discussion is needed on whether or not we're allowing these words for sir-lancebot#843

dusky shoreBOT Sep 6, 2021, 8:05 PM

#

GitHub

PROpen Hangman

vale ibex Sep 6, 2021, 8:08 PM

#

fixing spelling in force pushes so no one notices 👀

vale ibex Sep 6, 2021, 8:08 PM

#

dry folio lol, moderator discussion is needed on whether or not we're allowing these words...

yea, lets not have these

dry folio Sep 6, 2021, 8:09 PM

#

vale ibex yea, lets not have these

alright, i linked your message

trail pilot Sep 6, 2021, 8:11 PM

#

vale ibex yea, lets not have these

Yep, sorry for not looking at the words earlier and removing htem

#

them

#

@dry folio I just ran a regex replace so all the two letter words are removed

cold island Sep 6, 2021, 8:16 PM

#

green oriole Who is working on that again, are you working on botcore by any chance <@5123549...

In theory, takes me time to get to it though

clever wraith Sep 6, 2021, 8:34 PM

#

could people look at sir-lancebot#849

dusky shoreBOT Sep 6, 2021, 8:34 PM

#

GitHub

PROpen .quack

clever wraith Sep 6, 2021, 8:35 PM

#

i could use the reviews

austere hornet Sep 6, 2021, 8:35 PM

#

dry folio lol, moderator discussion is needed on whether or not we're allowing these words...

Yo what the hell

clever wraith Sep 6, 2021, 8:36 PM

#

austere hornet Yo what the hell

i mean, sex and sexual?

#

we dont want wholesome sir lancebot saying those things

austere hornet Sep 6, 2021, 8:37 PM

#

clever wraith we dont want wholesome sir lancebot saying those things

I know and I just realized that was our PR

#

I did not notice that

clever wraith Sep 6, 2021, 8:37 PM

#

lol

austere hornet Sep 6, 2021, 8:37 PM

#

Yeah ik

#

@trail pilot Did you remove those words?

#

Oh wait yeah you did sry tl;dr

trail pilot Sep 6, 2021, 8:48 PM

#

clever wraith i could use the reviews

Am I allowed to review?

#

Can anyone review

#

If so I'll gladly review

vocal prairie Sep 6, 2021, 8:51 PM

#

trail pilot Am I allowed to review?

Yes, of course!

#

Code reviews are always helpful

trail pilot Sep 6, 2021, 8:51 PM

#

Awesome!

clever wraith Sep 6, 2021, 8:56 PM

#

trail pilot Am I allowed to review?

yes

static canyon Sep 6, 2021, 9:00 PM

#

trail pilot Am I allowed to review?

You're allowed to review, but it won't count towards the mandatory review count (since you're not staff/core dev)

#

I'm not sure if contributor reviews count or not

brazen charm Sep 6, 2021, 9:01 PM

#

they do

trail pilot Sep 6, 2021, 9:02 PM

#

Contributor reviews meaning contributors with the contributor role in Python discord itself or just the contributor label on github?

brazen charm Sep 6, 2021, 9:02 PM

#

role

trail pilot Sep 6, 2021, 9:02 PM

#

Ah

trail pilot Sep 6, 2021, 9:03 PM

#

static canyon You're allowed to review, but it won't count towards the mandatory review count ...

Yeah that makes sense, thanks!

static canyon Sep 6, 2021, 9:03 PM

#

It's definitely still worth reviewing btw

brazen charm Sep 6, 2021, 9:04 PM

#

Just make sure you're thorough to the best of your ability, an approval with potential mistakes left in the code that could be caught with testing the functionality won't help anyone

fallen patrol Sep 6, 2021, 9:10 PM

#

Contributor reviews count

#

But they won't get the green check in github

#

But they do count towards merge

patent pivot Sep 6, 2021, 9:11 PM

#

policy-bot ftw

#

policy-bot has now learnt how to do team assignments

dry folio Sep 6, 2021, 9:13 PM

#

trail pilot <@!769221607888912487> I just ran a regex replace so all the two letter words ar...

perfect!

fallen patrol Sep 6, 2021, 9:14 PM

#

patent pivot policy-bot has now learnt how to do team assignments

Link to change log?

patent pivot Sep 6, 2021, 9:14 PM

#

lol

#

so

#

not a feature they added

#

a bug i fixed

#

1 sec

fallen patrol Sep 6, 2021, 9:14 PM

#

Did it get merged to main

#

Or only fixed on pydis

patent pivot Sep 6, 2021, 9:15 PM

#

https://github.com/python-discord/.github/commit/2bac4c430c3983acc267929ff198f13839c18578#diff-3c8e83b088928e76dafc082a91f74bc8c69b882a52deaa7b071d617d74e2f196

GitHub

Correct review request mode · python-discord/.github@2bac4c4

fallen patrol Sep 6, 2021, 9:15 PM

#

You really do make mistakes with config files.....

patent pivot Sep 6, 2021, 9:15 PM

#

in my defence

#

policy-bot has an endpoint that lints configs

#

and I run all the configs I write through them

fallen patrol Sep 6, 2021, 9:16 PM

#

No, you don't have a defense

#

King Arthur has black incompatible isort config still

patent pivot Sep 6, 2021, 9:16 PM

#

https://github.com/palantir/policy-bot#testing-and-debugging-policies

GitHub

GitHub - palantir/policy-bot: A GitHub App that enforces approval p...

A GitHub App that enforces approval policies on pull requests - GitHub - palantir/policy-bot: A GitHub App that enforces approval policies on pull requests

#

i merged that didn't i

fallen patrol Sep 6, 2021, 9:16 PM

#

I found anoth

#

Because I decided to finally look at isorts docs

patent pivot Sep 6, 2021, 9:17 PM

#

first i am hearing of it, lol

#

i mean ultimately, I haven't had a problem with it lol

fallen patrol Sep 6, 2021, 9:17 PM

#

Yeah

#

what's that Github Server?

dim pelican Sep 6, 2021, 9:56 PM

#

hey all, getting a weird error on my sir-lancebot clone: ImportError: cannot import name 'Thread' from 'discord' (/usr/local/lib/python3.9/site-packages/discord/__init__.py)

patent pivot Sep 6, 2021, 9:57 PM

#

hmmm, might need to update the discord.py dependency, we've bumped it to 2.0.0a

dim pelican Sep 6, 2021, 9:58 PM

#

Right, is that in the poetry.lock file? I've run poetry install and then docker-compose up which is when I hit that error

patent pivot Sep 6, 2021, 9:58 PM

#

try run docker-compose up --build

brazen charm Sep 6, 2021, 9:59 PM

#

Is there a reason for lancebot having different levels on its file and console handlers when debug is not on?

patent pivot Sep 6, 2021, 9:59 PM

#

uhhhhh

#

no

brazen charm Sep 6, 2021, 10:00 PM

#

I'll set both to info like bot does through the logger then

dim pelican Sep 6, 2021, 10:08 PM

#

patent pivot try run `docker-compose up --build`

now getting this error:
discord.ext.commands.errors.ExtensionFailed: Extension 'bot.exts.utilities.color' raised an error: TypeError: Expected discord.Colour, int, or Embed.Empty but received str instead.

#

Wait this is the file I'm working on... 🤦‍♂️

patent pivot Sep 6, 2021, 10:20 PM

#

lmfao

dim pelican Sep 6, 2021, 10:35 PM

#

But I don't know why I'm getting that error

trail pilot Sep 6, 2021, 10:47 PM

#

clever wraith Sep 6, 2021, 10:48 PM

#

is it fine if i just upload the working code

trail pilot Sep 6, 2021, 10:48 PM

#

Not sure what this is

clever wraith Sep 6, 2021, 10:48 PM

#

and u guys can just edit edit edit

patent pivot Sep 6, 2021, 10:55 PM

#

trail pilot

fine to ignore

dim pelican Sep 6, 2021, 11:01 PM

#

I think I figured out my first error, was trying to format an Embed incorrectly. But now I'm getting this:
TypeError: module() takes at most 2 arguments (3 given)

patent pivot Sep 6, 2021, 11:01 PM

#

what's the full traceback?

thorny obsidian Sep 6, 2021, 11:02 PM

#

@clever wraith We should be using trailing commas. Our style guide includes following PEP8 in cases where we don't call out that we're deviating from PEP8. Additionally, as mentioned in the review comment, it ensures that the git blame in the future is accurate.

clever wraith Sep 6, 2021, 11:03 PM

#

thorny obsidian <@456226577798135808> We should be using trailing commas. Our style guide includ...

alright

#

ill get on it soon

fallen patrol Sep 6, 2021, 11:07 PM

#

trail pilot

hm..... arguably the status embed should not be under the same requirements hmmm

dim pelican Sep 6, 2021, 11:11 PM

#

patent pivot what's the full traceback?

sir-lancebot    |   File "/usr/local/lib/python3.9/runpy.py", line 197, in _run_module_as_main
sir-lancebot    |     return _run_code(code, main_globals, None,
sir-lancebot    |   File "/usr/local/lib/python3.9/runpy.py", line 87, in _run_code
sir-lancebot    |     exec(code, run_globals)
sir-lancebot    |   File "/bot/bot/__main__.py", line 31, in <module>
sir-lancebot    |     bot.load_extension(ext)
sir-lancebot    |   File "/usr/local/lib/python3.9/site-packages/discord/ext/commands/bot.py", line 732, in load_extension
sir-lancebot    |     self._load_from_module_spec(spec, name)
sir-lancebot    |   File "/usr/local/lib/python3.9/site-packages/discord/ext/commands/bot.py", line 663, in _load_from_module_spec
sir-lancebot    |     raise errors.ExtensionFailed(key, e) from e
sir-lancebot    | discord.ext.commands.errors.ExtensionFailed: Extension 'bot.exts.utilities.color' raised an error: TypeError: module() takes at most 2 arguments (3 given)```

patent pivot Sep 6, 2021, 11:15 PM

#

how are you calling color

dim pelican Sep 6, 2021, 11:16 PM

#

class Color(commands.cog):
    """User initiated command to receive color information."""

    def __init__(self, bot: Bot):
        self.bot = bot

    @commands.command(aliases=["color", "colour"])
    @commands.cooldown(1, 10, commands.cooldowns.BucketType.user)
    async def color(self, ctx: commands.Context, *, user_color: str) -> None:
        """Send information on input color code or color name."""

patent pivot Sep 6, 2021, 11:18 PM

#

dim pelican ```py class Color(commands.cog): """User initiated command to receive color ...

commands.cog -> commands.Cog

dim pelican Sep 6, 2021, 11:19 PM

#

patent pivot `commands.cog` -> `commands.Cog`

ahhhhh typos. Thanks Joe, you are a genius

patent pivot Sep 6, 2021, 11:20 PM

#

sunglas

trail pilot Sep 6, 2021, 11:31 PM

#

Why does adding a trailing comma make the git blame more accurate? I heard it, hence I put it in the review comment, though I don’t know why it makes it more accurate

brazen charm Sep 6, 2021, 11:32 PM

#

if you add a new element, or swap things around the you don't have to add the comma and create a diff on that line

trail pilot Sep 6, 2021, 11:34 PM

#

Ah

brazen charm Sep 6, 2021, 11:35 PM

#

With a

a = [
    1,
]

file changed to contain a 2 on the second line in the list you can see how the diff changes

# no comma
 a = [
-    1
+    1,
+    2
 ]

# trailing comma
 a = [
     1,
+    2
 ]

clever wraith Sep 7, 2021, 12:03 AM

#

howdy people. changes requested to sir-lancebot#849 have been made

dusky shoreBOT Sep 7, 2021, 12:03 AM

#

GitHub

PROpen .quack

trail pilot Sep 7, 2021, 12:27 AM

#

@clever wraith Could you invite me to your testing server so I can test out some edge cases if you're fine with that

clever wraith Sep 7, 2021, 12:27 AM

#

sure but the bot is offlint

#

offline

#

Invited

clever wraith Sep 7, 2021, 1:08 AM

#

hey @patent pivot how long are ducks kept on quackstack?

patent pivot Sep 7, 2021, 1:08 AM

#

until reboot I think

#

cleared when we deploy quackstack

#

when we go for S3 then permanently

clever wraith Sep 7, 2021, 1:42 AM

#

couldnt that cause problems if someone spam requests

patent pivot Sep 7, 2021, 1:43 AM

#

clever wraith couldnt that cause problems if someone spam requests

yeah - that's why we're migrating to S3

#

this setup is not production ready

fallen patrol Sep 7, 2021, 4:03 AM

#

#

screams

#

two prs on king-arthur#10, king-arthur#11

dusky shoreBOT Sep 7, 2021, 4:07 AM

#

GitHub

PROpen Ensure compatibility with black
PROpen fix status embed

patent pivot Sep 7, 2021, 4:07 AM

#

!remind 18h t

stable mountainBOT Sep 7, 2021, 4:07 AM

#

Absolutely!

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

fallen patrol Sep 7, 2021, 4:13 AM

#

done

#

what even

#

is this

#

https://github.com/python-discord/king-arthur/blob/dcfc9a14f5384875b7cd0edc242ca0edacf9fb9c/.github/workflows/lint-build-deploy.yaml#L76-L79

stable mountainBOT Sep 7, 2021, 4:16 AM

#

.github/workflows/lint-build-deploy.yaml lines 76 to 79

build:
  if: github.ref == 'refs/heads/main'
  name: Build & Push
  runs-on: ubuntu-latest```

fallen patrol Sep 7, 2021, 5:04 AM

#

!remind 12h fix the skip of the thing and do the other thing, tldr king-arthur#10 probably need another change

stable mountainBOT Sep 7, 2021, 5:04 AM

#

No problem.

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

dusky shoreBOT Sep 7, 2021, 5:04 AM

#

GitHub

PROpen fix status embed

fallen patrol Sep 7, 2021, 5:06 AM

#

!remind cancel 2999

stable mountainBOT Sep 7, 2021, 5:06 AM

#

Aye aye, cap'n!

That reminder has been deleted successfully!

fallen patrol Sep 7, 2021, 5:07 AM

#

!remind 12h fix the skip of the thing and do the other thing, tldr king-arthur#10 probably need another change
see this section for what to add to the bottom of lint-build-deploy. https://github.com/python-discord/bot/blob/26cb61d15f30dfa170bb9ef61a167376ee0e3c52/.github/workflows/lint-test.yml#L135 Might need to refactor workflow to be like the below actually https://github.com/discord-modmail/modmail/blob/0ed105fd956a0455dc1ca1d19903149bacb0524f/.github/workflows/lint_test.yml#L171

stable mountainBOT Sep 7, 2021, 5:07 AM

#

Yep.

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

dusky shoreBOT Sep 7, 2021, 5:07 AM

#

GitHub

PROpen fix status embed

stable mountainBOT Sep 7, 2021, 5:07 AM

#

.github/workflows/lint-test.yml line 135

- name: Prepare Pull Request Payload artifact```
`.github/workflows/lint_test.yml` line 171
```yml
artifact:```

fallen patrol Sep 7, 2021, 5:07 AM

#

sheesh, three messages for one command.

green oriole Sep 7, 2021, 6:26 AM

#

cold island In theory, takes me time to get to it though

Yeah of course, I was just curious if 2.0 view things were part of the plan

fallen patrol Sep 7, 2021, 7:07 AM

#

is botcore public?

#

botcore#1

#

...@cold island?

#

going offline, if someone knows please ping me 😄

brisk brook Sep 7, 2021, 7:39 AM

#

Yes, because that's where the docker-compose file lives

#

That would run only the bot

#

But it depends on Redis and Postgres as well as the site I think, which may cause issues.

But yeah, try it.

gritty wind Sep 7, 2021, 7:54 AM

#

just try docker compose up -d first

#

Feel free to ask

#

Yea, but the compose should handle it for you

gritty wind Sep 7, 2021, 7:56 AM

#

gritty wind just try `docker compose up -d` first

This will add all the dependencies you’ll need

#

The whole concept of env vars in the bot is for production, you really don’t need to use an env file in development. In production, we use config-default which is publicly visible on git, so we can’t put all our tokens and what not in there. In development, you don’t have the same problem

#

You can still use env vars, we document all of them in the getting started guide

#

I remember this problem from a while back

#

@vale ibex is working on a fix right now iirc

#

If everything is online, you should be good

#

Is it online in docker?

#

Right, in that case the bot won’t start

#

It’ll time out soon

#

Chris is working on a fix, but in the mean time you do need to tweak the database manually slightly

#

Do you have any way to edit the database? Something like PG Admin

#

None of what?

#

If you don’t have any of the tools, it’s still possible to do this using just docker

vale ibex Sep 7, 2021, 8:04 AM

#

gritty wind <@!126811506632294400> is working on a fix right now iirc

Yea there's a pr on site for it

#

I see Mark made some comments while I was sleeping so ill take a look later

gritty wind Sep 7, 2021, 8:04 AM

#

Alright nice

patent pivot Sep 7, 2021, 8:05 AM

#

fallen patrol is botcore public?

nope, we'll be developing internally for now, but to my knowledge we don't have a repo

vale ibex Sep 7, 2021, 8:06 AM

#

The work around right now is to pull up postgres, run the sql manually then pull up the rest

#

Stop all the containers

#

And just run postgres

#

Then get a shell into the postgres container, run psql -Upysite

#

Then copy and paste the sql from here into the terminal https://raw.githubusercontent.com/python-discord/site/main/postgres/init.sql

#

Then you can start up the rest as normal

#

From docker desktop, if you click on a container its the terminal icon on the top right

#

I'm on mobile rn so can't remember what it's called

brisk brook Sep 7, 2021, 8:15 AM

#

It's the first button

#

>_

vale ibex Sep 7, 2021, 8:42 AM

#

try docker-compose up

gritty wind Sep 7, 2021, 9:01 AM

#

That’s correct for docker

#

Did you try docker-compose up before starting the bot again?

vale ibex Sep 7, 2021, 9:14 AM

#

Yea, took me about the same when I first set it up, wasn't a nice experience

#

annoyingly to make it any better, it's a lot of effort, since even using a template, IDs are not the same

#

We're working on a project that will hopefully improve this experience, but it's slow going atm

#

Did you update the urls?

#

in config.yml

#

urls:
    # PyDis site vars
    connect_max_retries:       3
    connect_cooldown:          5
    site:        &DOMAIN       "web:8000"
    site_api:    &API    !JOIN ["api.", *DOMAIN]
    site_api_schema:           "http://"
    site_paste:  &PASTE  !JOIN ["paste.", "pythondiscord.com"]
    site_schema: &SCHEMA       "http://"
    site_staff:  &STAFF  !JOIN ["staff.", *DOMAIN]

    paste_service:                      !JOIN ["https://", *PASTE, "/{key}"]
    site_logs_view:                     !JOIN [*SCHEMA, *STAFF, "/bot/logs"]

    # Snekbox
    snekbox_eval_api: "http://snekbox:8060/eval"

#

yea, change yours to this

#

they will be different, since in prod they are different to dev

#

Yea, we pinned to v7, to match d.py's pin

#

For the guild invite filter

#

literally just copy what I have above and put it in yours

#

that should be good then

#

now, the other thing you likely haven't done is redis

#

at the top, under bot::redis

vocal wolf Sep 7, 2021, 9:22 AM

#

mmmmmmmm triage time

vale ibex Sep 7, 2021, 9:22 AM

#

Set the values to this ```yml
redis:
host: "redis"
password: null
port: 6379
use_fakeredis: false

#

don't they need to be for strings?

#

you should only need to rebuild if you change dependancies

#

or the dockerfile etc

#

yup, that's gonna happen until my PR is merged

vale ibex Sep 7, 2021, 9:28 AM

#

vale ibex Stop all the containers

.

#

just do docker-compose up

#

you'll need to do it each time you reset the postgres container

#

site#583 shouldn't be much longer

dusky shoreBOT Sep 7, 2021, 9:29 AM

#

GitHub

PROpen Initialise metricity at runtime

vale ibex Sep 7, 2021, 9:31 AM

#

or ctrl+d twice

#

or close the window 😛

gritty wind Sep 7, 2021, 9:46 AM

#

you should be able to do just docker-compose up without rebuilding

#

As long as the postgres container isn't destroyed, you don't have to repeat that

#

It's only destroyed if you do docker compose down or if you rebuild

#

Closing your PC is fine

#

Anyways this shouldn't matter by tomorrow 🤞

vale ibex Sep 7, 2021, 9:50 AM

#

The arrows are default emoji, so you don't need to worry about that

gritty wind Sep 7, 2021, 9:50 AM

#

The help ones are all builtins

#

Except the trashcan

#

Which is basically all you need

vale ibex Sep 7, 2021, 9:50 AM

#

you can even set the trashcan emoji to an inbuilt one too like ❌

#

They won't work because the trashcan emoji missing raises an error

#

so the paginator doesn't run

#

you need to create a webhook in Discord for that channel

#

and then copy the webhook ID into the config

#

mods

vocal wolf Sep 7, 2021, 9:58 AM

#

why is suggestion so hard to spell when suggesting something in reviews

#

but I am

vale ibex Sep 7, 2021, 10:03 AM

#

To a duck-pond channel

vocal wolf Sep 7, 2021, 10:04 AM

#

@vale ibex

#

wanna review stuff

vale ibex Sep 7, 2021, 10:04 AM

#

working :p

vocal wolf Sep 7, 2021, 10:04 AM

#

darn

vale ibex Sep 7, 2021, 10:04 AM

#

very fun conference call atm

#

||/s||

vocal wolf Sep 7, 2021, 10:04 AM

#

I'll just request your review for later

vale ibex Sep 7, 2021, 10:04 AM

#

👌

vocal wolf Sep 7, 2021, 10:04 AM

#

vale ibex very fun conference call atm

oof

vale ibex Sep 7, 2021, 10:05 AM

#

site#583 evil

dusky shoreBOT Sep 7, 2021, 10:05 AM

#

GitHub

PROpen Initialise metricity at runtime

vale ibex Sep 7, 2021, 10:06 AM

#

It's already had a few passes, so likely close to done now

#

lol yea, I really like writing sql

#

yea, that's handled in the parse_db_url() func

#

yea, it'll raise a keyerror if it's not there, and fail validation if it's wrong

#

Because I've changed the line that Mark commented on

#

shrugR

vale ibex Sep 7, 2021, 10:32 AM

#

go into the channel settings, goto the integrations section and then webhooks

#

https://canary.discord.com/api/webhooks/<id>/<secret>

#

copy the webhook url and then take the id out of it

#

yup

#

ah 😄

#

docker-compose up --build bot

#

you shouldn't need to rebuild though

#

if you only changed config

#

remove the last bits

#

just have ":x:"

#

no < > either

#

only if you ever kill the portgres container manually

#

IE with docker-compose down or if you delete the container via docker desktop

#

or if you do docker-compose up --build

#

That's why we recommend doing docker-compose up --build bot since that only rebuilds the bot

#

rather than all containers

#

yea, docker-compose has a volume in it

#

so just code/config changes automatically get put in, without a rebuild

#

if you change the Dockerfile, or the dependencies in the pyproject.toml file then you'll need to rebuild

#

👌

#

!charinfo ❌

stable mountainBOT Sep 7, 2021, 10:53 AM

#

Character Info

\u274c : CROSS MARK - ❌

vale ibex Sep 7, 2021, 10:53 AM

#

try "\u274c"

short snow Sep 7, 2021, 10:55 AM

#

@clever wraith your zip file url is asking for password

vale ibex Sep 7, 2021, 10:55 AM

#

yea

short snow Sep 7, 2021, 10:55 AM

#

also can you generate template of your test server, so i can get thse emojis lemon_hyperpleased

vale ibex Sep 7, 2021, 10:56 AM

#

afaik templates don't include emoji

short snow Sep 7, 2021, 10:56 AM

#

category emojis

vale ibex Sep 7, 2021, 10:56 AM

#

ahh

short snow Sep 7, 2021, 10:56 AM

#

if you want to review PRs btw, i have a few of them for you

#

vale ibex Sep 7, 2021, 10:58 AM

#

Did you put the quotes there too?

#

What's on line 70-75 of your config.yml file

#

you have those numbers in thjere?

#

hm that's odd

#

I have the same in my config and it works fine

#

!paste

stable mountainBOT Sep 7, 2021, 11:03 AM

#

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.

vale ibex Sep 7, 2021, 11:03 AM

#

Could you put your config file in here

#

I just pasted that into my bot and it worked fine

#

Maybe its a weird line ending

#

copy from the paste service and paste it into your config

vocal wolf Sep 7, 2021, 11:09 AM

#

ok

#

I'm merging

#

the wikipedia guess game

#

it works well, I want this PR gone and shoved into main

#

@short snow mind if I squash it

short snow Sep 7, 2021, 11:10 AM

#

your choice 🙃

vocal wolf Sep 7, 2021, 11:10 AM

#

I'm squashing it

short snow Sep 7, 2021, 11:10 AM

#

okay

vocal wolf Sep 7, 2021, 11:11 AM

#

github broken

#

lmao

short snow Sep 7, 2021, 11:11 AM

#

lol, what happened

vocal wolf Sep 7, 2021, 11:11 AM

#

idk it bugged out, had to reload page

#

kept pressing merge and nothing happened

short snow Sep 7, 2021, 11:13 AM

#

xith here is your 🍪

vocal wolf Sep 7, 2021, 11:13 AM

#

@short snow Question: Do we actually need this? I'm confused as to why it's necessary as a PR.

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

green oriole Sep 7, 2021, 11:13 AM

#

It break if the status of the PR changes while you are trying to merge

green oriole Sep 7, 2021, 11:14 AM

#

vocal wolf <@!736234578745884682> Question: Do we actually need this? I'm confused as to wh...

Yeah, we do, think stuff like the latex cog we had to keep unloading

vocal wolf Sep 7, 2021, 11:14 AM

#

ugh the latex cog

short snow Sep 7, 2021, 11:14 AM

#

vocal wolf <@!736234578745884682> Question: Do we actually need this? I'm confused as to wh...

basically this was needed very much during the latex bug, because you had to keep unloading the cog everytime the bot restarted

#

and this can be a issue in future

vale ibex Sep 7, 2021, 11:14 AM

#

Check the first few lines

vocal wolf Sep 7, 2021, 11:14 AM

#

is it literally because of the latex cog that this is needed

short snow Sep 7, 2021, 11:14 AM

#

so it was decided to have this feature

green oriole Sep 7, 2021, 11:14 AM

#

We had to add an early return in the loading code to stop it from loading anymore

vocal wolf Sep 7, 2021, 11:14 AM

#

and nothing else

vale ibex Sep 7, 2021, 11:14 AM

#

I imagine you copied some of the other buttons on that page

green oriole Sep 7, 2021, 11:14 AM

#

That may happen later

vocal wolf Sep 7, 2021, 11:14 AM

#

god dammit latex

green oriole Sep 7, 2021, 11:15 AM

#

Like, if an important bug occurs we want to quarantine the cog

vocal wolf Sep 7, 2021, 11:15 AM

#

y u no not break

short snow Sep 7, 2021, 11:15 AM

#

quarantine the cog, i like it lol

green oriole Sep 7, 2021, 11:15 AM

#

Having to unload every time is a pain

vocal wolf Sep 7, 2021, 11:16 AM

#

@short snow shouldn't we ping devops and not all core devs when a cog is getting borked?

short snow Sep 7, 2021, 11:16 AM

#

i was said to ping the core devs here

#

by some core dev, lemme get the message

placid ermine Sep 7, 2021, 11:16 AM

#

vocal wolf ugh the latex cog

👀

vocal wolf Sep 7, 2021, 11:17 AM

#

I don't think core devs would enjoy that

#

we rarely ping core devs, if something is exploding, devops should be pinged instead

short snow Sep 7, 2021, 11:17 AM

#

#dev-contrib message

vocal wolf Sep 7, 2021, 11:18 AM

#

short snow https://discord.com/channels/267624335836053506/635950537262759947/8415214877563...

@gritty wind shouldn't we ping devops instead?

gritty wind Sep 7, 2021, 11:19 AM

#

The reasoning was dev-core are more relevant because they are managing the cogs

#

It’s unlikely that the described situation is a problem with our actual cache

vale ibex Sep 7, 2021, 11:20 AM

#

Do we need a ping?

gritty wind Sep 7, 2021, 11:20 AM

#

Most likely it would be to someone moving a cog, or one getting deleted, etc

vale ibex Sep 7, 2021, 11:20 AM

#

just throw something in dev-alerts

#

we watch that anyway for sentry

gritty wind Sep 7, 2021, 11:20 AM

#

You do?

vale ibex Sep 7, 2021, 11:20 AM

#

yea

vocal wolf Sep 7, 2021, 11:21 AM

#

we do

gritty wind Sep 7, 2021, 11:21 AM

#

If you want, you can forego the ping

green oriole Sep 7, 2021, 11:22 AM

#

Sentry would be fine, yeah

#

If it is really important, could do an at-here in #dev-alerts

gritty wind Sep 7, 2021, 11:22 AM

#

The importance of it will depend on why things are added to that list in the first place

#

The main driving factor for this was the latex cog

#

And I’d say keeping that unloaded was pretty critical since it was OOMing us

fervent sage Sep 7, 2021, 11:23 AM

#

lol that was funny

#

annoying but funny

brazen charm Sep 7, 2021, 11:31 AM

#

The bot has to be able to mute you, don't think there's more to it if everything else is set up

vale ibex Sep 7, 2021, 11:31 AM

#

Does your bot have permissions to mute?

#

@timid sentinel FYI, in bot#1813 I removed the previous nominations message as we discussed a few days ago

dusky shoreBOT Sep 7, 2021, 11:32 AM

#

GitHub

PROpen Declare TalentPool.cache on init of cog

vale ibex Sep 7, 2021, 11:32 AM

#

does it output an error in the console?

#

How do you know it failed to apply?

#

Could you paste the log here from when you did it?

#

Does the bot have a higher role than the mute role?

#

otherwise it cannot issue the role

short snow Sep 7, 2021, 11:35 AM

#

jsut give the bot a super bot role

short snow Sep 7, 2021, 11:52 AM

#

where is the lancebot logo located on branding repo?

#

nice ty

clever wraith Sep 7, 2021, 12:37 PM

#

ty for the review xith

brazen charm Sep 7, 2021, 1:45 PM

#

do you have any in the db?

vale ibex Sep 7, 2021, 1:45 PM

#

It does it at midnight

#

Not right now no

#

There's an issue for it tho

#

you could change the time it runs though

#

or change the logic to do it more often

#

bot.exts.fun.off_topic_names

#

the first function is the loop, so just change that

dim pelican Sep 7, 2021, 1:48 PM

#

Oh wow, is that set up for python-discord/bot?

#

My sir-lancebot setup is just my own "first" server with dedicated channels lol

#

Works is a loose term

vale ibex Sep 7, 2021, 1:55 PM

#

if you use docker it should just work

green oriole Sep 7, 2021, 2:10 PM

#

lol imagine

#

I guess outside of disabling fakeredis you don't have much to do

short snow Sep 7, 2021, 2:16 PM

#

we can have a contrib server now, int e is owner restricted :yay:

keen valley Sep 7, 2021, 2:23 PM

#

Can we Use bs4, for getting any info?

dim pelican Sep 7, 2021, 2:25 PM

#

keen valley Can we Use `bs4`, for getting any info?

In what context? For a bot command?

keen valley Sep 7, 2021, 2:25 PM

#

dim pelican In what context? For a bot command?

Yes, for Bot commands

dim pelican Sep 7, 2021, 2:26 PM

#

Hmm I'm going to let a core dev chime in on that one. I think if it is not malicious it should be fine, otherwise I would look for an API.

vale ibex Sep 7, 2021, 2:27 PM

#

Depends on what you use bs4 for, if its to parse scraped content, then it would depend on the website's rules

#

preferably use an API, as it's far less likely that an API will have breaking changes

keen valley Sep 7, 2021, 2:28 PM

#

vale ibex preferably use an API, as it's far less likely that an API will have breaking ch...

If any free API is not available, than should I go for bs4.beautifulsoup

vale ibex Sep 7, 2021, 2:29 PM

#

Again, it depends on whether the website allows scraping of it's content. Even then it's not a good idea, as a page's DOM is likely to change given enough time, compared to an API

short snow Sep 7, 2021, 2:30 PM

#

which issue are you trying to mention? If you are working on the bot

keen valley Sep 7, 2021, 2:30 PM

#

short snow which issue are you trying to mention? If you are working on the bot

I had not created the Issue till now, I was firstly planning for it

thorny obsidian Sep 7, 2021, 2:30 PM

#

vale ibex Depends on what you use bs4 for, if its to parse scraped content, then it would ...

oh yeah! check our the codewars kata issue. As far as I can tell, we can use bs4 because it at no point disallows automation

vale ibex Sep 7, 2021, 2:31 PM

#

which repo?

placid ermine Sep 7, 2021, 2:32 PM

#

codewars has an api for katas pithink

thorny obsidian Sep 7, 2021, 2:32 PM

#

lance

thorny obsidian Sep 7, 2021, 2:32 PM

#

placid ermine codewars has an api for katas <:pithink:652247559909277706>

the api is v1 and very limited

vale ibex Sep 7, 2021, 2:32 PM

#

i'll take a look

placid ermine Sep 7, 2021, 2:32 PM

#

oh yeah i used bs4 in my project too

thorny obsidian Sep 7, 2021, 2:32 PM

#

You would need to use scraping or manual compilation to get anything usable

placid ermine Sep 7, 2021, 2:32 PM

#

sad

vale ibex Sep 7, 2021, 2:34 PM

#

yea, it seems fine

#

as a precaution it would be good to include a mention and link to the FreeBSD 2-Clause License in the embed, since that's what challenges are licensed under

#

I haven't used codewars too much myself

#

are there many cases of offensive/troll challenges posted?

short snow Sep 7, 2021, 2:59 PM

#

@dim pelican the WTF Python PR should also parse the README.md with help of a markdown parser possibly https://mistune.readthedocs.io/en/latest/, and then show that markdown instead of the link in the embed description. This was the reason it was in WIP state

dim pelican Sep 7, 2021, 3:05 PM

#

short snow <@!698273827448291379> the WTF Python PR should also parse the README.md with he...

Copy that, so instead of sending the link in the embed description, send the relevant markdown section that matches the fuzzy matched search.

#

The last I saw was that the PR needed to change from fuzzywuzzy to rapidfuzz, so I will need to see how to get the markdown parsed and formatted to the embed correctly.

short snow Sep 7, 2021, 3:10 PM

#

yeah 👍

#

this is how i was gonna progress upon that, but that time never came:

example start with 
end with a line containing :bulb: Explanation
the last block ends before ### :arrow_forward: Minor Ones
you can search for the header and after that the example id, and including everything until the explanation block on the first page (pagination) and the explanation on the next++

#

there were a few exception cases which I don't remember, just go through the readme to check if possible, i may be wrong

thorny obsidian Sep 7, 2021, 3:20 PM

#

defcon is one of our raid mitigation tools

#

It'll allow us to reject accounts that are of a certain age. It's also what allows us to do a complete server shutdown with one command

dim pelican Sep 7, 2021, 3:22 PM

#

I've seen !shh in action twice

thorny obsidian Sep 7, 2021, 3:22 PM

#

!defcon shutdown and !defcon unshutdown

#

It'll remove write permissions for everyone from every channel and kick people from VC and remove speaking perms. It's incredibly convenient

#

A lot of it will move to Black Knight as we look to upgrade them

#

We want to expand what our defcon cog can do, especially in terms of quarantining accounts that meet specific parameters. But we don't want to reveal what those specific parameters are.

#

So it's harder for raiders to circumvent our raid mitigation. It lets us stay a bit ahead of the cat-and-mouse game for a bit longer.

celest charm Sep 7, 2021, 3:27 PM

#

so far raiders haven't taken advantage of our server specifics

#

that's weird

dim pelican Sep 7, 2021, 3:28 PM

#

I mean, are they expected to prepare more than let's make accounts and spam shit?

thorny obsidian Sep 7, 2021, 3:29 PM

#

We did have that one raider that specifically targeted our help channels. That was mildly annoying

dim pelican Sep 7, 2021, 3:31 PM

#

I'm glad you guys put some high quality thought into the anti-raid though. Better to have it and not need it than vice versa

austere hornet Sep 7, 2021, 3:33 PM

#

dim pelican I'm glad you guys put some high quality thought into the anti-raid though. Bette...

Absolutely agreed.

short snow Sep 7, 2021, 3:33 PM

#

shom is fast

fallen patrol Sep 7, 2021, 3:35 PM

#

thorny obsidian We did have that one raider that specifically targeted our help channels. That w...

I remember one of those

trail pilot Sep 7, 2021, 3:35 PM

#

placid ermine

oh you've worked with that before too?

trail pilot Sep 7, 2021, 3:37 PM

#

vale ibex as a precaution it would be good to include a mention and link to the FreeBSD 2-...

I'll add that to the footer

placid ermine Sep 7, 2021, 3:37 PM

#

trail pilot oh you've worked with that before too?

yeah

trail pilot Sep 7, 2021, 3:37 PM

#

Should I just say Challenges are licensed under FreeBSD 2-Clause License?

placid ermine Sep 7, 2021, 3:38 PM

#

trail pilot oh you've worked with that before too?

old picture but :P

short snow Sep 7, 2021, 3:39 PM

#

trail pilot oh you've worked with that before too?

https://github.com/ShakyaMajumdar/KataBot

trail pilot Sep 7, 2021, 3:39 PM

#

placid ermine old picture but :P

That's pretty cool!

#

this is how it is currently formatted

#

i cut off the description fairly early

#

not sure if i should do that

thorny obsidian Sep 7, 2021, 3:40 PM

#

hmmmm, I wonder how that would look with fields

#

Like a field for Languages and Tags

placid ermine Sep 7, 2021, 3:41 PM

#

imo the question itself should be most visible

trail pilot Sep 7, 2021, 3:41 PM

#

thorny obsidian Like a field for `Languages` and `Tags`

hm yea

thorny obsidian Sep 7, 2021, 3:41 PM

#

OOOH! A button for like "View the challenge"

#

🤩 buttons

trail pilot Sep 7, 2021, 3:42 PM

#

that would be nice! how does a button work with dpy 2.0 again haha

thorny obsidian Sep 7, 2021, 3:42 PM

#

Anyway, I think this is a super neat feature. I'll see if I can carve out time to review it

trail pilot Sep 7, 2021, 3:42 PM

#

thorny obsidian Anyway, I think this is a super neat feature. I'll see if I can carve out time t...

Thank you!

I'll try and see if I can add buttons

#

wdym?

thorny obsidian Sep 7, 2021, 3:43 PM

#

I do want to review the AoC thing today. Pretty pumped that got completed so quickly

trail pilot Sep 7, 2021, 3:44 PM

#

thorny obsidian I do want to review the AoC thing today. Pretty pumped that got completed so qui...

is it the new leaderboard?

#

This might help you out:
https://stackoverflow.com/questions/60686677/discord-py-emoji-all-servers-bot-in

Stack Overflow

discord.py emoji all servers bot in

I have a bot that sends embeds that include emojis specific to each embed and it works great on my test server where I uploaded the emojis but they don't show up on other servers since those server...

thorny obsidian Sep 7, 2021, 3:45 PM

#

trail pilot is it the new leaderboard?

The per-day leaderboard

trail pilot Sep 7, 2021, 3:46 PM

#

Ooh, that feature is pretty cool

#

Advent of Code are those challenges in december right?

#

from the first to the 25th of december?

thorny obsidian Sep 7, 2021, 3:46 PM

#

yup!

#

It's really fun

#

It's like the release of the qualifier but like... every day for 25 days

#

Plus I'm figuring out how to do a similar spoilers-allowed channel for easy solution discussion

trail pilot Sep 7, 2021, 3:47 PM

#

These discord buttons are really cool

#

I'm gonna see how I can incorporate them to the challenge command

trail pilot Sep 7, 2021, 3:51 PM

#

thorny obsidian It's like the release of the qualifier but like... every day for 25 days

Ooh, that sounds really fun

#

I’m looking back at the code for my qualifier for the code jam and it is atrocious haha

thorny obsidian Sep 7, 2021, 3:52 PM

#

AoC is a very similar style of "it just needs to works"

#

To thoroughly derail this channel, AoC is also one of the more unique events for this server because we tend to get some very good programmers coming out of the woodwork. Like, the people who participate on our leaderboard and in this server frequently also make it into the global leaderboard for AoC. Which is downright terrifying

trail pilot Sep 7, 2021, 3:58 PM

#

AoC sounds really fun, I’m guessing I’ll likely lag behind because I can’t do the first four days of it due to an exam

#

But I enjoy challenges like the code jam qualifier

thorny obsidian Sep 7, 2021, 3:59 PM

#

It's fine~ You can complete them as you go. I think I only made it to day 8 last year

trail pilot Sep 7, 2021, 4:04 PM

#

thorny obsidian It's fine~ You can complete them as you go. I think I only made it to day 8 last...

Ah alright, thanks!

#

Go buttons!

thorny obsidian Sep 7, 2021, 4:06 PM

#

awwww yeaaaaah

#

That's the CTA I like to see

trail pilot Sep 7, 2021, 4:10 PM

#

I'm gonna mess around with the UI and see if I can come up with something that looks really nice

clever wraith Sep 7, 2021, 4:10 PM

#

trail pilot Go buttons!

Goddamn that's nice

#

We need to start buttoning lancebot commands

trail pilot Sep 7, 2021, 4:10 PM

#

Agreed

thorny obsidian Sep 7, 2021, 4:11 PM

#

But I would like to see more info about the kata itself in the description and maybe condense some of the kata meta-info

trail pilot Sep 7, 2021, 4:12 PM

#

Yeah, I'm going to have the first slide be only about the difficulty and the description

#

and maybe have a pagination with the other embeds containing other information

austere hornet Sep 7, 2021, 4:18 PM

#

trail pilot Go buttons!

Nice!

austere hornet Sep 7, 2021, 4:18 PM

#

clever wraith We need to start buttoning lancebot commands

Might be a good idea

trail pilot Sep 7, 2021, 4:42 PM

#

thorny obsidian But I would like to see more info about the kata itself in the description and m...

#

this is the base, i have to clean some stuff up but yeah

stable mountainBOT Sep 7, 2021, 5:07 PM

#

fallen patrol !remind 12h fix the skip of the thing and do the other thing, tldr king-arthur#1...

It has arrived!

Here's your reminder: fix the skip of the thing and do the other thing, tldr king-arthur#10 probably need another change
see this section for what to add to the bottom of lint-build-deploy. https://github.com/python-discord/bot/blob/26cb61d15f30dfa170bb9ef61a167376ee0e3c52/.github/workflows/lint-test.yml#L135 Might need to refactor workflow to be like the below actually https://github.com/discord-modmail/modmail/blob/0ed105fd956a0455dc1ca1d19903149bacb0524f/.github/workflows/lint_test.yml#L171
[Jump back to when you created the reminder](#dev-contrib message)

thorny obsidian Sep 7, 2021, 5:10 PM

#

when will what?

#

so there is no set end date. We have our own internal guidelines for when a vote is actioned/retired, but it changes every once in a while

vale ibex Sep 7, 2021, 5:11 PM

#

trail pilot I'll add that to the footer

double checking, you shouldn't need it, since its the solutions that are licensed, not the questions themselves

thorny obsidian Sep 7, 2021, 5:11 PM

#

When you react with incident_unactioned or incident_actioned it'll archive the vote appropriately

solar swift Sep 7, 2021, 5:11 PM

#

whoa secret server 👀

trail pilot Sep 7, 2021, 5:12 PM

#

vale ibex double checking, you shouldn't need it, since its the solutions that are license...

Ah alright

solar swift Sep 7, 2021, 5:18 PM

#

thats quite nice

vale ibex Sep 7, 2021, 5:25 PM

#

should be better now that we specify docker runs on a x86 vm

#

So as long as you get docker installed, it should work 😄

#

!help voiceverify

stable mountainBOT Sep 7, 2021, 5:26 PM

#

Command Help

!voice_verify [_...]
Can also use: voice-verify, voiceverify

You cannot run this command.

*Apply to be able to use voice within the Discord server.

In order to use voice you must meet all three of the following criteria:

You must have over a certain number of messages within the Discord server
You must have accepted our rules over a certain number of days ago
You must not be actively banned from using our voice channels
You must have been active for over a certain number of 10-minute blocks*

vale ibex Sep 7, 2021, 5:27 PM

#

Yea, I can imagine. quite a few deps don't have wheels for arm

#

but we now tell docker to force the vm to run on x86, so it should be fine

#

we only added that a few days ago

#

not yet no

fallen patrol Sep 7, 2021, 5:29 PM

#

stable mountain

!remind 2h t

stable mountainBOT Sep 7, 2021, 5:29 PM

#

ROGER THAT

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

cold island Sep 7, 2021, 5:30 PM

#

Hmmm do we have an example somewhere of MaxConcurrencyReached being raised directly?

fallen patrol Sep 7, 2021, 5:30 PM

#

stable mountain

Lmfao @vale ibex a command that an admin cannot run 😛

austere hornet Sep 7, 2021, 5:31 PM

#

fallen patrol Lmfao <@126811506632294400> a command that an admin cannot run 😛

Yeah lmfao

fallen patrol Sep 7, 2021, 5:32 PM

#

cold island Hmmm do we have an example somewhere of `MaxConcurrencyReached` being raised dir...

No, but I believe that's from the dpy decorator

#

Iirc it is used on Lancebot for a few of of the games

vale ibex Sep 7, 2021, 5:32 PM

#

fallen patrol Lmfao <@126811506632294400> a command that an admin cannot run 😛

hah yea, it can only be ran in #voice-verification

fallen patrol Sep 7, 2021, 5:33 PM

#

And I think is locked to only those w/out the role

vale ibex Sep 7, 2021, 5:33 PM

#

I don't have the role 😄

austere hornet Sep 7, 2021, 5:33 PM

#

fallen patrol And I think is locked to only those w/out the role

Yep

austere hornet Sep 7, 2021, 5:33 PM

#

vale ibex I don't have the role 😄

Lmao

fallen patrol Sep 7, 2021, 5:33 PM

#

vale ibex I don't have the role 😄

Badmin smh

severe tangle Sep 7, 2021, 5:33 PM

#

@stable mountain down or discord bug?

fallen patrol Sep 7, 2021, 5:34 PM

#

!ping

stable mountainBOT Sep 7, 2021, 5:34 PM

#

You are not allowed to use that command here. Please use the #bot-commands channel instead.

severe tangle Sep 7, 2021, 5:34 PM

#

nvm, seems like a discord bug

cold island Sep 7, 2021, 5:34 PM

#

Probably just a cache thing

fallen patrol Sep 7, 2021, 5:34 PM

#

stable mountain You are not allowed to use that command here. Please use the <#26765994508681216...

Task failed successfully

severe tangle Sep 7, 2021, 5:34 PM

#

lmao

severe tangle Sep 7, 2021, 5:34 PM

#

cold island Probably just a cache thing

could be

austere hornet Sep 7, 2021, 5:35 PM

#

fallen patrol Task failed successfully

Lmao

vale ibex Sep 7, 2021, 5:35 PM

#

Alright, the issue with the site db on bot has been merged

#

in about 2 minutes yea 😄

trail pilot Sep 7, 2021, 5:36 PM

#

https://github.com/python-discord/sir-lancebot/issues/848
I'm not sure this should be assigned wtih the label category: fun

GitHub

Adding a challenge command · Issue #848 · python-discord/sir-lanceb...

Description The .challenge command would pull a random kata (challenge) from codewars.com, and the user can filter with either the query or the kyu (difficulty), 1 being the hardest, 8 being the ea...

#

It's more like utilities imo

austere hornet Sep 7, 2021, 5:41 PM

#

trail pilot It's more like `utilities` imo

Yeah I can agree with you

fallen patrol Sep 7, 2021, 5:41 PM

#

It's technically fun if it's a game

trail pilot Sep 7, 2021, 5:42 PM

#

fallen patrol It's technically fun if it's a game

It'snot really a game

#

it's just grabbing a kata

cold island Sep 7, 2021, 5:44 PM

#

fallen patrol Iirc it is used on Lancebot for a few of of the games

I see the thread pool executor for images, but not the dpy decorator

fallen patrol Sep 7, 2021, 5:44 PM

#

Hmm

cold island Sep 7, 2021, 5:45 PM

#

I think I found something online

fallen patrol Sep 7, 2021, 5:46 PM

#

This is the decorator I was thinking of

#

https://discordpy.readthedocs.io/en/stable/ext/commands/api.html#discord.ext.commands.max_concurrency

cold island Sep 7, 2021, 5:46 PM

#

yeah that's it, thanks

fallen patrol Sep 7, 2021, 5:47 PM

#

https://github.com/python-discord/sir-lancebot/blob/8a410f3abd39a1b48c514d32651a50d4bdced492/bot/exts/utilities/latex.py#L75 is the only place I found it

stable mountainBOT Sep 7, 2021, 5:47 PM

#

bot/exts/utilities/latex.py line 75

@commands.max_concurrency(1, commands.BucketType.guild, wait=True)```

fallen patrol Sep 7, 2021, 5:56 PM

#

Huh

#

That bucket seems wrong actually

cold island Sep 7, 2021, 5:59 PM

#

Well it says one at a time per server no?

fallen patrol Sep 7, 2021, 6:03 PM

#

cold island Well it says one at a time per server no?

yeah, but that should be a command bucket given that the purpose of that bucket is to keep the RAM usage down

#

king-arthur#10

dusky shoreBOT Sep 7, 2021, 6:06 PM

#

GitHub

PROpen fix status embed

fallen patrol Sep 7, 2021, 6:28 PM

#

king-arthur#10

dusky shoreBOT Sep 7, 2021, 6:28 PM

#

GitHub

PROpen Refactor workflows, add concurrency options, separate build and deploy

fallen patrol Sep 7, 2021, 6:28 PM

#

first status embed from king arthur btw

vale ibex Sep 7, 2021, 6:44 PM

#

Nice 😄

austere hornet Sep 7, 2021, 6:49 PM

#

Hey @vale ibex , when you have a moment, could you please review sir-lancebot#847? I would greatly appreciate it. Thanks so much!

dusky shoreBOT Sep 7, 2021, 6:49 PM

#

GitHub

IssueOpen Add Mad Libs game

vale ibex Sep 7, 2021, 6:50 PM

#

Could you describe what madlibs is, and what you expect the output of the game to be?

#

In the issue

austere hornet Sep 7, 2021, 6:51 PM

#

vale ibex In the issue

Sure, one sec

vale ibex Sep 7, 2021, 6:51 PM

#

No rush, I'm about to have dinner so likely won't be able to look right away

#

but in future, please try to fill out each part of the issue with what is it you plan to do.

austere hornet Sep 7, 2021, 6:54 PM

#

vale ibex but in future, please try to fill out each part of the issue with what is it you...

Ok, I will try my best. Thanks!

austere hornet Sep 7, 2021, 7:10 PM

#

Hmm, good point...

#

Well if that happens it's not really my problem...that would be something the mods would take care of imo

austere hornet Sep 7, 2021, 7:15 PM

#

vale ibex Could you describe what madlibs is, and what you expect the output of the game t...

Done, let me know if I'm still missing something

dim pelican Sep 7, 2021, 7:16 PM

#

Also Madlibs gets pretty spammy with consecutive inputs

austere hornet Sep 7, 2021, 7:21 PM

#

dim pelican Also Madlibs gets pretty spammy with consecutive inputs

True

stable mountainBOT Sep 7, 2021, 7:29 PM

#

fallen patrol !remind 2h t

It has arrived!

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

fallen patrol Sep 7, 2021, 7:29 PM

#

done ^

thorny obsidian Sep 7, 2021, 7:34 PM

#

Nah, not unless you delete the docker container. You just need to start the bot

vale ibex Sep 7, 2021, 7:38 PM

#

The issue is fixed now anyway

#

so no need to manually migrate metricity each rebuild

patent pivot Sep 7, 2021, 7:58 PM

#

@fallen patrol would rather not have separated workflows

#

we'll probably migrate all repos to single file workflows in future given the new reusable actions features

fallen patrol Sep 7, 2021, 7:59 PM

#

patent pivot we'll probably migrate all repos to single file workflows in future given the ne...

wdym?

patent pivot Sep 7, 2021, 7:59 PM

#

you've split the workflow into 3 files

fallen patrol Sep 7, 2021, 7:59 PM

#

reusable actions?

#

link

patent pivot Sep 7, 2021, 8:00 PM

#

https://github.blog/changelog/2021-08-25-github-actions-reduce-duplication-with-action-composition/

The GitHub Blog

phanatic

GitHub Actions: Reduce duplication with action composition

fallen patrol Sep 7, 2021, 8:01 PM

#

huh wow

patent pivot Sep 7, 2021, 8:02 PM

#

but yeah, basically on my domain projects at least we're doing single file workflows

fallen patrol Sep 7, 2021, 8:02 PM

#

patent pivot you've split the workflow into 3 files

i split it to get the indicators at the bottom of every pr, since it was a nonconditional skip that would be at the bottom of every pr; will revert

patent pivot Sep 7, 2021, 8:02 PM

#

skipped jobs are fine by me, the UI much better displays single file workflows in actions tab

#

and uhhhh

#

eventually my domain projects might use something like https://danger.systems/

#

so the actions under the PR aren't hugely important

fallen patrol Sep 7, 2021, 8:04 PM

#

huh, okay

#

in that case

#

gonna do something like this

#

https://github.com/discord-modmail/modmail/blob/main/.github/workflows/lint_test.yml#L171-L197

patent pivot Sep 7, 2021, 8:06 PM

#

tbf we could also remove status embed

#

lol

viscid coral Sep 7, 2021, 8:06 PM

#

https://github.com/python-discord/sir-lancebot/issues/862#issuecomment-914569280 Me not good with Python

patent pivot Sep 7, 2021, 8:06 PM

#

and turn off the GitHub ones too

#

for KA they're not hugely necessary

fallen patrol Sep 7, 2021, 8:07 PM

#

patent pivot tbf we could also remove status embed

i mean, nearly every public project on pydis has it

#

even kubernetes has it iirc

patent pivot Sep 7, 2021, 8:07 PM

#

I know, I'm just not a huge fan of the artefact stages everywhere

patent pivot Sep 7, 2021, 8:07 PM

#

fallen patrol even kubernetes has it iirc

yes, but kube doesn't actually go to #dev-log lol

fallen patrol Sep 7, 2021, 8:08 PM

#

it doesn't?

#

#dev-log message

#

(it does)

patent pivot Sep 7, 2021, 8:08 PM

#

No, it doesn't

#

The status embed does, the actual git traffic goes to our internal dev ops channel lol

fallen patrol Sep 7, 2021, 8:09 PM

#

patent pivot tbf we could also remove status embed

ah, yes I was talking about the status embed in response to this

#

changed, gibe thoughts

patent pivot Sep 7, 2021, 8:15 PM

#

yeah looks better

#

artefact stage is sad though

#

why are we running a separate flow for status embeds

#

it makes sense for multi manifests but not single

fallen patrol Sep 7, 2021, 8:18 PM

#

patent pivot why are we running a separate flow for status embeds

if I slap it in the bottom of any one of the stages, it could pose a problem ¯_(ツ)_/¯

#

only if you want it to only inform about linting, ig

patent pivot Sep 7, 2021, 8:18 PM

#

separate stage that always runs or something

fallen patrol Sep 7, 2021, 8:18 PM

#

ohhh

#

you mean the second file

#

so the reason it can't be is documented in the comments

#

eh, it isn't

fallen patrol Sep 7, 2021, 8:20 PM

#

patent pivot why are we running a separate flow for status embeds

if a fork is the pr, then what occurs is the fork does not have access to the respository secrets

#

that means that it won't run with the proper secrets for the countless forks that create a pr.

patent pivot Sep 7, 2021, 8:20 PM

#

hmm

fallen patrol Sep 7, 2021, 8:20 PM

#

there's not a safe way to get around that

#

https://docs.github.com/actions/automating-your-workflow-with-github-actions/creating-and-using-encrypted-secrets

GitHub Docs

Encrypted secrets - GitHub Docs

fallen patrol Sep 7, 2021, 8:23 PM

#

patent pivot artefact stage is sad though

this could be moved to the bottom of the lint-test job

cold island Sep 7, 2021, 8:23 PM

#

Easy PR for review
https://github.com/python-discord/site/pull/585

GitHub

Update embed description length to 4096 by mbaruh · Pull Request #5...

Max embed description length was raised to 4096, but the API wasn't updated, causing logging to fail when trying to clean long embeds.

fallen patrol Sep 7, 2021, 8:23 PM

#

as it only reports on lint test for all other pydis embeds

fallen patrol Sep 7, 2021, 8:24 PM

#

cold island Easy PR for review https://github.com/python-discord/site/pull/585

reviewed 😛

cold island Sep 7, 2021, 8:24 PM

#

Thanks 😄

fallen patrol Sep 7, 2021, 8:24 PM

#

actually I had already looked at it before you linked it here 👀

cold island Sep 7, 2021, 8:25 PM

#

Do you have alerts for all messages sent in #dev-log ? lol

fallen patrol Sep 7, 2021, 8:25 PM

#

nope, but since I'm doing the king arthur workflows rn, I had been watching

#

@patent pivot to further prove the point ; this happened when I renamed the workflow to what was configured on the status_embed.yaml which is on main #dev-log message
but it did not run when the workflow was renamed back to its current name #dev-log message

patent pivot Sep 7, 2021, 8:27 PM

#

I know it doesn't work, yeah

fallen patrol Sep 7, 2021, 8:28 PM

#

sad_cat_thumbs_up

cold island Sep 7, 2021, 8:39 PM

#

It's used in one place shrug

brazen charm Sep 7, 2021, 8:44 PM

#

cpr = "!cd ${GIT_PREFIX:-.} && git fetch upstream refs/pull/${1}/head && git checkout FETCH_HEAD #"

vale ibex Sep 7, 2021, 9:11 PM

#

!cd ${GIT_PREFIX:-.} will cd to the project root dir
git fetch upstream refs/pull/${1}/head will fetch the given pull request number from upstream
git checkout FETCH_HEAD # will checkout that pulled PR

fallen patrol Sep 7, 2021, 9:11 PM

#

lmao

#

@vocal wolf be yoda, you will

vocal wolf Sep 7, 2021, 9:12 PM

#

?

fallen patrol Sep 7, 2021, 9:13 PM

#

https://github.com/python-discord/sir-lancebot/pull/605#issuecomment-914205979

cold island Sep 7, 2021, 9:27 PM

#

Hmm how would I use VSC to attach to the bot container and debug it?

#

I'll just try the tutorial

brazen charm Sep 7, 2021, 9:36 PM

#

feels like running it locally would be the simplest

cold island Sep 7, 2021, 9:46 PM

#

Hmmm can I run bot localy and site in docker?

tawdry vapor Sep 7, 2021, 9:48 PM

#

yes I've been doing that for years

#

You can select which services to start with docker-compose

#

docker-compose up -d web

cold island Sep 7, 2021, 9:49 PM

#

And are there changes I need to do in the config?

#

Almost 5 years

tawdry vapor Sep 7, 2021, 9:49 PM

#

You just need to change the API url to localhost

#

You can double check the contrib guide, it explains how

clever wraith Sep 7, 2021, 9:51 PM

#

hey @brisk brook, what did you mean regarding https://github.com/python-discord/sir-lancebot/pull/849#discussion_r703760977

GitHub

`.quack` by Kronifer · Pull Request #849 · python-discord/sir-lance...

Relevant Issues

Closes #652
Description

Implemented a .quack command that can take a seed and optional ducky type if you want to generate a duck or manduck. If the seed given is a word, that wor...

brisk brook Sep 7, 2021, 9:52 PM

#

My question is, why are you using .get()?

#

It's usually used when you think that a key is missing, but you never have any if-statements checking for it

brazen charm Sep 7, 2021, 9:52 PM

#

tawdry vapor You can select which services to start with docker-compose

I use the docker override file to completely ignore it

clever wraith Sep 7, 2021, 9:53 PM

#

brisk brook It's usually used when you think that a key is missing, but you never have any i...

fair ill get on that

fallen patrol Sep 7, 2021, 9:53 PM

#

huh

cold island Sep 7, 2021, 9:59 PM

#

tawdry vapor You can double check the contrib guide, it explains how

Hmm it only says this https://pythondiscord.com/pages/guides/pydis-guides/contributing/bot/#run-on-the-host

Python Discord - Contributing to Bot

A guide to setting up and configuring Bot.

#

Site is running, checked localhost:8000, but bot can't connect

brazen charm Sep 7, 2021, 10:00 PM

#

what does the config look like?

cold island Sep 7, 2021, 10:01 PM

#

urls:
    # PyDis site vars
    site:        &DOMAIN       "localhost:8000"
    site_api:    &API    !JOIN ["api.", *DOMAIN]
    site_paste:  &PASTE  !JOIN ["paste.", "pythondiscord.com"]
    site_schema: &SCHEMA       "http://"
    site_staff:  &STAFF  !JOIN ["staff.", *DOMAIN]

    paste_service:                      !JOIN ["https://", *PASTE, "/{key}"]
    site_logs_view:                     !JOIN [*SCHEMA, *STAFF, "/bot/logs"]

tawdry vapor Sep 7, 2021, 10:01 PM

#

It is explained in the config section

#

Set urls.site:

    If running the webserver in Docker, set it to "web:8000".
        If the site container is running separately (i.e. started from a clone of the site repository), then COMPOSE_PROJECT_NAME has to be set to use this domain. If you choose not to set it, the domain in the following step can be used instead.
    If running the webserver locally and the hosts file has been configured, set it to "pythondiscord.local:8000".
    Otherwise, use whatever domain corresponds to the server where the site is being hosted.

#

I guess you fall under the last point

#

I use localhost and it works. Maybe check docker ps to see how the service is being exposed on the network

cold island Sep 7, 2021, 10:03 PM

#

1f8190b9efc4 ghcr.io/python-discord/site:latest "python manage.py ru…" 44 minutes ago Up 45 minutes 127.0.0.1:8000->8000/tcp bot_web_1

tawdry vapor Sep 7, 2021, 10:06 PM

#

Can you access it in your browseR?

cold island Sep 7, 2021, 10:06 PM

#

yep

tawdry vapor Sep 7, 2021, 10:06 PM

#

what's the error when the bot fails to connect

cold island Sep 7, 2021, 10:07 PM

#

aiohttp.client_exceptions.ClientConnectorError: Cannot connect to host api.localhost:8000 ssl:default [None]
2021-09-08 00:56:26 | bot | CRITICAL | Could not connect to site API. Is it running?

tawdry vapor Sep 7, 2021, 10:07 PM

#

Maybe you are missing site_api_schema: "http://"? I have it in my config.

#

Not sure if it is actually being used though

stable mountainBOT Sep 7, 2021, 10:07 PM

#

patent pivot !remind 18h t

It has arrived!

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

cold island Sep 7, 2021, 10:08 PM

#

I'll add it to config.yaml, but it should be pulling it from config_default

#

same thing

tawdry vapor Sep 7, 2021, 10:10 PM

#

Does http://api.localhost:8000/bot/ work in your browser?

cold island Sep 7, 2021, 10:10 PM

#

tawdry vapor Sep 7, 2021, 10:11 PM

#

Yeah that's how it should be. I am running out of ideas. If your browser can connect I see no reason why the bot cannot

brazen charm Sep 7, 2021, 10:11 PM

#

Are you on win? I also get a DNSError if I try to use localhost but the domain from the hosts file works

cold island Sep 7, 2021, 10:12 PM

#

I'm on windows yeah

tawdry vapor Sep 7, 2021, 10:12 PM

#

Try 127.0.0.1 first

brazen charm Sep 7, 2021, 10:12 PM

#

that had the same error on my machine

tawdry vapor Sep 7, 2021, 10:12 PM

#

Works fine for me on Windows 🤷‍♂️

brazen charm Sep 7, 2021, 10:13 PM

#

no idea then, but worth a try at least

cold island Sep 7, 2021, 10:13 PM

#

tawdry vapor Try 127.0.0.1 first

no dice

tawdry vapor Sep 7, 2021, 10:15 PM

#

Can you connect at all from python e.g. requests.get("http://api.localhost:8000/bot/")

cold island Sep 7, 2021, 10:17 PM

#

tawdry vapor Can you connect at all from python e.g. `requests.get("http://api.localhost:8000...

socket.gaierror: [Errno 11001] getaddrinfo failed

tawdry vapor Sep 7, 2021, 10:18 PM

#

I don't know why that happens, sorry

brazen charm Sep 7, 2021, 10:20 PM

#

I think that is because of the subdomain, but the bot fails before that afaik

brazen charm Sep 7, 2021, 10:26 PM

#

brazen charm I think that is because of the subdomain, but the bot fails before that afaik

Looks like it connects if I leave out the subdomain so there could just be some weirdness around that
I'd just set the domains in the hosts file and that should work I believe

cold island Sep 7, 2021, 10:27 PM

#

127.0.0.1   pythondiscord.local
127.0.0.1   api.pythondiscord.local
127.0.0.1   staff.pythondiscord.local
127.0.0.1   admin.pythondiscord.local

Is in my hosts file

#

For the bot? no, I only have the bot token

brazen charm Sep 7, 2021, 10:28 PM

#

change the localhost to the pythondiscord.local and I think that shouldwork

cold island Sep 7, 2021, 10:28 PM

#

brazen charm change the localhost to the pythondiscord.local and I think that shouldwork

In the config? that didn't work

brazen charm Sep 7, 2021, 10:28 PM

#

What's the whole traceback?

cold island Sep 7, 2021, 10:28 PM

#

sec

#

Traceback (most recent call last):
  File "C:\Program Files\Python39\lib\runpy.py", line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "C:\Program Files\Python39\lib\runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "C:\Users\_\Desktop\pydis\bot\bot\__main__.py", line 15, in <module>
    bot.instance.run(constants.Bot.token)
  File "C:\Users\_\AppData\Local\pypoetry\Cache\virtualenvs\bot-uy3Tm0_t-py3.9\lib\site-packages\discord\client.py", line 723, in run
    return future.result()
  File "C:\Users\_\AppData\Local\pypoetry\Cache\virtualenvs\bot-uy3Tm0_t-py3.9\lib\site-packages\discord\client.py", line 702, in runner
    await self.start(*args, **kwargs)
  File "C:\Users\_\AppData\Local\pypoetry\Cache\virtualenvs\bot-uy3Tm0_t-py3.9\lib\site-packages\discord\client.py", line 665, in start
    await self.login(*args, bot=bot)
  File "C:\Users\_\Desktop\pydis\bot\bot\bot.py", line 256, in login
    await self.cache_filter_list_data()
  File "C:\Users\_\Desktop\pydis\bot\bot\bot.py", line 87, in cache_filter_list_data
    full_cache = await self.api_client.get('bot/filter-lists')
  File "C:\Users\_\Desktop\pydis\bot\bot\api.py", line 80, in get
    return await self.request("GET", endpoint, raise_for_status=raise_for_status, **kwargs)
  File "C:\Users\_\Desktop\pydis\bot\bot\api.py", line 75, in request
    await self.maybe_raise_for_status(resp, raise_for_status)
  File "C:\Users\_\Desktop\pydis\bot\bot\api.py", line 67, in maybe_raise_for_status
    raise ResponseCodeError(response=response, response_json=response_json)
bot.api.ResponseCodeError: Status: 401 Response: {'detail': 'Invalid token.'}

brazen charm Sep 7, 2021, 10:30 PM

#

You'll need BOT_API_KEY=badbot13m0n8f570f942013fc818f234916ca531 in your env vars

cold island Sep 7, 2021, 10:31 PM

#

Worked, tysm 👌

trail pilot Sep 7, 2021, 10:32 PM

#

@thorny obsidian I think you were interested in seeing the product with the UI added for challenges: I basically added a link button along with a few dropdown options to switch

thorny obsidian Sep 7, 2021, 10:33 PM

#

ooooh neat

trail pilot Sep 7, 2021, 10:33 PM

#

I'm probably gonna do some code cleanup and then I think I'll unmark it from the draft label

vocal wolf Sep 7, 2021, 10:38 PM

#

trail pilot <@!212644551926611969> I think you were interested in seeing the product with th...

good stuff

trail pilot Sep 7, 2021, 10:39 PM

#

does it automatically request a review from a code owner?

brazen charm Sep 7, 2021, 10:39 PM

#

yes

trail pilot Sep 7, 2021, 10:39 PM

#

Ah alright, thank you

fallen patrol Sep 7, 2021, 10:44 PM

#

trail pilot <@!212644551926611969> I think you were interested in seeing the product with th...

suggestion, reverse dropdown menu and button

trail pilot Sep 7, 2021, 10:44 PM

#

yep, i reversed it a few minutes ago

#

right after the video cause it looked weird

cold island Sep 7, 2021, 11:08 PM

#

tawdry vapor yes I've been doing that for years

never going back

clever wraith Sep 8, 2021, 1:44 AM

#

Damn going to school will significantly impede my ability to program all day

trail pilot Sep 8, 2021, 2:03 AM

#

clever wraith Damn going to school will significantly impede my ability to program all day

I started school and I can still program for an hour or two a day

cold island Sep 8, 2021, 2:32 AM

#

Staying at school is what will allow you to keep programming in your free time. Listen to your teachers ok_handbutflipped

clever wraith Sep 8, 2021, 2:33 AM

#

ik

vocal wolf Sep 8, 2021, 8:48 AM

#

@static canyon Thanks for the !i l PR, it has been merged.

patent pivot Sep 8, 2021, 10:16 AM

#

@vocal wolf no solution yet - we're blocking that on https://github.com/python-discord/kubernetes/issues/96

GitHub

Latency issues and failing checks on site resulting in service unav...

Site is intermittently killed by Kubernetes and respawned, and the last time this happened, both site pods were killed at the same time. The following key findings were made: A long time passes by ...

vocal wolf Sep 8, 2021, 10:40 AM

#

patent pivot <@196664644113268736> no solution yet - we're blocking that on https://github.co...

alrioght

austere hornet Sep 8, 2021, 1:23 PM

#

clever wraith Damn going to school will significantly impede my ability to program all day

Yeah same it sucks 😢

keen valley Sep 8, 2021, 2:11 PM

#

Did https://pythondiscord.com/ allow web scrapping?

Python Discord | Home

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.

patent pivot Sep 8, 2021, 2:12 PM

#

generally, if a site does not have a present robots.txt, web scraping is considered permitted

#

so we do permit that, yes

#

in fact

green oriole Sep 8, 2021, 2:13 PM

#

And we actually need to have a robots.txt file

patent pivot Sep 8, 2021, 2:13 PM

#

please hold

#

i hav an idea

#

i retract my idea let's just add one to django