-
Notifications
You must be signed in to change notification settings - Fork 29
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
option to disable keypress listener on linux. #36
Comments
index 200e4c7..96588c9 100644
--- a/requirements.txt
+++ b/requirements.txt
@@ -5,5 +5,4 @@ PyAutoGUI~=0.9.53
chess~=1.9.3
stockfish~=3.28.0
packaging~=21.3
-keyboard~=0.13.5
-PyQt5~=5.15.7
\ No newline at end of file
+PyQt5~=5.15.7
diff --git a/src/gui.py b/src/gui.py
index 3b1fab5..c91a1c1 100644
--- a/src/gui.py
+++ b/src/gui.py
@@ -12,7 +12,6 @@ from webdriver_manager.chrome import ChromeDriverManager
from overlay import run
from stockfish_bot import StockfishBot
from selenium.common import WebDriverException
-import keyboard
class GUI:
@@ -346,11 +345,6 @@ class GUI:
if not self.opened_browser:
continue
- if keyboard.is_pressed("1"):
- self.on_start_button_listener()
- elif keyboard.is_pressed("2"):
- self.on_stop_button_listener()
-
def on_open_browser_button_listener(self):
# Set Opening Browser button state to opening
self.opening_browser = True
diff --git a/src/stockfish_bot.py b/src/stockfish_bot.py
index 415c599..db08110 100644
--- a/src/stockfish_bot.py
+++ b/src/stockfish_bot.py
@@ -11,7 +11,6 @@ import re
from grabbers.chesscom_grabber import ChesscomGrabber
from grabbers.lichess_grabber import LichessGrabber
from utilities import char_to_num
-import keyboard
class StockfishBot(multiprocess.Process):
@@ -192,9 +191,6 @@ class StockfishBot(multiprocess.Process):
((int(move_start_pos[0]), int(move_start_pos[1])), (int(move_end_pos[0]), int(move_end_pos[1]))),
])
while True:
- if keyboard.is_pressed("3"):
- break
-
if len(move_list) != len(self.grabber.get_move_list()):
self_moved = True
move_list = self.grabber.get_move_list() quick and dirty attempt to get rid of
UPD: it works just fine with a fresh account. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Linux users can't just run it with sudo, this will cause issues with graphical apps that use X (chrome, for example).
The text was updated successfully, but these errors were encountered: