Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Typo in "main.py" and Added "import colorsys" in background.py #21

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

renderman21
Copy link

Running the game, there is a sudden error in the console that most of them are "undefined". And when importing the files for main.py, there is a dot (.) before the referenced file, thus throwing off an error

Apparently, there is a dot (.), and it doesn't run properly.
There is a colorsys in line 9 and it needs to be defined properly.
@firefish111
Copy link

firefish111 commented Jul 6, 2021 via email

@phoebe-leong
Copy link

phoebe-leong commented Jul 7, 2021

youll have to edit all of the directory references

@phoebe-leong
Copy link

on your local version

@firefish111
Copy link

youll have to edit all of the directory references

but it works on Windows as well, have you not used cmd?

@firefish111
Copy link

@phoebe-leong this guy is right though, it's supposed to be ./ not just .

@renderman21
Copy link
Author

no no, . means the current directory, it needs that otherwise it will attempt to open a file in path, or what is system32 if you use that shitty abomination called Microsoft Windows

On Tue, Jul 6, 2021, 06:28 ph3b3 @.***> wrote: the . at the start of directorys is a unix thing i think — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#21 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AH5R7YUBGIECCP5FY3ZLG2LTWKIBHANCNFSM47DEJEBA .

Ah okay okay, I am using Linux in testing the game. And the . before the referenced python files such as player.py utils.py, etc., is a hidden file. Guessed I didn't account for Windows though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants