plover icon indicating copy to clipboard operation
plover copied to clipboard

Error disabling timeout

Open AndrewHess opened this issue 2 years ago • 4 comments

Describe the bug When I try to disable the timeout for the connection to my keyboard, I get an error and the timeout remains enabled. The following is logged to plover.log

2022-12-05 16:17:33,746 [Dummy-1] ERROR: engine update failed
Traceback (most recent call last):
  File "/Users/runner/work/plover/plover/build/osxapp/Plover.app/Contents/Frameworks/Python.framework/Versions/Current/lib/python3.9/site-packages/plover/engine.py", line 147, in run
  File "/Users/runner/work/plover/plover/build/osxapp/Plover.app/Contents/Frameworks/Python.framework/Versions/Current/lib/python3.9/site-packages/plover/engine.py", line 189, in _update
  File "/Users/runner/work/plover/plover/build/osxapp/Plover.app/Contents/Frameworks/Python.framework/Versions/Current/lib/python3.9/site-packages/plover/config.py", line 396, in update
  File "/Users/runner/work/plover/plover/build/osxapp/Plover.app/Contents/Frameworks/Python.framework/Versions/Current/lib/python3.9/site-packages/plover/config.py", line 202, in validate
TypeError: float() argument must be a string or a number, not 'NoneType'

To Reproduce

  1. From the Plover UI go to Configure -> Machine
  2. Select GeminiPR for the machine
  3. In options, scroll to timeout and disable the "Use timeout" field
  4. Select Ok to apply the change

This gives a error message, and reopening that setting page shows the timeout is still enabled.

Expected behavior The timeout should be disabled.

The reason I want to do this is so that I won't need to manually reconnect my machine every time I log into my computer. But I had trouble finding documentation on this Timeout setting, so if this doesn't do what I suspect, I'm open to alternatives.

Operating system MacOS 13.0 Plover 4.0.0dev12

Keyboard Moonlander Mark 1 from ZSA

AndrewHess avatar Dec 05 '22 21:12 AndrewHess

Hi Andrew, I’m encountering the same issue - did you find a solution?

Peter-Col avatar Aug 12 '23 07:08 Peter-Col

Nope I haven't

AndrewHess avatar Aug 14 '23 04:08 AndrewHess

Thanks for letting me know!

On 14 August 2023 at 2:14:50 pm, Andrew Hess @.***) wrote:

Nope I haven't

— Reply to this email directly, view it on GitHub https://github.com/openstenoproject/plover/issues/1579#issuecomment-1676645769, or unsubscribe https://github.com/notifications/unsubscribe-auth/AWAB372G627SR3SAMIH6WU3XVGQ3VANCNFSM6AAAAAASUYUCZ4 . You are receiving this because you commented.Message ID: @.***>

Peter-Col avatar Aug 14 '23 09:08 Peter-Col

I don't think that "timeout" value is what allows reconnecting the machine on sleep.

Maybe refer to #1560 . (I haven't looked at those.)

user202729 avatar Aug 19 '23 03:08 user202729