-
Notifications
You must be signed in to change notification settings - Fork 7
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
Two quick commands after each other break #9
Comments
Hi Charlie,
Great to hear you get some use out of it. Unfortunately, I am not using
this myself and linux primarily at the moment. I will have to see where
I can work on this again to make a revised, better tested version. I
believe it was meant to debounce too fast commands. I will try and give
it a glance, but I can't promise anything.
Kind Regards,
Rudi
…On 2019-09-08 20:40, Charlie Bouthoorn wrote:
For example, tapping twice quickly after another gives clickclick
command not found, Eish. I think it's because the
client_sock.recv(1024) receives too much at once.
Another case would be
{"action":"mouse-move","x":1.811187744140625,"y":1.63995361328125}{"action":"mouse-move","x":2.9740753173828125,"y":0.71514892578125}
command not found, Eish
Other than that, this works great! Thank you for making an open-source
"It just works" equivalent. Other apps either didn't work, required
WiFi, or were paid, so I appreciate this project 😄
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub [1], or mute the
thread [2].
Links:
------
[1]
#9
[2]
https://github.com/notifications/unsubscribe-auth/AAOQPNVFOKEL34IEEEZAUPLQIVBJXANCNFSM4IUUMITQ
|
@C-Bouthoorn Please pull the latest version of the server. I have added some code to try and curb the multiple commands coming through. You may need to do this to get it to work: sudo rm -rf core core.pyc #deletes possibly compiled version
git pull origin master
sudo ./bluetooth_server.py This seems to make the mouse movement a bit smoother since it doesn't "drop" commands due to invalid JSON. Just a note on This is quite fine, since most of these command as tiny. The issue is actually on the phone app, it's sending multiple commands in one string. This fix should hopefully make it a bit better experience. |
It seems to fix the case where two movements follow each other, but the following is still broken:
|
For example, tapping twice quickly after another gives
clickclick command not found, Eish
. I think it's because theclient_sock.recv(1024)
receives too much at once.Another case would be
{"action":"mouse-move","x":1.811187744140625,"y":1.63995361328125}{"action":"mouse-move","x":2.9740753173828125,"y":0.71514892578125} command not found, Eish
Other than that, this works great! Thank you for making an open-source "It just works" equivalent. Other apps either didn't work, required WiFi, or were paid, so I appreciate this project 😄
The text was updated successfully, but these errors were encountered: