#dev-contrib
1 messages · Page 68 of 1
but is it ok if I submit an issue first?
@long brook yeah, you're even required to open an issue first
no I was saying that since I didn't know how I would implement it.
I presumed you had tell how you would do it
It would be nice if you had an idea of how to implement it
yea
We could discuss it here, if you wanna think about solutions
(That doesn’t include me though, since I’ll be doing some physics)
well every phrase that contains an action, is swapped to the beginning
i'm too young to do physics
im in bio
How would you know where exactly to swap it?
Well, it would be nice if you could have an idea of the implementation first
I'm thinking about something like this
for example
About something like this i'm thinking.
I’d recommend studying how Yoda actually talks, so you have a solid goal to work toward
Aahha yes
is the contributors role only for people who work on branding?
no
Not in the slightest
o ok, Anubhav didn't have it
I think kutiekat and I were the first ones to get it from working with branding🤔 (don’t quote me on that, though)
We have a description of the role on the website, hold on I'll get it
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.
yea
the snake video command is returning weird videos now
Yup
one just about faking pregnancy
I think we can disable snake video command unless we find any solution to it
Bcz it's really sending weired videos
We can't really unload one command that's the thing
I guess we can unload the whole cog in the meantime
Hmm that is also fine
do we have a commad that gives us the usage of all commands?
Or just comment the command
Alright, kwzrd just merged it
do we have a commad that gives us the usage of all commands?
@eternal owl wdym?.helpshould give you every commands
Ohk
frequency
Nope I guess
race for the issue
yes we do
👀
Seasonal
seasonal
oh, seasonal
are python stats public?
uh
some
https://stats.pythondiscord.com/ has things like eval usage & in use help channels
Public statistics for the Python Discord server
Looks like snake command is fixed again
i mean,all commands
but if you see any stats we are recording and are thinking "oh shit! that would cool to have there!"
ah, no they aren't public
but I can get you them now
one second
alright 👍
sniped
Yeh I am also curious to know
past 7 days
you are a fake and a fraud
I'm a slow sniper
joe doesn't always win
I mean, can we get stats of a single cmd?
That’s actually not the graph I had expected
Accept destroys everything
True true
whats the least used?
Btw what is the use of accept command?
let me find out
i was curious of .snake movie but thats for seasonal, lol
oh god
Probably !free hahah (though it’s not a command
maybe we can use stats to get rid of cmds?
Lul
Not too unexpected
Unwatch which command is this
second graph looks funny
i guess majority are staff commands
discovery @sullen phoenix
fuck I queried last 90 days
@patent pivot no wonder why the graph took 20 seconds to load for me
ahh yeah
Haha
Ehmmm
👀
I just want to look at the panel
hahahahaha
So apparently scrolling is denied haha
lol
!otn a scrolling-is-denied-or-maybe-mobile-is-funkk
Hahaha rip network
ModMail iirc
cool
because turns out a destructured json database is pretty slick at storing destructured json responses lul
whats going to handle forms backend? new microservice?
yeah
flask or fast API?
it might turn out... not being a microservice lol
neitehr
starlette
fastapi is built on top of starlette
I don't see what features it brings
we don't need public docs... I'd even rather not have public docs for a forms system with anti-tamper
I also like class based views, something fastapi does not provide
when are we starting on the new microservice?
time to grind react and typescript
however, when I make projects for the server I do make things open like... immediately
so you'll see it for sure
oh
flowchart time? flowchart time
+-----------+ +----------+ +------------+ +-----------+
| PyDis Bot +------->+ StatsD +------->+ Graphite +<-------+ Grafana |
+-----------+ +----------+ +------------+ +-----------+
that's the stats stack
Oo I thought hidden talent
lol
😂
it is a little more advanced than that actually
Doesn't quite work on non-wide screens >_>
lol
yeah that's how the stack actually looks on mobile
Discord🤦
hmmmm
its light theme
Check mine
+-------------+
| Metricity |
+------+------+
|
v
+------------+ +------+-----+
+------------->+ PyDis Site +----->+ PostgreSQL +<-------------+
| +------------+ +------------+ |
| |
+-----+-----+ +----------+ +------------+ +-----+-----+
| PyDis Bot +------->+ StatsD +------->+ Graphite +<-------+ Grafana |
+-----------+ +----------+ +------------+ +-----+-----+
|
+-------------+ +------------+ |
| Exporters +<----+ Prometheus +<-------------+
+-------------+ +------------+
whats white, lol
there we go
how do you make them
Oops
well, I think that gives enough of an overview on how our stats stack operates
preposterous
So far how long we are storing data in db?
hmmm
depends on what data you mean
give me an item of data we store and I'll tell you the lifespan
Ban
we are legion, we never forget
lol
Ohk so we are storing infractions from 3 years lol

let me see
exploring data is always fun, xD
13,266 infractions
+---------+
| Hello |
+----+----+
|
+----v----+
| Bye |
+---------+
That’s actually amazing
6,901 of those are from the last 6 months
Did that on my phone using it
Whattt so many
lol
is there multiple instances of bot running too?
You are quick
Yeh just seen
Where is fire😂😂
is there multiple instances of bot running too?
@lime mural no, just one instance of bot
hmm what about metricity bot?/
what if it goes down 🤔
it... goes down
the vast majority of bot outages have been a discord fault, in one instance there was a linode fault with networking
both of these would not have been resolved by having multiple instances
hmm, doesn't the metricity bot get overwhelmed with the amount of updates it has to keep up with, in a server this big?
nope
it gets by pretty well
I'm going to bump it to 1.5 soon anyway to ditch presence events
I am actully working on something similar, for a different server, like metricity
but using express and vuejs
and discord.py
i might use discord.js just to learn it tho
I don't know y but my all dbs are getting corrupted bcz my hard disk got fked up😔
using metricity bot/repo as a ref, inspiration
lol
be careful
metricity CPU spikes
I need to find a nice optimisation soon
user sync process is intensive
got a 5$ DO server for now
Sorry one out of scope question is it required for api to have website?
rest api, yes
Without website it won't work?
I meant we can't host without website?
submitted my issue
you don't need to have a frontend tho @green mesa
you can host without website
Ok
if it's internal, if public need that docs mate
i'm so fast tonight
lol
gcc be like: not so fast man, told you you'll crash
by gcc do you mean the compiler?
Now that Python barely consumes CPU, let's go wild
lol
This needs reviews. It's been ready for months. https://github.com/python-discord/bot/pull/886
Sorry, I thought that I have today time to review this, but I got one unexpected school homework
@patent pivot I've created an abstraction for the metricity connection locally, which works fine. I've added unit tests which mock this abstraction, to cover for example the case where the user exists in the site database but not in metricity. However, if I push what I have now, the pipeline will probably fail on the coverage check again because I now haven't tested the database layer instead. I tried using in-memory sqlite to mock the metricity database and write tests for it, but there seems to be syntactical differences in SQL between postgres and sqlite, so they are not quite interchangeable. I'm now thinking that maybe I should just exclude the db abstraction from coverage. What do you think?
warning: LF will be replaced by CRLF in a commit. I assume this is ok?
or would that break the code in linux
I'm not even sure why it says that, I only see LF's in there, and I assume that's how it was in the previous commit as well
@cold island might be the way your git is configured
we don't really want any CRLFs in pydis code
in fact, we have a precommit to convert them, iirc
I read that that warning is funky and might mean the opposite. I didn't find any CRLF after the commit
okay, maybe
And the precommit didn't say anything
as long as you're committing \n and not \r\n you should be good
@obtuse arrow that sounds fine by me, yeah
@cold island If you're on windows, you should probably set core.autocrlf to true. That will convert CRLF in your working copy to LF in the repo when you commit, and the other way around when you check out.
@patent pivot Cool
If you're on windows, you should probably set core.autocrlf to true. That will convert CRLF in your working copy to LF in the repo when you commit, and the other way around when you check out.
@obtuse arrow already seems to be true
Then you should be good.
You'll get CRLF in your working copy and LF in the repo.
@patent pivot @crude gyro Pushed an update to the PR now.
Sweet stuff, I'll have a look at it during the day tomorrow 👍
Cool
I heard that if the bto is not verfied then the bot is gonna crash but, joe is verfied so the python bot is not gonna crash right?
our bots are only in this guild
and a few test guilds
joe is a verified bot dev from elsewhere
ye, I just asked from the people in discord dev portal
and they said that this update will only affect bots that are verfied
and over 76 servers
this is what they said
no. unverified bots aren't affected by this update
yeah, our bots are not verified
Is there a way to merge these 2 commits after they are pushed? Both have subtle changes that be made into 1 commit
hey @tawdry vapor , the bulk_create and bulk_update methods can also take in another argument called batch_size, this indicates how many objects are to be created/saved in SQL query. This can help updating/creating faster. what do you think?
https://github.com/python-discord/site/pull/378#issuecomment-696694622
these are old benchmarks, and 2000 users per query was the fastest
@eternal owl if that’s your own feature branch that you’re sure no one else is using, you could squash the commits into one and force push
will the third commit make it possible to do so? or will I have to squash all 3?
I assume it would be possible, but don’t take my word on that
I don't know if it'd really make a difference given the amount of users we create/update @eternal owl
alright 👍
We can keep it in mind if query times start creeping up
yea
@eternal owl https://stackoverflow.com/questions/39023360/git-squash-commits-in-the-middle-of-a-branch
thx vester, I will look at that :p
I don’t know if it’s considered best practice, but it should work
its prolly mb, I should have done it in just 1 commit, lol
Alright
Yeah, that's the intended way of doing that, but you should do that only if you haven't pushed yet
Thanks
nice response on the issue, lol
Lol 'no.'
Lol I also wnt to check but issue is no longer available there
Btw what was in the issue?
Just curious to know
Yup, hence the atomization
Weired issue and obvious answer😂😂😂
@green oriole @obsidian patio extremely sorry for flooding ur emails bc of my tiny commits with pr 475 lol, btw it's ready for review now
Hahah no worries. I’ll be quite busy these following days, but I’ll review it when I get time
No problem, I got used to it haha, I'll try to review this weekend
(I am the one spamming his emails)
(you sure do)
.issue 796
[404] #796 Issue/pull request not located! Please enter a valid number!
.issue 796 bot
A'ight
yeah it didn't get approved by a coredev yet
but ak is now one
so he can do it
lol
Haha
Anybody can give https://github.com/python-discord/site/pull/395#pullrequestreview-504399747 second review? I want to start working with resources displaying.
Don't worry, we will come around to review it, it is quite a consequent PR
@green mesa before you get started if you don't mind I'd like to talk to you about when and what to commit
In your previous PRs on seasonalbot, you seemed to make quite a few commits, without having any substantial commit message in it, it would be nice if you could pay more attention to it with @stable mountain, because we are often searching the history, compared to @dusky shore
Sure
Ohk I will try my best
Do we have some sort of guide how to write commit message which is meaningful?
It's mentioned in the contrib doc https://github.com/python-discord/bot/blob/master/CONTRIBUTING.md
point #4
Ty
I'd also like to note that pretty much all variables you use in the code should be readable
Variables like a aren't readable
This also is quite an interesting read, I'd recommend you to take a look at it https://chris.beams.io/posts/git-commit/
If you have any question or even hesitation, feel free to reach out one of the core dev, we will happily help you, we prefer to answer questions than spend more time reviewing 
Wondering if creating pull requests without being assigned are allowed
on seasonalbot, you should first be assigned.
on the rest, i'd at a minimum express your intention to work on an issue via a comment to avoid the accident of multiple people working on the same thing
@eternal owl I have pushed changes according to what u asked that day at bug ec pr whenever u r free can u have a look at it 
okay 👍
Tnx
hey @tawdry vapor , regarding the pagination warning, the response results seemed to be in order tho,
But to remove the warning, I am explicitly ordering by the id field
changed this line queryset = User.objects.all() to queryset = User.objects.all().order_by("id") in the ModelViewSet
!echo hello
hello
hello
its alive
Was it down?
Might have been a small blip, but it seems ok now
Ah ok
Hmm
Do know why that didn't work that time
its meant to?
how
My keyboard ahh
it sends a msg every 10s if you don't give the right answer
okay thats weird
yup
lets try again
!ping
0.000 ms
10.378 ms
101.864 ms
prolly a small hiccup
Discord is really slow
ok
Right, the broken ping
Yeh
whats the expected ping?
Around 10-20 iirc
I'll have to fix that real quick. @green oriole I'll fix it. Can you just have a look so it works before I open a PR?
Okayy
yea it's fine now
I think long embeds have been fixed in the discord android beta version
I am also a beta tester
If you're updated, i'm not sure then
can anyone give me feedback on my git commit messages?
https://github.com/python-discord/site/pull/378
I suddenly realized that I'm on mobile and the embed error is fixed \o/
They look pretty good, but I'm seeing two things
please start with uppercase
@neon phoenix right, what about the actual messages
For the love of god, please don't make the first line longer than 50 characters
i like how you used imperative mood
that is how it should be
❌ Added a feature
✅ Add a feature
I think 50 is a little too strict, I've not seen any GUI fail to display the full summary if it's 72 or below
And also sometimes the commit messages aren't really descriptive, like this one https://github.com/python-discord/site/pull/378/commits/5388034d9a00e41f18ba5f476a2c3e347d4bd569, it doesn't say what part you changed, which is the most important
For the love of god, please don't make the first line longer than 50 characters
@green oriole right, i guess thats because of larger commits, I will try pushing smaller commits
If you can get a nicer message with 60 than with 50, I think it's always worth it
72 displays just fine
And also sometimes the commit messages aren't really descriptive, like this one https://github.com/python-discord/site/pull/378/commits/5388034d9a00e41f18ba5f476a2c3e347d4bd569, it doesn't say what part you changed, which is the most important
@green oriole well, what should I say instead?
!e print(len('refactor tests to use updated changes and add tests for UserListPagin'))
damn
@green oriole :white_check_mark: Your eval job has completed with return code 0.
69
Finally
just make a change.org petition asking for more space for git messages
So please no more than 69 chars
okay
this commit could be split into 2? (refactor tests to use updated changes and add tests for UserListPagin)
right?
And the message for this commit could be like "Update the documentation for the user endpoint"
updated
it should be in imperative mood
like
Update docs and add a number and merge this
what number?
it's just an example
I swear that's on my spellchecker
could use ,

using linux @green oriole ?
Yup
hello from ms-dos 1.25
discord linux doesn't come with a spell checker:(
how's life with long file names
Ah, I'm on mobile haha
discord mobile gets a spell checker?
give me a commit message for this?
just removed the brackets
does normalize error response work?
lol
or
Convert error message from list[str] to str
but this is too accurate
whatever man
.issue 1222 bot
This is a very minor change (a single character). Just needs to be tested, but there's not much of a code review needed
@green oriole Think we finally got this one working (though the times still seem a bit low) https://canary.discordapp.com/channels/267624335836053506/267659945086812160/763827470512160788
That 0.003 would've been a 3 second period previously
It still look really fast though
Yeah. It does look almost too fast
I'm not even sure how the gateway can send the message this fast
I've only received a single message above 0.000
It probably can't
0.000s is just instant
!ping
0.000 ms
11.373 ms
105.654 ms
!ping
181.403 ms
16.524 ms
103.273 ms
lmao

that seems better
What changed?
it was doing secs / 1_000 to get ms
It returned 0.000 after the bot had restarted with my PR merged
no, that was a different build
Can I do this one please
https://github.com/python-discord/seasonalbot/issues/495
Sure thing!
I think this can be approved?
Alright
cuz I am already assigned to other issues on the Bot and Site repo
Fair enough
hey @tawdry vapor , the smart syncing of users should also be merged with the site PR, together
Yeah I'm waiting for the site to deploy before merging the bot PR
okay
btw, thanks a lot for spending time on reviewing my PRs
and your critical observation helps me in becoming a better developer
Are there any plans to add a timeit end point to snekbox?
or should it be implemented purely on bot side?
No plans as far as I know, but anyone is welcome to work on it
Implementation is still up for discussion
!e
import webbrowser
webbrowser.open("https://www.google.com/""")
You are not allowed to use that command here. Please use the #bot-commands channel instead.
@crisp merlin anyways the public eval command doesn't have access to the internet
@patent pivot @crude gyro No rush or anything, but just pinging you about my PR again, in case you forgot.
sorry @obtuse arrow , been away practically all week on off sites and other crazy stuff
No worries!
Your reminder will arrive in 1 day!
Btw, just a general question since you're concerned about the commit history, do you prefer if I resolve conflicts as soon as they occur, or should I wait until reviews are done and fix them all at once before the final merge?
whatever is more convenient for you, imo. usually solving them once is enough, even when you do it early.
very rare that conflicts re-occur
Ok, I generally prefer to fix them immediately, but I was just asking in case you were concerned about the number of commits.
https://github.com/python-discord/site/issues/396
Thoughts on having the owner field as a foreign key instead of just storing the ID?
And rename owner to "organizers" too ?
replied in the issue
If they leave the server without any infraction iirc
Don’t take my word for it though
yeah, that right
if they appeal to delete the data and decide to leave server forever then also infraction and all data get deleted right
should be on the privacy document
and data reviews
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.
if they appeal to delete the data and decide to leave server forever then also infraction and all data get deleted right
@green mesa no, not really, only the personal data is stripped, but your user id isn’t
we do not delete infractions
umm then i think i misunderstood joe 
Your username is deleted, user ID isn't
how do I purge all data from the postgresql server running in docker cont?
infractions stay
how do I purge all data from the postgresql server running in docker cont?
@eternal owldocker rm site_postgresshould work, assumingsiteis the compose project name
alright
Hi data man here
o/
If they leave the server and file a data request then we will scrub their username and anonymise the data, which is our implementation of GDPR
infractions are not personal data and are not tied to you, they are tied to a discord ID, which is not end user data
oh okay
hello
so coming to the user events, I guess I will set the organizer to null when that user(Organizer) is removed from DB? or do you want to delete the event also?
ahh that's a good Q
if a user files a data deletion request I am kind of with the opinion we should delete their events (as we delete their reminders)
okay
Well, they shouldn’t be able to run the event if they left, so that make sense I think
if a user files a deletion request it's rare we delete them
we just anonymise
I can adjust the data retention policy to withhold any users who have events
👍
yeah keep events and assume the users will never be deleted
if there is a situation where that is necessary leave that to me and I'll make the decisions
But uhm, who is going to run the event if they leave?
pybot?
We'll figure that out manually
But uhm, who is going to run the event if they leave?
@green oriole that depends if anyone else wants to take over
because there are situations where we could reassign events
we don't need to make that a feature of this system though
Fair enough
👍
Having a commend to transfer the ownership of an event sounds useful though
Maybe for admins
It does, if it's not too much hassle we could do that, but only administrators should be able to reassign
But it's not really for regular users
yeah
Well, the user owner/organiser can transfer its own event
you can always use the django admin
I'm don't really want that
Alright
I'd rather it be an admin task, because if people are abusing the feature I'd like to know, and it people are creating bad users events I'd also like to know
sorry for disturbing discussion, I wnt to run site and bot together without docker how can i do so
to run site, you need to run postgres server, I guess docker is the easiest option
are you on windows?
it's not fun setting up without docker
get WSL 2
as if I do 😂
With compose it is really easy
never worked with that
I started a docker course but never finished it
docker-compose up and docker-compose down
docker-compose up to start, CTRL + C to stop
ah yes
only this
if you don't detach
Well, you need -d in this case f1re
yeah
also docker-compose up --build
You need build to update dependencies, not code though
first build then up?
It will build itself, don’t worry
Just add the build argument if the dependencies changed
ok where in compose script?
Wdym?
The compose script is already in the repo
Well, script
It is more of a config than a script
ok
write that command when you're in the repo's parent directory
how can i delete my virtual env in vsc?
when i cloned the repo all files are untracked y so?
There should all be tracked, except for the ignored ones, such as .env and comfig.yml
If no file is tracked, something went wrong
ok reforking
lol now came to know the issue i also have one repo name site so what happen is new fork was named site1 automatically by github
thats y all file are showing untracked
now everything is getting installed
Ignoring importlib-metadata: markers 'python_version < "3.8"' don't match your environment
whats this error?
how did you create ur env?
Interesting
That implies pip is skipping a line of the requirements file as it's only marked as required for certain python versions
I have one in my project for dataclasses on 3.6
The site is running 3.7
ooo
Then Anubhav is running on 3.8 or 3.9
i have both 3.7 and 3.8
What project is this
site
You need to add the --python 3.7 argument when syncing
Warning: Your Pipfile requires python_version 3.7, but you are using 3.8.0 (C:\Users\HP\.\s\S\python.exe).
i can set after installation?
delete the env
"importlib-metadata": {
"hashes": [
"sha256:90bb658cdbbf6d1735b6341ce708fc7024a3e14e99ffdc5783edea9f9b077f83",
"sha256:dc15b2969b4ce36305c51eebe62d418ac7791e9a157911d58bfb1f9ccd8e2070"
],
"markers": "python_version < '3.8'",
"version": "==1.7.0"
},
in pipfile.lock
ok
pipenv --rm
or navigate to .virtualenvs and get rid of it
and then do pipenv sync --dev
rm seems easy to me
yeh its getting removed
--rm git also uses and pipenv also
Using C:/Users/HP/AppData/Local/Programs/Python/Python37/python.exe (3.7.7) to create virtualenv…
so you want to work on the site or just run it?
just wnt to run it, after learning some more django i will try to contribute to site
y we are using 3.7 though?
@green mesa because nobody updated it yet 🤷♂️
hm, are you not using docker?
you shouldn't need to do this then

oh yea :/ , I use docker only for postgres
and run the site in debug mode in pycharm
oo postgres docker is different and site docker is different 😅
You’ll need it for your IDE completion though
its still installing thing
Ignoring pyuwsgi: markers 'sys_platform != "win32"' don't match your environment
It is normal
all dependencies installed now docker time UwU?
Well, you didn’t really need the- hang on
You don’t want to contribute to the site for now, do you?
Ah, you actually need it them
so i can work on main pybot
without build?
It will build itself
ok
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_version
docker.errors.DockerException: Error while fetching server API version: (2, 'CreateFile', 'The system cannot find the file specified.')
[10340] Failed to execute script docker-compose```
did you install docker-compose ?
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.
Let’s start again
start again
yes i have docker installed
well that wont work unless u got postgres running
Didn’t you just said you don’t want to use it?

he has to, atleast to run the postgres server
Starting in debug mode.
Starting server.
Starting in debug mode.
Waiting for PostgreSQL database.
Not ready yet, retrying.
Not ready yet, retrying.
Not ready yet, retrying.```
yes thats the problem
https://pythondiscord.com/pages/contributing/site/
have you gone through all these steps?
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.
only postgres step is left
pip install docker-compose ?
na i didnt did this
ok
and the site wont run unless postgres is setup
ok
pip install docker-compose?
@eternal owl you can install it using pip what
wow I never knew
yes
I have been using wget to download the binary ever since
@eternal owl still same error
which error
After running pip install docker-compose also
Do you have postgres running?
You don't need to if you use docker
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_version
docker.errors.DockerException: Error while fetching server API version: (2, 'CreateFile', 'The system cannot find the file specified.')
[10340] Failed to execute script docker-compose
This error
dont know y this error i am getting
Is docker running?
yeh
Does docker build . work ?
unable to prepare context: unable to evaluate symlinks in Dockerfile path: CreateFile C:\Users\HP\Desktop\site\Dockerfile: The system cannot find the file specified.
Well, are you 100% sure that the docker engine is installed and running?
yeh its running
docker symbol is coming
and it says docker desktop is running
Hello from Docker!
This message shows that your installation appears to be working correctly.
To generate this message, Docker took the following steps:
1. The Docker client contacted the Docker daemon.
2. The Docker daemon pulled the "hello-world" image from the Docker Hub.
(amd64)
3. The Docker daemon created a new container from that image which runs the
executable that produces the output you are currently reading.
4. The Docker daemon streamed that output to the Docker client, which sent it
to your terminal.
To try something more ambitious, you can run an Ubuntu container with:
$ docker run -it ubuntu bash
Share images, automate workflows, and more with a free Docker ID:
https://hub.docker.com/
For more examples and ideas, visit:
https://docs.docker.com/get-started/
PS C:\Users\HP\Desktop\site> ```
i just ran docker run hello-world
i restarted docker and compose is working now
Database is uninitialized and superuser password is not specified.
You must specify POSTGRES_PASSWORD to a non-empty value for the
superuser. For example, "-e POSTGRES_PASSWORD=password" on "docker run".
You may also use "POSTGRES_HOST_AUTH_METHOD=trust" to allow all
connections without a password. This is *not* recommended.
See PostgreSQL documentation about "trust":
https://www.postgresql.org/docs/current/auth-trust.html
failed to resize tty, using default size
The terminal process "C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe -Command docker run --rm -it -p 5432:5432/tcp postgres:12-alpine" terminated with exit code: 1.```
where to specify?
What are you trying to run? The site comes with a development config for the database that you should use
Why when syncing users I get TypeError: list indices must be integers or slices, not str?
is it normal
Deleting 'wiki\markitup\skins\markitup\images\bg-editor-textile.png'
Deleting 'wiki\markitup\skins\markitup\images\bg-editor-wiki.png'
Deleting 'wiki\markitup\skins\markitup\images\bg-editor-xml.png'
Deleting 'wiki\markitup\skins\markitup\images\bg-editor.png'
Deleting 'wiki\markitup\skins\markitup\images\handle.png'
Deleting 'wiki\markitup\skins\markitup\images\menu.png'
Deleting 'wiki\markitup\skins\markitup\images\submenu.png'
Deleting 'wiki\markitup\skins\simple\readme.txt'
and when server start at 0.0.0.0:8000 it says
This site can’t be reached The webpage at http://0.0.0.0:8000/ might be temporarily down or it may have moved permanently to a new web address.
The server deleting the statics is normal
The site can't be reached because you didn't set the host files, I think
i did that day only but let me cross check also
This deleting is a bit annoying...
I guess we can filter that out
Also, the website should be at http://pythondiscord.local
I have used localhost:8000 recently for development
Well, that should work too
if you use pycharm to run site, then the deletion process won't show up
Because that's how static collection works
You delete the current statics, and collect them again
I have seen some Django sites who use syncing for static files, instead of collecting all of them again. But they had a lot of static files and now their deployment progress is faster.
Well, we don't really have the need to just sync statics right now, we don't really have too many of them
I just realised, the newer django versions use pathlib instead of the os module
in the settings file
finally
i changed host file also same error
Would there be interest in a bot command to create a help voice server that the bot deletes when empty? I think that's something I could do but not sure how/where to contribute that in the repo
You should open an issue on the meta repo first https://git.pydis.com/meta
ok
Creating server with command that is available for everyone is not a good idea
other idea was have the bot create a voice channel when there was more than x amount of people in the help vc
Auto scaling in such way is good, but creating servers... nope.
ok, I'll go with more the second one then, could work pretty well with a Voice-Help category and have the bot check number of people in each channel in that category
Can someone review my PR here- https://github.com/python-discord/seasonalbot/pull/496
It's been almost 24 hours so,
on it
tysm
some people wait a few days to even weeks for a review, so 24 hours is not long
how can I add review comment on the lines that were not changed in the PR?
it doesn't allow me to
you can't
ah that sucks
you can get a permalink and then comment on it though
right
i got my first site PR merged yesterday but for some reason, my name doesn't appear on the contributors list
https://github.com/python-discord/site/graphs/contributors
pythondiscord.com - A Django and Bulma web application. - python-discord/site
I just found that in bot repo contributors statistics I'm at second place
could anyone review this if they're free? Its a very small fix so https://github.com/python-discord/seasonalbot/pull/497
@eternal owl I noticed same thing when my zodiac pr got merged but after few days it automatically detected by GitHub
Everytime I need to run docker compose up?
Well, everytime you want to start the containers
Ok
you can start it and leave it alone, even after u close the terminal
ye, a lot happens
Everything is automated
After everything 😐 same site cannot reached
How can I know docker default ip
Docker machine command is not working
https://stackoverflow.com/questions/60078434/docker-machine-command-not-found i think time to update guide
have you installed docker-machine? and to get docker-default ip, u can visit the hosts file
I have to install toolbox of docker right for machine?@eternal owl
@crude gyro
Here's your reminder: review Dementati PR.
[Jump back to when you created the reminder](#dev-contrib message)
what does cat C:\Windows\System32\Drivers\etc\hosts give for you? use git bash
maybe try installing docker-machine and try to get the ip
i installed toolbox and then i get UwU
PS C:\Users\HP\Desktop> docker-machine version
docker-machine.exe version 0.16.1, build cce350d7
nice
PS C:\Users\HP\Desktop\site> docker-machine ip default
Docker machine "default" does not exist. Use "docker-machine ls" to list machines. Use "docker-machine create" to add a new one.```
PS C:\Users\HP\Desktop\site> docker-machine ls
NAME ACTIVE DRIVER STATE URL SWARM DOCKER ERRORS```
#dev-log message why does it say 'Pull request closed' even though its merged -_-
Thats how github works
What'd be a point of having an open pull request if it has been merged
oh k
when I try to run site tests from pycharm, i get RuntimeError: Model class django.contrib.contenttypes.models.ContentType doesn't declare an explicit app_label and isn't in an application in INSTALLED_APPS.
But they work fine via terminal pipenv run test
@eternal owl on master ?
nop, on my own branch
Mhm
maybe I should update my branch 🤔
You need to pass the --build flag to docker compose to update the dependencies inside the container then @eternal owl
yes that I will do, I am talking about pycharm
tests run fine when I do pipenv run test
but they don't when I run them via pycharm
this is the config
I still haven't pulled latest changes, I will do that once I commit my changes, so no, the pipenv env is not updated
should that even matter?
Sounds like it is the culprit
cuz the tests run fine via terminal
It seems like the pycharm and the terminal interpreter isn't the same
Can you run pipenv --which and check that it is the same path than your config?
@eternal owl I found this: https://stackoverflow.com/a/49610002
it works, thanks a bunch @cold moon
@eternal owl replied to your question in the issue
cool cool cool
Im almost done with the UserEvents model, writing tests now
now i am stuck with one thing hyperv and virtualbox cant be started together
and docker machine is using virtual box
docker desktop uses hyperv
Now as site voice gate is ready, may I work on https://github.com/python-discord/bot/issues/1190 and https://github.com/python-discord/bot/issues/1193 ?
For our Voice Gate project, we need a new cog which will interact with the endpoints specified in python-discord/site#398. This issue cannot be solved until python-discord/site#398 is ready. Tasks ...
everything is configured only docker-machine ip default is not working
The ip docker uses should be in your hosts file under the docker entries
The 192.168.43.76 is your docker ip
@obtuse arrow 🎊
@patent pivot query mine plz
lol
The guide should be fixed to get the IP from there, or host.docker.internal added to the dev allowed hosts so it can be reached without going into the hosts file at all (at least on win)
can anyone help me on how to start with contribution
@brazen charm
You can't connect to 0.0.0.0; the docker ip should be added to your hosts file for the pydis domain names and connect to that
i have made changes through notepad
and saved there
but django start at this ip
with docker-compose up
Sure thing @snow scroll ! I'd recommend you to start with @dusky shore, are you familiar with GitHub? You should try to find an issue you'd like to work on, with no one already working on it https://git.pydis.com/seasonalbot. Welcome to our projects!
0.0.0.0 is a special ip, the server is listening for connections at the docker ip since it's running there. If you added the required config to the hosts file you should be able to reach it through pythondiscord.local and the other names
@green oriole yes I am significantly familiar with GitHub, thanks for your help!
Can any of core devs assign me to bot repo 1193 and 1190?
Done
Thanks 
@patent pivot Woot woot
But in Voice Gate cog issue, there is mentioned to check does user qualify, but there isn't said what is requirements to qualify?
I'll add them now
one sec
@cold moon issue updated
Ideally, this should be quite easily configurable, so probably in the constants file
when i manually typed out ip then i get this
pipenv run start always starting at 0.0.0.0:8000
current host file https://paste.pythondiscord.com/idodopavog.shell
i can make changes in settings.py?
ok
Use the pythondiscord.local name you added to the hosts file, no ips
Adding the entry to the hosts file is the last thing you should do with ips for the site
@brazen charm already i added https://paste.pythondiscord.com/idodopavog.shell
but django only start at 0.0.0.0
Ignore ips now
Put pythondiscord.local into your browser and you should get the site
admin
both?
as for both id and pass
finally site is working
i am very happy UwU
bcz of Numerlor i dont have to change anything in settings.py
You can also use an env var to override the allowed host setting @green mesa
ooo
y it happens when i check contribution section i am not able to check like normal
all those are stored in the production database
it ask to login and when i logged in ,it redirected to somewhere else
you can make your own in /admin
But after Dewikification is done, all pages should be shown locally too.
yep
lol
ohh nice, i didn’t even notice for frontend
grinding hard😅

