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

Debug flag / efficiency increases #4

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

snarkiergoblin
Copy link
Contributor

Added a debug flag to make debugging easier
Moved background colour determination to outside the while True, which should increase efficiency
Added where I intend to check for birds - will implement later

@snarkiergoblin snarkiergoblin changed the title Few changes Debug flag / efficiency increases Oct 2, 2019
…debugging screenshots

Added various features:

Removed math import and replaced with int() conversion, which should speed up the program and reduces the required number of imports

Added a time between the start of the program and the bot starting, to allow the user to switch tabs

Added screenshot saving capability, to help debugging and implementation of automatic resolution detection/screenshot positioning
@Raamyy
Copy link
Owner

Raamyy commented Oct 3, 2019

The background colour is used inside the while loop as the background colour changes during the gameplay (night mode).

@snarkiergoblin

@snarkiergoblin
Copy link
Contributor Author

Hi,

Sorry 🙃 I probably shouldn't try to program while I'm running on almost no sleep?

I hope this has fixed it, please do let me know if it hasn't.

Thanks :))

@Raamyy
Copy link
Owner

Raamyy commented Oct 4, 2019

No problems!

Try working on the duck feature.

@snarkiergoblin
Copy link
Contributor Author

Will do, I'm suddenly pretty busy this week so I'll have time next week :)

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.

2 participants