#dev-contrib

1 messages ยท Page 165 of 1

clever wraith
#

I need the help channel id because people can participate in multiple help channels at once

molten perch
#

Oh, then you wouldn't totally repurpose it. ๐Ÿ˜
I think it's fine then, if you're willing to perform those additional parsing.

molten perch
clever wraith
#

I just realised how vague my initial message was lemon_sweat sorry about that! I'll go ahead w/ it then

austere hornet
#

What is sir-lancebot#1013? Seems like someone opened a PR for no reason

dusky shoreBOT
magic arch
#

Looks like they're changing the uptime message to use Discord's relative timestamps?

#

I see no issue made for it though, plus they seem to have removed the pull request template stuff

#

That too

#

I do like the idea though, using that feature seems great here

gritty wind
#

Generally, this has violated all our contribution rules (no issue, no communication, missing template, failing lint)

#

Let the core devs handle it, weโ€™ll work it out

austere hornet
vocal wolf
#

@magic arch "I'll go through the topic suggestions"

#

"oh no"

magic arch
#

loll

vocal wolf
#

ok most of these are like

#

terrible

#

this'll be fine

magic arch
#

Sounds about right tbh

#

You got this py_strong

fallen patrol
vocal wolf
#

what I mean by that is there was a bunch of just straight up spam

magic arch
#

oh yes it was great to watch

vocal wolf
magic arch
#

already on it lemon_wink

vocal wolf
#

for some reason I couldn't find kat's username in the review list until now lol

#

ty for giving them the contributing guidelines

#

:D

stable mountainBOT
#

bot/resources/utilities/py_topics.yaml line 43

- Which Package do you use the Most and Why?```
blazing magnet
blazing magnet
vocal wolf
magic arch
#

dang already had that comment set myself too

magic arch
vocal wolf
#

@blazing magnet k re-review

blazing magnet
vocal wolf
magic arch
#

boom take my comments

austere hornet
#

Whoa another capitalization mistake

magic arch
#

Yup, blame the anonymous person who submitted it

short snow
vocal wolf
#

I do not know

magic arch
#

love the amount of immediate activity on this though

vocal wolf
#

@patent pivot devops person #dev-log isn't showing PR/review comments for sir-lance

austere hornet
magic arch
#

they are a pretty big topic

short snow
#

something even happened for the black's github webhook

magic arch
#

maybe GitHub is just going through some stuff right now, it happens

patent pivot
#

hmmmmmm

magic arch
#

status page shows webhooks all good though so idk

short snow
#

you can check if they are firing from the webhook logs

patent pivot
#

cloudflare hsa banned us

vocal wolf
#

wat

#

what did we do

short snow
#

uhh

patent pivot
#

i think they've just banned workers

austere hornet
patent pivot
#

cloudflare ip ranges

magic arch
#

ooh how fun, now what?

vocal wolf
patent pivot
#

lol

#

we probably just wait

vocal wolf
#

temp or perma

blazing magnet
#

ha, guess we can't speak too highly of CF now sadge

short snow
#

lol

magic arch
#

bye bye cloudflare

vocal wolf
patent pivot
#

cloudflare haven't done anything tbh

#

it's just we're on shared IPs because workers

#

we'll just have to hold on a bit unfortunately for the workers to free up

magic arch
#

just blame them anyways

#

also fun fact apparently i get the auto merge button on github mobile

#

even though I shouldn't for pydis prs

austere hornet
magic arch
#

nope, I can review and all that but no pushing to protected branches

#

at least I don't think so

austere hornet
#

Oh

magic arch
#

what I see on web:

#

what I see on iOS:

austere hornet
#

Oh hmm

magic arch
thorny obsidian
#

Only devops, core devs, and admins can merge to main

magic arch
#

yeah so I definitely shouldn't have the auto-merge button

full fractal
#

@clever wraith hey this is your carrier pigeon reminding you of this PR's existence: bot#1929

dusky shoreBOT
austere hornet
#

Lol

short snow
#

it is working ๐ŸŽ‰

patent pivot
#

yep

austere hornet
fallen patrol
austere hornet
#

(Still) looking for approvals (or not) on sir-lancebot#989 and sir-lancebot#1000, thanks! (sorry if I've been annoying about this)

austere hornet
# dusky shore

Wait what, why did #1000 go before #989, I sent #989 before #1000, is that a bug? Or is that supposed to happen?

magic arch
#

There's an open issue for that, sir-lancebot#743, basically the problem is that the issues/PRs are passed to a set at some point in making that embed, so they lose their order

dusky shoreBOT
magic arch
#

Looks like there's a lot going on in sir-lancebot#778, might be fixed in there

dusky shoreBOT
magic arch
#

Never mind, it's not, I think it's stalled until that PR is merged though

fallen patrol
dusky shoreBOT
fallen patrol
#

sigh i get that number wrong all the time

magic arch
#

968 I think is what you're going for

fallen patrol
#

sir-lancebot#968 ye

dusky shoreBOT
austere hornet
brisk brook
magic arch
#

To clarify it can't actually merge, mostly a visual thing

austere hornet
magic arch
#

Have you met the gh mobile ui? ๐Ÿคฃ

austere hornet
#

I do have the GH app on mobile (android) but never came across that kind of thing so idk

patent pivot
#

@tender ridgerys it is using it, which is why I dismissed the review.

#

oh, not here anymore, will ping on github

#

ah the edit clears that up

placid ermine
#

lmfao yall trolling at this point

#

how does it happen thrice in a row

patent pivot
#

what lol

placid ermine
#

you not seeing i rewrote with an api + akarys not seeing you not seeing that i rewrote with an api + you not seeing akarys not seeing you not seeing that i rewrote with an api

patent pivot
#

ah lol

austere hornet
#

Wait Akarys left the server? :(

white light
brazen charm
#

!d tut-for not exactly easily discoverable but you should be able to get them with docs

stable mountainBOT
#

4.2. for Statements

The for statement in Python differs a bit from what you may be used to in C or Pascal. Rather than always iterating over an arithmetic progression of numbers (like in Pascal), or giving the user the ability to define both the iteration step and halting condition (as C), Pythonโ€™s for statement iterates over the items of any sequence (a list or a string), in the order that they appear in the sequence. For example (no pun intended)...

magic arch
#

I've noticed a handful of people having issues with https connections to paste.pydis.com, @patent pivot the cloudflare man, any thoughts on what might be causing this the past few days?

shrewd geyser
#

Seems like I'm being blocked from using the pasting service l. As request from Ben I'm posting more information

patent pivot
#

i've noticed it to - i'm not sure but it certainly doesn't seem to be on our end

shrewd geyser
#

I'm using a Motorola phone

patent pivot
#

cert authority invalid is a weird one that should never happen

shrewd geyser
#

I can provide any specific information if you need it

patent pivot
#

yeah so Tizzy was reporting a Fortinet thing earlier

#

we don't use Fortinet anywhere in our stack and I have no idea where it would be injected

#

do you have an anti-virus/firewall on your phone? are you on a network that has Fortinet?

shrewd geyser
#

Not sure about both

#

I'm currently at school so I'm using school wifi

patent pivot
#

Right, I suspect your school is using Fortinet then and Fortinet has for some reason blocked us

#

I'll have a look into it, since it's odd that both yourself, Tizzy and Hemlock were having problems getting onto paste.pythondiscord.com

shrewd geyser
#

Alright

magic arch
#

I haven't had issues with it personally but yeah, noticed a few people the past few days particularly in pygen mentioning this

patent pivot
#

I suspect Fortinet has blocked us for one reason or another, and so all users of that, either directly or indirectly are being affected

shrewd geyser
#

Is it possible for you to contact them and resolve the issue?

patent pivot
#

yeah, I'm going to talk to Fortinet now

shrewd geyser
#

Thanks!

patent pivot
#

gave them a call, now talking to their labs team

thorny obsidian
#

Phishing for python knowledge! ๐Ÿฅ

patent pivot
#

alright, submitted a lab re-evaluation request, hopefully shouldn't take too long

fallen patrol
#

lmao

#

lmaooooo

#

idk why its funny

#

i think because of the cf banning cf and then this

patent pivot
fallen patrol
#

yeah

patent pivot
#

it could be seen as phishing

fallen patrol
#

wouldn't be surprised if toptal did something

patent pivot
#

toptal didn't do anything

#

it was an automated flag

#

based on the content of something on the hastebin

fallen patrol
#

i'm surprised hastebin[.]cc isn't flagged vs hastebin[.]com

patent pivot
#

if it went to manual review, it would have been ignored, but not every flag can go to review, so most are automatically filtered

fallen patrol
#

makes sense

austere hornet
#

That would be cool imo

#

And maybe even be able to ping that role

patent pivot
shrewd geyser
patent pivot
#

๐Ÿ‘

brisk brook
patent pivot
#

doesn't really work like that

#

robots.txt is for scrapers - not firewall assessments

#

(and we don't have a robots.txt on hastebin, lol)

brisk brook
#

That would be a huge vector for abuse ๐Ÿ˜‚

magic arch
#

Malicious sites would just be like "nah you shouldn't look at us, we're a good site"

vocal wolf
eternal owl
#

oh, I forgot about that, I need to add a command cooldown so we don't get rate limited, will do it soon, thanks for reminding xith!

#

@vocal wolf

vocal wolf
eternal owl
#

will do

vocal wolf
#

nais

brisk brook
#

najs

austere hornet
#

@patent pivot You said "privately" twice at the end of this comment in case you didn't notice: #dev-log message

patent pivot
#

oops

#

meant to be publicly and privately

vale ibex
#

hey @cold moon do you have some spare time at some point to resolve the issue on site#608? there's been a migration merged since your PR, so there are multiple leafs nodes now and django doesn't like it

vale ibex
#

If not, I can sort it out

#

Just means I can't approve and merge it

fallen patrol
cold island
#

Otherwise you can't merge it anyway

vale ibex
#

Yea I made one

#

I can't merge it right now, but if I was to commit on that branch, I wouldn't be able to approve and merge it when ready

cold island
#

Ah

#

I can commit it if needed

#

If that's the only blocker

vale ibex
#

Alright cool, let's see what KS says, since my pr still needs reviews anyway so not a huge rush

#

And my pr isn't really testable until the prolem resolves, since you can't start site in its current state

#

Unless you edit migrations

cold island
#

How does github do the thing where you can merge main before merging but can still merge

vale ibex
#

Uhhh

#

I don't get the question

cold island
#

Lol

#

Before merging to main

dim pelican
#

You know, the thing

cold island
#

You can update the branch

vale ibex
#

Yea

cold island
#

Which creates a commit

vale ibex
#

That's just git merge main

cold island
#

But you can still merge the PR

vale ibex
#

Without merging main?

cold island
#

No, after

#

You said you don't want to fix it because then you can't approve

#

OK I see how the question is confusing now

#

I meant approve not merge

vale ibex
#

Yea, the issue is that there are two migrations based on the same parent node

#

So it needs a code change to resolve

cold island
#

Right but a merge commit is also code changes, for example with merge conflicts

vale ibex
#

The merge commit that gets added doesn't have you as an author, since it pulls in the commits from main

#

So you're not counted as a commiter

#

The only part I disagree with github on is that of you resolve conflicts you can still count as a reviewer

#

Since during that resolution you can do whatever you like

dusky shoreBOT
short snow
#

did you just rebase?

#

ok just remove your last commit and then do the literal changes again

#

that sounds the easiest way to fix this

#

or you do a soft reset and remove all the unrelated changes

short snow
#

git reset --hard HEAD~2 if you want to do the former, or use --soft if you wanna do the latter

#

then you can just edit the changes

#

so you can just reset hard if you want

#

even if you mess up the hard reset, there is reflog ๐Ÿ™ƒ

static canyon
#

When I go to push a commit I made, it's showing these other commits and I don't know why

#

I only want to be pushing the top one

#

CC @fallen patrol is this because of what I did for pushing to your repo instead of pydis'?

fallen patrol
#

did you switch your branch before you made your commits?

#

it doesn't look like you did

#

and you fixed issue 2053 while based off of my fix branch

static canyon
#

Eh fuck

#

So... how do I fix this?

#

I did do a git checkout origin/main, then I used PyCharm's interface to create a new branch

#

I assumed that would work, but apparently not

short snow
#

you basically want to move some commit to a different branch right?

static canyon
#

Idrk tbh

#

I think I created the branch on arl's repo instead of pydis'

#

Honestly have no clue though tbh

short snow
#

isn't arl's repo one of the remotes, so it should be okay ig

static canyon
#

Yeah, it is

#

Arl's is one of my remotes

static canyon
short snow
#

remove those commits from this branch or make a new branch on pydis repo and remove them?

static canyon
#

Hmm

#

New branch on pydis and remove ig

short snow
#

git checkout -b origin/branch-name arl-remote/branch

#

may work

#

and then rebase to remove the commits

static canyon
#

fatal: 'arl/fix-issue-2053' is not a commit and a branch 'origin/fix-issue-2053' cannot be created from it

#

I guess I need to use the commit hash instead?

short snow
#

uhh, the origin can be removed from first argument, not sure about the second one

#

lemme check

#

ah ok

#

fetch arl's remote first

#

and then run it again

#

git fetch --all should do the job

short snow
# short snow and then rebase to remove the commits

by rebase you can just do git rebase -i HEAD~{n} where n is the last commit number you want to remove, according to your screenshot, you want to remove 2->5 so you can do HEAD~5 and then replace the the pick with drop

#

you may get some conflcits but that should work ๐Ÿคž

static canyon
#

So git fetch --all then git checkout -b origin/fix-issue-2053 arl/fix-issue-2053 then the rebase?

short snow
#

origin shouldn't be needed, but yeah

static canyon
#

Okay thanks

#

Lemme try

#

Hmm, still same error

#
(bot-6mLjCNeK-py3.9) C:\Users\tizzy\bot>git checkout -b origin/fix-issue-2053 arl/fix-issue-2053
fatal: 'arl/fix-issue-2053' is not a commit and a branch 'origin/fix-issue-2053' cannot be created from it
#

Same without the origin/

short snow
#

can you check if that branch exists? and the remote is there

#

testing locally, oe sec

static canyon
short snow
#

there is no such branch on arl's repo pithink

static canyon
#

I mean I haven't pushed at all so I guess that's why?

short snow
#

ok just do the rebase on that branch

#

and push to origin

static canyon
#

What commit do I rebase from?

short snow
#

oh well well this is getting pretty confusing, not sure if I am doing it correctly, how big is your commit

static canyon
#

It's tiny

#

Can redo it in like 20 seconds

#

So if I have to lose it it's not a big deal

short snow
#

maybe that would be better, i don't think i would be able to fix this

static canyon
#

So how would I "start over"?

short snow
#

if you are okay with that, or wait for someone else to help you out

#

i think i know a way but i don't want to complicate it more and then mess up, (undo merge, so it removes those "extra" commits got from main, and then do the rebase to remove the commits from arl, and then merge again)

static canyon
#

I mean I'm more than happy to just restart

#

As I said, it's like 20secs of edits

#

It's a tiny PR

#

I just need to know how to not mess up again

short snow
#

maybe don't merge with main branch until you are ready with the current code

#

basically when you merge with main, it adds in some commits according to the history

#

so to the timline changes

#

so if you want to remove a commit from 1 week behind which is actually your 2nd last commit on your branch, you would need to go very behind to remove it once you merge with main

static canyon
#

I didn't merge main though

#

Well, not into my branch

short snow
static canyon
#

I've got no clue what you're saying at this point

short snow
#

uhh, i will just stop lol, ignore me

static canyon
#

lol

#

So how do I restart?

short snow
#

make a new branch from main, copy in the patch

static canyon
#

I mean that's what I originally did

#

I checked out main, created a branch, implemented the fix, committed, went to push

vale ibex
#

What does your main look like?

short snow
#

uhh well then i dunno how that messed up

#

reset main, it maybe messed up?

#

also looks like that branch points to arl's repo, so i don't think its checkout out from pydis repo main

short snow
#

oh well gtg, good luck with that ๐Ÿ™ƒ

static canyon
vale ibex
#

could you checkout main and git log

static canyon
#

git checkout origin/main?

vale ibex
#

is your main not tracking origin/main?

static canyon
#

๐Ÿคท

#

Just doing that to be sure

vale ibex
#

could you do git branch -vv

static canyon
#
(bot-6mLjCNeK-py3.9) C:\Users\tizzy\bot>git branch -vv
* (HEAD detached at origin/main)    ece780ab Merge pull request #1721 from python-discord/enhance/backend/time-util-refactor
  fix-bot-2034                      d30776a1 [origin/fix-bot-2034: gone] Infraction mod-log improvements
  fix-issue-1924                    d5dc9ca4 fix: make editing follow the same duration rules
  fix-issue-2053                    5bd9e1ae Add missing arguments to `notify_infraction` call
  incident-archive-msg-improvements 1e0c0cfe [origin/incident-archive-msg-improvements] Fix tests
  main                              56fe0d5c [origin/main: ahead 4, behind 33] Merge branch 'fix-issue-1924' of https://github.com/onerandomusername/bot into main
vale ibex
#

alright git checkout main

#

then git reset --hard origin/main

#

That should leave you with a main branch that is up to date with origin/main and no extra commits

#

you can then branch from that and apply your fix

static canyon
#

Okay, thanks

#

Perfect, that worked, tyvm ๐Ÿ‘

vale ibex
#

๐Ÿ‘

static canyon
#

Hopefully after this bot#1951 will stop breaking things lemon_sweat

static canyon
#

This is the 3rd PR fixing issues with it lol

pallid mulch
stable mountainBOT
#

bot/exts/backend/error_handler.py line 252

async def handle_check_failure(ctx: Context, e: errors.CheckFailure) -> None:```
timid sentinel
#

