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

few improvements #12

Merged
merged 1 commit into from
Oct 21, 2023
Merged

few improvements #12

merged 1 commit into from
Oct 21, 2023

Conversation

conradRz
Copy link
Contributor

@conradRz conradRz commented Oct 20, 2023

  • lines 18-21 --- on Windows, this pull request introduces new functionality. It checks if the user is inactive (no mouse/keyboard usage) and if so, it goes to sleep. I use it, as quite often I fall asleep or leave, and the less prolific's API is used, the less it costs, the less likely they will ban us/or disable it

  • line 120 --- it works as it is. It goes straight to the study, therefore there is no # TODO

  • lines 145-170 --- script pausing functionality implementation. Most likely this is impossible to get done under PyAutoGui, as it's too dumb, I checked. Shame, as PyAutoGui would make it cross platform (Linux, Mac)

  • lines 162-167 --- unnecessary

  • lines 191-204 --- implementation of script sleeping on the Windows platform

line 18-21 on Windows, this pull request introduces new functionality. It checks if the user is inactive (no mouse/keyboard usage) and if so, it goes to sleep. I use it, as quite often I fall asleep or leave, and the less prolific's API is used, the less it costs, the less likely they will ban us/or disable it

- line 120 it works as it is. It goes straight to the study, therefore there is no # TODO

- line 145-170 script pausing functionality implementation. Most likely this is impossible to get done under PyAutoGui, as it's too dumb, I checked. Shame, as PyAutoGui would make it cross platform (Linux, Mac)

lines 162-167 unnecessary

lines 191-204 implementation of script sleeping on the Windows platform
@UnMars UnMars merged commit a8376b4 into UnMars:main Oct 21, 2023
1 check passed
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