dev-contrib

[brisk brook]

See this in our test server: #476196062214750219 message

Not quite sure why you can't reproduce it

Could it be well, timezones 😅. I live in Sweden with UTC+01:00 right now I believe

I mentioned the theoretical fix in the review though, we should aim to have all datetimes be aware now with how discord.py disnake treats them.

[vale ibex]

was this PR created before we migrated to d.py 2.0?

[brisk brook]

I don't have a lot of time, I was gonna test the botcore PR and then shut off my computer for the day, would you like me to try to reproduce it tomorrow?

[vale ibex]

I have no concept of how long ago that was

since merging in main might have fixed it

(if the naive time was from discord py)

[brisk brook]

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

The migration was merged October 17 and the PR created December 10, so that can't be it 🤔

[vale ibex]

Pr desc also has

Since discord.py moved to timezone aware datetimes, the discord timestamp converter failed, therefore giving no confirmation message, this has now been fixed.

so it's likely unrelated

[exotic ember]

yes please

[brisk brook]

Okay I have requested a review from myself, I will take a look at it tomorrow on my Notifications page and let you know!

Hey @vale ibex for the bot-core PR, can you remove the tag and (when it's merged) apply it on the merge commit like what was done for v2.1.0?

If not, what should be the status-quo moving forward?

[vale ibex]

lol yea, I've made a note to myself not to push tags until they're in main

I'm just going to move the tag to the tip of main when this is merged

[brisk brook]

Sounds good!

[vale ibex]

I had assumed how tags worked, which was wrong

[last patio]

good evening

[thorny spade]

i am here

to begging

for a review

to python-discord/sir-lancebot#1041

[dusky shore]

GitHub

truncate output.

[wind ruin]

just wondering - are the review rules public or only staff can see them?

[thorny spade]

idk

[wind ruin]

nvm

[vocal prairie]

the policy-bot rules are public iirc

[patent pivot]

all entirely public yeah

one sec

view at https://policy-bot.pythondiscord.com/details/python-discord/site/686

and then click the python-discord/site: main in top left to get to the rules section

in any repo with policy-bot, the file will be at .github/review-policy.yml https://github.com/python-discord/bot/blob/main/.github/review-policy.yml

though sometimes that's a hook to another repo, like our .github repository

[trim cradle]

@wild prism I saw the issue, I'll respond Soon ™️

but if it's within the next 7 hours, it means I'm not doing work.

[vale ibex]

This is great news

the update to github's PR files changed tab has fixed the memory leak issue i was having

[wild prism]

flex regex, shirk work

[brisk brook]

I apologize for all pings, but in case you have the test server muted, I pinged you in bot-commands. It's not something urgent, but I realize that you may miss it if it's completely muted.

[exotic ember]

no worries, ping me all you want

I have the test server always muted

@brisk brook do you have master merged?

[brisk brook]

Yeah let's continue here 😅. I don't know, I am looking into why the PR isn't showing up in my list (I used a previous branch for it)

[exotic ember]

@short snow deleted the branch iirc

[brisk brook]

Oooooooh yeah because I am missing two PRs which both seem to be theirs. By the looks of it the whole fork was removed.

Gonna open an issue on the GitHub Desktop repository, but how can you continue working on the PR if the branch it is on is removed @exotic ember?

[exotic ember]

I can just push to a new branch and open another PR

[brisk brook]

Ah, right. Odd that the PR is still up though

[brisk brook]

Yeah hmm I am not sure how to pull the changes then? How do you usually pull changes?

[exotic ember]

are there any changes since you last reviewed? I just pulled the PR using the github cli yesterday, then merged the latest master locally

[brisk brook]

The last commit I had was 24c1566583fd83b07000106f627cf0abd0f7001e

[exotic ember]

yeah thats what i have, then i merged master locally

[brisk brook]

I think the easiest for me would be if you pushed that to a branch on the bot-repo, can you do that when you get the time to?

[exotic ember]

yep, ill ping you when I do

[trim cradle]

@wild prism I tried to approve

but I can't find approve button

help

[wild prism]

uh ehm

[trim cradle]

I did it 😄

anyway nice work

[wild prism]

ty for the good editing

[trim cradle]

ty for being amazing 😄

[exotic ember]

@brisk brook sorry it took so long, I've pushed to https://github.com/python-discord/bot/tree/fix/modpings

[short snow]

Oh sorry, I didn't realize that deleting the fork would be problematic for people working on it

[exotic ember]

no problem

[cold island]

Do we care? @vale ibex

[vale ibex]

Yea, there's a lot of tags that mention discord.py

I mentioned in the PR desc that I didn't scope those

[cold island]

@vale ibex have you tried playing with the help command buttons? it seems to stutter, work, but then complain it failed

[vale ibex]

I have not

let me look at possible breaking changes in views

[cold island]

@vale ibex approved

[vale ibex]

nice

just going to squash and rebase onto main

luckily nothing got merged into main that conflicted

went through each commit too, and they didn't add things that would break

[tawdry vapor]

For some reason my test bot is really unresponsive. Any ideas why?

2022-03-05 15:49:03 | bot.pagination | TRACE | Got reaction: ➡
2022-03-05 15:49:13 | bot.pagination | DEBUG | Got next page reaction - changing to page 2/6

The first time I clicked the reaction it didn't even log that it received it

[vale ibex]

hmmm

[tawdry vapor]

And see how it took 10 seconds for it to respond

[vale ibex]

what does ping say about your connection?

[tawdry vapor]

Command processing time
321.743 ms
Python Discord website status
Healthy
Discord API latency
77.635 ms

[vale ibex]

that seems reasonable

[tawdry vapor]

It took a while for it to respond with the ping embed even

I think it's something wrong with HTTP requests.

Cause it also hangs when attempting to connect to our API

2022-03-05 15:52:49 | bot | INFO | Attempting site connection: 1/3
2022-03-05 15:53:12 | bot.exts.info.pep | TRACE | Started refreshing PEP URLs.

Or maybe DNS?

[gritty wind]

You mean patching in our own handlers and stuff to patch the library’s http handler was a bad idea? :DDDDDD

[tawdry vapor]

It's not only requests to Discord

[gritty wind]

Based on the api comment, it seems to be the entire http handler. Since I imagine this isn’t a problem for most disnake users my guess is that it’s the custom http stuff which is what I was referring to

[vale ibex]

I don't experience this issue though, which is odd

Are you on a branch with the disnake changes Mark?

[tawdry vapor]

No

I'm not sure what you're talking about patching the HTTP handler. The api client has its own session

Are we patching aiohttp somewhere?

[gritty wind]

Not patching aiohttp, but rather patching the client from the discord library with our own connectors and resolvers

[tawdry vapor]

Hmm I think the site connection is misleading

There's no log for a successful connection.

And the next log is from a cog that was waiting for the bot to be ready

So the delay is more than likely just the time taken to log into discord

I know what you're talking about with the connector now. The API client does use the same connector though.

[vale ibex]

is the delay all the time or just near start up?

[tawdry vapor]

Always

[vale ibex]

since the user syncer could be causing slow downs

[tawdry vapor]

Any command I run

it's just slow

[vale ibex]

huh, odd

[gritty wind]

Where is this? I’d like to test it

[tawdry vapor]

My bot on the test server

[vale ibex]

#476196003733569547 message

[gritty wind]

I mean what Pr/branch 😅

I jumped in about half way through the conversation

[tawdry vapor]

https://github.com/python-discord/bot/pull/1680 I don't think it matters it was happening with other branches too

[gritty wind]

Oh gotcha

[tawdry vapor]

I'm going to try to make logging more verbose for libraries to see if there is any insight on network requests

I am quite suspect of it being a DNS issue

[cold island]

!echo .

[stable mountain]

.

[cold island]

it didn't explode!

[vale ibex]

huzzah

!int e discord.version

[stable mountain]

In [4]: discord.__version__
Out[4]: 2.4.0```

[cold island]

discord? not disnake?

[vale ibex]

I kept the discord var in the int e globals

it just points to disnake

[cold island]

ah

[tawdry vapor]

It seems to pretty consistently get delayed by 10 seconds which is weird

2022-03-05 16:12:53 | bot.exts.moderation.infraction._scheduler | TRACE | Awaiting the infraction #4 application action coroutine.
2022-03-05 16:13:03 | discord.gateway | DEBUG | For Shard ID None: WebSocket Event: {'t': 'GUILD_MEMBER_UPDATE', 's': 10

[fallen patrol]

that looks like its still using dpy?

[tawdry vapor]

Yes

I did update my branch to disnake a few minutes ago and it's still the same problem

[fallen patrol]

what's the ref so i can check it out locally?

[tawdry vapor]

Haven't pushed yet but it's bot#1680

[dusky shore]

GitHub

Add command to resend infraction DM

[tawdry vapor]

It doesn't matter. I could be on main and it'd be the same problem

It's just something weird with my system; I dunno. Like I said, I suspect DNS cause I've seen what it looks like to have DNS server down. Requests take a long time but eventually load.

Might try disabling aiodns or all the custom connector stuff in general

[balmy sluice]

bad time to port

[sleek steppe]

Funny how dpy comes back the day we switch to disnake 😄

[austere hornet]

Yeah

[cold island]

We still get the latest Discord features, and leave behind an alpha release, so it's not without merit

And since it was mostly changing the namespace it wasn't too difficult

[fallen patrol]

We are not stopping development anytime soon, either

[thorny spade]

the magic thing has happened

so will you migrate to dpy again

[vocal wolf]

Is it just me or are the disnake docs kind of laggy?

[fallen patrol]

I thought we fixed that

[austere hornet]

Seems fine to me

[fallen patrol]

Since every token is on the same page, they will be slightly laggy... but we did remove some of the code that was causing a lot of lag

[fallen patrol]

are you on stable or latest?

[vocal wolf]

latest

[fallen patrol]

they seem fine once they've loaded

which page are you on, specifically?

[vocal wolf]

the lag seems to have just

disappeared

I was on ApplicationCommandInteraction

[fallen patrol]

ah, maybe the laggy css was cached

[vocal wolf]

perhaps

[gaunt cypress]

Why don't we use gist?

[thorny obsidian]

use gist for what?

[gaunt cypress]

Pasting stuff?

[cold island]

You're welcome to if you want

Not sure what you mean

[sleek steppe]

we have our own pastebin, if that's what you're asking

!paste

[stable mountain]

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.pythondiscord.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.

[gaunt cypress]

Ik we have but "why not use gists"

[thorny obsidian]

which account will the gist live under? We can't create gists for orgs. Gists are meant to be somewhat permanent and I'd feel uncomfortable with uploading arbitrary user pastes onto github associated with a user account

[gaunt cypress]

Ahh I see thanks

[green oriole]

The timing on that is just gold

[placid ermine]

it really is

[digital plank]

what are the future plans btw

[crude gyro]

we're gonna need some time to discuss that before we can comment on it, there's a discussion now among the core devs.

[austere hornet]

👍

[brazen rover]

Wouldn't it be nice if there was a subscribe feature for the discord bot?
When you ask a question in the help chanels it often takes a while for someone to see your question and answer something.
The bot already tells you via PMs that you opened a help channel.
I think it would be nice if you could tell the bot that you want to get notified via PM once the first person answered something in the help channel?
That way you could do something different while waiting for an answer, like working on code (which we of course love)

[austere hornet]

I think if you run !helpdm on in #bot-commands that would do it, no?

[placid ermine]

helpdm is for the helper, not the helpee

[austere hornet]

Ohhh

Hmm, then that might actually be nice to add

[brazen rover]

!helpdm help

[stable mountain]

Bad argument

help is not a recognised boolean option

Command Help

!helpdm <state_bool>
*Allows user to toggle "Helping" dms.

If this is set to on the user will receive a dm for the channel they are participating in.

If this is set to off the user will not receive a dm for channel that they are participating in.*

[gritty wind]

Having no clue what this feature is, wouldn't this imply it'll work for both

!src helpdm

[stable mountain]

Command: helpdm

Allows user to toggle "Helping" dms.

Source Code

Go to GitHub

bot/exts/help_channels/_cog.py:647

[gritty wind]

Nope, explicit ignore

Not sure why though

[clever wraith]

will the newest 2.0 be updated for pydis bot?

i can actually implement my modal without a fork now :H

[vocal prairie]

We recently migrated to disnake.

The decision on discord.py is yet to be determined, that's a core dev discussion.

[clever wraith]

??? why disnake

[sleek steppe]

Pretty much we migrated to disnake an hour before danny announced the continuation of dpy, and we were preparing for a switch to disnake for a while anyways

[dry folio]

it's been so long since i've done anything remotely related to bot development 😔

i have no idea what's going on with the discord api wrappers currently, but now that d.py continues development, maybe it'll be good to switch back

[clever wraith]

In our opinion, all trust for d.py is long gone

[mint nebula]

they betrayed us

[patent pivot]

we'll be discussing it internally with core devs, no updates immediately

[fervent sage]

fyi for everyone reading who doesn't know, we (maintainers of disnake) have no intent of stopping development in any way, the lib isn't gonna just disappear because dpy is back, so that shouldnt be a worry :P

[crude gyro]

I thought you maintained nextcord

I'm so confused

@fervent sage are you involved in both of the forks?

[fervent sage]

Yeah

I'm a NC and disnake maintainer because I'm weird lol

Though I mostly handle infrastructure for disnake like websites and domains and stuff rather than doing a ton of coding

[cold island]

@vale ibex what is the website where you write a json and it shows you the resulting embed?

[wind ruin]

https://leovoel.github.io/embed-visualizer/

there's also https://cog-creators.github.io/discord-embed-sandbox/ to build them, which produces discord.py code for you

[cold island]

That's what I was looking for, thanks 👍

[vale ibex]

that's the one 😄

[brisk brook]

I somewhat forgot about this over the weekend, I tried it out on this branch which seems to have fixed the issue. 👍

[cursive relic]

Can I find the list of existing topics from somewhere of .topic command

[vale ibex]

https://github.com/python-discord/sir-lancebot/blob/main/bot/resources/utilities/py_topics.yaml

[cursive relic]

Thx 😄

[surreal veldt]

Good evening if I want to make a pull request, are these steps mandatory?

[gritty wind]

These are the requirements for running the project locally

For most code changes, you'll want to test your code before you PR

(In general, not just to our projects)

If you're doing content changes, it's up to you

Personally I want to make sure things look how I expect, but at the end of the day it's not necessary

There'll also be a deploy preview showing what the site will look like with your changes

[surreal veldt]

If I do the requirements or without?

[gritty wind]

The requirements are just for running the project on your local machine. The preview will be built automatically provided you don't have errors that prevent it from running

[fallen patrol]

sigh.

That monkeypatch...

#dev-contrib message
I'm seriously considering implementing this into disnake

[gritty wind]

It's been greaaaat

Dev-log is loving it

Err error-log

[fallen patrol]

I would like to also add that acking has been disabled

this affects clients, not bots

oh acking is back up thankfully

[fallen patrol]

yeah i bet

[trim cradle]

I don't think we've ever measured how long it takes for a given message to iterate through every regex in the blacklist, or has that since been done?

[gritty wind]

A single message? Not very long

I could int e it

[trim cradle]

if you feel so inclined 😛

though if you want to int e to print out the list of patterns, I could experiment and report back

[gritty wind]

0.0004418427124619484s on a single message on all filters (not just regex)

Check your DMs

[austere hornet]

What happened to the status of @humble leaf? I remember it was "The-Not-Quite-So-Bot-as-Sir-Lancebot" or something like that at one point, did it get removed?

[patent pivot]

it still is - it's in the repo image

[sleek steppe]

they're talking about @humble leaf's discord status, which is missing for some reason

[patent pivot]

ahhhh

hmmmm

M-x deploy restart sir-robin

[radiant merlin]

:white_check_mark: Restarted deployment sir-robin in namespace default.

[patent pivot]

there we go

it was probably knocked around a bit as a result of the discord outage earlier

[fallen patrol]

^ that just happens sometimes

[fallen patrol]

to explain it a bit futher, status tend to sometimes be removed when the api is deployed. It would also explain why @viscid badge doesn't have a status either

[austere hornet]

Yeah that's what I meant sorry, should have been more clear

[austere hornet]

I see, thanks

[clever wraith]

Heya, is there any guide on how to setup the database or how to bypass the check saying connection doesn't work? (I won't need the database for my changes.)

[molten perch]

Hey! I presume you're talking about the site project? (https://github.com/python-discord/site)
As this is a channel for Python Discord related projects.

[clever wraith]

Yep

[molten perch]

In that case I would suggest to read that guide: https://www.pythondiscord.com/pages/guides/pydis-guides/contributing/site/
Let me know if you have any questions or difficulties.

[clever wraith]

Already did so, the issue is that for some reason docker doesn't start? Says the daemon is not running.

Which isn't as I get an error when starting the service.

[molten perch]

Do you use Docker Desktop?

[clever wraith]

No, but I probably will try to get on Windows and see how it goes there 😅

[molten perch]

Oh, alright then. However you could try to run it manually?
With the: dockerd command. (https://docs.docker.com/config/daemon/#start-the-daemon-manually)

[clever wraith]

Yeah could've tried that

[cursive relic]

Is there possibility to do !resources <tag> or something similar? If not, would it be possible

[dim pelican]

To look up the tags we have?

[cursive relic]

Eh I don't think that would work, the site wasn't exactly what I thought it to be

[rapid swallow]

i think there's an issue open for this

[dim pelican]

Oh to search the resources directly you mean

[cursive relic]

Ye

[rapid swallow]

the command will provide you a link with the filtered according to the tag

[cursive relic]

Oh there was issue for it, didn't know it

Thx for info 😄

[rapid swallow]

i could be wrong though

https://github.com/python-discord/bot/issues/2079

[rapid swallow]

my bad, I'm never wrong

[rapid swallow]

oh wait

[rapid swallow]

@cursive relic this is what you're suggesting, correct?

[cursive relic]

Yep, that's exactly what I thought about

Thank you!

[surreal veldt]

could someone give me a brief break down of how a normal message would be put into a markdown file?

[thorny obsidian]

What do you mean by normal message?

[surreal veldt]

For example if I want to put a pin in #discord-bots to a markdown file

[thorny obsidian]

So, it would just like regular discord markdown with an embed except now you also have access to # ## and ###

[brisk brook]

https://github.com/python-discord/sir-lancebot/pull/1036 should have received enough approvals to be merged now

[fallen patrol]

wow, @slow bone now has a banned name

https://github.com/discord/discord-api-docs/commit/bf3e2ec9d0d926c5940e9020b4fe7f7f1cc84dc0

Usernames cannot contain the following substrings: '@', '#', ':', '```', 'discord'

[rapid swallow]

is channel creation counted in 2 edits/10 minutes ratelimit for editing channels?

seems like it

[rapid swallow]

how do I check the number of edits left before being rate limited? I dont see anything for it hmm

[gritty wind]

@rapid swallow rate limits on all requests are sent in a X-… header

If you’re wondering if they’re documented anywhere

Haha imagine discord doing something useful like that

[rapid swallow]

ok but how do i access this withh discord.py?

[tawdry vapor]

Why do you need it?

I believe d.py already delays requests to respect the headers.

I don't think it's accessible unless you use the http client to request endpoints yourself

[rapid swallow]

it's for modmail's ?tag command

?help tag

[slow bone]

`?tag [tag]`

Append a tag at the beginning of the channel name.

Using the command without any argument will reset it.

Permission level: SUPPORTER [2]

[rapid swallow]

when you try to edit the name of a channel 3rd time in a 10 minute window, it just silently fails

i thought it'd be neat if the command could show some kinda useful message

[gritty wind]

It doesn’t fail it gets postponed

[rapid swallow]

oh

[gritty wind]

You’d need to entirely bypass the lib to get that feature

[rapid swallow]

I used coomands.cooldown() but thing is channel creation is also included in this

[gritty wind]

Aka access the http client directly and send out the request manually

[rapid swallow]

how about switching out discord.py in favor of hikari?

I don't see any problems which can arise from that

[gritty wind]

A dirty fix for this is to just

[rapid swallow]

this is what it looks like

[gritty wind]

Send a message confirming receipt and saying it might take up to 10 mins to reflect

[rapid swallow]

I guess I could also pretend that this is not a problem

that's how i solve all of my problems

[tawdry vapor]

What problems?

[rapid swallow]

the ones which go away when you ignore them

[thorny spade]

hi there

https://github.com/python-discord/sir-lancebot/issues/1023

can anyone assign me @MaskDuck on github, since id like to work on this

[fallen patrol]

part of this is actually a @slow bone implementation detail

it makes the channel and then sets the topic even though creating the channel can have the topic set when makingit (╯°□°）╯︵ ┻━┻

and channel creation actually has an extremely high ratelimit, of 500 requests in every 600 seconds

anyways

https://github.com/Rapptz/discord.py/blob/462ba84809f43349296c44cbe6468f631a00edab/discord/http.py#L135-L143

[stable mountain]

discord/http.py lines 135 to 143

bucket = route.bucket
method = route.method
url = route.url

lock = self._locks.get(bucket)
if lock is None:
    lock = asyncio.Lock()
    if bucket is not None:
        self._locks[bucket] = lock```

[fallen patrol]

https://github.com/Rapptz/discord.py/blob/462ba84809f43349296c44cbe6468f631a00edab/discord/http.py#L591-L592

[stable mountain]

discord/http.py lines 591 to 592

def edit_channel(self, channel_id, *, reason=None, **options):
    r = Route('PATCH', '/channels/{channel_id}', channel_id=channel_id)```

[fallen patrol]

# make a route instance for modifying channels
route = discord.http.Route('PATCH', '/channels/{channel_id}', channel_id=channel.id)
# see if a lock exists for that 
lock = self.bot.http._locks.get(route.bucket)

# if this returns True, then you can't make a request
# if this is False, then you are able to make a request
return lock and lock.locked()

[patent pivot]

Looks like someone else has already registered interest

[fallen patrol]

this essentially will tell you if there is a ratelimit or not for a specific endpoint

is it hacky af? yes.
is modmail already hacky af? also yes.

[thorny spade]

😦

[fallen patrol]

(note that this code isn't tested but I'm currently refactoring ratelimits so I thiink i have a pretty good idea of it)

[patent pivot]

Does anyone want to migrate moderator ping config to site?

From redis, to align with new persistence policies

[thorny spade]

wdym

maybe let mods do this

[patent pivot]

we're not storing data that needs proper persistence in redis anymore

[thorny spade]

yes?

[patent pivot]

it doesn't require any info from the moderators, the mods set their config through bot commands, it's just that right now that data gets stored in redis, not on site

so anyone can do the migration

[thorny spade]

so we need to move ping config from bot to site?

[patent pivot]

no, the storage needs to be moved from redis to site

[thorny spade]

hm

i am not really experienced on redis and site

so maybe not me

[patent pivot]

yeah just offering it up to anyone

[thorny spade]

https://github.com/python-discord/bot/issues/2007

is that this issue

[patent pivot]

uhh it's related

same system

[gritty wind]

Zig said we might not necessarily want to move this

We’re fine with losing it

[patent pivot]

ahh alright, that may ring a bell, I’d still rather move it and don’t mind doing that myself - I’d rather just put in the work than just accept it’s not persistent, when realistically it probably should be

[cold island]

It might be better to wait until kosa is done with the modpings pr

[surreal veldt]

So I want to contribute to the python site. I forked it and cloned it to a local folder on my computer. Now I was told to create a new branch. Anyone mind explaining to me how to do this?

[dim pelican]

git branch -b <your branch name> will create a new git branch on your forked repo

Then when you make a pull request, you can use that new branch for the new features, and keep the main branch un touched, so you can pull updates from the origin main

[surreal veldt]

And, what do I replace branch name with? Is it the folder name? Also I'm using VS code terminal

[dim pelican]

A good branch name should be short, and try to describe what change you are making

[surreal veldt]

Are there any in-depth videos? I tend to understand a lot easier from them

[dim pelican]

Also just to back track a little bit, are you working off an issue on the repo? Or is this part of the pinned message migration project?

[surreal veldt]

The pinned message migration project

[dim pelican]

Workflow: https://www.youtube.com/watch?v=1SXpE08hvGs

Ah. Maybe git branch -b pin_migrate_raven?

[surreal veldt]

Oh so I can put any name, however, how does it know where to create the new branch in?

[dim pelican]

It depends on what git repo you are currently in

It sounds like you are in your locally cloned fork of the repo

[surreal veldt]

So I just open the site folder I cloned from the website in VS code and run the commands right?

[dim pelican]

After you checkout the new branch you created, since you want the original branch to not have the changes

git status should show some info regarding what branch you have checked out

➜ git status
On branch main
Your branch is up to date with 'origin/main'.

[surreal veldt]

got it thanks. Also I'm watching a video; it says git branch -c, which one is correct?

[dim pelican]

I believe -b creates and checks out a new branch, -c checks out a branch that is already present? Let me double check

https://git-scm.com/docs/git-branch
-c is copy:

-c
--copy

Copy a branch, together with its config and reflog.

I think I got my wires crossed and meant to say git checkout -b new_branch

Yeah: https://git-scm.com/docs/git-checkout#Documentation/git-checkout.txt-emgitcheckoutem-b-Bltnew-branchgtltstart-pointgt

[surreal veldt]

And once I add the file in the new branch, I do git add filename right?

[dim pelican]

Yup!

And before you commit, make sure you have the pre-commit hooks installed

iirc: pre-commit install

[surreal veldt]

Where do I do run that command?

[dim pelican]

On the command line / terminal

(brb)

[surreal veldt]

Wrong screenshot whoops

[dim pelican]

No worries 👍

So after the pre-commit hooks are installed, and you have added the file, you can do git commit and make a descriptive, short commit message

[surreal veldt]

What do you think of this? I have it as preview in VS Code

[dim pelican]

When you make your PR there will be a more in-depth review, but my quick one is:

• most customizable could be more customizable
  -...you can follow this: For... looks like it could be worded differently?
  -In the first ctx.send: This is the help page...

[surreal veldt]

Wow I just noticed I made many typos lol

[dim pelican]

Haha, thats why code reviews exist!

[surreal veldt]

Is there anything now to change? Going to be going in a bit more detail later on but more or less it'd be like that

Also going to be referencing Stella (the author of the walkthrough)

[dim pelican]

So far it looks good to me, there will likely be some more improvements suggested during the PR review process

[placid ermine]

the indentation inside the codeblock's if-else looks a bit wonky

[sleek steppe]

You can also use hyperlinks here 😄

[dim pelican]

I think it is just the MD preview in VS Code, they haven't created the PR yet

[hoary haven]

@surreal veldt please use links like this:
[text here](https://url.com)

and that section might benefit from using a bulleted list, and changing the wording to be in the pattern of:

• get command object from ...
• get command name

^ something like that

who is Stella? do they need to be referenced by name on this guide?

[surreal veldt]

i mean they created the helpcommand walkthrough

I believe they are referenced in a tag, so I thought we can do it on the site too?

!customhelp

[stable mountain]

Custom help commands in discord.py

To learn more about how to create custom help commands in discord.py by subclassing the help command, please see this tutorial by Stella#2000

[surreal veldt]

well command name and object are different

[hoary haven]

oh cool

[hoary haven]

yes, what i meant to say is, if going for a bulleted list,

• "get the command name"

reads better than

• "to get the command name-"

and the actual text of "get the command" will be the hyperlink

[surreal veldt]

Yeah that's true, I'm at this now.

[hoary haven]

nice nice

thanks so much for working on this @surreal veldt @clever wraith
much appreciated

[clever wraith]

[surreal veldt]

Alright so I believe I'm nearly done however I wanted to ask here, I was planning to put something like this additionally.
Can i have feedback please?

You can also set custom values to display

#subclass HelpCommand
...
brief = command.brief
name, aliases, description = brief
#where brief is a list, as shown below```
```py
@bot.command(brief = [name here, aliases here, description here```

That's what I currently have implemented for my help command ^

[brisk brook]

Is that meant to be code that works?

[surreal veldt]

Yeah, it does work

But I wanted to know whether it's worth adding or not

[brisk brook]

The first codeblock doesn't?

I am a bit confused at what this suggests, you can already grab the name and list of aliases by other means

command.brief is already a thing that has a set purpose

[surreal veldt]

No but the purpose is to show you can have custom values for each command some being outside the functionality of what you are restricted to be able to in the command deco

For example maybe a example (command usage example) value

[brisk brook]

So we are using the fact that nothing is done to brief so that we can pass any kind of values to it?

[surreal veldt]

a list is passed

i can show an example

[brisk brook]

Yes but discord.py wants it to be a string

What happens to command that don't have briefs?

[surreal veldt]

Example command:

@bot.command(brief=['Example', 'Just an example command', '!example @member'])
async def example(ctx, member : disnake.Member):
  pass

Help command:

@bot.command()
async def helP(ctx, command_name):
  command = bot.get_command(command_name)
  brief = command.brief
  name, description, example_usage = brief
  print(name, description, example_usage)

Output from help command (provided !help example is done) : Example, Just an example commamd, !example @member

[surreal veldt]

It can be a list

[brisk brook]

It can be a list because you pass it a list. It's meant to be a string: https://discordpy.readthedocs.io/en/latest/ext/commands/api.html#discord.ext.commands.Command.brief

[surreal veldt]

okay yeah so the method is not efficient is what you're saying?

[brisk brook]

Yeah my point is that it isn't an official way to use brief. It's a bit of a hack you can put in your code on your own risk but I don't think we should put it in a guide

[surreal veldt]

Got it. Also, I'm now at this and can't really think of more to add. Any suggestions?

And would a article be wanted about this? I assume not
#discord-bots message

[hoary haven]

can you reword the first sentence to also be a hyperlink as opposed to just posting the URL

is the walkthrough by stella a pre-requisite reading before the rest of your content can be understood?

[placid ermine]

minor, but the await in the else is more indented than the one in the if

[surreal veldt]

It is?

Visual studio code showed it was aligned iirc will check

[surreal veldt]

Oh that

[surreal veldt]

Yeah I'd say so

[surreal veldt]

And this in case it was missed?

[surreal veldt]

Oh sorry i misread, yes it can be helpful but I also think its better they go through that first before looking for the custom implementation

[hoary haven]

okay, if so then maybe a mention of that would be helpful.

A basic walkthrough by Stella#2000 on subclassing the HelpCommand will provide some foundational knowledge required before attempting a more customizable help command.

^ something like that

i have questions that i'll defer to others on:

• should we ask Stella#2000 for permission before putting their username on our website?
• as for your message on replit vs heroku, i'm not sure, will defer to others.

[surreal veldt]

I can ask Stella yeah

[green oriole]

neither repl.it nor heroku are suitable for hosting a discord bot

they won't even let you run code 24/7 without hackery

even so, they suffer from critical flaws such as shared IPs

[surreal veldt]

I just asked them and they said they are fine with it

[surreal veldt]

Anything else I'm missing or can add now?

[cursive relic]

Is there yet any plans for using slash commands?

[molten perch]

Not until the permission rework.

[surreal veldt]

How do I do the pre-commit install command? I'm in the directory with the branch and I did git add but when I do pre-commit install it says pre-commi : The term 'pre-commit' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.

[molten perch]

Because, it is installed in your poetry environment.
You can either run poetry run task precommit or enter the poetry environment with poetry shell and then run pre-commit install.
(the first one is easier 😉 )

(you also gotta run poetry install before doing so, to create the poetry env)

[surreal veldt]

poetry : The term 'poetry' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.

[molten perch]

Yes, I presumed you had poetry installed.
I advise you to follow this guide: https://www.pythondiscord.com/pages/guides/pydis-guides/contributing/site/
(But you can install poetry by following the docs here: https://python-poetry.org/docs/#installation)

[surreal veldt]

So I have to install poetry?

[molten perch]

Well, I would advise you to install it.

[surreal veldt]

but I think I'm lost but you already said I have it installed

oh nevermind, I just read that wrong

pip install poetry is a different command that this? Or would it achieve the same thing

[molten perch]

You can install it via pip, but it is not recommended. (See the docs)
Be aware that it will also install Poetry’s dependencies which might cause conflicts with other packages.

[surreal veldt]

ah well previously i already did pip install poetry

[molten perch]

On your current workstation?

[native yoke]

@surreal veldt lmk when you're done with ur markdown file. My pin has something that fits will with yours so I was thinking we could just make it into one file

[surreal veldt]

I am done with it

[native yoke]

have u Pr'd it and has it been accepted?

or actually. maybe I could contribute directly to ur fork

if thats alright

[surreal veldt]

No not PRd yet request that from @trail pilot actually

[trail pilot]

Yep, I'll be taking it over

[native yoke]

:o

[trail pilot]

I'll try to get the PR in by today (or at least make it)

[native yoke]

ok sounds good

[surreal veldt]

Your pin referring to the on_command_error?

[cursive relic]

Thx 👍

[native yoke]

Yea

[surreal veldt]

Is that revelant to help command?

[dense bison]

tem alguem do brasil ???

[wild prism]

!rule 4

[stable mountain]

Rules

4. Use English to the best of your ability. Be polite if someone speaks English imperfectly.

[dense bison]

anyone from Brazil?

I want to tak some doubts

[wild prism]

check out #❓｜how-to-get-help

[native yoke]

Ahh true my mistake. Wasn’t reading what u wrote properly

[cursive relic]

Why is there no posts in #reddit recently , is it on purpose?

[patent pivot]

no - the reddit relay seems to be broken right now

but there aren't any logs indicating why - if someone wants to document the process with log calls then that might fix it

[cursive relic]

Which bot handles the reddit posts?

[patent pivot]

lancebot iirc

[cursive relic]

.src Reddit

[dusky shore]

Cog: Reddit

Track subreddit posts and show detailed statistics about them.

Source Code

Go to GitHub

bot/exts/utilities/reddit.py:27

[cursive relic]

Ah, thx

[patent pivot]

https://github.com/python-discord/sir-lancebot/blob/main/bot/exts/utilities/reddit.py

yea

[surreal veldt]

By the way @trail pilot was the PR made?

[trail pilot]

Not yet, I've been a bit busy with school

[thorny obsidian]

There should be an associated issue that's approved by core devs before you make a pull request

It makes sure the idea is solid and something we want without wasted effort on a PR. The issue doesn't need to be too involved, but you're also welcome to ask here (although we do prefer GH)

[gritty wind]

I don’t think multiple queries will fit, we already trim many entries. You could paginate like we do with the help command for instance, but I’m not sure if the benefit gained over just calling the command multiple times is there

[fallen patrol]

#dev-log message
we merged an update that removed that enum attribute

[fallen patrol]

@patent pivot this is sufficient for licensing right? https://github.com/onerandomusername/monty-python/blob/main/LICENSE_THIRD_PARTY

[gritty wind]

You may need to put the license in the files were you’ve used the code as well, but it’s pretty vague. Some people will tell you to add your own license to the top of all your files as well, so i don’t think there’s an agreed upon standard. Technically speaking you also need to get your users to agree to this license when agreeing to your terms and conditions, but most discord bots don’t exactly have terms and conditions

[fallen patrol]

but most discord bots don’t exactly have terms and conditions
I'm about to make one

sadly

[vocal wolf]

@sleek steppe lmao that was fast

[sleek steppe]

lol I forgot I didn't approve yet and saw the github notification

[trim cradle]

If a message contains a py codeblock, it would be nice if @stable mountain gave you a reaction that evaluates the code if you click on that reaction.

[austere hornet]

Oh, so you're saying instead of:

!e

print("Hello, world!")

Do:

print("Hello, world!")

(some reaction added)

If so, that's not a bad idea. But I think it might be unnecessary because, well, why not just use !e?

[trim cradle]

I'm not suggesting that the eval command be removed. this would just be another way to get that functionality.

[austere hornet]

Ah I see, ok. Yeah might be a good idea. Don't see a reason not to do it.

[surreal veldt]

Isn't that what it does currently?

[trim cradle]

there's similar behavior if you edit a message that was evaluated

[placid ermine]

i think the new context menu thingies would be more suitable than a reaction

@fallen patrol's bot, on another server

[fallen patrol]

to add on to this, all of the existing features are great, I kept every other eval invoke method and just added... 3 more ways?

[placid ermine]

!otn a backwards compatible bots

[stable mountain]

:ok_hand: Added backwards-compatible-bots to the names list.

[fallen patrol]

so that's basically monty in a nutshell

anyways

okay I'm adding that to it

[trim cradle]

huh, I didn't know this was a thing

how do we start having that

[placid ermine]

dunno if dpy 2.0 has it

[gritty wind]

We’re not going to have it just yet, discord’s implementation is kind of shit

They’re rewriting it and we’ll look at it again once they make it better

[placid ermine]

O

[placid ermine]

discord discord or discord.py?

[gritty wind]

Discord discord

[placid ermine]

o ok

[fallen patrol]

I have absolutely no permissions on monty: all app commands are runnable by all users

even though yes, disnake supports checks for slash commands, the interface by discord is bad, and v2 permissions will be out soon probably anyhow

[gritty wind]

It's interface is bad probs because discord's api is bad

[fallen patrol]

edited :3

the discord interface is shit for permissions

v2 perms will be better\™️

[gritty wind]

Designed by a toddler I'd say

[fallen patrol]

imagine giving them that much credit

[narrow valve]

Good afternoon everyone. I am new to docker and kubernetes, Could anyone suggest me few resources for beginner level?

[fallen patrol]

did you know, the way ratelimits are set up, you can actually send 9 messages in one second

[surreal veldt]

Implementation of the permission system for application commands or application commands in general?

[gritty wind]

A bit of both

The permissions are 100% a nightmare

[surreal veldt]

What exactly is a bad about the UI?

[gritty wind]

The system itself gets really messy when you have more than 5 commands

Like you can’t find anything

[surreal veldt]

Ah I'm yet to see 5 application commands

[gritty wind]

I think since the system’s been announced we’ve had well over 5 suggested

Lol

[signal forge]

Howdy everyone. I'm getting the following error when trying to run the bot locally on my test server created with the template provided in the directions. Docker is up and running and working without issue.

Traceback (most recent call last):
File "/usr/local/lib/python3.9/site-packages/disnake/http.py", line 467, in static_login
data: user.User = await self.request(Route("GET", "/users/@me"))
File "/usr/local/lib/python3.9/site-packages/disnake/http.py", line 420, in request
raise HTTPException(response, data)
disnake.errors.HTTPException: 401 Unauthorized (error code: 0): 401: Unauthorized

he above exception was the direct cause of the following exception:

Traceback (most recent call last):
File "/usr/local/lib/python3.9/runpy.py", line 197, in _run_module_as_main
return _run_code(code, main_globals, None,
File "/usr/local/lib/python3.9/runpy.py", line 87, in _run_code
xec(code, run_globals)
File "/bot/bot/__main__.py", line 13, in <module>
bot.instance.run(constants.Bot.token)
File "/usr/local/lib/python3.9/site-packages/disnake/client.py", line 856, in run
eturn future.result()
File "/usr/local/lib/python3.9/site-packages/disnake/client.py", line 835, in runner
await self.start(*args, **kwargs)
File "/usr/local/lib/python3.9/site-packages/disnake/client.py", line 798, in start
await self.login(token)
File "/bot/bot/bot.py", line 259, in login
await super().login(*args, **kwargs)
File "/usr/local/lib/python3.9/site-packages/disnake/client.py", line 658, in login
data = await self.http.static_login(token.strip())
File "/usr/local/lib/python3.9/site-packages/disnake/http.py", line 471, in static_login
raise LoginFailure("Improper token has been passed.") from exc
disnake.errors.LoginFailure: Improper token has been passed.

[gritty wind]

What do you have in your bot's .env

The error indicates the bot token is invalid

[signal forge]

I've got what is required per the instructions. I created a .env file and added:

BOT_TOKEN=lskdjfaaksjdfaksdjhfk.asodfhjsakljdfh

^Randomness to avoid sharing my token 🙂

[sleek steppe]

Are you sure you passed the correct token?

[signal forge]

I can regenerate it again, but yup it was generated via the Discord Developer panel in the browser.

[gritty wind]

The tokens are usually a little longer than that

Mine is 59 characters for instance

Unless you've changed it

[signal forge]

Mines 59 as well

[gritty wind]

Hm, it should work then

Uhh, could you screenshot the dashboard?

[signal forge]

With my token displayed?

I don't think that's safe.

[gritty wind]

Hidden, just where you're getting it

There are other tokens on the dashboard, but none of them are that length, so it's probably not that

[gritty wind]

Alright, all that looks correct

My next step would be checking that the correct variable is being loaded

You could print it right before it's passed on line

[signal forge]

oh great idea... so on like 258

of bot.py

[gritty wind]

Yeah that seems right

[signal forge]

How would I print it though

what variable

BOT_TOKEN/

How do I print out the BOT_TOKEN if it's pulled from .env

[gritty wind]

It's loaded as constants.Bot.token

or you could try to get it from args

[signal forge]

yup constants.Bot.token is incorrect.

It's not what is provided in .env

Do I need to resave .env after changing the token?

PyCharm autosaves everything thought

[gritty wind]

It should autosave

Where's the env file in relation to the rest of the project?

You could screenshot the folder viewer for this

[gritty wind]

The arrow's a nice touch :D

But yeah that's in the correct position

Is the env loading thing actually installed? Try importing it by running poetry run python -c "import dotenv"

[signal forge]

zsh: command not found: poetry

[gritty wind]

Oh, how are you installing and running the code

[signal forge]

I just started Docker

[gritty wind]

Ahh, okay. It should be loaded by docker then

[signal forge]

am not installing and running the code anywhere else

[gritty wind]

Dockerfile or just docker?

docker compose or just docker*

brain fart

[signal forge]

Assuming you have Docker installed and running, enter the cloned repo in the command line and type docker-compose up

that's what i did

[gritty wind]

In that case it's docker compose, so it should've just worked

Hm

[wild prism]

you didnt regenerate the token, did you?

[signal forge]

I did

I regenereated it then replaced the token in the .env file

oh

it's printing my public key

i'm going to search the files of the project with my public key as the search term

i might have provided it somewhere i shouldn't have

[hoary haven]

hhm, did you change anything around after cloning the repo?

[signal forge]

I think I provided my public key in the .env file when it should have been my bot token.

I had to regenerate my bot token

replace the public key with it in .env

but it's still picking up my public key even though it's been removed from the .env file

[gritty wind]

You may need to recreate the container to get the .env file to reflect

[signal forge]

oh darn

[gritty wind]

it should be quicker than the first time though

[signal forge]

so delete the container in docker

then recreate it?

[gritty wind]

docker-compose down should delete it, then docker-compose up --build should rebuild it

[signal forge]

Currently building

fingers crossed

variable has been updated

Bot is live!

How freaking cool.

There is so much to this bot that I find fascinating.

I'm still new to programming, but am looking forward to reviewing the source code and perhaps helping out on issues for beginners.

[hoary haven]

getting the bot up & running is a first major step, congrats

[signal forge]

🙂 It's so damn cool.... such an interesting bot

[wild prism]

if you want something a little easier, Lancebot is more geared to beginners
but you're welcome to do @stable mountain as well, and setting it up is half the battle already

[signal forge]

Having to rebuild the Docker container for changed to implement is a pain. Is there an easier way to quicken the time it takes for changes made to be reflected in the server without having ot restart Docker?

[gritty wind]

For code changes it shouldn't require a rebuild

Anything under the bot directory that is

What are you working on?

[sleek steppe]

you can use the !ext command instead of restarting the docker container as well

[signal forge]

I'm just working on this issue for fun as a learning experience

https://github.com/python-discord/bot/issues/2079

so I created a new resources.py file in bot -> exts -> info

Working on creating this new bot command.

[gritty wind]

As long as you have a discord cog, you can do !ext reload <cog_name> to get it to load your changes

Or simply !ext reload * to reload everything

Or you can stop and restart the bot (without rebuilding the docker container), that'll work too

[signal forge]

@gritty wind where would I run the !xt command?

!ext

[gritty wind]

Anywhere on your server (just make sure you have the admin role configured and set up)

!ext reload infractions

[stable mountain]

:ok_hand: Extension successfully reloaded: bot.exts.moderation.infraction.infractions.

[gritty wind]

This is what you should expect to see

[signal forge]

@gritty wind Awesome

I didn't have the admin role

How long have you been involved with the Python projects on GitHub?>

You seem to have a good understanding of the Bot codebase 🙂

[gritty wind]

Uhhh I made my first contribution when I first became a helper, so... maybe a year and a half?

[austere hornet]

Re: site#694
I think the embed color in the #roles channel in the server should also be changed for the Project Leads role. I also think the forms team and events team should have separate embeds because the roles don't have the same color so it doesn't look very good imo

[dusky shore]

GitHub

Update roles page

[signal forge]

What learning resources can you suggest for learning about Cogs.

[vocal prairie]

https://tutorial.vcokltfre.dev/tutorial/05-cogs/ is great

[signal forge]

Nice! There doesn't seem to be many in depth tutorials around discord.py / sidnake

[fallen patrol]

@patent pivot could y'all please include a reason of why you're switching back to dpy aside from it was unarchived

I'm getting asked why

[patent pivot]

existing projects and infrastructure are in discord.py, so path of least work for us is keeping things that way. also I think going forward it will continue to be the most prominent Python library which aides us in terms of bringing on new Core Developers and new Contributors.

nothing against the forks.

[fallen patrol]

doubt this

from what you've said in places, but w/e

[patent pivot]

not sure what you're referencing? I suspect personally that there may be a loss of motivation now that discord.py master is being maintained, but it's not the primary reason that Python Discord is staying with discord.py

It's more us praising boring technology!

https://engineering.atspotify.com/2013/02/25/in-praise-of-boring-technology/

We've got stuff that works - our existing infrastructure is built around discord.py and so right now it makes sense for us to stick with that.

[fallen patrol]

dpy could be archived at any time again, and I would not be surprised if that happens

if you're fine with boring technology that gets archived and unarchived at any time, then use dpy

[gritty wind]

Discord.oh guarantees as much stability as other forks 🤷‍♂️

Because just as forks have said they’ll maintain the lib for now, so has d.py

[patent pivot]

sure, and should that be the case then we may have to make another decision, but everything is very young right now, there are no guarantees on any forks right now or discord.py. right now, we use discord.py and so it makes sense to keep things that way for now.

[last patio]

good article

i think even if discord.py would fall out of maintenance again the v10 support will probably last a while

[patent pivot]

yea, +1

[patent pivot]

thanks

it talks about postgres, and uses the word "great success", so you know it's a good article

#dev-announcements message

p much these messages
#dev-contrib message
#dev-contrib message
#dev-contrib message

[last patio]

exactly

[fervent sage]

a loss of motivation to who?

[patent pivot]

Having seen similar things in other spaces, not all Python, once the main repository is restored the forks will dwindle in activity. I'm not saying that will happen, but there's a chance it does, just like there is a chance that Discord.py archives again. Especially considering Discord.py is the second most used Discord library even 6 months after deprecation with no signs of slowing.

[fallen patrol]

A large thing that causes forks to dwindle in activity is when people switch away from them

especially prominent users who people listen to

[patent pivot]

it's the decision we unanimously made amongst the Core Developers for now, we've got a lot of infrastructure vested in this and this was the decision we felt was right for us now. that's not to say we won't be monitoring the fork situation, and as mentioned I suspect there will be a point down the line where we have to make a decision like this again.

[fallen patrol]

Why do you believe there's another point where you'll need to make a decision?

You just said that the forks will dwindle in activity

(which I don't believe will occur)

[last patio]

even if it doesn't, we have repos like https://github.com/python-discord/bot-core that need a common, stable, standard base to build on

and using the standard track is probably the best choice for that

[fallen patrol]

Curious why this, tho

[patent pivot]

because libraries change, not just discord.py, we've made decisions like this in the past. moving site from flask to django, changing database from rethinkdb to postgres. it's inevitable at some point some infrastructure decision we have made will have to be altered down the line

[patent pivot]

well, like what happened 6 months ago, if there is something unstable with regards to the future of discord.py I think that rejuvinates the forks

[fervent sage]

I think the entire future of discord.py is unstable given that Danny already ragequit once

[fallen patrol]

the fact that he came back goes against literally every reason he left in the first place.

[fervent sage]

Given what he says hasn't changed at the start of the gist it's clear he's coming back out of vengence/anger the forks are succeeding

[twilit juniper]

… seems a bit too quick to change back to dpy. Some of these forks have proven that they have good code quality and can get new features implemented in half the time dpy could. As some have already said it seems that he’s coming back for the wrong reasons and will just rage quit again. I also think by using dpy we kind of promote using it and while it’s a good library the community is rather toxic and isn’t very helpful when it comes to helping new people which completely goes against what python discord is.

[wind ruin]

it seemed more like "no further immediate changes" than "adequate the code to discord.py v2 new standards"? or at least I hope so

[crude gyro]

our bots are absolutely crucial to the running of this community, so like @patent pivot said, we're generally gonna pick the most boring software we can find for the job. With d.py back, that's going to be d.py. If a fork overtakes it in popularity or utility, we'd probably reconsider.

we would never have switched to disnake in the first place if it had been viable to continue using d.py for another six months.

we didn't switch because we were making some sort of endorsement or statement, we switched because we felt it was the path of least resistance, and the most boring option available to us, and because we're adverse to excitement in our core service platform.

[fervent sage]

Excitement like... The library deciding on 0 days notice to discontinue?

This makes no sense to me

[crude gyro]

yes, that was very exciting, and I wasn't happy about it. I'm also not ready to write d.py off forever just because it archived once, just like I'm not ready to write off the forks just because the upstream is back.

I wish the whole thing was less of a circus.

this wasn't a trivial decision to arrive at, but I still stand behind it.

[fallen patrol]

From the way everyone was talking, I had figured y'all had written off the forks (also how the announcement was written)

If it was more clear that you weren't writing off the forks and they're still very much on the table, that would be easier for us

[crude gyro]

they're not on the table for our immediate future, but I personally am still excited to see where the forks go.

[brisk brook]

Hmm, my viewpoint on this is honestly just a neutral acceptance.

I don't like discord.py: I've moved on from it and don't want anything to do with its culture (both in terms of the helping and maintainership) so I'll not use it for any personal projects. I would've probably voted for just going through with the decision made (because it was made).

The thing is, with slightly different timing all bots would still be on discord.py and everything would be continuing as normal. The decision to move to a fork was one that was forced - it simply had to be made. If discord.py had been unarchived before the final decision, and announcement (say, 4 months ago) we'd not be here discussing anything because all bots would just remain on that. I understand that all bots are going back to discord.py because that is what would've happened if discord.py was unarchived before a fork was chosen.

It is what it is.

[cold island]

Not 4 months, had it been announced earlier that day we wouldn't merge the PR. Seriously. I'm not trying to belittle the work and effort done in the forks, but it wouldn't make sense for us to switch in the first place when everything we had was using dpy and that it was then getting all of the new features and the up to date API version.

[brisk brook]

Oops yes sorry. 4 months ago is 2 months after discord.py was archived. I meant 4 months into its archival.

[cold island]

Given that we hadn't completed the switch yet on all our projects, we decided to stop the switch.

[brisk brook]

That's roughly when the fork discussions started picking up more seriously if my sense of time is correctly.

[signal forge]

I'm adding a Cog to the Bot and have a function that scrapes data from a website then adds that information to a list. Do I use scheduling.py to have this Cog run every X hours to ensure the scraped data is kept up to date?

[brisk brook]

Which issue is this?

You'll want to use discord.ext.tasks to run it every X hours. Hold on

https://discordpy.readthedocs.io/en/latest/ext/tasks/index.html

There are examples here ^

[signal forge]

Thanks!

[vocal wolf]

If you're going to make a PR, was an issue approved first?

[thorny obsidian]

I think this is on their local fork, not being added to ours

[signal forge]

Yes it was approved.

[vocal wolf]

Very nice. Thanks!

[signal forge]

🙂

[vocal wolf]

@obsidian patio Twemoji merged

[obsidian patio]

Thanks Xith

.twemoji 🥓

[dusky shore]

Bacon

1f953
Download svg

[trim cradle]

If we have !resources as a proper command (not just a tag) with dynamic behavior based on what comes after, where should that command live?

Asking for @signal forge

[dim pelican]

I'd vote for exts/utilities

[tawdry vapor]

exts/info

[clever wraith]

just a small note.
we have this statement in #roles

@Voice Verified
This role allows people to speak in our voice channels.
To get this role send the command !voice-verify in #voice-verification. Please note that there are requirements to getting this role. They are listed in the channel where you run the command.

however #voice-verification does not exist(or is removed for historic reasons), this should be changed to #bot-commands perhaps.

[patent pivot]

it does, it's just that once you are voice verified you can't see it

[clever wraith]

oh!!

alright makes sense.

[patent pivot]

it's only relevant when pending verif, once you've done that it doesn't matter anymore :)

[clever wraith]

ah i see. alright great! thanks for clairification joe!

[signal forge]

For what it's worth, I've begun tinkering with this using exts/info.

[cursive relic]

should there be open issue relating to that #reddit sometimes doesn't post

[austere hornet]

I asked the same question and have been told there's something wrong on Reddit's side not on ours, so I don't think an issue is necessary. Afaik, Reddit is aware and they are working on resolving the issue. It did post yesterday though so maybe it's fixed, we'll see if it will post tonight.

[cursive relic]

there's option to add logging to see what could be wrong (or what goes wrong), because there's not any logging yet

eh. nvm I didn't read your message properly. Sorry

[austere hornet]

👍

All good

[trim cradle]

@signal forge

Also, I added a method to scrape the Topics from https://www.pythondiscord.com/resources/. I plan to use this information for validation
I think this is overkill. if a URL for the resources page has an invalid ?topic= argument, it just doesn't do any topic filtering.

[vale ibex]

Good point, I did this earlier 😄

[signal forge]

Good point, though I think there is some benefit for users to be able to see from within Discord what topics exist on the site.

Plus I'm kind of excited and want to leverage this first contribution as much as possible as a learning experience 🙂

[trim cradle]

I hope it is a learning experience for you 😄 but added complexity in the command is complexity that the whole project has to maintain, and I don't think the costs outweigh the benefits here.

[signal forge]

True... then in that case it should be ready shortly. I'm at work but hope to continue working on it tonight.

[trim cradle]

It sounds like you're interested to take on something more substantial as well. I'm sure we can find another task that will push you a bit further than this one does.

[signal forge]

🙂 Totally.... I'm in a non-technical career but am fascinated with programming and want to help anyway I can.

[crude gyro]

that's a great way to learn. open source is a gateway drug.

thanks for the contribution

[signal forge]

No problem, Lemon! Is it ok if I'm assigned to the issue?

[crude gyro]

yeah, link it?

oh this one? https://github.com/python-discord/bot/issues/2079#issuecomment-1066746493

assigned.

[signal forge]

Thanks so much, @crude gyro

[crude gyro]

thanks yourself

[green oriole]

I randomly found that in my picture gallery and I have to say this is the most important announcement ever put out

oh wait this is staff-announcements

that's considerably less fun

also yes this is from august

[fallen patrol]

@vale ibex Ik you're migrating back off of disnake but https://github.com/python-discord/bot/pull/2113/commits/56d7fdd52622fb215b3b3ef34e74681c8ae3a40b it was actually renamed MessageInteraction for typings

as dpy wasn't well designed for typings with the different types of interactions

[vale ibex]

Ahhh right, couldn't find anything in the changelog so assumed it was removed

[signal forge]

Howdy, everyone. Forgive the beginner question, but I am working on an issue for the bot. I forked and cloned the project and am pushing my branch to my local repository on GitHub. Should I also push my new branch to the actual GitHub repository, or do I just do that when I am complete with my work locally? I think it would make sense to push my branch to the actual repository so other people can merge it into their forks to test it out, right?

I'm super new to Git, so thanks for bearing with me 🙂

[summer garden]

You won't be able to push to the pydis repo. You can either open a draft pull request with your work, so that others can poke around and test it, or finish working and open a normal pull request. If everything goes well the changes will be merged to the repo

[eternal hull]

What's the best way to check if an issue regarding a certain problem already exists so I don't end up making a duplicate one?

[austere hornet]

Well, simple: if you want open an issue for a game you want to write for @dusky shore , for example, go to https://git.pydis.com/sir-lancebot/issues and search for that game and if nothing comes up, then you're free to open an issue. If something does come up, you can feel free to comment on that issue. Same goes for all of the other repos.

Hope this helps!

[eternal hull]

Also, what do I do after I submit an issue?

It's something for @stable mountain

[austere hornet]

Just wait until a core dev approves it and assigns you to the issue if it is approved

[eternal hull]

Ah ok

[hoary haven]

other users will pitch in and give ideas for or against, possibly discuss other options. eventually a core dev will decide on a direction for implementation, or close the issue if we decide it's not something we want at the moment

[eternal hull]

Will I get like a message if it gets approved?
Or would I just have to keep checking in?

[eternal hull]

Understood

[hoary haven]

you'll get a ping and probably some notifications from github :)

[austere hornet]

You can also check #dev-log to see if someone commented on your issue

[eternal hull]

Noted

That's cool

oh that's really cool

[austere hornet]

Yeah 😁

[hoary haven]

yeah, even if your issue isn't approved, lurk these channels and you can learn a lot by observing

[eternal hull]

I will

thanks again

[hoary haven]

would they be any way for us to add git docs to !docs?

[patent pivot]

as in the CLI? not really

we could add manpages of popular commands but it's not quite as intuitive and the UX won't be great

[placid ermine]

i think its out of scope

[fallen patrol]

[thorny spade]

https://github.com/python-discord/sir-lancebot/issues/1015

can anyone approve

thanks

[cold island]

Personally not in favor, commented in the issue

[patent pivot]

on the plus side with regards to deleting /etc/ (<#dev-announcements message>) we now know we can bootstrap hosts from scratch lol

[fallen patrol]

@cold island relative timestamps do work on mobile lol, what OS are you on?

there is specific locations where they don't work

[patent pivot]

🎉 https://search.censys.io/certificates/297721dea88b2a27387f37579b40c89ee34f3453b24d86ecc0a2abad9250e84e

[fervent sage]

subject deez nuts

[patent pivot]

yeah look under alt names lol

[fervent sage]

ye ik

but the deez nuts joke was more important

[patent pivot]

banks shoots, banks scores

[cold island]

Relative timestamps just show up as dates on android

[fallen patrol]

this is my bots uptime command

on android

both an exact and relative timestamp, visible on android

[cold island]

🤷‍♂️

I'll try updating the app later

Nope there don't seem to be avaliable updates

[patent pivot]

are you on a beta

[rapid swallow]

maybe we can have both

[fallen patrol]

it's been like this for months for me

but yes

[patent pivot]

yeah, it's not on stable

and most android users are on stable

[fallen patrol]

probably will have to wait another few months and then some for it then

[surreal veldt]

Has the pin article been pull requested? CC: @trail pilot

[last patio]

@surreal veldt this one? https://github.com/python-discord/site/pull/697

[surreal veldt]

The one regarding discord.py help commands

[last patio]

ah I see

[surreal veldt]

okay yeah I just checked and doesn't seem so

[trail pilot]

Not yet, I think I'll be able to PR it by tomorrow or Thursday since I have a lot more free time then

[surreal veldt]

oh okay then in that case do you mind if I try to do it myself

not to get it done faster, just for personal knowledge gain

[trail pilot]

nope, I don't mind!

[surreal veldt]

thank you!

[trail pilot]

Np!

[signal forge]

Can you link to the issue? I just want to take a look at it. I won't submit any PR or anything.

[surreal veldt]

The issue wasn't made

But it's regarding a project for putting some pins in #discord-bots to the python site

[wild prism]

that one's a bit outside the normal workflow since it's a special project being run by @trail pilot

[signal forge]

Ah... I see 🙂 Thanks @wild prism

[surreal veldt]

Anyone, if they have some time, mind walking me through the steps of installing pre-commit hooks

[brisk brook]

It should just be poetry run task precommit!

[dim pelican]

I think pre-commit install should also work:

:Desktop\python\sir-lancebot  main [$?2✓] 🐍

➜ pre-commit install
pre-commit installed at .git\hooks\pre-commit

[surreal veldt]

I know I asked this before, but I didn't have time to follow up.
Anyways, it raises

  Command C:\Users\tenuk\AppData\Local\pypoetry\Cache\virtualenvs\site-C_3__m3n-py3.9\Scripts\python.exe -W ignore - errored with the following return code 1, and output:

  The system cannot find the path specified.
  input was : import sys```

I need to put it to PATH right?

[brisk brook]

What exactly are you thinking of adding to PATH?

[surreal veldt]

poetry

[brisk brook]

I am gonna be honest, personally I don't use the pre-commit hooks. If you haven't yet made any changes I think you could start with that and take a look at pre-commit once you have made the changes you want and are looking to commit it.

[brisk brook]

Wouldn't that have a different error like Couldn't find specified program or path 'poetry'....?

[surreal veldt]

yeah i have no idea

i'm kind of clueless in these type of things

[surreal veldt]

i don't know what to do here but I can send the full error

it's when i run poetry run task precommit

[dim pelican]

Got it, and what is your current working directory when you run it?

[surreal veldt]

uhhh

what do you mean?

[dim pelican]

From the command line, you should be in a directory (what is the path to the left of your cursor?)

For example, I am currently on my Desktop, in the python directory:
(I use a modified shell, but the concept is the same)

[surreal veldt]

Is it this? Command C:\Users\tenuk\AppData\Local\pypoetry\Cache\virtualenvs\site-C_3__m3n-py3.9\Scripts\python.exe -W ignore - errored with the following return code 1, and output:

[dim pelican]

If I navigate to where I have cloned Sir-Lancebot, it would show like this:

[dim pelican]

That looks like it was part of the traceback

[surreal veldt]

okay yeah nevermind it's this

C:\Users\tenuk\site

[dim pelican]

👍

And what did you run from there?

[surreal veldt]

sorry I don't get this

[surreal veldt]

this is the command I did where it error'd

[dim pelican]

Try poetry run task pre-commit

Or pre-commit install

[surreal veldt]

it's the same error

PS C:\Users\tenuk\site> poetry run task pre-commit

  EnvCommandError

  Command C:\Users\tenuk\AppData\Local\pypoetry\Cache\virtualenvs\site-C_3__m3n-py3.9\Scripts\python.exe -W ignore - errored with the following return code 1, and output:

  The system cannot find the path specified.
  input was : import sys

  if hasattr(sys, "real_prefix"):
      print(sys.real_prefix)
  elif hasattr(sys, "base_prefix"):
      print(sys.base_prefix)
  else:
      print(sys.prefix)


  at ~\.poetry\lib\poetry\utils\env.py:1195 in _run
      1191│                 output = subprocess.check_output(
      1192│                     cmd, stderr=subprocess.STDOUT, **kwargs
      1193│                 )
      1194│         except CalledProcessError as e:
    → 1195│             raise EnvCommandError(e, input=input_)
      1196│
      1197│         return decode(output)
      1198│
      1199│     def execute(self, bin, *args, **kwargs):```

i assume there's something wrong with how I installed poetry maybe?

[dim pelican]

And you have done poetry shell?

[surreal veldt]

that raises the same error

but when I do just poetry it gives a list of commands

[dim pelican]

Weird....

[surreal veldt]

how do I uninstall poetry so I can try again?

maybe my files got messed up

[vale ibex]

open up C:\Users\tenuk\AppData\Local\pypoetry\Cache\virtualenvs\

and delete the folder site-C_3__m3n-py3.9

Done that?

@surreal veldt ?

[surreal veldt]

when i do Users>tenuk I don't see AppData

[vale ibex]

it's a hidden folder

just paste that entire path into the address bar of windows explorer

[vale ibex]

yea

and press enter

[surreal veldt]

sorry I meant to send both at once

[vale ibex]

oh, that's likely the problem

[vale ibex]

since this error is trying to access that path

[surreal veldt]

so do you know what to do?

[vale ibex]

What about just C:\Users\tenuk\AppData\Local\pypoetry\Cache

[surreal veldt]

nope

[vale ibex]

hmm interesting

what is poetry version

Actually, it'll need to be poetry --version

[surreal veldt]

i may or may not have messed up my laptop files not sure lol

[surreal veldt]

Poetry version 1.1.13

[vale ibex]

That's the same version I have, no idea what's happened to your files

could you run python -c "import os,sys;print(os.path.dirname(sys.executable))"

that'll output the path to your base python install

does it end in 39?

[surreal veldt]

PS C:\Users\tenuk\site> python -c "import os,sys;print(os.path.dirname(sys.executable))"
C:\Users\tenuk\AppData\Local\Microsoft\WindowsApps\PythonSoftwareFoundation.Python.3.9_qbz5n2kfra8p0
PS C:\Users\tenuk\site> 

[vale ibex]

ah

you have python installed from windows store

I'm not saying that's causing this issue, but it's definitely not helping

and you will run into issues in future

I suggest downloading the Windows installer (64-bit) from the bottom of this page https://www.python.org/downloads/release/python-3910/

make sure to select add Python to PATH when going through the installer

[surreal veldt]

I am pretty sure I downloaded it from the link you sent

[vale ibex]

Ah, then your PATH likely isn't in the correct order

Since the windows store version is what's being used when you run python

[surreal veldt]

is there any other way to confirm this?

[vale ibex]

this confirms that the windows store version is higher.

to confirm that you have the other version installed, you can go to C:\Users\Chris\AppData\Local\Programs\Python\

and see if a Python39 folder is there

that's the default path for it

[surreal veldt]

not found

oh wait

have to replace Chris

[vale ibex]

oh lol yea

[vale ibex]

32bit version isn't great

I'd suggest installing 64 bit version and make sure to select add it to your path

[surreal veldt]

uninstall?

[vale ibex]

you can uninstall 32 bit if you want

Python39-32 means it's Python 3.9 (32-bit) you have isntalled

[surreal veldt]

should i delete 310 also?

[vale ibex]

3.10 is worth keeping

[surreal veldt]

i was using 3.9

[vale ibex]

Just deleting the folder doesn't uninstall it.

if you want to remove it, do it via control panel

[surreal veldt]

so i should remove 32bit version from path and add 64 right

oh also I don't know if this helps but I have this in PATH

[vale ibex]

that's fine

that's what allows you to use poetry in powershell

If you install 64bit from python.org, it will add to path for you

so long as you tick the box

[surreal veldt]

okay I installed it

what do I do to put this over windows store?

[vale ibex]

if you restart all instances of powershell and run python -c "import os,sys;print(os.path.dirname(sys.executable))" what shows up?

if it's still windows store, then you'll need to raise the new install above that in your PATH var

[surreal veldt]

before I ran it in VS code terminal but now in powershell and it says PS C:\Users\tenuk> python -c "import os,sys;print(os.path.dirname(sys.executable))" C:\Users\tenuk\Desktop\python PS C:\Users\tenuk>

[vale ibex]

ok that works

you installed python on your desktop?

[surreal veldt]

yeah just installed the 64bit on laptop

[vale ibex]

I mean C:\Users\tenuk\Desktop\python means it's in your desktop folder

shouldn't cause issues, just odd

anyway, if you open C:\Users\tenuk\Desktop\python is there a python.exe in there?

[vale ibex]

cool

[surreal veldt]

pythonw.exe is there

[vale ibex]

python.exe is too

if you goto your site folder now in powershell

run poetry env use C:\Users\tenuk\Desktop\python\python.exe

[surreal veldt]

how do I go to my site folder in powershell?

[vale ibex]

the same way you got to it here

[surreal veldt]

oh okay i that wrong

[surreal veldt]

it gives that error again

[vale ibex]

alright, you're likely going to have to reinstall poetry now, since I have no idea where it's storing that env that it's trying to use

when you reinstall, it should correctly use your python version

[surreal veldt]

do i use pip uninstall?

[vale ibex]

did you install it with pip?

[surreal veldt]

i think so yeah

'twas a while back

[vale ibex]

Ah, that's highly not recommended

[surreal veldt]

yeah...i heard that

but is it fine to uninstall with pip?

[vale ibex]

I am assuming you installed it with pip on the windows store version

so you'll need to uninstall it with that version too

the other option is to see if it appears in control panel

[surreal veldt]

well

[trail pilot]

the issue with all the pins to migrate would be here: site#695

[dusky shore]

GitHub

Migrating #discord-bots pins to the site

[signal forge]

Nice thanks!

[surreal veldt]

how do I go back to the old version now that I installed the 64 bit?

[surreal veldt]

here?

[vale ibex]

No, in the control panel when you're trying to add or remove programs

[surreal veldt]

oh in control panel > uninstall a program, I don't see it

[vale ibex]

alright

if you run (Invoke-WebRequest -Uri https://install.python-poetry.org -UseBasicParsing).Content > install-poetry.py that will download the install script

[surreal veldt]

install?

oh okay

[vale ibex]

you can then run python install-poetry.py --uninstall

and that might work for the pip installed one

[surreal veldt]

do i need to go back to the before py version?