InWhitelistCheckFailure seems to be a subclass of ContextCheckFailure, which is handled

static canyon
#

It's a bpo thing

#

The way it works is by using page headings

#

But list methods don't have page headings

#

So there's no way to programmatically find them

#

One of the staff here opened an issue on bugs.python.org about it. The only way it can work is if the psf adds those headers

brazen charm
#

lists explicitly don't expose the symbols in the index we use, tuples and others aren't really documented directly

vale ibex
brazen charm
vale ibex
brazen charm
#

I don't think it's that big of an issue to link to it manually, something like that would have to be hardcoded with the url, names and how the things should be looked up (as there isn't even a good way to get them out of the html), while the current system is completely automatic

fallen patrol
magic arch
fallen patrol
#

ah that's why it didn't make sense

#

!d list.sort also I don't see the issue, what's missing?

stable mountainBOT
#

sort(*, key=None, reverse=False)```
This method sorts the list in place, using only `<` comparisons between items. Exceptions are not suppressed - if any comparison operations fail, the entire sort operation will fail (and the list will likely be left in a partially modified state).

[`sort()`](https://docs.python.org/3/library/stdtypes.html#list.sort "list.sort") accepts two arguments that can only be passed by keyword ([keyword-only arguments](https://docs.python.org/3/glossary.html#keyword-only-parameter)):

*key* specifies a function of one argument that is used to extract a comparison key from each list element (for example, `key=str.lower`). The key corresponding to each item in the list is calculated once and then used for the entire sorting process. The default value of `None` means that list items are sorted directly without calculating a separate key value.
static canyon
#

None of the other methods (e.g. list.append) have headers

cursive relic
#

Is this supposed to happen with the !server ?

gritty wind
#

It used to be formatted in a highlighted string iirc, but yeah people asked for that info

#

Itโ€™s only in certain channels, namely: here and bot-commands

cursive relic
#

What does the features mean? Just the general features in server?

gritty wind
#

Itโ€™s the features enabled for this server in the discord backend

cursive relic
#

I see

gritty wind
#

Youโ€™ll need to refer to discordโ€™s docs for specifics

patent pivot
#

i'm tempted to revert the PR which made that available in all public channels

#

it just complicates the embed with information which is only going to grow as discord adds new features

stable mountainBOT
#

bot/exts/info/information.py lines 180 to 185

if ctx.channel.id in (
    *constants.MODERATION_CHANNELS,
    constants.Channels.dev_core,
    constants.Channels.dev_contrib,
    constants.Channels.bot_commands
):```
patent pivot
vale ibex
#

That just added #bot-commands

patent pivot
#

I added it there so that people stopped using it here to see that info - now I think we should just remove bot commands and dev contrib

#

we've already talked about how cluttered that embed is

#

and it has just generated questions here

vale ibex
#

Ah right, I miss understood your message

#

I read it like we enabled it in all public chanels

#

but yea, we can remove the public channels that are in there

patent pivot
#

ah no, just contrib & bot commands

#

will PR

vale ibex
#

are there any valid use cases to see those features?

brazen charm
#

an optional arg would work if people actually want it, though I doubt there are many valid uses for that beyond being curious

patent pivot
#

for public there aren't any afaik, for staff maybe to see if we have a certain feature? but we also have eval access

#

so

vale ibex
#

either

#

yea, the only use I can see is if we are wondering if they've given us access to a specific feature at some point

#

so we can probably keep it in core dev or smth just for that

patent pivot
#

yea

#

though I think having it in the #-spam channels is nice, don't want to have the same problem where we run commands in a discussion channel

vale ibex
#

Sure

patent pivot
#

!server

stable mountainBOT
#
Server Information

Created: <t:1483877013:R>
Roles: 93
Member status: status_online 60,833 status_offline 241,483

Members: 302,316

Helpers: 137
Moderation Team: 35
Admins: 15
Owners: 3
Contributors: 45
Leads: 8

Channels: 238

Category: 30
News: 8
Staff: 67
Stage_Voice: 2
Text: 121
Voice: 10

patent pivot
#

neat

vale ibex
#

@exotic ember what button is site#645 changing?

dusky shoreBOT
vale ibex
#

I can't spot any differences between the deploy preview and the production site

#

I've managed to find a drop down called sub-articles on the contributing pages, but that looks fine to me even on small view ports

#

So I'm not sure if this is the right one, or what should be tested, since it's not specified on the PR, nor the issue

last patio
#

wait we have deploy previews

#

okay I phrased that wrongly

#

I know we merged deploy previews

#

but some part of my brain lacks the attention to actually realize the comment is there linking to it

exotic ember
vale ibex
#

Ahhh, i see

#

I didn't try clicking it

exotic ember
#

bad commit message :blobsweat:

vale ibex
#

๐Ÿ˜…

gritty wind
fallen patrol
stable mountainBOT
#
I'll allow it.

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

timber pelican
#

where is it?

#

!d list.pop - cannot find documentation

vale ibex
lethal zenith
#

hi

brittle prawn
rapid swallow
gritty wind
#

I've left a comment in my review ๐Ÿ˜›

#

@vale ibex I see you've now had the joy of trying to find stuff in the logs

#

I'm making the PR now

vale ibex
gritty wind
#

It should cut the spam 10-fold

vale ibex
#

more annoying part was grafana refreshing every 5 sec which scrolls back to the top lol

gritty wind
#

Yeah no idea why that's default heh

#

I'll change it actually

vale ibex
#

it makes sense for a dashboard that watch lists

#

but ofc that's not our use case of trawling logs lol

gritty wind
#

It may or may not have been half the motivation for All App Logs

#

That and fixing all our flaming infra

#

sir-lancebot#1016

dusky shoreBOT
vale ibex
gritty wind
#

Hey guys, I'm making this announcement to inform you of a problem with our AoC logging system which negatively impacts all our logging output. I propose we reduce the logging output. All those in favor stay silent. All those against send your home address, contact details, and complaints to complaints@chrisjl.dev.

Signed,
Yours truly

#

Issue approved

vale ibex
#

uhhhh

#

ModuleNotFoundError: No module named 'setuptools'

#

wut

#

poetry pls

gritty wind
#

Hey guys, I'm writing this follow up announcement to inform you that I'm making the executive decision to assign myself to the issue
Signed

  • Me again
#

lol

#

watcha up to

vale ibex
#

your lint action

gritty wind
#

Wait this is actions

#

lmfao

#

Just gonna remove isort, ez

#

_distutils_hack this is a good sign

vale ibex
#

lmfao

gritty wind
#

Did you rerun

vale ibex
#

ye

#

just to see if it was ephemeral

#

but it was not lol

#

what's changed?

#

Ah, I'm guessing pip install poetry now has a new version?

#

we should probably pin that version

gritty wind
#

Did 1.2 release?

#

hjasihjdh
hype

#

hm

#

No I'm not seeing an update

#

This isn't even poetry related

#

It's precommit which is doing this

vale ibex
#

oh, I had assumed pre-commit used the poetry python env

#

and that was missing setuptools

gritty wind
#

Not for isort, no

vale ibex
#

Ah

gritty wind
#

It actually tries to build it from source

#

but yeah, what's changed then

vale ibex
#

we pin isort to 5.8.0, so that shouldn't have changed

#

maybe we can bump that up to latest and see if it fixes it?

gritty wind
#

I mean.. isort itself hasn't had an update in like half a month, so yeah lol

#

Latest is

vale ibex
#

5.10.1

gritty wind
#

failing all actions

vale ibex
#

oh lol

gritty wind
#

Okay lets see if there are any other changes that need to be made if we bump that version

#

Going to add it to my PR

#

We don't even include isort in our poetry config

#

Why

vale ibex
#

we use flake8-isort

#

so that it auto-fixes

#

iirc

gritty wind
#

Oh, I thought that was just a flake8 rule, didn't know it fixed

vale ibex
#

not suree, could be wrong on that

#

it's been a while

gritty wind
#

No you seem to be right

#

I just wasn't familiar

#

Okay, it's still failing

vale ibex
#

Would putting setup tools into the dev deps work here?

gritty wind
#

See my last commit lol

vale ibex
#

oh lol

#

That works

gritty wind
#

Our last working action was 4 days ago

#

I wonder what's changed since

#

Ok, yeah I didn't expect that to work

vale ibex
gritty wind
#

It seems to be a bug with python packaging in general

gritty wind
vale ibex
#

true

#

Don't we use isort in @stable mountain too?

gritty wind
vale ibex
#

what's the difference

gritty wind
#

Nothing appears different in the config

#

I guess we should test python too

vale ibex
#

ill rebase one of my prs to main

gritty wind
#

that'll drop our period from 4 days to 2 hours tops

vale ibex
#

trigger an action

gritty wind
#

Sounds good

vale ibex
#

alright it worked on bot

#

but that might be due to a cached env

gritty wind
#

Did it

#

yeah cache

#

Wait why was lance not cached

#

damn it we should encode cache date into the name

#

I'll add it to my setup-python action

#

It's not clear why lance has a missing cache though

#

Up until I made the bump PR, .pre-commit-config had not changed

vale ibex
#

Do we have an org-wide storage limit on cache?

#

might have just been pushed out, since it's been 4 days

gritty wind
#

Afaik the limit is only per repo

vale ibex
#

Not sure then

gritty wind
#

Otherwise cache misses will be infinite on bigger orgs

#

precommit-0-Linux-/home/runner/work/sir-lancebot/sir-lancebot/.cache/pre-commit-cache-3.9.10-a5c2422f4ab1f919b208ee0f5ee072ceb9c51fce08af3df0cde53ccdeb3463ab
precommit-0-Linux-/home/runner/work/sir-lancebot/sir-lancebot/.cache/pre-commit-cache-3.9.9-a5c2422f4ab1f919b208ee0f5ee072ceb9c51fce08af3df0cde53ccdeb3463ab

#

Python version changed

vale ibex
#

ah

#

makes sense

gritty wind
#

Does it?

vale ibex
#

that could explain other things then

gritty wind
#

Why did the version change

vale ibex
#

we pin to 3.9.*

gritty wind
#

Oh that makes sense

#

Well... what now

#

I've gotta run now

vale ibex
#

Can look into it later, in a work training call atm

cursive relic
#

The #reddit didn't get post today? Or is it just random timezone thing

vale ibex
#

Yea, it didn't

#

It's happened before too

#

Not sure what causes it, that cog doesn't have much logging

cursive relic
#

Should there be more?

#

Also I opened a issue x time ago, how can I find it because I don't remember what it was about

vale ibex
#

Yes, we should have logging to tell us something didn't work

#

I'm not sure the cause here, so can't speak to where that logging should be

cursive relic
#

Could you link the src?

vale ibex
#

.src reddit

dusky shoreBOT
#
You're bad at computers.

Your input was invalid: Unable to convert reddit to valid command or Cog.

Usage:```
.source [source_item]

vale ibex
#

.src Reddit

dusky shoreBOT
#
Noooooo!!

Your input was invalid: Unable to convert Reddit to valid command or Cog.

Usage:```
.source [source_item]

vale ibex
#

huh, could have sworn the cog was reddit

cursive relic
#

I can look it up when I get to laptop

gritty wind
#

hm it is reddit

#

.c

#

.c list

dusky shoreBOT
#

Utilities
status_online bookmark
status_online challenges
status_online cheatsheet
status_online colour
status_online conversationstarters
status_online emoji
status_online epoch
status_online githubinfo
status_online issues
status_online pythonfacts
status_online realpython
status_online reddit
status_online stackoverflow
status_online timed
status_online wikipedia
status_online wolfram
status_online wtf_python

gritty wind
#

.src Reddit

dusky shoreBOT
#
Do you mind?

Your input was invalid: Unable to convert Reddit to valid command or Cog.

Usage:```
.source [source_item]

gritty wind
#

no clue

vale ibex
#

.int e print(bot.get_cog('reddit'))

dusky shoreBOT
#
None

vale ibex
#

.int e print(bot.get_cog('Reddit'))

dusky shoreBOT
#
None

vale ibex
gritty wind
#

Well, that's not good lol

#

The cog list function reads them from the constant, not necessarily what's loaded

#

.cog load Reddit

#

.cogs load Reddit

dusky shoreBOT
#

:x: Extension bot.exts.utilities.reddit is already loaded.

gritty wind
#

.int e bot.reload_extension("bot.exts.utilities.reddit")

dusky shoreBOT
#
[No output]
gritty wind
#

.src Reddit

dusky shoreBOT
#
Cog: Reddit

Track subreddit posts and show detailed statistics about them.

Source Code
gritty wind
#

The cog wasn't loaded

#

I have no clue man

vale ibex
#

weird

#

that's likely why it didn't webhook a message then

#

wonder what caused it to be unloaded

#

.uptime

dusky shoreBOT
#

I started up 3 days ago.

vale ibex
#

since it posted 2 days ago, and hasn't rebooted since

gritty wind
#

This isn't the first time the reddit cog failed to post mysteriously, perhaps it just takes the weekends off?

vale ibex
#

lol maybe

#

I wonder if other cogs randomly unload

gritty wind
#

Moderation cog ๐Ÿคก

vale ibex
#

is there an on_unload event or similar?

#

could be useful to log state on that case

#

I wonder if it's caused by discord causing us to relogin every evening

gritty wind
#
    def cog_unload(self) -> None:
        """Cancel season-related tasks on cog unload."""
vale ibex
#

Ah yea ofc

#

I have written one of those myself Facepalm

gritty wind
#

lol can't be expected to remember everything

#

or can you hyperlemon

cursive relic
#

so the cog was just not loaded?

gritty wind
#

Yes, but it should've been

vale ibex
#

hmmmmm

#

I wonder if it's somethinng that we need to do with the login method of the bot

gritty wind
#

So have ext unload write to a cache, have the cog unload pop it. If it hasn't popped in 30 seconds, panic
To detect unexpected unloads, have a task which checks ext list from bot.cogs, and compare it to an internal cache of loaded cogs

#

Do we patch the login method ourselves

vale ibex
#

we do in python, not in lance

gritty wind
#

hm no

#

yeah

vale ibex
#

the issue could be that we don't unset bot._guild_available on logout

#

So when we reconnect to the websocket every night, bot.wait_until_guild_available won't reflect the cache

#

A fix would be to unset that Event at the start of login()

gritty wind
vale ibex
#

lol, this doesn't exist in d.py

#

it's our own thing

gritty wind
#

lmfaooooo

#

really? huh

#

good to know

#

Does on_guild_unavailable not cover it?

vale ibex
#

Yea, all the bot._guild_available wait_until_guild_available stuff is ours

#

hmmm, not sure if that is fired on ws reconnect

#

it might be

cursive relic
#

non relevant to current discussion, but is there any plans yet of changing the lib from dpy to something else?

vale ibex
#

yes, we will need to eventually

#

we are currently waiting to see what becomes the defacto standard

#

or at least waiting to see if one will become one

#

since all the forks/libs out there have only recently become relevant, so we want to ensure that the one we pick will be maintained well.

cursive relic
#

I see, is there any uh top x list that you are looking for

#

I mean, is there a list of possibilities that are like "ranked"

vale ibex
#

No, we have not ranked any of the existing libs

#

Since time will change that anyway

cursive relic
#

right, that makes sense

vale ibex
#

We are in now rush to swap, since what we have works for now

cursive relic
#

Thx for for the info ๐Ÿ˜„

vale ibex
#

once a new defacto standard appears, we will switch to it

#

if external forces make us choose before that, then we will have a core dev discussion on which one to pick

cursive relic
#

I see

vale ibex
#

so it might be good to patch disconnect() to also clear that Event

#

The other option could be to drop Discord.py's loggers to info over night, and see what it's doing

rapid swallow
#

!src reddit

stable mountainBOT
#
Bad argument

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

rapid swallow
#

does @dusky shore handle #reddit now?
If I am not wrong, once upon a time, @stable mountain used to handle #reddit

#

and there were talks about merging the cogs or something

#

I don't remember exactly because this was about a year ago

vale ibex
#

Yea, it was moved over the lance a few months ago

gritty wind
gritty wind
vale ibex
#

.src reddit

dusky shoreBOT
#
Command: reddit

View the top posts from various subreddits.

Source Code
vale ibex
#

ah that picks up the command

#

ofc

gritty wind
#

Theres a command? lmfao

vale ibex
#

.reddit top

dusky shoreBOT
#

Here are the top r/Python posts of all time!

#

Lad wrote a Python script to download Alexa...
reddit_post_photoโ€ƒreddit_upvote12337โ€ƒreddit_commentsโ€‚149โ€ƒreddit_usersiEslam

This post has:
9777 upvotes, 967 downvotes and 452 comments!
reddit_post_textโ€ƒreddit_upvote9229โ€ƒreddit_commentsโ€‚444โ€ƒreddit_usersKrukerfluk

I redesign the Python logo to make it more modern
reddit_post_photoโ€ƒreddit_upvote7861โ€ƒreddit_commentsโ€‚271โ€ƒreddit_usersjessjwilliamson

Automate the boring stuff with python - tinder
reddit_post_textโ€ƒreddit_upvote6719โ€ƒreddit_commentsโ€‚337โ€ƒreddit_usersbackprop88

Just finished programming and building my own...
reddit_post_photoโ€ƒreddit_upvote6617โ€ƒreddit_commentsโ€‚472โ€ƒreddit_usersjanky_british_gamer

stable mountainBOT
hoary haven
#

@mint nebula hey, i transferred your issue to the bot repo. bot#2060
did you want to take a shot at it?

dusky shoreBOT
hoary haven
#

(still needs approval though; if you want to wait before starting any work on it)

gritty wind
#

Are you looking for approval from anyone specific?

#

If the change is just to add Claimant: <name> to the bottom of the embed or whatever, that's pretty inconsequential

#

Giving it my scale seal of approval

hoary haven
#

also to change the behavior of the pin
can we pin that embed itself, as opposed to the claimant's first message?

mint nebula
gritty wind
#

I mean I guess, though it's less convenient

#

What's the motivation

hoary haven
#

both users and @stable mountain sometimes delete that first message

vale ibex
#

It's worth mentioning that we don't store a reference to that embed at all, so a new redis cache would be needed

gritty wind
#

Right, but what's the benefit of pinning the embed

vale ibex
gritty wind
#

We don't need a cache, can't you just call pin wherever you call send

vale ibex
clever wraith
gritty wind
#

Ah, I see

clever wraith
#

the embed would be edited as soon as they claim

#

oh, wait, yeah

#

the available message is sent when the channel moves to the available category

vale ibex
vale ibex
hoary haven
#

ok

clever wraith
#

i would be opposed to pinning the embed

hoary haven
#

pinning is not mentioned in the issue i realize now

clever wraith
#

Its nice to be able to quickly see the claimants message

vale ibex
#

i would only be up for that if we only do it when the opener message is deleted

gritty wind
#

That seems like a lot of work, I'm still not clear what the motivation is

vale ibex
#

Yea, me neither

gritty wind
#

Also, can we send a new embed with the claimant name to solve the cache issue

#

Or rather, it's a way to more cleanly keep that record

vale ibex
#

It's what we do currently for the available one

gritty wind
#

I thought we sent a new seperate embed on channel claim

#

I don't follow

#

what do we do currently

vale ibex
#

Sending a new embed after pinning the message seems like better ux too

#

especially if the opener message is long

gritty wind
#

Yeah

vale ibex
#

so we'd just add an in use embed to that

gritty wind
#

I would be a little opposed to including more content in the embed though, other than author name

#

So it'll be a pretty small embe

#

!embed Claimed by: Mr. Christopher

stable mountainBOT
#

Claimed by: Mr. Christopher

vale ibex
#

excuse me

#

it's doctor

#

smh

gritty wind
#

!embed Claimed by: Mr. doctor

stable mountainBOT
#

Claimed by: Mr. doctor

vale ibex
#

much better

gritty wind
#

What are you a doctor off

vale ibex
#

But yea, we can shove the occupied emoji in there as an thumbnail

#

say who claimed it

#

no need for description

#

easy

hoary haven
#

would you categorize this as a "first good issue?" bc then i could give it a shot. that also means it might take a month :)

vale ibex
#

hah yea already have

#

it's just going to be putting a ctx.send in the claim_channel call

rapid swallow
#

a doctorate in doctors

cerulean cloak
#

im new tp coding in python so do i download python or use vs code

magic arch
fallen patrol
#

although it miight need to be debug, not sure what level it logs incoming events at

fallen patrol
dusky shoreBOT
thorny obsidian
#

That's still the general aim, although I'm definitely fine hearing alternatives

cold moon
#

@vale ibex Voice mute PR should be OK now.

vale ibex
#

@gritty wind found the issue btw sir-lancebot#1017

dusky shoreBOT
vale ibex
#

Also bot#2061

dusky shoreBOT
cursive relic
vale ibex
#

It means we're waiting for the author to do something

#

it's the opposite of ready for review

cursive relic
#

oh I seee

short snow
#

@placid ermine you can just embed the image, so that the backgoround is always dark

placid ermine
#

oh good idea

vale ibex
#

&latex ```latex
\section{First Section}

\subsection{A subsection}
\begin{equation}
L' = {L}{\sqrt{1-\frac{v^2}{c^2}}}
\end{equation}
```

#

command I used if you want it for testing

placid ermine
#

'ight

placid ermine
#

so the embed idea didnt quite work out

#

i'll try the PIL route

cold island
#

This is now using an API?

vale ibex
#

Yea

#

wait @placid ermine what does a jpg format look like from the API?

#

Since jpg doesn't support transparency, it might already have a decent background

placid ermine
#

aaaAA the post request succeeds but then the get fails with jpg for some reason

vale ibex
#

lol

#

also just noticed that we're sending the payload with the GET request too

#

I imagine this isn't needed

placid ermine
#

yeah i noticed too

placid ermine
vale ibex
#

looks good ๐Ÿ‘Œ

#

Just out of interest, how long does making that background take?

#

Do we need to be worried about it blocking the event loop?

placid ermine
vale ibex
#

cool, sounds fine to me ๐Ÿ˜„

#

Didn't expect it to be much

placid ermine
#

yeah

hoary haven
#

ok i never finished setting up bot last time i poked in here, so trying again

#

docker is running

#
time="2022-01-26T13:21:37-05:00" level=warning msg="The USE_METRICITY variable is not set. Defaulting to a blank string."
services.metricity.environment.USE_METRICITY must be a string, number or null```
dim pelican
#

Did you do docker-compose up --build first?

hoary haven
#

i didn't..
also i'm fairly certain that last time i set up bot successfully (on a different machine) i didn't do that, and it just โœจ worked โœจ

dim pelican
#

I think I have mine set to false

hoary haven
#

added USE_METRICITY=false to .env and got

services.metricity.environment.USE_METRICITY must be a string, number or null```
so same thing except for the warning from before
vale ibex
#

Have you set USE_METRICITY to anything in your .env?

hoary haven
#

didn't earlier, and tried it just now set to false

vale ibex
#

and it's still giving error?

dim pelican
#

Is it there twice?

hoary haven
#

no it's not there twice

vale ibex
#

could you tell me what docker --version and docker-compose --version output?

dim pelican
#

lemon_thinking I'm stumped, I'll leave it to the docker lord Chris

vale ibex
#

We use a special syntax in the docker compose to set a default

#

my suspicion is that you're on an old compose version that doesn't support it

hoary haven
#

Docker version 20.10.7, build f0df350
Docker Compose version v2.0.0-beta.6

vale ibex
#

interesting

#

I'm not on compose v2

#

let me look at the changelog

hoary haven
#

from docker-compose.yml

vale ibex
#

Yea, that's the default, it's not changed

hoary haven
#

i will unselect docker compose v2

vale ibex
#

Could you try just docker compose up

#

no -

hoary haven
#

i'll do what you said first

#

same error as before

#

switching to
docker-compose version 1.29.2, build 5becea4c

#

looks like it's working

vale ibex
#

Very weird, I'm looking at the changelog and that shouldn't be broken in v2

hoary haven
#

should i care about this?

vale ibex
#

So I've just bumped up to compose v2 and it works just fine, however I get Docker Compose version v2.2.3

#

rather than 2.0.0-beta.6

#

so it might have been a bug in an old beta

vale ibex
#

We use compose volumes to share code changes, so we don't need to rebuild images all the time in dev

hoary haven
#

i just unselected this checkbox

vale ibex
#

it's not very performant in wsl2

vale ibex
hoary haven
#

oh lol

vale ibex
#

so for some reason you've got an old version

#

I have the same v1 build as you though

hoary haven
#

ok ty. i'm gonna add some emoji snowflakes to the config.yml
and then in a bit try to run bot again but locally

vale ibex
#

๐Ÿ‘Œ

hoary haven
#

what's the right way to stop a container? other than using the UI button in the docker application?

vale ibex
#

ctrl+c in the terminal you started it in

hoary haven
#

as far as i can tell i do have setuptools installed though

dim pelican
#

Try poetry shell first, then poetry install?

vale ibex
#

heh, this is the issue we fixed in the github actions yesterday, didn't expect anyone to see it so quickly locally. You're gonna have to do $env:SETUPTOOLS_USE_DISTUTILS = 'stdlib'

hoary haven
#

oki oki. it's running though w/ some errors lol. we're almost there

hoary haven
#

now that discordpocalypse is over for now
https://paste.pythondiscord.com/zoloqurura.yaml

i figure some things i can ignore
but not that bottom part?
i also don't really know what i am doing with webhooks. i just made one and chucked the same webhook ID in the config.yml

#

i gave my bot the "Admin Bots" role

edit: i just realized i still had my bot on the staff server so i kicked it

vale ibex
#

the errors there suggest missing channels and webhook ids

#

The "Optional config.yml" file there has a ๏ฟฝ character for everything you need to populate

hoary haven
vale ibex
#

Can you send me the full config.yml?

trail pilot
#

hmm, what would you all think for topics in #pedagogy

#

Via .topic

austere hornet
vale ibex
austere hornet
#

Speaking of issues I'm still waiting on approval on sir-lancebot#989 and sir-lancebot#1000 (maybe)?

thorny obsidian
#

I don't have anything off the top of my head. The big thing to mention is the Events tab, @Announcements role, aaaaand... maybe the !subscribe command?

#

Can you? AFAIK it's not available through the API

brittle hatch
#

how is a % b calculated? Can't find how it's done in python

#

I need it in details

sleek steppe
brittle hatch
#

ok

thorny obsidian
#

Ah, looks like they finally added support in November. Yeah, we can pull from the scheduled events tab. We'll have to add a check for only public events

#

If the event is tied to a private voice channel, yes. Only people who can see the channel can see the event

#

No, not necessarily. I don't trust Discord enough to have it be the source of truth

#

If an event needs to be created I can just make a PR

#

You don't need to create a custom event creation modal. If a non-Discord Events Tab event needs to be created, I can just manually PR it

#

Yeah, we can try pulling events from the Discord Events Tab.

vale ibex
#

Nice small PR to fix an issue @hoary haven encountered with the example config.yml file in the bot contrib guide.
site#646

dusky shoreBOT
thorny obsidian
#

Looks good! I would slightly reword it to, although the second sentence is quite long now. Feel free to adjust further.

An integral part of the Python Discord experience are the community-wide events we run. Whether it's competing in our yearly Code Jam, joining our Advent of Code leaderboard, or learning about APIs via a collaborative digital canvas, there's an event for you to join!

Head over to the Discord server and !subscribe to the @Announcements role to be notified of future events - meanwhile, you can check out the scheduled events tab on the Discord server or read on for further information.

austere hornet
dusky shoreBOT
hoary haven
short snow
#

rather than having that, we could have just kept the pinned bot message, it tells the samething...

rapid swallow
#

hello mina

short snow
#

anyway, that looks nice ok_handbutflipped

hoary haven
#

i'm not changing any pinning behavior

#

just adding a new message that's sent

#

oh i left that out of the screenshot, but yeah claimant's first message is still pinned

short snow
#

nvm nvm, i literally forgot how the pinned message looks

rapid swallow
#

the system message for pinned message does not state the author of the message

short snow
#

yeah, i thought it did ๐Ÿคฆ

hoary haven
#

(bonus for moderators we'll be able to find the appropriate channel belonging to someone when they cross-post)

#

i think our other embeds have proper punctuation so maybe i need to add a . at the end

placid ermine
#

Channel; claimed: By mina.

tawdry vapor
#

The extra message may detract from the actual question, but I suppose it won't be much more detracting that the system pin message already is.

hoary haven
#

we also have the option of doing a regular message instead of an embed. then users that ever lose their help channel can use the discord inbox (I bring this up bc the help channel guide has a section on how to find your channel again)

short snow
#

embeds are more highlighted IMO, but in this case it doesn't really matter though as users mostly delete there help messages early, so yeah regular message may work ๐Ÿ‘

fallen patrol
#

we could add a command for users to be mentioned in their open help thread to be able to find it

wild prism
#

idk if users who don't know how to use discord's search will be using a bot command like that (or checking their inbox)

fallen patrol
#

then again yeah

hoary haven
#

the inbox feature is pretty easy to use but for some reason i didn't know about it until way after learning how to use from: and mentions:

but i think i'll go with the embed. will PR soonโ„ข๏ธ

hoary haven
#

what is our max character count per line for docstrings?

vale ibex
#

It varies per project, but we don't have a different line limit for docstrings to other lines

#

In the project root, open up the tox.ini file and it'll be the max-line-length value

hoary haven
#

oki

hoary haven
#

bot#2062 github wanted mark to review but ofc anyone is welcome

dusky shoreBOT
sleek steppe
#

Yeah those are just auto review requests for code owners, they are requested for the review but don't have to in order to merge

hoary haven
#

@molten perch @sleek steppe ooh thanks guys!

sleek steppe
dim pelican
hoary haven
#

pretty chuffed myself aww

vale ibex
#

voice gate is currently down due to the voice_mute changes in site & bot. I'd appreciate some reviews on these two bug fix PRs
bot#2064 site#647

vale ibex
#

To any core devs reading, these two PRs can be merged separately if needed, as it's already in a broken state

vocal wolf
#

is github down for anyone else?

magic arch
#

yup

#

just the home page apparently though (see lounge)

magic arch
austere hornet
#

That's weird, everything is working for me on GitHub. Is this the dashboard or am I being dumb?

magic arch
#

Yep, that's the one

austere hornet
#

Ok, but why is it broken for everyone but me, I'm confused...

magic arch
#

No clue, tis the nature of GitHub sometimes

austere hornet
#

Okey dokey ๐Ÿ˜„

magic arch
crude gyro
#

@cold moon @exotic ember @patent pivot @gritty wind

I'm not sure which one of you worked on the redirects app in our site the most, but judging from the blame you were all at least somewhat involved?

I'm hitting a wall trying to get it to redirect the old resource pages (like /resources/videos) to the new page with GET parameters.

e.g., I want to redirect /resources/videos to /resources/?type=video, and it just.. doesn't seem to work. Does it just not support that?

#
# This just redirects to /resources
resources_videos_redirect:
  original_path: resources/videos/
  redirect_route: "resources/?type=video"

# This still just redirects to /resources
resources_interactive_redirect:
  original_path: resources/interactive/
  redirect_route: "resources:index"
  redirect_arguments: ["type=interactive"]
#

I think the redirect app is the right place for redirects that maintain backwards compatibility, and cf workers are for convenience redirects

patent pivot
#

I've added a couple of redirects we missed in migration, I'm unsure of the internals though unfortunately

crude gyro
#

I'm guessing that these arguments are forwarded to my view, instead of being added to the URL itself. But perhaps I could just.. allow those arguments into my view method and add them to the request from there. that seems somewhat reasonable.

gritty wind
#

All of my work was exclusively tied to static builds, so it was mostly hacking the app to make it work for that

#

Fwiw, I think the redirects system is very complex, and difficult to work with, at some point we should deprecate it

#

Even if it breaks backward compatibility to some extent

#

@patent pivot could you pull data at all about traffic to those paths?

stable mountainBOT
#
I'll allow it.

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

drifting elm
#

Hello. I'm trying to compile Python3.8.9 from source on Windows 10 but getting compiling errors. I've tried a couple of version of python so far and got the same error below:

fatal error RC1116: RC terminating after preprocessor errors

Can anyone here help?
There is an issue for this https://bugs.python.org/issue45220

gritty wind
drifting elm
#

sorry

#

Is there another discord server to discuss general python internal? Like compilation?

#

@gritty wind

gritty wind
#

Afaik not on discord. We do help with stuff like that, just not in this channel

drifting elm
#

alright

#

thx

patent pivot
#

maybe we have the site push used redirects into a table we can pull data on or something

gritty wind
#

Yeah that sounds simple enough to do

cold moon
#

Would be too hard to put StatsD into site?

#

This would make more sense for stats than custom table

patent pivot
#

i think we need a table with auto-increment pk, we'll then store the hash of the requesting IP, the path they are trying to access and the referrer that sent them, maybe also IP geoloc (to country)

#

in theory we can still do that with a cf worker if we want

patent pivot
#

so in that respect - it is deprecated

patent pivot
#

yes but that's stupidly annoying to aggregate

#

why use a hacky solution like that when we have a database which we can plug directly into metabase and get all the data immediately without annoying hacks

sleek steppe
#

Hey, I was wondering if there's any reason why Lancebot doesn't have the bot-core dependency? If there isn't one, I'd be happy to add it ๐Ÿ™‚ .

stable mountainBOT
# exotic ember !remind 5h
It has arrived!

Here's your reminder: @cold moon @exotic ember @patent pivot @gritty wind

I'm not sure which one of you worked on the redirects app in our site the most, but judging from the blame you were all at least somewhat involved?

I'm hitting a wall trying to get it to redirect the old resource pages (like /resources/videos) to the new page with GET parameters.

e.g., I want to redirect /resources/videos to /resources/?type=video, and it just.. doesn't seem to work. Does it just not support that?
[Jump back to when you created the reminder](#dev-contrib message)

cinder forum
#
  1. On pydis bot why put stuff on config-default.yml, why not just .py?
#
  1. I don't see a requirements.txt where can I find what needs to be installed?
sleek steppe
# cinder forum 2) I don't see a requirements.txt where can I find what needs to be installed?
  1. We use config-default.yml (and config.yml) for configuration, as yml is an easier syntax for configs rather than python files. See https://www.pythondiscord.com/pages/guides/pydis-guides/contributing/bot/#configyml

  2. We use poetry (https://python-poetry.org/) which is a dependency manager which makes it a lot easier for development. The dependencies are listed in pyproject.toml. See https://www.pythondiscord.com/pages/guides/pydis-guides/contributing/bot/ for a guide on how to contribute to the pydis bot.

gritty wind
#

We were using pipenv previously but it didnโ€™t work for mac users

rapid swallow
gritty wind
#

And it was kinda slower

#

Itโ€™s preference anyway, they are pretty similar

thorny obsidian
#

pipenv five years to lock too, which while it didn't happen frequently was still annoying

placid ermine
#

heyo, could i get another staff/contributor approval on sir-lancebot#1010

dusky shoreBOT
cold island
placid ermine
#

code has been yoinked with minimal changes from the eval source

#

hm

cold island
#

eval as in

#

!src eval

stable mountainBOT
#
Command: eval

Run Python code and get the results.

Source Code
cold island
#

?

stable mountainBOT
#

bot/exts/core/internal_eval/_internal_eval.py lines 96 to 98

except Exception:
    # 400 (Bad Request) means there are too many characters
    log.exception("Failed to upload `โ€‹internal eval`โ€‹ output to paste service!")```
cold island
#

I think in the case of int eval we just don't want sentry issues opened for int eval code

#

the snekbox command probably handles it better

#

hmm... it doesn't

gritty wind
#

Tbf we only lock like every other month lol

cold island
placid ermine
#

without the transparency mask the transparent pixels just appear black

cold island
#

Are there any known ratelimits for the API?

#

I'm wondering if a per-user cooldown is needed here

placid ermine
#

nothing's mentioned in the docs

#

well there's this

cold island
#

hmmmm

#

It might be a good idea to contact them

#

I don't expect the usage to be huge, but I don't know what kind of load they're expecting

placid ermine
#

@short snow said another server uses it

gritty wind
#

Does the other server have 300K users ๐Ÿ™ƒ

placid ermine
#

i mean, 300k users aren't using our bots

#

i dont think we'll use it more than a math server, even with more members

gritty wind
#

Yeah thatโ€™s very true

#

We should still check with them though, a programming server shouldnโ€™t compromise on program ethics ๐Ÿ˜ƒ

placid ermine
#

fair enough

cold island
#

Yeah I think it's just good manners to at least give them a heads up

cold island
#

@placid ermine would you like to send them an email, or should someone else?

short snow
placid ermine
cold island
#

Sure

tawdry vapor
#

Logging could be improved if you want though

fringe sphinx
#

also, the second dropdown doesn't slide out, it just kinda exists once you click. is that supposed to happen?

vale ibex
#

I wonder if the speed of the slide scales with how long the dropdown is

#

would explain why it's so fast

fringe sphinx
exotic ember
#

it does slide, the dropdown is just so ridiculously long the part you see animates instantly

fringe sphinx
#

oh maybe you're right

exotic ember
#

also the animation is ease-out, which starts fast and slows down towards the end

slim widget
#

Has anyone suggested developing a configuration wizard for the bot?

slim widget
#

Alright, I'll look into that, thanks ๐Ÿ‘

rapid swallow
slim widget
#

Yeah, with like buttons and stuff ๐Ÿ‘€

#

I finally got around to setting up my own test-server.

#

And thought it would be handy to have something you could use in to add the channel IDs etc to the config file. As well as check that the channels, roles, bot, etc. all have the right permissions.

#

But I recognise this is a challenging task.

rapid swallow
#

it's not a configuration wizard

sleek steppe
#

Just a reminder that bot#1602 (the !timeit command) is ready for review ๐Ÿ˜„

dusky shoreBOT