openstenoproject / plover

Open source stenotype engine

Home Page:http://opensteno.org/plover

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Error disabling timeout

AndrewHess opened this issue · 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

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

Nope I haven't

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.)