#dev-contrib | Python | Page 78

gritty wind Jan 20, 2021, 1:39 PM

#

So even if you setup one manually, it won't affect the pipenv

short snow Jan 20, 2021, 1:39 PM

#

but if u are not into your virtual enivornment?

gritty wind Jan 20, 2021, 1:39 PM

#

Yeah it's based on your project

short snow Jan 20, 2021, 1:39 PM

#

since, when u update your pipfile, u need to run pipenv lock

gritty wind Jan 20, 2021, 1:40 PM

#

You don't, it does automatically if you install
It's based on the project, so as long as you are in the same directory as the pipfile, it'll work correctly

short snow Jan 20, 2021, 1:43 PM

#

bump

gritty wind Jan 20, 2021, 1:45 PM

#

An issue was opened for it from sentry

#

Feel free to tackle if you think you've got a fix

#

Hmm or maybe I'm misremembering

short snow Jan 20, 2021, 1:46 PM

#

.issue 1369

dusky shoreBOT Jan 20, 2021, 1:46 PM

#

[404] #1369 Issue/pull request not located! Please enter a valid number!

short snow Jan 20, 2021, 1:46 PM

#

is there

#

but not any for this

gritty wind Jan 20, 2021, 1:46 PM

#

It opened one for the other issue right above that

#

.issue 1369 bot

dusky shoreBOT Jan 20, 2021, 1:46 PM

#

GitHub

IssueOpen [bot] #1369 The reminder command doesn't properly handle overflowing durations

short snow Jan 20, 2021, 1:47 PM

#

yeah that is open

#

but not for battleship

gritty wind Jan 20, 2021, 1:47 PM

#

Right, uhh I think the error handler might've eaten this one then

#

In that case, do you wish to open an issue for it, or should I?

short snow Jan 20, 2021, 1:47 PM

#

u can go ahead

balmy sparrow Jan 20, 2021, 1:51 PM

#

oh i didn't, this is the one pipenv seems to have made

gritty wind Jan 20, 2021, 1:51 PM

#

In that case pipenv should be available within the environment

#

Does running pipenv after deactivating work?

balmy sparrow Jan 20, 2021, 1:52 PM

#

Let me try, one second

#

Vsc tells me that pipenv isn't on path

#

do i add the pipenv executable to path?

gritty wind Jan 20, 2021, 1:53 PM

#

I guess you could try?

balmy sparrow Jan 20, 2021, 1:53 PM

#

gritty wind Does running pipenv after deactivating work?

this worked

📎 unknown.png

#

oh, i've seen people here have some issue due to line endings or something? what is it supposed to be? vsc says I'm using CRLF right now

gritty wind Jan 20, 2021, 1:55 PM

#

That's windows' default

#

Most programs use LF

#

That being said

#

If you commit CRLF files, git will automatically convert to LF

green oriole Jan 20, 2021, 1:56 PM

#

and MacOS use CR because standards

gritty wind Jan 20, 2021, 1:56 PM

#

It's just that the linting will change them all to LF as well, so it might appear as though you have changed all files

#

That won't make it to github though, it'll be fixed before then

#

You can set your git cloning settings so that it clones in LF

balmy sparrow Jan 20, 2021, 1:57 PM

#

Oh right

#

iirc while installing git it had asked me to pick what to use

#

i have no idea what i picked

gritty wind Jan 20, 2021, 1:58 PM

#

Then its probably a good idea you set it up now and reclone

#

before you start working

green oriole Jan 20, 2021, 1:58 PM

#

If you left the default, it is CRLF

#

Although it is a setting you can set per repo or globally

balmy sparrow Jan 20, 2021, 1:58 PM

#

gritty wind Then its probably a good idea you set it up now and reclone

Alright

gritty wind Jan 20, 2021, 1:59 PM

#

green oriole Although it is a setting you can set per repo or globally

I'm not sure how it would even work per repo, because it is done at clone time, and you can't set repo overwrites till you clone

#

git config --global core.autocrlf input will use whatever the repo uses for all your git clones

balmy sparrow Jan 20, 2021, 2:01 PM

#

alright done, thanks for guiding me through this :D

green oriole Jan 20, 2021, 2:09 PM

#

gritty wind I'm not sure how it would even work per repo, because it is done at clone time, ...

You can pass a --config argument to git clone, or change the checkout ending https://git-scm.com/docs/git-clone#Documentation/git-clone.txt---configltkeygtltvaluegt

gritty wind Jan 20, 2021, 2:11 PM

#

Oh thanks for telling me

balmy sparrow Jan 20, 2021, 2:59 PM

#

is there no lancebot equivalent to !ping of @stable mountain ?

green oriole Jan 20, 2021, 3:03 PM

#

There isn't, I don't think there's really a need for it

balmy sparrow Jan 20, 2021, 3:04 PM

#

fair enough

late wolf Jan 20, 2021, 3:23 PM

#

But there's uptime lemon_happy

gritty wind Jan 20, 2021, 3:23 PM

#

.uptime

dusky shoreBOT Jan 20, 2021, 3:23 PM

#

I started up 2 days ago.

late wolf Jan 20, 2021, 3:24 PM

#

hmm why does isn't it more specfic

balmy sparrow Jan 20, 2021, 3:26 PM

#

~~sir lancebot's enemies must never find out when he sleeps~~

balmy sparrow Jan 20, 2021, 4:38 PM

#

.issue 1355 bot

dusky shoreBOT Jan 20, 2021, 4:38 PM

#

GitHub

IssueOpen [bot] #1355 Make sure that messages from private channels can't be ducked

balmy sparrow Jan 20, 2021, 4:38 PM

#

I've commented here, could I get assigned? Looks simple enough

gritty wind Jan 20, 2021, 4:38 PM

#

Are you anand2312?

balmy sparrow Jan 20, 2021, 4:38 PM

#

yep

gritty wind Jan 20, 2021, 4:38 PM

#

I've assigned you a while ago, but didn't know who to ping lol

#

Yeah go ahead and start

balmy sparrow Jan 20, 2021, 4:39 PM

#

Oh, thanks (I've changed my nickname here now)

#

I have a doubt; how do I get the helper role ID?

#

is it constants.Roles.helpers?

gritty wind Jan 20, 2021, 4:40 PM

#

Yeah

#

that's the general staff role

balmy sparrow Jan 20, 2021, 4:41 PM

#

Also, this is what I had in mind:

    @staticmethod
    def is_helper_viewable(channel: TextChannel) -> bool:
        """Check if helpers can view a specific channel."""
        helper_role = channel.guild.get_role(constants.Roles.helpers)
        overwrites = channel.overwrites_for(helper_role)
        overwrites.view_channel ...

channels where the overwrites are either None or True are both viewable if I understand permissions correctly

#

do i check for both cases? or are all the channels explicitly set to one option (true/false)

gritty wind Jan 20, 2021, 4:46 PM

#

That would work, but I'm unsure how discord handles implicit permissions

#

I'm assuming the moderation channels are invisible to everyone, with explicit read permissions for moderators

balmy sparrow Jan 20, 2021, 4:46 PM

#

If it isn't explicitly set to either True or False, discord.py sets it to None - https://discordpy.readthedocs.io/en/latest/api.html?highlight=overwrites_for#discord.PermissionOverwrite

gritty wind Jan 20, 2021, 4:47 PM

#

In that case that should work

balmy sparrow Jan 20, 2021, 4:47 PM

#

oh, what would be the "neat" way to do write that though?

#

(overwrites.view_channel is None) or overwrites.view_channel is what immedietely came to mind but that doesn't look nice

gritty wind Jan 20, 2021, 4:49 PM

#

If it's none, helpers can't read it, and we want to treat it the same as False, no?

gritty wind Jan 20, 2021, 4:49 PM

#

balmy sparrow If it isn't explicitly set to either True or False, discord.py sets it to None -...

Actually no that won't work

#

You'd have to check the overwrites for @everyone, and @helpers so it works in all channels helpers can see

#

but from there, you can just return
everyone.view or helpers.view

#

The intersection of those two permissions are the channels viewable by helpers

#

You can then group the none with the false

balmy sparrow Jan 20, 2021, 4:52 PM

#

Right, let me try something like that

gritty wind Jan 20, 2021, 4:54 PM

#

@patent pivot 1370 is a duplicate of

#

.issue 1309 bot

dusky shoreBOT Jan 20, 2021, 4:55 PM

#

GitHub

IssueOpen [bot] #1309 Voice ban bug for non-members

patent pivot Jan 20, 2021, 4:55 PM

#

smh

gritty wind Jan 20, 2021, 4:55 PM

#

Which in itself is a duplicate of

#

.issue 1016 bot

dusky shoreBOT Jan 20, 2021, 4:55 PM

#

GitHub

PRMerged [bot] #1016 Some bug fixes

gritty wind Jan 20, 2021, 4:55 PM

#

Or at least that's why my pr on 1309 was closed

patent pivot Jan 20, 2021, 4:55 PM

#

lol

#

hm wait

#

yeah

gritty wind Jan 20, 2021, 4:56 PM

#

Did the issue occur again within the last 20 days?

patent pivot Jan 20, 2021, 4:57 PM

#

yeah

gritty wind Jan 20, 2021, 4:57 PM

#

Interesting, that might mean 1309 isn't a duplicate

#

but in that case you should have a pr ready to go 🤓

#

Hmm yeah I don't see how 1016 solves the issue

balmy sparrow Jan 20, 2021, 5:32 PM

#

gritty wind The intersection of those two permissions are the channels viewable by helpers

I think this is how it works; and leaving it as None means the user can see the channel unless they have some role which is explicitly denied those permissions
Also, intersection of channels everyone can see and channels helpers can see - should just be the channels the helpers can see right?

📎 Screenshot_2021-01-20_212759.png

gritty wind Jan 20, 2021, 5:34 PM

#

Well here's how I believe the setup to be:
python general - everyone
staff-social - helpers
mod-social - mods

None of these channels have explicit permissions except for the role listed besides them. The channels visible to helpers explicitly are only the staff channels. If the permission returns None on the other two channels, there'd be no way to determine if the channel should be used or not

#

If you combine everyone and helper role, you can make it work for public channels such as pygen, and staff channels, but not mod channels

balmy sparrow Jan 20, 2021, 5:38 PM

#

gritty wind but from there, you can just return `everyone.view or helpers.view`

Right, so I'm just gonna do this. Thanks.

#

Also, should I be making a branch on my fork to which I commit these changes?

gritty wind Jan 20, 2021, 5:39 PM

#

Yeah, that's usually a good idea

#

This gist does an alright job of explaining the fork-and-pull workflow

#

https://gist.github.com/Chaser324/ce0505fbed06b947d962

#

It's completely cli, but you can do most of the steps on the github page too

balmy sparrow Jan 20, 2021, 5:41 PM

#

Oh alright

tawdry vapor Jan 20, 2021, 5:55 PM

#

@patent pivot GH bot seems to not be detecting reviews again

patent pivot Jan 20, 2021, 5:56 PM

#

It is

#

📎 image0.png

tawdry vapor Jan 20, 2021, 5:57 PM

#

Not the policy bot

#

I'm talking about your bot

#

The one that adds labels

patent pivot Jan 20, 2021, 5:57 PM

#

oh yeah the bot is gone, we discontinued that

tawdry vapor Jan 20, 2021, 5:57 PM

#

😭

patent pivot Jan 20, 2021, 5:57 PM

#

I think it was decided that if we wanted to do that we'd use actions

tawdry vapor Jan 20, 2021, 5:58 PM

#

Well, I liked those labels

#

Seems premature to discontinue the bot when we don't have an alternative in place yet

patent pivot Jan 20, 2021, 5:58 PM

#

If they were used I can certainly bring it back

balmy sparrow Jan 20, 2021, 6:07 PM

#

When I run pipenv run lint, it said it failed flake8; how do I see what went wrong?

gritty wind Jan 20, 2021, 6:08 PM

#

It should be printed with the rest of the error, how are you running it?

balmy sparrow Jan 20, 2021, 6:08 PM

#

📎 unknown.png

#

or is the pipenv not found what's failing

gritty wind Jan 20, 2021, 6:08 PM

#

Ok it failed in running flake8

#

Yeah

#

so, is pipenv currently installed globally?

balmy sparrow Jan 20, 2021, 6:09 PM

#

yeah

gritty wind Jan 20, 2021, 6:09 PM

#

so can you just type pipenv in console

#

without the python -m

balmy sparrow Jan 20, 2021, 6:09 PM

#

oh no I meant it is on my main py3.8 install, it doesn't work without the python -m, says pipenv isn't a recognized command

gritty wind Jan 20, 2021, 6:10 PM

#

So the flake8 thing iirc just calls pipenv run flake8, so if it isn't installed it'll error out

#

How were you installing flake8 again?

balmy sparrow Jan 20, 2021, 6:11 PM

#

I'd installed it earlier with pip install

#

Was I supposed to install again now?

gritty wind Jan 20, 2021, 6:11 PM

#

Oh again was just a figure of speech

#

And flake8 was a brain fart lol

#

how did you install pipenv when you got it to work

balmy sparrow Jan 20, 2021, 6:12 PM

#

Deleted my old python install, reinstalled 3.8, pip installed pipenv

#

Then set up pipenv environment in pycharm

gritty wind Jan 20, 2021, 6:12 PM

#

Hmm in that case it should've installed the pipenv command to your base system

#

Is python installed on the user or system?

balmy sparrow Jan 20, 2021, 6:13 PM

#

What's the difference?

#

Oh, I think I installed python now on the system (it's path is now in C:/Program Files/Python/python38)
does on user mean when it is installed in the Users/... folder?

gritty wind Jan 20, 2021, 6:14 PM

#

Yeah

#

I was trying to check, but I can't tell if it actually makes a difference

balmy sparrow Jan 20, 2021, 6:15 PM

#

I know where the pipenv executable is, should I add it to path?

gritty wind Jan 20, 2021, 6:15 PM

#

Is it under C:/Program Files/Python/Python38/Scripts

balmy sparrow Jan 20, 2021, 6:16 PM

#

Nah it's in the Users folder I think. Does this mean I have another install of python on the user as well

#

yep it's in User/.../AppData/Roaming/Python/python38/Scripts

gritty wind Jan 20, 2021, 6:16 PM

#

Possibly, but the more relevant part is that it did the pipenv install on your user, not system, which is why it isn't added to path

#

You could manually add it to path, but that might cause issues

gritty wind Jan 20, 2021, 6:17 PM

#

gritty wind Is it under `C:/Program Files/Python/Python38/Scripts`

Do you see a pip.exe here?

balmy sparrow Jan 20, 2021, 6:17 PM

#

pip, pip3.8, pip3 all there

gritty wind Jan 20, 2021, 6:18 PM

#

Perfect, could you try running the install from that then

(in an admin cmd)
C:/Program Files/Python/Python38/Scripts/pip.exe install pipenv```

#

That should hopefully get it installed and added to path

balmy sparrow Jan 20, 2021, 6:18 PM

#

Let me try that

#

After that would I have to do pipenv install and everything on the project again?

gritty wind Jan 20, 2021, 6:19 PM

#

I think your pipenv environment will still exist

#

but you should hopefully be able to do pipenv command directly now, instead of adding python -m

#

Which should also fix the linting issue

rapid swallow Jan 20, 2021, 6:20 PM

#

do windows users need to use pipenv-win or something?

#

nvm, I think that's for pyenv

gritty wind Jan 20, 2021, 6:20 PM

#

I'm using pipenv fine, but his problem seems to be not having it installed on sys

rapid swallow Jan 20, 2021, 6:21 PM

#

yeah

#

but if pipenv is not in path, they might have some other underlying issue

balmy sparrow Jan 20, 2021, 6:21 PM

#

🙁

📎 unknown.png

#

says requirement already satisfied

rapid swallow Jan 20, 2021, 6:22 PM

#

I think you are installing pipenv to the virtual environment

gritty wind Jan 20, 2021, 6:22 PM

#

Ok i'm a bit out of ideas now, but you can try adding to path ig

tawdry vapor Jan 20, 2021, 6:22 PM

#

The simple fix is to just add the user scripts directory to PATH... It doesn't matter if it's installed to the global site, the user site, or your desktop. Just add it to PATH and it would've fixed the issue.

rapid swallow Jan 20, 2021, 6:22 PM

#

and not as a global package

gritty wind Jan 20, 2021, 6:23 PM

#

rapid swallow I think you are installing pipenv to the virtual environment

He's using the absolute path to the base pip

rapid swallow Jan 20, 2021, 6:23 PM

#

gritty wind He's using the absolute path to the base pip

no idea what that means 😄
but does it install packages globally?

gritty wind Jan 20, 2021, 6:24 PM

#

Not necessarily, but it's using his actual pip, so not a virtual env

balmy sparrow Jan 20, 2021, 6:25 PM

#

Editing environment variables should take effect immedietely right?

gritty wind Jan 20, 2021, 6:25 PM

#

After restarting the terminal, yeah

#

(restart as in close and open)

balmy sparrow Jan 20, 2021, 6:25 PM

#

Oh right lmao

balmy sparrow Jan 20, 2021, 6:27 PM

#

tawdry vapor The simple fix is to just add the user scripts directory to PATH... It doesn't m...

Nice, that worked. Thank you!

📎 unknown.png

balmy sparrow Jan 20, 2021, 7:03 PM

#

https://github.com/python-discord/bot/blob/eccbebf8f29998dcee72b690ed405221363a984f/bot/exts/fun/duck_pond.py#L110
https://github.com/python-discord/bot/blob/eccbebf8f29998dcee72b690ed405221363a984f/bot/exts/fun/duck_pond.py#L83
Is it fine if I make the type hints uniform? There seem to be two functions where it's type hinted as discord.Message and the rest are Message

patent pivot Jan 20, 2021, 7:08 PM

#

Actually, cc @hardy gorge, you mentioned we could replace the GitHub label bot with GitHub Actions, is that possible?

#

and if so, where do we start

hardy gorge Jan 20, 2021, 7:08 PM

#

It should be, depending on the kind of labels you want

patent pivot Jan 20, 2021, 7:09 PM

#

same as we had before, needs 2 reviews, needs 1 review, waiting for author

hardy gorge Jan 20, 2021, 7:10 PM

#

Shouldn't be too difficult, although I've had a few glitches before. I'll write something tomorrow or so.

gritty wind Jan 20, 2021, 7:49 PM

#

Hey, is rijusougata13 on github here?

#

Or aruna2019, I'm unclear which one it is

quiet gull Jan 20, 2021, 8:31 PM

#

yes

#

i am here

green oriole Jan 20, 2021, 8:32 PM

#

Hey

#

Well, I think the PR is good to go

quiet gull Jan 20, 2021, 8:33 PM

#

will it be merged then?

green oriole Jan 20, 2021, 8:34 PM

#

I think so, I am going to leave an approval and ask someone to squash the commits

quiet gull Jan 20, 2021, 8:35 PM

#

do i have to create a new pr because it was disapproved and didnt passed the cases.

green oriole Jan 20, 2021, 8:35 PM

#

No no, you are all good

#

The check didn't pass because I asked for change

quiet gull Jan 20, 2021, 8:35 PM

#

ok thank you for helping me in this

green oriole Jan 20, 2021, 8:36 PM

#

Sure thing

#

If you wish to contribute to our repos again, can you please have a read over these documents https://chris.beams.io/posts/git-commit/ ?

#

!pep8

stable mountainBOT Jan 20, 2021, 8:36 PM

#

PEP 8 is the official style guide for Python. It includes comprehensive guidelines for code formatting, variable naming, and making your code easy to read. Professional Python developers are usually required to follow the guidelines, and will often use code-linters like flake8 to verify that the code they're writing complies with the style guide.

You can find the PEP 8 document here.

quiet gull Jan 20, 2021, 8:37 PM

#

sure i will check it out

green oriole Jan 20, 2021, 8:37 PM

#

Thank you

#

@gritty wind do you mind re-reviewing this?

vocal wolf Jan 20, 2021, 8:45 PM

#

@green oriole lol I'll review this

#

also squash/merging is apparently not enabled in sir-lancebot

gritty wind Jan 20, 2021, 8:52 PM

#

Hey @quiet gull it isn't mandatory, but have you considered adding this part too?

Additionally, you can add a try/except around the input section, to report invalid input and stop the bot from failing the game.```

quiet gull Jan 20, 2021, 9:01 PM

#

@gritty wind can i do this like-

#

try:
return grid[number-1][index]
except:
return "invalid input"

gritty wind Jan 20, 2021, 9:06 PM

#

Preferably you'd wrap it around the entire code that could generate an error, which is pretty much the whole function.

If you put it around line 262, you can wrap the whole function. Also, shouldn't be there, you should send a message to the player (you can see an example on line 247 and 248).

#

You can also see how to structure your try/except/else in that code block so you can ask for input again if the input is invalid

gritty wind Jan 20, 2021, 9:25 PM

#

If you're not interested in implementing this suggestion, feel free to say so and we'll get your contribution merged 😄

tawdry vapor Jan 20, 2021, 11:48 PM

#

Still looking for reviews on these two https://github.com/python-discord/bot/pull/1346 https://github.com/python-discord/snekbox/pull/88

GitHub

Fix Race Conditions in Help Channels by MarkKoz · Pull Request #134...

The following race conditions were fixed:

User claiming a second channel while their first channel is in the middle of being claimed.
Multiple people claiming the same channel (this was accounted ...

GitHub

Use protobuf to parse the nsjail config and remove redundancies by ...

Fixes #74
Protobuf is used to parse the config because it's the proper way to do it and it's less maintenance than a custom parser. A Python module is generated from the protoc tool...

cold island Jan 21, 2021, 12:48 AM

#

A nice issue to familiarise oneself with the bot's code:
https://github.com/python-discord/bot/issues/1373

GitHub

Command suggester wrongly invoked for commands under cooldown · Iss...

When a command is under cooldown and is invoked again, the error handler will wrongly try to suggest an alternative as if the command does not exist, when it should not do anything at all.

short snow Jan 21, 2021, 4:31 AM

#

@gritty wind

📎 Screenshot_from_2021-01-21_09-55-39.png

#

📎 Screenshot_from_2021-01-21_09-55-28.png

#

should i ask them, which tokens would they want?

#

like react to 🟠 🔵 🟢 🔴 🟡 etc.?

gritty wind Jan 21, 2021, 4:46 AM

#

short snow

Nice

gritty wind Jan 21, 2021, 4:46 AM

#

short snow should i ask them, which tokens would they want?

Hmm, sure go for it

short snow Jan 21, 2021, 4:49 AM

#

also, i didn't understand your last review comment

gritty wind Jan 21, 2021, 4:50 AM

#

The one about returning?

short snow Jan 21, 2021, 4:50 AM

#

gritty wind Hmm, sure go for it

well if that would be good or not, since it would make the game longer and add another 30sec check per user

#

If you return at this point, you can simplify a large portion of this function. You'd be able to remove all the code that checks for a full column, and all the code that tries to break out of the loop, and the return at the end.

#

this one

gritty wind Jan 21, 2021, 4:52 AM

#

short snow well if that would be good or not, since it would make the game longer and add a...

Honestly that isn't even my main concern, instead it just feels like needless complexity, and I'm not sure how much of a demand there is for it

#

Use your better judgment here, I don't think its necessary, but if you want to do it, go for it

short snow Jan 21, 2021, 4:52 AM

#

nvm, would just leave that out

#

and how about using ducks instead of the tokens

gritty wind Jan 21, 2021, 4:53 AM

#

         for row_num, square in reversed(list(enumerate(column))):
                    if not square:
                        self.grid[row_num][column_num] = player_num
                        coords = row_num, column_num
                        break
                else:
                    await self.channel.send(f"Column {column_num + 1} is full. Try again")
                    full_column = True
            if not full_column:
                break
        return coords

short snow Jan 21, 2021, 4:53 AM

#

ducky_wizard and ducky_lemon

gritty wind Jan 21, 2021, 4:53 AM

#

So here is what I'm thinking

#

There are only 2 things that happen here, you either return coords, or you loop in the while loop again

#

These four lines

full_column = True
            if not full_column:
                break
        return coords

Are not really necessary

#

(As well as the break)

#

You can cut it down to:

         for row_num, square in reversed(list(enumerate(column))):
             if not square:
                 self.grid[row_num][column_num] = player_num
                 return row_num, column_num
         await self.channel.send(f"Column {column_num + 1} is full. Try again")

gritty wind Jan 21, 2021, 4:57 AM

#

short snow and how about using ducks instead of the tokens

Can you take a screenshot with that?

short snow Jan 21, 2021, 4:57 AM

#

ok

#

pycharm takes ages to load, so wait for sometime

gritty wind Jan 21, 2021, 4:58 AM

#

Take your time lol

short snow Jan 21, 2021, 5:18 AM

#

gritty wind Take your time lol

📎 Screenshot_from_2021-01-21_10-47-52.png

gritty wind Jan 21, 2021, 5:19 AM

#

I think that works, but maybe should be optional

#

It can be part of the command

short snow Jan 21, 2021, 5:22 AM

#

theme?

gritty wind Jan 21, 2021, 5:22 AM

#

You can call it that

short snow Jan 21, 2021, 5:24 AM

#

fruits - 🥒 🍉

lemons - lemon_s_autumn lemon_s_winter

ducks - ducky_wizard ducky_lemon

python - pydis_hacktoberfest_2020 python

gritty wind Jan 21, 2021, 5:24 AM

#

What if you just take two optional emojis in the command?

#

That way you can pick your tokens, without adding any time to the game

short snow Jan 21, 2021, 5:27 AM

#

what they put nsfw emjois? or emojis that the opponent won't like?

gritty wind Jan 21, 2021, 5:28 AM

#

On the second part, that one feels a bit irrelevant, as the token isn't too big a deal, and if it bothers you that much, you just don't join the game

#

On the first part, can't the bot only send default emojis, and emojis from this server anyways?

short snow Jan 21, 2021, 5:28 AM

#

hmm yeah

#

🖕 what about this?

gritty wind Jan 21, 2021, 5:29 AM

#

If someone wants to be childish, let them

gritty wind Jan 21, 2021, 5:29 AM

#

gritty wind On the second part, that one feels a bit irrelevant, as the token isn't too big ...

And I think this also hits some of the same points as ^

short snow Jan 21, 2021, 5:30 AM

#

lol ok

#

should board come before or emojis?

#

in the arguments order

gritty wind Jan 21, 2021, 5:34 AM

#

board size then emojis

#

I think it's easier to enter a number to get to the emojis, than it is to enter two emojis to get to a number

short snow Jan 21, 2021, 5:36 AM

#

ok

short snow Jan 21, 2021, 6:24 AM

#

the only problem: non server emojis

📎 Screenshot_from_2021-01-21_11-53-17.png

gritty wind Jan 21, 2021, 6:24 AM

#

Yup, we don't want non-server emojis anyways

#

First check if the bot has access to the emoji

short snow Jan 21, 2021, 6:24 AM

#

📎 Screenshot_from_2021-01-21_11-54-34.png

gritty wind Jan 21, 2021, 6:24 AM

#

And if doesn't send an error

short snow Jan 21, 2021, 6:24 AM

#

ok, how is that done?

#

lemme look up the docs one min

gritty wind Jan 21, 2021, 6:25 AM

#

https://discordpy.readthedocs.io/en/latest/api.html?highlight=emoji#discord.Emoji.available

short snow Jan 21, 2021, 6:25 AM

#

you were too fast, ty

gritty wind Jan 21, 2021, 6:25 AM

#

lol no worries, always feel free to ask

#

For whomever merges lance#564, make sure to squash the commits

gritty wind Jan 21, 2021, 7:26 AM

#

@tawdry vapor I'm looking over 1346. In terms of the channel getting closed at the same time as the dormant close, we were discussing the possibility of making the bot not close the channel if it detects typing (in #community-meta).

Would that be an alternative solution, and if so, wouldn't it be a simpler implementation?

tawdry vapor Jan 21, 2021, 7:30 AM

#

No it just leads to a bigger 3-way race condition

#

The close task will be scheduled regardless so the typing event would be racing to cancel it

gritty wind Jan 21, 2021, 7:32 AM

#

Right, but you don't have to cancel the scheduled event, you can add the check in the close function, and reschedule if it needs to

tawdry vapor Jan 21, 2021, 7:33 AM

#

That still has the exact same race conditions currently present

gritty wind Jan 21, 2021, 7:35 AM

#

Fair enough then

short snow Jan 21, 2021, 8:45 AM

#

@gritty wind changes pushed

#

and changes mentioned

#

also, we could link them in the footer (The open trivia db)

gritty wind Jan 21, 2021, 8:53 AM

#

short snow <@!386599446571384843> changes pushed

Alright, thanks. I'll give it another look in a bit

#

Don't worry about those for now, its just a reference for later. I personally don't understand much about licencing myself so

short snow Jan 21, 2021, 9:06 AM

#

aight!

short snow Jan 21, 2021, 11:46 AM

#

to open issues on features, are we supposed to tell them here, or we can directly open issues?

#

or to make some commands, more better

patent pivot Jan 21, 2021, 11:46 AM

#

feel free to open issues, it's nice to discusss on github

short snow Jan 21, 2021, 11:48 AM

#

ok.

#

was thinking of:

displaying summary of the options of wikipedia command.
hangman game.

gritty wind Jan 21, 2021, 11:49 AM

#

Those would be two separate issues, correct?

short snow Jan 21, 2021, 11:49 AM

#

yeah

gritty wind Jan 21, 2021, 11:50 AM

#

What are the options for the wikipedia command?

short snow Jan 21, 2021, 11:53 AM

#

gritty wind What are the options for the wikipedia command?

.wiki python

gritty wind Jan 21, 2021, 11:53 AM

#

.wiki python

dusky shoreBOT Jan 21, 2021, 11:53 AM

#

Wikipedia results for `python`

1 Python (programming language)
2 Monty Python
3 Burmese python
4 PYTHON
5 Ball python
6 Reticulated python
7 History of Python
8 Monty Python and the Holy Grail
9 Python (genus)

#

Enter number to choose

short snow Jan 21, 2021, 11:54 AM

#

these options

#

1-9

gritty wind Jan 21, 2021, 11:54 AM

#

Oh like the results

dusky shoreBOT Jan 21, 2021, 11:54 AM

#

Sorry, but you cannot do that, I will only accept an positive integer, You have 2/3 chances left

short snow Jan 21, 2021, 11:54 AM

#

using paginator

gritty wind Jan 21, 2021, 11:54 AM

#

Okay, go for it

dusky shoreBOT Jan 21, 2021, 11:54 AM

#

Sorry, but you cannot do that, I will only accept an positive integer, You have 1/3 chances left

gritty wind Jan 21, 2021, 11:54 AM

#

how do I exit this

dusky shoreBOT Jan 21, 2021, 11:54 AM

#

Sorry, but you cannot do that, I will only accept an positive integer, Please try again by using .wiki command

gritty wind Jan 21, 2021, 11:54 AM

#

ok its done

short snow Jan 21, 2021, 11:56 AM

#

and bump

#

lol

cold island Jan 21, 2021, 12:10 PM

#

Hmm, why would it be saying that?

📎 unknown.png

gritty wind Jan 21, 2021, 12:11 PM

#

Is it installed in your pipenv?

#

pipenv run pip list

cold island Jan 21, 2021, 12:11 PM

#

Well it's the bot env

gritty wind Jan 21, 2021, 12:12 PM

#

Hmm it should be available when run then

cold island Jan 21, 2021, 12:12 PM

#

I tried installing the precommit hook

gritty wind Jan 21, 2021, 12:12 PM

#

can you try manually running it pipenv run pre-commit

#

or is it pipenv run python -m pre-commit

#

one of those

cold island Jan 21, 2021, 12:12 PM

#

that is the command I used

brazen charm Jan 21, 2021, 12:12 PM

#

The first thing that came to mind was an unactivated env but the first line looks like it is; did you install the dev dependencies?

cold island Jan 21, 2021, 12:13 PM

#

I ran pipenv install

brazen charm Jan 21, 2021, 12:13 PM

#

I believe it only does normal packages by default, try sync/install with --dev

gritty wind Jan 21, 2021, 12:13 PM

#

cold island that is the command I used

Ah, in that case try following Numerlor's idea

cold island Jan 21, 2021, 12:14 PM

#

Ok, that worked. Thanks! @brazen charm @gritty wind

rose heron Jan 21, 2021, 5:13 PM

#

🇭 how can my bot react with this. stuck at emojies again

#

got em nvm

obsidian patio Jan 21, 2021, 5:14 PM

#

Hey! This channel is for discussing the Python Discord repos. Feel free to claim a help channel ( #❓｜how-to-get-help or #discord-bots) if you need help 😄

trim cradle Jan 21, 2021, 5:17 PM

#

Please review the #rules, especially 3 and 7, which pertain to following staff instructions and keeping channels on their respective topics. Thanks!

surreal venture Jan 22, 2021, 4:42 AM

#

Can anyone give a PR review for any pull request?

cold island Jan 22, 2021, 4:45 AM

#

yes

surreal venture Jan 22, 2021, 4:57 AM

#

Okay

short snow Jan 22, 2021, 7:40 AM

#

@eternal owl sure, will do that on the wikipedia, on mobile so no login to github.

eternal owl Jan 22, 2021, 8:00 AM

#

👍

short snow Jan 22, 2021, 9:12 AM

#

is there a issue open on the formating of the reddit webhook already?

subtle kraken Jan 22, 2021, 9:16 AM

#

whats wrong with the formatting ?

short snow Jan 22, 2021, 9:17 AM

#

#reddit message the bold

#

the one with Im_joesph and it breaks the formatting of the others then

green oriole Jan 22, 2021, 9:23 AM

#

Damn @brazen charm that was fast haha

#

I should have marked the PR as draft

#

We are looking into the size issue

brazen charm Jan 22, 2021, 9:26 AM

#

There is an issue for unescaped brackets in the post bodies, but solving that probably isn't going to be exactly trivial unless every link in them is escaped (and broken)

short snow Jan 22, 2021, 9:26 AM

#

hmmm k

brazen charm Jan 22, 2021, 9:28 AM

#

Normal styling Markdown should be simpler but it'll still need to handle escaped markdown etc. if it's done properly

brazen charm Jan 22, 2021, 10:00 AM

#

can we get https://github.com/python-discord/bot/pull/1346 merged?

GitHub

Fix Race Conditions in Help Channels by MarkKoz · Pull Request #134...

The following race conditions were fixed:

User claiming a second channel while their first channel is in the middle of being claimed.
Multiple people claiming the same channel (this was accounted ...

patent pivot Jan 22, 2021, 10:01 AM

#

yeppers

#

review from yourself and scale is good enough for me

brazen charm Jan 22, 2021, 10:09 AM

#

Thanks

gritty wind Jan 22, 2021, 11:11 AM

#

When was the JSON check removed from pre-commit?

#

I was looking at
https://github.com/python-discord/sir-lancebot/pull/562/files
And these should not be allowed to pass linting as is

short snow Jan 22, 2021, 12:27 PM

#

@Scaleios#8200 yeah even i did that, but if we keep the difficulty level to medium, then the questions which come out are relatively good

(sorry for ghost ping in media)

#

(for open trivia db)

late wolf Jan 22, 2021, 1:38 PM

#

Why no ping command for @dusky shore ?

patent pivot Jan 22, 2021, 1:39 PM

#

what's the purpose of it?

#

we have ping on @stable mountain to do a rough infra check, and even then most of our infra monitoring is done through other tools

late wolf Jan 22, 2021, 1:39 PM

#

infra check?

#

You use ping to do infraction check?

patent pivot Jan 22, 2021, 1:40 PM

#

infrastructure

short snow Jan 22, 2021, 1:40 PM

#

how about putting Loading... message before .spookify and such commands since they take time, and when they are done, the msg are just edited

patent pivot Jan 22, 2021, 1:40 PM

#

!ping

stable mountainBOT Jan 22, 2021, 1:40 PM

#

Pong!

Command processing time

88.663 ms

Python Discord website latency

1.183 ms

Discord API latency

116.843 ms

short snow Jan 22, 2021, 1:41 PM

#

so atleast the user knows about that

cold moon Jan 22, 2021, 1:41 PM

#

@dusky shore Starts typing

patent pivot Jan 22, 2021, 1:41 PM

#

short snow how about putting ` Loading...` message before `.spookify` and such commands sin...

that's where we use typing indicators

#

wew ks is too fast

short snow Jan 22, 2021, 1:41 PM

#

but, sometimes it stops too

#

like it just happened for ks123's

late wolf Jan 22, 2021, 1:41 PM

#

It didn't stop for me

#

¯_(ツ)_/¯

cold moon Jan 22, 2021, 1:41 PM

#

Yes, this takes very long time to generate images

short snow Jan 22, 2021, 1:42 PM

#

sometimes

vale ibex Jan 22, 2021, 2:45 PM

#

I've been looking at constants.py in sir lancebot and am wondering, what are the benefits from inheriting from NamedTuple?

#

Is it to make the class immutable?

late wolf Jan 22, 2021, 2:51 PM

#

Its the same as collections.namedtuple

hardy gorge Jan 22, 2021, 3:01 PM

#

vale ibex I've been looking at constants.py in sir lancebot and am wondering, what are the...

You create an immutable class that you can type hint, yeah

#

It's a pretty common practice for something like that

#

You also have typed dicts in the typing module

vale ibex Jan 22, 2021, 3:05 PM

#

Makes sense, thanks 🙂

green oriole Jan 22, 2021, 7:21 PM

#

Hey @pulsar salmon!

#

Our current setup should be working without any issue but here is the issue

#

We currently have around 27k infractions in our database, and if we try to search for something like " ", it will try to send back 15k or so infractions, and it won't work

#

Well, it would technically work but it will run into physical limitations first

#

So we need to make sure that we won't send more than 50 infraction back

#

If you want to test this, you can create 6 dummy infractions and try to limit the query to 5 infractions, and test that

pulsar salmon Jan 22, 2021, 7:33 PM

#

Thanks @green oriole I will try doing that.

short snow Jan 22, 2021, 11:35 PM

#

@gritty wind since the discord emoji check doesn't work for default emojis, i would need to implement it on my own which is a 4000 character regex

#

so should i still do the part or skip it?

gritty wind Jan 22, 2021, 11:46 PM

#

The easier solution would just be to union it to discord.emoji, and one letter strings

#

Since that's what all emojis will fall into

#

If it's > 1 character, you can fail the conversion, and if it is 1 character, but like a letter or whatever, it isn't a big deal

#

(I'm also seeing a bunch of much shorter regexs, but this isn't such a crucial feature to warrant getting into that)

short snow Jan 22, 2021, 11:49 PM

#

hmmm, lemme try that

#

didn't think of that

short snow Jan 23, 2021, 12:06 AM

#

@gritty wind code pushed, the earlier one had problems in the emoji check

tawdry vapor Jan 23, 2021, 4:29 AM

#

This needs a review please https://github.com/python-discord/snekbox/pull/88

GitHub

Use protobuf to parse the nsjail config and remove redundancies by ...

Fixes #74
Protobuf is used to parse the config because it's the proper way to do it and it's less maintenance than a custom parser. A Python module is generated from the protoc tool...

short snow Jan 23, 2021, 8:13 AM

#

#sir-lancebot-playground message this should be taking the full name right?

short snow Jan 23, 2021, 9:28 AM

#

https://httpstatusdogs.com/

HTTP Status Dogs

HTTP Status Dogs. Hypertext Transfer Protocol Response status codes. And dogs.

#

should we add this to http_cat?

#

like:

#

.http cat <error_code>
.http dog <error_code>

#

bump.

cold moon Jan 23, 2021, 9:35 AM

#

Dogs lemon_hyperpleased I'd really like to see .http dog command.

short snow Jan 23, 2021, 9:37 AM

#

the problem it is not included in python library, so we can use aiohttp and a regex to get them all

cold moon Jan 23, 2021, 9:40 AM

#

https://httpstatusdogs.com/img/200.jpg We can just change this status code

short snow Jan 23, 2021, 9:41 AM

#

am checking for available ones

#

like when it is not available we say, that wrong statuscode

#

.http_cat 999

#

#bot-commands message for this

#

we check in the list of correct ones right?

cold moon Jan 23, 2021, 9:42 AM

#

.source http_cat

dusky shoreBOT Jan 23, 2021, 9:42 AM

#

Command: http_cat

Sends an embed with an image of a cat, potraying the status code.

Source Code

Go to GitHub

cold moon Jan 23, 2021, 9:42 AM

#

We can use same strategy

short snow Jan 23, 2021, 9:42 AM

#

so similar, we would need to create our one list usng aiohttp, to get the codes

#

there are different from the cat ones

#

some are missing ig

#

should i open a issue on it?

#

or just make a pr

cold moon Jan 23, 2021, 9:59 AM

#

Sir Lancebot contributing guidelines requires issue

clever wraith Jan 23, 2021, 10:12 AM

#

@daring peak

#

hi

daring peak Jan 23, 2021, 10:12 AM

#

this is not where I meant

#

I mean help channel

clever wraith Jan 23, 2021, 10:12 AM

#

oh

daring peak Jan 23, 2021, 10:12 AM

#

#❓｜how-to-get-help

eternal owl Jan 23, 2021, 10:20 AM

#

https://github.com/python-discord/site/issues/391
is this still a go?

GitHub

Get rid of subdomains in our site · Issue #391 · python-discord/site

There is no reason we should use subdomains in our app. Subdomains are a complexity that makes authentication management much more difficult, because OAuth across subdomains does not work well. Spe...

patent pivot Jan 23, 2021, 10:47 AM

#

Hmmmm

#

Potentially not, since were splitting out the features that need subdomains into different apps

#

staff subdomain => logcord
api subdomain => separated API

gritty wind Jan 23, 2021, 10:48 AM

#

What about the new apps using sub domains? Forms for instance

#

I’m not aware of any using the site’s auth though

patent pivot Jan 23, 2021, 11:03 AM

#

gritty wind What about the new apps using sub domains? Forms for instance

that specific issue is just for removing site subdomains, yeah

#

we use a tonne of subdomains internally which are fine

#

but like, we don't want api.pythondiscord.com, pythondiscord.com and staff.pythondiscord.com to all be handled by the same app

#

because that's chaos

#

especially for local dev setups

short snow Jan 23, 2021, 11:23 AM

#

weird all my cogs get loaded but the bot is offline

#

zeero errors

#

nah was my net problems

short snow Jan 23, 2021, 11:46 AM

#

#discord-bots message hmm need some help in this, dunno why it happens

gritty wind Jan 23, 2021, 11:52 AM

#

What are you doing when it’s raised?

short snow Jan 23, 2021, 11:53 AM

#

nvm, found it, was using the for loop variable instead of the list to which the results were appended

short snow Jan 23, 2021, 12:16 PM

#

Scaleios can u have a look at my pr when u are free of c4

gritty wind Jan 23, 2021, 12:20 PM

#

Yeah, I’m planning to get on that soon

daring peak Jan 23, 2021, 2:07 PM

#

hello fellow pythonistas

#

In #sir-lancebot-playground the command .scarymovie isnt working

#

please check it out yourself and see

#

should I open an issue in github? or wait until a response is given to the above messages?

gritty wind Jan 23, 2021, 2:15 PM

#

Its in the halloween cog, so I'm assuming it just isn't loaded right now

balmy sparrow Jan 23, 2021, 2:24 PM

#

gritty wind Its in the halloween cog, so I'm assuming it just isn't loaded right now

it seems to trigger typing, and it's help command works, I don't think that can happen if the cog is unloaded.

gritty wind Jan 23, 2021, 2:29 PM

#

Hmm, that's true

#

I can't test if there's anything broken with it because I don't have an API key, perhaps someone in devops can check sentry

daring peak Jan 23, 2021, 3:47 PM

#

balmy sparrow it seems to trigger typing, and it's help command works, I don't think that can ...

yea its shown that "sir lance bot is typing..." and then after 2 secs it just disappears

cold island Jan 23, 2021, 3:50 PM

#

the halloween cog only executes commands in October iirc

gritty wind Jan 23, 2021, 3:52 PM

#

That's what I thought at first, but the cog is loaded, and there don't appear to be any seasonal checks in the scarymovie file

cold island Jan 23, 2021, 4:11 PM

#

hmm ok so it's just the reactions that are limited to October

#

we don't seem to do logging for lancebot, so it'll have to be debugged

eternal owl Jan 23, 2021, 4:14 PM

#

@gritty wind regarding the quiz questions, do you think it will be a good idea to use Open Trivia DB ?

gritty wind Jan 23, 2021, 4:28 PM

#

eternal owl <@!386599446571384843> regarding the quiz questions, do you think it will be a g...

To do so would be to trust the moderation of the people running it, and I can't make that call on my own

pulsar salmon Jan 23, 2021, 5:56 PM

#

Hey, does anyone know how to implement LimitOffsetPagination to limit the number of queries returned? I am a beginner and trying to contribute to resolve an issue. I feel like I should use the paginate_queryset method but I am feeling abit confused navigating through the viewset code. Any advice/ suggestion will be great. Thanks

sharp timber Jan 23, 2021, 9:42 PM

#

On GenericAPIView subclasses you may also set the pagination_class attribute to select LimitOffsetPagination on a per-view basis.
seems relevant from a cursory look

#

from here: https://www.django-rest-framework.org/api-guide/pagination/#limitoffsetpagination

late wolf Jan 24, 2021, 6:48 AM

#

For the internal cog, self.interpreter is not being used, so why is it there?

exotic ember Jan 24, 2021, 6:53 AM

#

most likely a missed line during a refactor

late wolf Jan 24, 2021, 6:58 AM

#

why do we even have helper interpreter?

exotic ember Jan 24, 2021, 7:05 AM

#

not sure what you mean by that?

late wolf Jan 24, 2021, 7:08 AM

#

https://github.com/python-discord/bot/blob/abe439e801eac508cd1b4daa0b67dcd5c5ab549a/bot/interpreter.py#L15
Where are we even using this?

exotic ember Jan 24, 2021, 7:11 AM

#

we probably aren't

late wolf Jan 24, 2021, 7:12 AM

#

then why does it exist lol

exotic ember Jan 24, 2021, 7:19 AM

#

well, we probably were using it, then did a refactor that removed the need for it, but didn't get rid of it

#

dead code so to speak

late wolf Jan 24, 2021, 7:19 AM

#

can I open pr removing it?

exotic ember Jan 24, 2021, 7:23 AM

#

sure

short snow Jan 24, 2021, 7:24 AM

#

till when does a issue get approved/rejected?

brazen charm Jan 24, 2021, 7:25 AM

#

It'd probably be worth looking at making int e use it, the cog is a bit of a mess and abstracting the run logic away would help

gritty wind Jan 24, 2021, 7:27 AM

#

short snow till when does a issue get approved/rejected?

It'll depend on the issue, but usually things get approved once any discussion about their necessity or specs is out of the way

short snow Jan 24, 2021, 7:28 AM

#

ok

#

had about porting commandnotfound suggestions, status dogs, and improving wiki command

gritty wind Jan 24, 2021, 7:29 AM

#

If they both have issues open, they will be dealt with sooner or later
(cc @vocal wolf as our resident triager :D)

vocal wolf Jan 24, 2021, 7:30 AM

#

Sorry, I've been really busy lately, I'm getting around to issues/PRs though.

gritty wind Jan 24, 2021, 7:30 AM

#

Lol it wasn't a dig, those are pretty recent issues

vocal wolf Jan 24, 2021, 7:31 AM

#

I've been focusing on reviving really old issues/PRs

#

working backwards lol

#

I'll check out the recent ones tho

short snow Jan 24, 2021, 7:33 AM

#

ok np.

short snow Jan 24, 2021, 8:42 AM

#

@vocal wolf there is a typo in the dog status issue. it is
Since dogs have some codes missing, we will use
and not
Since dogs have some comes missing, we will use, will fix that

vocal wolf Jan 24, 2021, 8:42 AM

#

ok_handbutflipped

short snow Jan 24, 2021, 11:54 AM

#

does anyone have the regex for getting the message inside a single/block codeblock

#

need for the automatic linking issue on sir-lancebot

exotic ember Jan 24, 2021, 12:00 PM

#

!source snekbox

stable mountainBOT Jan 24, 2021, 12:00 PM

#

Bad argument

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

#

Command Help

!source [source_item]
Can also use: src

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

exotic ember Jan 24, 2021, 12:01 PM

#

is it not called that anymore? Anyways the source for !eval should have something like that

patent pivot Jan 24, 2021, 12:01 PM

#

!source eval

stable mountainBOT Jan 24, 2021, 12:01 PM

#

Command: eval

Run Python code and get the results.

Source Code

Go to GitHub

patent pivot Jan 24, 2021, 12:01 PM

#

or do you mean linking to the snekbox repo

exotic ember Jan 24, 2021, 12:02 PM

#

I wanted to link to the snekbox cog

#

thought that was possible

subtle kraken Jan 24, 2021, 12:03 PM

#

!src Snekbox

stable mountainBOT Jan 24, 2021, 12:03 PM

#

Cog: Snekbox

Safe evaluation of Python code using Snekbox.

Source Code

Go to GitHub

subtle kraken Jan 24, 2021, 12:03 PM

#

@exotic ember

#

case sensitive

exotic ember Jan 24, 2021, 12:03 PM

#

ah

vocal wolf Jan 24, 2021, 12:25 PM

#

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

#

In need of reviews

vocal wolf Jan 24, 2021, 12:42 PM

#

Another one in need of reviewing:

#

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

rapid swallow Jan 24, 2021, 12:59 PM

#

vocal wolf https://github.com/python-discord/sir-lancebot/pull/415

Would it be possible to support searching for comics by name?

vocal wolf Jan 24, 2021, 12:59 PM

#

No.

rapid swallow Jan 24, 2021, 1:00 PM

#

no, like, it would be a cool feature

#

so can you try to implement it?

vocal wolf Jan 24, 2021, 1:00 PM

#

Well yes, it would. But it's not actually possible.

rapid swallow Jan 24, 2021, 1:00 PM

#

yeah...

vocal wolf Jan 24, 2021, 1:00 PM

#

Without like recording the names and their respective numbers

rapid swallow Jan 24, 2021, 1:00 PM

#

since no one even knows the real name of the strips

vale ibex Jan 24, 2021, 1:01 PM

#

yea, we'd need to store a reference to each comic to allow that, since xkcd haven't implemented a search

rapid swallow Jan 24, 2021, 1:01 PM

#

We might be able to get that information from some website

vale ibex Jan 24, 2021, 1:01 PM

#

Also, I'm not sure if XKCD titles are unique?

vocal wolf Jan 24, 2021, 1:01 PM

#

I think it would be just easier to go by number. That's what we usually reference them by.

rapid swallow Jan 24, 2021, 1:02 PM

#

vale ibex Also, I'm not sure if XKCD titles are unique?

I think they are unique, but not everyone knows their actual names

vale ibex Jan 24, 2021, 1:02 PM

#

There is this though https://github.com/adtac/relevant-xkcd

GitHub

adtac/relevant-xkcd

There's always a relevant XKCD. Contribute to adtac/relevant-xkcd development by creating an account on GitHub.

#

which might be cool to implement

#

So rather than a title, it takes a concept

vocal wolf Jan 24, 2021, 1:03 PM

#

vale ibex There is this though https://github.com/adtac/relevant-xkcd

I feel like that would be unnecessary for one command.

vale ibex Jan 24, 2021, 1:04 PM

#

oh yea for sure

rapid swallow Jan 24, 2021, 1:04 PM

#

vale ibex So rather than a title, it takes a concept

this would be super cool :0

vocal wolf Jan 24, 2021, 1:05 PM

#

@rapid swallow If they do implement a search feature, I'll be sure to put that into the bot.

rapid swallow Jan 24, 2021, 1:05 PM

#

alright 👍

vocal wolf Jan 24, 2021, 1:06 PM

#

ok_handbutflipped

#

@vale ibex If you don't mind, you can re-review and hit approve

late wolf Jan 24, 2021, 1:07 PM

#

the current server command only shows online and offline members but this
https://github.com/python-discord/bot/pull/1166 pr, shows all online, idle, offline and dnd.
Do we really want to show all of those instead of showing only online and offline?

vocal wolf Jan 24, 2021, 1:08 PM

#

!server

stable mountainBOT Jan 24, 2021, 1:08 PM

#

Server information
Created: 4 years and 16 days ago
Voice region: europe
Features: BANNER, COMMUNITY, DISCOVERABLE, PARTNERED, VIP_REGIONS, WELCOME_SCREEN_ENABLED, VANITY_URL, MEMBER_VERIFICATION_GATE_ENABLED, PREVIEW_ENABLED, INVITE_SPLASH, ANIMATED_ICON, RELAY_ENABLED, NEWS

Channel counts
Category channels: 26
News channels: 8
Text channels: 154
Voice channels: 15
Staff channels: 61

Member counts
Members: 136,870
Staff members: 88
Roles: 86

Member statuses
status_online 32,583
status_offline 104,290

brazen charm Jan 24, 2021, 1:08 PM

#

The screenshots are outdated

vocal wolf Jan 24, 2021, 1:09 PM

#

@late wolf

📎 unknown.png

#

here's where the command is currently at

late wolf Jan 24, 2021, 1:09 PM

#

right

vocal wolf Jan 24, 2021, 1:09 PM

#

@vale ibex ty ❤️

rapid swallow Jan 24, 2021, 1:10 PM

#

I could nitpick

#

but uhhh

vocal wolf Jan 24, 2021, 2:38 PM

#

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

#

Another PR that needs reviewing

eternal owl Jan 24, 2021, 2:40 PM

#

I will take a look

cold moon Jan 24, 2021, 2:52 PM

#

What discussion is going on in dev-voice currently?

vocal wolf Jan 24, 2021, 2:53 PM

#

cold moon What discussion is going on in dev-voice currently?

Was redis stuff, now it's just silence

eternal owl Jan 24, 2021, 2:53 PM

#

keyboard silence Derp_Slamming_Keyboard

#

do we want another reaction to cancel the game(call it a draw) before the game actually ends?

📎 unknown.png

#

if it is User vs AI, then only the user needs to react, if it is User vs User, both of them have to react

#

@cold moon

cold moon Jan 24, 2021, 3:06 PM

#

eternal owl do we want another reaction to cancel the game(call it a draw) before the game a...

Not sure is this necessary. Game will be cancelled after some time. This may be future improvement.

eternal owl Jan 24, 2021, 3:06 PM

#

yea, I think 120s is kinda too long? its not a game where we would spend that long to think lemon_pleased ?

cold moon Jan 24, 2021, 3:07 PM

#

Maybe then 60s?

eternal owl Jan 24, 2021, 3:08 PM

#

yea

#

even 30s should do I think

cold moon Jan 24, 2021, 3:08 PM

#

I make change now

eternal owl Jan 24, 2021, 3:08 PM

#

oki

#

I will make a comment just to keep track

cold moon Jan 24, 2021, 3:10 PM

#

Change made

eternal owl Jan 24, 2021, 3:11 PM

#

I haven't tested this with a second player tho

#

lemme do a quick test with a friend

late wolf Jan 24, 2021, 3:13 PM

#

What yall dicussing?

vocal wolf Jan 24, 2021, 3:15 PM

#

currently keyboards are being slapped

#

mostly silence

eternal owl Jan 24, 2021, 3:15 PM

#

approved @cold moon 👍

vocal wolf Jan 24, 2021, 3:17 PM

#

@cold moon anything else to commit? If not, we can merge.

cold moon Jan 24, 2021, 3:19 PM

#

vocal wolf <@!446310592425361418> anything else to commit? If not, we can merge.

Ready for merge

vocal wolf Jan 24, 2021, 3:21 PM

#

alright

short snow Jan 24, 2021, 3:34 PM

#

Should there be ability to choose your tokens on the ttt command.

#

In dev-voice we can’t join but in media we can, hmmm

#

Y so?

eternal owl Jan 24, 2021, 3:35 PM

#

short snow Should there be ability to choose your tokens on the ttt command.

so the player who invokes the command gets to play first

#

they are treated as player 1

eternal owl Jan 24, 2021, 3:36 PM

#

short snow Y so?

dev-voice is restricted to contribs

#

the ppl with the contributors role

late wolf Jan 24, 2021, 3:55 PM

#

oh @Scaleios#8200 became a core dev but it wasn't stated in #dev-announcements

late wolf Jan 24, 2021, 4:20 PM

#

Hey why aren't we able to exit out of .ttt?

green oriole Jan 24, 2021, 4:23 PM

#

What do you mean? Cancel the game?

late wolf Jan 24, 2021, 4:24 PM

#

No cancel the game

gritty wind Jan 24, 2021, 4:25 PM

#

cold moon Not sure is this necessary. Game will be cancelled after some time. This may be ...

It was a possible suggestion. Do you want to give it a shot?

green oriole Jan 24, 2021, 4:25 PM

#

Hmm

#

Why not

eternal owl Jan 24, 2021, 4:42 PM

#

I can make an issue if you are interested on working on it

#

#dev-contrib message

#

@late wolf

hallow venture Jan 24, 2021, 5:10 PM

#

Hey

signal hedge Jan 24, 2021, 6:24 PM

#

`@client.event
async def on_member_join(member):
await client.get_channel('#795719602691637319').send(f"{member.name} has joined")

@client.event
async def on_member_remove(member):
await client.get_channel(795719602691637319).send(f"{member.name} has left")
`

#

can i use this

#

for simple welcome message

#

?

gritty wind Jan 24, 2021, 6:25 PM

#

Hey if you'd like help with your own discord bot, check out #❓｜how-to-get-help or #discord-bots

#

This channel is for pythondiscord projects

signal hedge Jan 24, 2021, 6:25 PM

#

oke

short snow Jan 24, 2021, 9:25 PM

#

eternal owl so the player who invokes the command gets to play first

That’s not what i mean, in the tt command, the tokens are default to x and o, how about making them be choosed by player (optional) for example: .ttt ducky_yellow ducky_wizard

late wolf Jan 25, 2021, 2:37 AM

#

eternal owl <@!707472976454483988>

Until vsc decides to co-operate and change the goddamn font, i can't work on anything.
So, I can work on it when vsc decides to co-operate if someone else hasn't already.

late wolf Jan 25, 2021, 3:22 AM

#

Hey, can I get the table creation query for the infraction table?

eternal owl Jan 25, 2021, 4:03 AM

#

https://github.com/python-discord/site/blob/master/pydis_site/apps/api/models/bot/infraction.py
here is the model @late wolf

GitHub

python-discord/site

pythondiscord.com - A Django and Bulma web application. - python-discord/site

late wolf Jan 25, 2021, 4:04 AM

#

thankyou

#

but i was asking for the table structure

eternal owl Jan 25, 2021, 4:05 AM

#

you can make out from the django model, right?

eternal owl Jan 25, 2021, 4:05 AM

#

late wolf Until vsc decides to co-operate and change the goddamn font, i can't work on any...

whats wrong with vscode, lol

eternal owl Jan 25, 2021, 4:06 AM

#

short snow That’s not what i mean, in the tt command, the tokens are default to x and o, ho...

I think keeping the default is nice, but I guess having the X and O be a different color from the numbers should make it better visually

late wolf Jan 25, 2021, 4:07 AM

#

eternal owl whats wrong with vscode, lol

Oh just got it fixed, i downloaded the font but didn't install it and i was getting mad at vsc for not changing the font, lemon_sweat

late wolf Jan 25, 2021, 4:07 AM

#

eternal owl whats wrong with vscode, lol

cant

eternal owl Jan 25, 2021, 4:09 AM

#

just reinstall vscode and delete all the prev configs

late wolf Jan 25, 2021, 4:11 AM

#

eternal owl https://github.com/python-discord/site/blob/master/pydis_site/apps/api/models/bo...

I can't figure out the table structure from this

late wolf Jan 25, 2021, 5:10 AM

#

Hey i never requested review from @ks123#9814 but why did github automatically do that?

gritty wind Jan 25, 2021, 5:10 AM

#

Codeowners

#

He was a codeowner of a file you edited, so it requested him

late wolf Jan 25, 2021, 5:10 AM

#

ooh

#

oh forgot to remove the import

gritty wind Jan 25, 2021, 5:14 AM

#

late wolf I can't figure out the table structure from this

This is why I use docker for all the DBs but

#

-- auto-generated definition
create table api_infraction
(
    id          serial                   not null
        constraint api_infraction_pkey
            primary key,
    inserted_at timestamp with time zone not null,
    expires_at  timestamp with time zone,
    active      boolean                  not null,
    type        varchar(9)               not null,
    reason      text,
    hidden      boolean                  not null,
    actor_id    bigint                   not null
        constraint api_infraction_actor_id_ed96dde9_fk_api_user_id
            references api_user
            deferrable initially deferred,
    user_id     bigint                   not null
        constraint api_infraction_user_id_8714e9f5_fk_api_user_id
            references api_user
            deferrable initially deferred
);

alter table api_infraction
    owner to pysite;

create index api_infraction_actor_id_ed96dde9
    on api_infraction (actor_id);

create index api_infraction_user_id_8714e9f5
    on api_infraction (user_id);

create unique index unique_active_infraction_per_type_per_user
    on api_infraction (user_id, type)
    where (active = true);```

late wolf Jan 25, 2021, 5:14 AM

#

thx

#

you should have pinged me on that

short snow Jan 25, 2021, 6:20 AM

#

still left to go through the full pr m_cht, but yeah did some part of it.

late wolf Jan 25, 2021, 9:27 AM

#

So before when we did !help tempban we didn't get a response back, but rn we do.
Why was this changed?

subtle kraken Jan 25, 2021, 9:28 AM

#

iirc previously if you couldn't run a command, it simply wouldn't give you help
now that info is in the help embed (that you can't run the command)

#

it is how help system works in d.py as well, if you can't run a command chances are you don't need to know it exists

cold island Jan 25, 2021, 10:17 AM

#

That's how it was before, then it just didn't give the help embed, and now it's back to the previous behavior

short snow Jan 25, 2021, 10:54 AM

#

#python-discussion message shouldn't !server command be whitelisted to bot commands and ot,meta only?

late wolf Jan 25, 2021, 11:17 AM

#

oh why doesn't help(list.append) work in snekbox but works in repl??

📎 unknown.png

green oriole Jan 25, 2021, 11:18 AM

#

Right, I think it is because it is part of the site module, which isn’t loaded by snekbox

#

!e

import site
site.main()

help(list.append)```

stable mountainBOT Jan 25, 2021, 11:19 AM

#

@green oriole :white_check_mark: Your eval job has completed with return code 0.

001 | Help on method_descriptor:
002 | 
003 | append(self, object, /)
004 |     Append object to the end of the list.

green oriole Jan 25, 2021, 11:19 AM

#

There you go

late wolf Jan 25, 2021, 11:19 AM

#

oooh so we need to add site to snekbox allowed packages??

green oriole Jan 25, 2021, 11:22 AM

#

We use the -S option to not load it, so I assume that there’s a reason for us to not load it

#

There are some details here https://github.com/python-discord/snekbox/commit/dd307c793588cdbfb5f759a91f337299a842f05d

balmy sparrow Jan 25, 2021, 12:31 PM

#

@exotic ember has requested changes on https://github.com/python-discord/bot/pull/1372 , but the ensuing discussion seems to suggest that I don't have to be making any changes. Should I be doing something? Or have I misunderstood it, and I need to be making changes?

exotic ember Jan 25, 2021, 12:35 PM

#

@balmy sparrow I misread the function, your PR is fine. Give me a bit and I'll get it merged

balmy sparrow Jan 25, 2021, 12:40 PM

#

Cool, thanks :D

late wolf Jan 25, 2021, 12:46 PM

#

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

This is also ready to be merged

green oriole Jan 25, 2021, 12:47 PM

#

Merged!

late wolf Jan 25, 2021, 12:49 PM

#

Thx

late wolf Jan 25, 2021, 1:07 PM

#

What exactly is Sentry in terms of @stable mountains code?

cold moon Jan 25, 2021, 1:08 PM

#

Sentry is errors catching service

green oriole Jan 25, 2021, 1:08 PM

#

It is https://sentry.io !

Sentry

Application Monitoring and Error Tracking Software

Self-hosted and cloud-based error monitoring that helps software teams discover, triage, and prioritize errors in real-time.

late wolf Jan 25, 2021, 1:09 PM

#

thx, leme check it out

patent pivot Jan 25, 2021, 1:12 PM

#

good service

late wolf Jan 25, 2021, 1:14 PM

#

For our redirect_output decorater why do we not use commands.check() but make it accept the function?

late wolf Jan 25, 2021, 1:34 PM

#

Also, why do bot about rather than just about?
The goup has only command, why use group if we are only gonna have one command?

short snow Jan 25, 2021, 1:35 PM

#

and bot doesn't show up in the help command, why?

subtle kraken Jan 25, 2021, 1:37 PM

#

both of those commands are hidden
might be why

short snow Jan 25, 2021, 1:37 PM

#

but why can we use them then?

subtle kraken Jan 25, 2021, 1:37 PM

#

hidden != usable

green oriole Jan 25, 2021, 1:37 PM

#

They are pretty useless tbh

subtle kraken Jan 25, 2021, 1:37 PM

#

to be fair, I haven't heard of bot group before

#

I think its a subject to be deleted

green oriole Jan 25, 2021, 1:37 PM

#

We could add some informations here

#

But I would happily nuke it

late wolf Jan 25, 2021, 1:38 PM

#

subtle kraken both of those commands are hidden might be why

Right, why are they hidden?

late wolf Jan 25, 2021, 1:38 PM

#

green oriole But I would happily nuke it

Maybe instead of using group we could use subcommands?

subtle kraken Jan 25, 2021, 1:38 PM

#

as I just said, I haven't even known of their existence so its hard for me to tell

late wolf Jan 25, 2021, 1:38 PM

#

or just make it about

subtle kraken Jan 25, 2021, 1:38 PM

#

group is subcommand system

subtle kraken Jan 25, 2021, 1:39 PM

#

green oriole We could add some informations here

can't imagine what kind of info could go there to be fair

late wolf Jan 25, 2021, 1:39 PM

#

subtle kraken `group` is subcommand system

But it supports multiple subcommands, whereas subcommand only supports one IIRC

subtle kraken Jan 25, 2021, 1:39 PM

#

theres no difference

#

there is only one subcommand system in d.py

#

which is group

#

subcommand decorator means its a subcommand of group

#

group means its parent command and group of commands

late wolf Jan 25, 2021, 1:40 PM

#

oh right

#

i was hallucinating

subtle kraken Jan 25, 2021, 1:40 PM

#

group is bot subcommand is about

exotic ember Jan 25, 2021, 1:40 PM

#

@balmy sparrow while testing I found another potential issue, sorry for the false promise of a quick merge lemon_pensive

late wolf Jan 25, 2021, 1:41 PM

#

my b

balmy sparrow Jan 25, 2021, 2:06 PM

#

exotic ember <@!271586885346918400> while testing I found another potential issue, sorry for ...

haha that's alright, i'll check in a sec

late wolf Jan 25, 2021, 2:11 PM

#

Can I open a pr, making the about command not hidden and not making it a group

brazen charm Jan 25, 2021, 2:15 PM

#

I think should be discussed more in depth in an issue first, it's probably not necessary at all; I doubt it had a significant amount of uses recently

late wolf Jan 25, 2021, 2:15 PM

#

k

gritty wind Jan 25, 2021, 2:16 PM

#

You could open it, but if we’re discussing removing it entirely, it probably won’t be approved for a while, if at all

#

I’m on mobile, so I can’t search it atm

#

What cog is it?

late wolf Jan 25, 2021, 2:18 PM

#

!src Utils

stable mountainBOT Jan 25, 2021, 2:18 PM

#

Cog: Utils

A selection of utilities which don't have a clear category.

Source Code

Go to GitHub

late wolf Jan 25, 2021, 2:19 PM

#

gritty wind You could open it, but if we’re discussing removing it entirely, it probably won...

I will wait until it gets discussed then.

clever wraith Jan 25, 2021, 2:19 PM

#

Hey Guys , so i managed to run the docker

#

And the image is also running now

#

2021-01-25 14:17:18,964 |     9 |                 gunicorn.error |     INFO | Booting worker with pid: 9

#

Now what to do , i mean where is the snekbox code

gritty wind Jan 25, 2021, 2:21 PM

#

late wolf !src Utils

Thanks

gritty wind Jan 25, 2021, 2:22 PM

#

clever wraith Now what to do , i mean where is the snekbox code

If that’s the snekbox container you started, the http end point is documented in the Readme

clever wraith Jan 25, 2021, 2:22 PM

#

clever wraith And the image is also running now

After docker this https://github.com/python-discord/bot/blob/master/bot/exts/utils/snekbox.py ?

GitHub

python-discord/bot

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

#

it is running on http

gritty wind Jan 25, 2021, 2:23 PM

#

Wait if you started the docker compose of the bot, you should be able to interface with snekbox directly or through the bot

clever wraith Jan 25, 2021, 2:23 PM

#

gritty wind Wait if you started the docker compose of the bot, you should be able to interfa...

i started the snekbox server

#

https://github.com/python-discord/snekbox

GitHub

python-discord/snekbox

Easy, safe evaluation of arbitrary Python code. Contribute to python-discord/snekbox development by creating an account on GitHub.

#

This

gritty wind Jan 25, 2021, 2:24 PM

#

Ok, so what do you need to do now?

clever wraith Jan 25, 2021, 2:24 PM

#

clever wraith After docker this https://github.com/python-discord/bot/blob/master/bot/exts/uti...

I just want a eval bot @gritty wind , so this is the only required file ?

gritty wind Jan 25, 2021, 2:25 PM

#

Yeah, but you’d need to add a bot to load it and run

#

And if it has imports from the rest of the project, you’d need those too

clever wraith Jan 25, 2021, 2:25 PM

#

Ik discord.py , but the python bot code is so dependant

#

on other files

gritty wind Jan 25, 2021, 2:25 PM

#

The whitelist and channel stuff is needed, but it can be ripped out

#

Let me try to highlight the necessary parts

clever wraith Jan 25, 2021, 2:26 PM

#

Ohh kk , thnx in advance 🙂

gritty wind Jan 25, 2021, 2:31 PM

#

You’d need all the imports on lines 1 - 9 (12 if you want to reuse the discord components). 22-39 are input parsing so you’d need those too unless you want to write your own. 46-49. You’d want most of the class itself, but you should remove line 283 (maybe even 282), and you should set the URL of snekbox on line 61. You can keep everything after the class too

#

This is only the cog though, you’d need something to start the bot itself

late wolf Jan 25, 2021, 2:31 PM

#

Happen to ping me if you guys reach to a conclusion on what to do with the bot about command

green oriole Jan 25, 2021, 2:44 PM

#

subtle kraken can't imagine what kind of info could go there to be fair

Hmm. Maybe a GitHub link?

subtle kraken Jan 25, 2021, 2:44 PM

#

!src

stable mountainBOT Jan 25, 2021, 2:44 PM

#

Bot's GitHub Repository

Repository

Go to GitHub

subtle kraken Jan 25, 2021, 2:44 PM

#

duplicate of ^

green oriole Jan 25, 2021, 2:45 PM

#

Yeah

late wolf Jan 25, 2021, 2:48 PM

#

#dev-contrib message
this too

wintry trellis Jan 25, 2021, 6:25 PM

#

Hello!
With reference to https://pythondiscord.com/pages/contributing/bot/#using-docker, I believe Docker isn't mandatory, right?

Python Discord | Contributing to Bot

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

gritty wind Jan 25, 2021, 6:26 PM

#

Docker i not mandatory for any of the projects

#

That being said, it does make certain aspects easier

#

Like the databases, the site, snekbox, etc

#

But you can run everything on your host

brazen charm Jan 25, 2021, 6:27 PM

#

The bot is probably the project where it helps the least, but the site can be a pain to set up without docker depending on the os

green oriole Jan 25, 2021, 6:28 PM

#

I would honestly recommend you to use Docker for all of our projects, at least when possible

wintry trellis Jan 25, 2021, 6:31 PM

#

👍

wintry trellis Jan 26, 2021, 12:13 AM

#

Nice to meet you once again!
With reference to https://pythondiscord.com/pages/contributing/bot/#environment-variables, is it fine if we use only the BOT_TOKEN and BOT_API_KEY variables?
If so .env file will become:

BOT_TOKEN=token
BOT_API_KEY=key

Also, how to acquire the BOT_API_KEY?

Python Discord | Contributing to Bot

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

cold island Jan 26, 2021, 12:20 AM

#

wintry trellis Nice to meet you once again! With reference to https://pythondiscord.com/pages/c...

For most features you technically only need to specify the bot token

wintry trellis Jan 26, 2021, 12:21 AM

#

I see, thank you.

cold island Jan 26, 2021, 12:22 AM

#

that is assuming you're running the bot using docker

wintry trellis Jan 26, 2021, 12:25 AM

#

For now, I am running it locally

#

I assume I need to use all the variables then?

#

And that might be the reason for the cause of this error?

from async_rediscache import RedisSession
ModuleNotFoundError: No module named 'async_rediscache'

cold island Jan 26, 2021, 12:48 AM

#

looks like you just don't have the package installed.

What is your reason for starting up the bot?

sharp timber Jan 26, 2021, 8:17 AM

#

Oh this, I had this issue

#

There's something wrong in the dependency install step, I think, in the docs, because I got around this I believe by disabling the included redis setup. That, or there's something I missed, which is very likely

green oriole Jan 26, 2021, 8:18 AM

#

Hmmm

#

Do you use Docker ?

sharp timber Jan 26, 2021, 8:18 AM

#

I was

#

docker-compose up was throwing this

green oriole Jan 26, 2021, 8:18 AM

#

Hmm

sharp timber Jan 26, 2021, 8:18 AM

#

I can restart into my linux if you're free to help troubleshoot?

green oriole Jan 26, 2021, 8:19 AM

#

Sure thing

sharp timber Jan 26, 2021, 8:20 AM

#

kk, here goes nothing

#

alright, syncing up my system and its a slight bit of a mess so ~10m

sharp timber Jan 26, 2021, 8:43 AM

#

resetting back to the original state

#

Currently getting an libffi.so error

#

just isntalled updates, so gonna reboot, see if I can force upgrade my python to 3.9 and try again

#

python's at 3.9 now so the reboot changed osmething, but still broken on libffi, so forcing a reinstall via pyenv

#

Well, this sucks

#

Why does pip install --upgrade pipenv leave me at pipenv 2018.*.* pithink

#

conflicting pipenv versions, fixing

#

Ok, I had to manually purge a version fo pipenv I don't know how got on my system >.> but its settup up now

#

Fighting virtualenv again ;-;

#

Why does it keep calling into python 3.8. My python 3.8 doesn't work.

#

Ok, after manually navigating into ~/.local and purging the broken virtual environments it seems to be working

#

reinstalling dependencies with pipenv sync --dev

#

Alright, I'm not getting the async redis error anymore for the moment, but I am throwing a "Connect call failed: cannot assign requested address" @green oriole if you have some insight here

#

I threw up a use_fakeredis last time I used this to get around it, but that feels wrong

cold island Jan 26, 2021, 9:11 AM

#

Are you talking about the bot or lancebot?

sharp timber Jan 26, 2021, 9:11 AM

#

bot

cold island Jan 26, 2021, 9:11 AM

#

Also the bot uses 3.8 iirc

sharp timber Jan 26, 2021, 9:12 AM

#

My 3.8 is broken, so I bumped the pipfile up to 3.9. Was getting this error before all that happened, so I don't think it's related

#

Why on earth is it still using 3.8 for the bot if I changed that

cold island Jan 26, 2021, 9:13 AM

#

Might be related to your config file

sharp timber Jan 26, 2021, 9:13 AM

#

I just realized the docker container is firing up with 3.8

#

that probably isn't related though

cold island Jan 26, 2021, 9:13 AM

#

Yeah it's probably fine

sharp timber Jan 26, 2021, 9:13 AM

#

The 3.8 error is a ctypes .so error and this is just a binding error

cold island Jan 26, 2021, 9:14 AM

#

What is your config under the URLs section

sharp timber Jan 26, 2021, 9:15 AM

#

Should be basically the default

#

web:8000, etc

#

The erroring bind is defined at the top, 6379, for redis

#

host localhost

cold island Jan 26, 2021, 9:16 AM

#

If you're using docker then the only variable I can think of is the config. Maybe try copying the version on Notion

sharp timber Jan 26, 2021, 9:18 AM

#

Where under notion would it be?

#

found it

#

Home expands, that was not obvious

#

ah, so the config has fakeredis on

#

ModuleNotFoundError: No module named 'arrow'

cold island Jan 26, 2021, 9:22 AM

#

Try rebuilding the image?

#

Pull first

sharp timber Jan 26, 2021, 9:23 AM

#

how do I force it to

cold island Jan 26, 2021, 9:23 AM

#

docker-compose pull

#

docker-compose up --build

sharp timber Jan 26, 2021, 9:23 AM

#

TIL

#

And that will probably fix it

#

errored on api.localhost, which makes sense, let me reset that part of my config

#

Ugh, what did I do to fix that

#

It's searching for api.web:8000 which isn't dns-set for me

#

but it's no longer breaking on redis or arrow, which is good, thanks

green oriole Jan 26, 2021, 9:29 AM

#

You’ll have to start the site before the bot

#

The bot will start faster than the site, and error out

cold island Jan 26, 2021, 9:29 AM

#

Hm? Why? It should start both

green oriole Jan 26, 2021, 9:29 AM

#

try dcoker-compose up -d web and docker-compose up --build bot

green oriole Jan 26, 2021, 9:30 AM

#

cold island Hm? Why? It should start both

The startup sequence for the site is longer than for the bot

#

I opened an issue about that

sharp timber Jan 26, 2021, 9:30 AM

#

Doesn't apply here, I'm getting a dns error despite starting the site manually first

cold island Jan 26, 2021, 9:30 AM

#

green oriole I opened an issue about that

Weird, I never had this issue

green oriole Jan 26, 2021, 9:30 AM

#

Hmm

sharp timber Jan 26, 2021, 9:31 AM

#

cant connect to api.localhost:8000 basically

green oriole Jan 26, 2021, 9:31 AM

#

You are starting the site using GHRC container, right?

sharp timber Jan 26, 2021, 9:31 AM

#

docker-compose up web

green oriole Jan 26, 2021, 9:31 AM

#

hmm

#

They should use the same virtual network

#

Are your git repo up to date?

sharp timber Jan 26, 2021, 9:31 AM

#

Should be

#

I just did a git pull upstream master git push origin master

green oriole Jan 26, 2021, 9:33 AM

#

Hmm

#

Oooh

#

Got it, you are using localhost

sharp timber Jan 26, 2021, 9:34 AM

#

I'm on 134a2c61...

green oriole Jan 26, 2021, 9:34 AM

#

This works if the site is running on the host

#

Otherwise you should use web

sharp timber Jan 26, 2021, 9:35 AM

#

I changed the site: &domain to web:8000, but it continued to error

#

OK, this time it didn't

#

I changed nothing but now its failing with "improper token" and I at least know probably what that error is

green oriole Jan 26, 2021, 9:38 AM

#

Cool

sharp timber Jan 26, 2021, 9:41 AM

#

I mistakenly erased my .env file, so time to fix that sigh

#

TIL if your window is too small there's no indicator that your token is cut off

#

alright, came up just fine

wintry trellis Jan 26, 2021, 10:55 AM

#

cold island looks like you just don't have the package installed. What is your reason for s...

I'm sorry I did not understand your question.

#

I installed the package async-rediscache

wintry trellis Jan 26, 2021, 11:02 AM

#

wintry trellis And that might be the reason for the cause of this error? ``` from async_redisca...

I found a solution for this error. A new error seems to appear now and that is, from bot import api, constants ImportError: cannot import name 'api' from partially initialized module 'bot' (most likely due to a circular import)

#

Not sure where I am going wrong.

short snow Jan 26, 2021, 11:07 AM

#

@green oriole "The Automatically linking issues using repo#number in messages" pr is ready for review.

green oriole Jan 26, 2021, 11:07 AM

#

Saw that, thanks!

short snow Jan 26, 2021, 11:11 AM

#

one more question, for testing prs, like currently am testing iceman's, do we just copy paste their code into ours like on a branch called testing or something, and then test it.
or ...?

brazen charm Jan 26, 2021, 11:13 AM

#

checkout their branch

short snow Jan 26, 2021, 11:13 AM

#

what if it is not the main repo?

brazen charm Jan 26, 2021, 11:17 AM

#

I use this git alias from Mark to do it quickly cpr = "!cd ${GIT_PREFIX:-.} && git fetch upstream refs/pull/${1}/head && git checkout FETCH_HEAD #", to do it manually you just have to fetch the pull head and checkout

short snow Jan 26, 2021, 11:18 AM

#

ok thankyou!

#

can override_in_channel decorator be used for on_message listner?

vocal wolf Jan 26, 2021, 11:32 AM

#

I just use github's CLI

#

gh pr checkout 567

short snow Jan 26, 2021, 11:35 AM

#

short snow can `override_in_channel` decorator be used for `on_message` listner?

and does it accept categories?

short snow Jan 26, 2021, 12:17 PM

#

#bot-commands message the inline coceblock check fails here (from bot/filters)

gritty wind Jan 26, 2021, 1:15 PM

#

wintry trellis I found a solution for this error. A new error seems to appear now and that is, ...

You have the code
‘from bot import ...’
In a file titled bot, which causes a circular import

brazen charm Jan 26, 2021, 1:31 PM

#

The bot package has to be ran as a module if you're not doing that

short snow Jan 26, 2021, 1:36 PM

#

python3 -m bot or pipenv run start ⤴️

cold island Jan 26, 2021, 1:47 PM

#

wintry trellis I'm sorry I did not understand your question.

What are you trying to use the code for?
If you just want to see how the bot works or write some feature, then I strongly suggest you do it with docker. It sets everything up for you

#

And there should be no reason to change the code just to make it work - it should already work as is.

gritty wind Jan 26, 2021, 1:49 PM

#

They were repurposing the snekbox code for their own bot

gritty wind Jan 26, 2021, 1:49 PM

#

brazen charm The bot package has to be ran as a module if you're not doing that

So I don’t think this is applicable

cold island Jan 26, 2021, 1:50 PM

#

Well, then you don't need all of the code and dependencies, just the eval cog.

green oriole Jan 26, 2021, 1:51 PM

#

!source eval

stable mountainBOT Jan 26, 2021, 1:51 PM

#

Command: eval

Run Python code and get the results.

Source Code

Go to GitHub

green oriole Jan 26, 2021, 1:51 PM

#

You only need to use this file, although you’ll have to change some contants and things like that.

gritty wind Jan 26, 2021, 1:51 PM

#

They’ve had that since yesterday, but they were struggling to decouple it from the bot

short snow Jan 26, 2021, 2:04 PM

#

ig they are writing their own snekbox instead

short snow Jan 26, 2021, 2:05 PM

#

short snow can `override_in_channel` decorator be used for `on_message` listner?

and the messages below that, 😄

trim cradle Jan 26, 2021, 4:06 PM

#

Can someone send me their bot config file for the bot test server? I lost anything that wasn't checked in when I wiped my computer.

late wolf Jan 26, 2021, 4:08 PM

#

What's wrong with the docs command
#bot-commands message

#

it isn't doing anything

green oriole Jan 26, 2021, 4:09 PM

#

trim cradle Can someone send me their bot config file for the bot test server? I lost anythi...

It is on Notion https://www.notion.so/pythondiscord/config-yml-1673fcd1232048b5a9bd2cc47c2f2769

wintry trellis Jan 26, 2021, 4:21 PM

#

cold island What are you trying to use the code for? If you just want to see how the bot wor...

I wish to write some features
And using docker right now

cold island Jan 26, 2021, 4:22 PM

#

A question. I'm looking at the bot's code, and I see quite a few cogs with class attributes, but they are only used in the context of self. Is there any reason why they can't be instance attributes initialized in init?

#

It's not like we ever have two instances of the same cog

green oriole Jan 26, 2021, 4:23 PM

#

Do you have any example?

cold island Jan 26, 2021, 4:23 PM

#

Pretty much any cog under moderation

#

Defcon, incidents, etc

green oriole Jan 26, 2021, 4:25 PM

#

I don’t see any class attributes in incidents

#

defcon hasn’t been updated in a while though, and is bound to reach the sun

cold island Jan 26, 2021, 4:26 PM

#

Well I'm updating it right now lol

wintry trellis Jan 26, 2021, 4:26 PM

#

gritty wind They were repurposing the snekbox code for their own bot

Was this meant for me or... ?

green oriole Jan 26, 2021, 4:27 PM

#

I had a quick look, and it seems like the only class attributes are redis caches, which need to be class attributes

#

What are you doing to the poor defcon?

cold island Jan 26, 2021, 4:27 PM

#

green oriole I had a quick look, and it seems like the only class attributes are redis caches...

Why?

green oriole Jan 26, 2021, 4:28 PM

#

We use the attribute name in order to know their unique name

cold island Jan 26, 2021, 4:28 PM

#

green oriole What are you doing to the poor defcon?

Finishing my open pr

cold island Jan 26, 2021, 4:28 PM

#

green oriole We use the attribute name in order to know their unique name

Whose unique name?

green oriole Jan 26, 2021, 4:29 PM

#

The redis cache name

#

For example, if you have a cache assigned to Zig.zebra, it guess that it should load the cache of this specific entry

cold island Jan 26, 2021, 4:30 PM

#

Ok

late wolf Jan 26, 2021, 4:38 PM

#

Does anybody know anything about this?
#dev-contrib message

cold island Jan 26, 2021, 4:45 PM

#

Can you give a github link to the function?

late wolf Jan 26, 2021, 4:49 PM

#

right

#

Here
https://github.com/python-discord/bot/blob/3d014d95f57633a2f8fdf40f61ed1b0ade7343dc/bot/decorators.py#L65

cold island Jan 26, 2021, 4:56 PM

#

And what is your proposed change?

late wolf Jan 26, 2021, 4:57 PM

#

I wasn't proposing a change, I was asking why do aren't we using commands.check() but rather take the command function itself

wraith kayak Jan 26, 2021, 5:23 PM

#

When I type docker-compose up, this error shows:

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "docker-compose", line 3, in <module>
  File "compose\cli\main.py", line 67, in main
  File "compose\cli\main.py", line 123, in perform_command
  File "compose\cli\command.py", line 69, in project_from_options
  File "compose\cli\command.py", line 132, in get_project
  File "compose\cli\docker_client.py", line 43, in get_client
  File "compose\cli\docker_client.py", line 170, in docker_client
  File "site-packages\docker\api\client.py", line 188, in __init__
  File "site-packages\docker\api\client.py", line 213, in _retrieve_server_versi
on
docker.errors.DockerException: Error while fetching server API version: (2, 'Cre
ateFile', 'The system cannot find the file specified.')
[5156] Failed to execute script docker-compose

vale ibex Jan 26, 2021, 5:24 PM

#

Do you have docker running?

thorny obsidian Jan 26, 2021, 5:24 PM

#

I've gotten this error several times... every single time it was because I forgot I didn't have docker actually running

vale ibex Jan 26, 2021, 5:24 PM

#

systemctl status docker to see if it is

#

systemctl start docker to start it if it isn't

#

(assuming linux)

wraith kayak Jan 26, 2021, 5:25 PM

#

I am on windows cmd

vale ibex Jan 26, 2021, 5:25 PM

#

yea, that's from when docker-compose isn't able to connect to docker via docker socket

#

Do you have docker desktop installed?

#

https://www.docker.com/products/docker-desktop

wraith kayak Jan 26, 2021, 5:26 PM

#

Yep

vale ibex Jan 26, 2021, 5:26 PM

#

Do you have it running?

wraith kayak Jan 26, 2021, 5:26 PM

#

No

vale ibex Jan 26, 2021, 5:26 PM

#

That'll be why 🙂

wraith kayak Jan 26, 2021, 5:27 PM

#

Yeah 😄

vale ibex Jan 26, 2021, 5:28 PM

#

Just look out for the happy whale in your taskbar

📎 unknown.png

cold island Jan 26, 2021, 6:39 PM

#

late wolf I wasn't proposing a change, I was asking why do aren't we using `commands.check...

It's not used on a command

wraith kayak Jan 26, 2021, 6:49 PM

#

when I do pipenv run start, a new cmd opens but the bot is not running. Why?

#

I double checked the bot token

gritty wind Jan 26, 2021, 6:50 PM

#

Can you try running another command through pipenv

#

For example

#

pipenv run pip list

wraith kayak Jan 26, 2021, 6:50 PM

#

This is what I got

Package    Version
---------- -------
pip        20.3.3
setuptools 51.3.3
wheel      0.36.2

gritty wind Jan 26, 2021, 6:51 PM

#

Did that also open in a new window?

#

Also you may need to run install first

wraith kayak Jan 26, 2021, 6:51 PM

#

No in the same window

gritty wind Jan 26, 2021, 6:53 PM

#

The whole opening in a new cmd thing is weird, but the error may be caused by not installing. Try running pipenv install --dev

wraith kayak Jan 26, 2021, 6:54 PM

#

The result is still negative 😦

gritty wind Jan 26, 2021, 6:55 PM

#

Does anything get printed?

wraith kayak Jan 26, 2021, 6:55 PM

#

Nothing

gritty wind Jan 26, 2021, 6:56 PM

#

Ok hmm

#

Try pipenv shell to log into the virtual environment

#

Then try python -m bot

wraith kayak Jan 26, 2021, 6:58 PM

#

Error:
No module named bot .__main__; 'bot' is a package and cannot be directly executed

gritty wind Jan 26, 2021, 6:58 PM

#

What directory are you in?

#

Right, and where is the repo?

wraith kayak Jan 26, 2021, 7:00 PM

#

Inside acer and file named bot

gritty wind Jan 26, 2021, 7:00 PM

#

Ok go the bot folder

#

Make sure you see Pipfile in that folder

#

(Also make sure to exit your current environment first by typing exit)

wraith kayak Jan 26, 2021, 7:01 PM

#

gritty wind Make sure you see `Pipfile` in that folder

Yes, I see

gritty wind Jan 26, 2021, 7:01 PM

#

Run pipenv install --dev and pipenv run start in that folder

stable mountainBOT Jan 26, 2021, 7:06 PM

#

:incoming_envelope: :ok_hand: applied mute to @wraith kayak until 2021-01-26 19:16 (9 minutes and 59 seconds) (reason: chars rule: sent 3252 characters in 5s).

gritty wind Jan 26, 2021, 7:06 PM

#

Hey mods, can we get that lifted

cold island Jan 26, 2021, 7:07 PM

#

!unmute 728987928591728802

stable mountainBOT Jan 26, 2021, 7:07 PM

#

:incoming_envelope: :ok_hand: pardoned infraction mute for @wraith kayak.

cold island Jan 26, 2021, 7:07 PM

#

!paste

stable mountainBOT Jan 26, 2021, 7:07 PM

#

Pasting large amounts of code

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

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

wraith kayak Jan 26, 2021, 7:07 PM

#

Ok my bad sorry

#

https://paste.pythondiscord.com/bufowucevi.coffeescript

gritty wind Jan 26, 2021, 7:10 PM

#

This error is caused by not having redis running locally

#

You can either run it through docker

#

Or use fakeredis

patent pivot Jan 26, 2021, 7:10 PM

#

hmmmm

#

we should fix that

#

or at least make it more evident what is going on

gritty wind Jan 26, 2021, 7:10 PM

#

We could default to fake_redis, and make it an env var :D

patent pivot Jan 26, 2021, 7:11 PM

#

+1

gritty wind Jan 26, 2021, 7:12 PM

#

gritty wind Or use fakeredis

If you want to do this right now, make a copy of config-default.yml, call it config.yaml, and change use_fakeredis on line 10 to true

#

(How to setup your config file is explained here: https://pythondiscord.com/pages/contributing/bot/#configure-the-bot but it's pretty much what I just said)

wraith kayak Jan 26, 2021, 7:14 PM

#

True is case sensitive?

gritty wind Jan 26, 2021, 7:15 PM

#

I don't think so

patent pivot Jan 26, 2021, 7:15 PM

#

not in yaml, no

wraith kayak Jan 26, 2021, 7:17 PM

#

Looks like that is solved now but now I get this error

aiohttp.client_exceptions.ClientConnectorError: Cannot connect to host api.pytho
ndiscord.local:8000 ssl:default [Connect call failed ('127.0.0.1', 8000)]

For reference:

config.yaml

urls:
    # PyDis site vars
    site:        &DOMAIN       "pythondiscord.local:8000"

gritty wind Jan 26, 2021, 7:17 PM

#

Yup, that's because the site isn't running

#

The easiest way to get all the dependencies running is to use docker

#

(You can manually set everything up, but that would take a bit of time)

#

Do you have docker?

wraith kayak Jan 26, 2021, 7:18 PM

#

I tried installing in linux but not working successfully so decided to do it manually.

gritty wind Jan 26, 2021, 7:19 PM

#

Well docker can be a bit of a pain sometimes, though I admit I haven't heard of it not working on linux. Either way, the only other app that needs to be running for the bot to start is site

#

Setting up the site project is detailed here https://pythondiscord.com/pages/contributing/site/, though if you want to give docker another shot, we can try and help you debug it

wraith kayak Jan 26, 2021, 7:21 PM

#

Let me try to setup docker again on linux and will get back right here. Thanks for helping me 😄

wraith kayak Jan 26, 2021, 7:37 PM

#

The main error when I write this command is:

sudo add-apt-repository \
   "deb [arch=amd64] https://download.docker.com/linux/ubuntu \
   $(lsb_release -cs) \
   stable"

Error:

Traceback (most recent call last):
  File "/usr/bin/add-apt-repository", line 12, in <module>
    from softwareproperties.SoftwareProperties import SoftwareProperties, shortcut_handler
  File "/usr/lib/python3/dist-packages/softwareproperties/SoftwareProperties.py", line 28, in <module>
    import apt_pkg
ModuleNotFoundError: No module named 'apt_pkg'

gritty wind Jan 26, 2021, 7:41 PM

#

What OS are you on, and what steps have you run so far?

#

I haven't setup docker on a linux distro before, but I'm looking at this page https://docs.docker.com/engine/install/

wraith kayak Jan 26, 2021, 7:42 PM

#

Ubuntu 18.
I am currently in the 3rd step under SET UP THE REPOSITORY

green oriole Jan 26, 2021, 7:43 PM

#

I had a similar issue, I solved it by used the automatic install

gritty wind Jan 26, 2021, 7:44 PM

#

I guess another solution would be to check the apt_pkg install

#

So apt install python3-apt

#

might need a sudo or two in there

wraith kayak Jan 26, 2021, 7:51 PM

#

Oh I successfully installed. 🙂

#

Now I think set up a path to the bot files and do docker-compose up --no-deps bot?

gritty wind Jan 26, 2021, 7:55 PM

#

Well the whole reason we went through the trouble of installing docker was so you can use the deps

#

Doesn't make sense to disable them now

#

you should be able to start everything up with docker-compose up

#

You can add -d to detach it from your terminal too

wraith kayak Jan 26, 2021, 7:55 PM

#

Ok, thanks 😄

green oriole Jan 26, 2021, 7:56 PM

#

You shouldn't use --no-deps actually

#

I should update the guide

gritty wind Jan 26, 2021, 7:57 PM

#

I think that's mentioned under the bot only section

#

so its fine there

#

https://pythondiscord.com/pages/contributing/bot/#site-and-bot
The one above it doesn't mention it

green oriole Jan 26, 2021, 7:57 PM

#

Hmm I see

spiral orchid Jan 26, 2021, 8:09 PM

#

Hi, I'm working on Infraction result limit. I'd setup the server and ready to roll, but getting confused regarding db structure, like how to update db with dummy data.

#

tried with admin panel but didn't figure out

gritty wind Jan 26, 2021, 8:11 PM

#

I would say the easiest way would probably be to hand out actual infractions if you have the bot setup.

If not, the data in the DB is probably not too important, just that you have a few infractions. Are you familiar with SQL and inserting into tables at all?

spiral orchid Jan 26, 2021, 8:12 PM

#

Not much with SQL

#

Can you explain how to setup the bot

green oriole Jan 26, 2021, 8:14 PM

#

You can use Django admin to insert them

#

You'll need to add a user first

spiral orchid Jan 26, 2021, 8:15 PM

#

In the API administration user can't be added

green oriole Jan 26, 2021, 8:16 PM

#

Hmm really

#

You are using the Admin account?

spiral orchid Jan 26, 2021, 8:17 PM

#

📎 unknown.png

gritty wind Jan 26, 2021, 8:20 PM

#

patent pivot we should fix that

Opened an issue for it, but left it in planning stage for now. Added a couple solutions there

green oriole Jan 26, 2021, 8:20 PM

#

What if you go to https://api.localhost:8000/bot/users

spiral orchid Jan 26, 2021, 8:23 PM

#

📎 unknown.png

gritty wind Jan 26, 2021, 8:25 PM

#

You left the api part off the url

#

http://api.pythondiscord.local:8000/bot/users

spiral orchid Jan 26, 2021, 8:26 PM

#

📎 unknown.png

green oriole Jan 26, 2021, 8:32 PM

#

Hmm if you set the Auth header you can do a post request from here

spiral orchid Jan 26, 2021, 9:31 PM

#

There are total 6 infraction and the default page size is 4

📎 unknown.png

wraith kayak Jan 26, 2021, 9:32 PM

#

Successfuly installed docker in linux and is running but now I get the following error :

ERROR: Couldn't connect to Docker daemon at http+docker://localhost - is it running?

If it's at a non-standard location, specify the URL with the DOCKER_HOST environment variable.

spiral orchid Jan 26, 2021, 9:32 PM

#

I think that is it or something else need to be changed

gritty wind Jan 26, 2021, 9:33 PM

#

wraith kayak Successfuly installed docker in linux and is running but now I get the following...

Try:
sudo service docker status

#

If it says offline, sudo service docker start

wraith kayak Jan 26, 2021, 9:36 PM

#

I just did sudo docker-compose up and looks like it worked

#

But this the output I get at last

System check identified no issues (0 silenced).
web_1       | January 26, 2021 - 21:35:24
web_1       | Django version 3.0.11, using settings 'pydis_site.settings'
web_1       | Starting development server at http://0.0.0.0:8000/
web_1       | Quit the server with CONTROL-C.
```Is it?

gritty wind Jan 26, 2021, 9:38 PM

#

You're up

wraith kayak Jan 26, 2021, 9:38 PM

#

but the bot isn't online 🥲

gritty wind Jan 26, 2021, 9:39 PM

#

I'm not sure if this is looking at the full container, or just the site one

#

disconnect your terminal and check the status

wraith kayak Jan 26, 2021, 9:40 PM

#

It says active running in green

gritty wind Jan 26, 2021, 9:41 PM

#

For all the containers? Is the bot listed there

#

Oh that's the docker enginer, not the actual containers

#

docker container ls

wraith kayak Jan 26, 2021, 9:42 PM

#

oh

#

     STATUS              PORTS                      NAMES
e23a19f89b0e        ghcr.io/python-discord/site:latest      "python manage.py ru…"   8 minutes ago       Up About a minute   127.0.0.1:8000->8000/tcp   bot_web_1
b39417e58c7e        postgres:12-alpine                      "docker-entrypoint.s…"   8 minutes ago       Up 2 minutes        5432/tcp                   bot_postgres_1
8589936a46cf        redis:5.0.9                             "docker-entrypoint.s…"   8 minutes ago       Up About a minute   127.0.0.1:6379->6379/tcp   bot_redis_1
0bc29b6c54d4        ghcr.io/python-discord/snekbox:latest   "gunicorn -c config/…"   8 minutes ago       Up 2 minutes        127.0.0.1:8060->8060/tcp   bot_snekbox_1

gritty wind Jan 26, 2021, 9:44 PM

#

Hmm yeah the bot itself is missing