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

KeyError: 'inactivity_timeout' when trying to use l-band spartn corrections: #120

Closed
semuadmin opened this issue Feb 28, 2024 · 0 comments · Fixed by #121
Closed

KeyError: 'inactivity_timeout' when trying to use l-band spartn corrections: #120

semuadmin opened this issue Feb 28, 2024 · 0 comments · Fixed by #121
Assignees
Labels
bug Something isn't working

Comments

@semuadmin
Copy link
Contributor

There's a similar problem when trying to use l-band spartn corrections:

$ ./venvs/pygps/bin/pygpsclient
Exception in thread Thread-4 (_read_thread):
Traceback (most recent call last):
File "/usr/lib/python3.11/threading.py", line 1038, in _bootstrap_inner
self.run()
File "/usr/lib/python3.11/threading.py", line 975, in run
self._target(*self._args, **self._kwargs)
File "/home/drd/venvs/pygps/lib/python3.11/site-packages/pygpsclient/stream_handler.py", line 120, in _read_thread
inactivity_timeout = settings["inactivity_timeout"]
~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^
KeyError: 'inactivity_timeout'

It looks like the code that launches the reader thread doesn't provide an inactivity timeout in on_connect() in spartn_lband_frame.py

(That might have been on version 1.4.9 since I installed an older version to get around the first problem.)

Originally posted by @divenal in #118 (comment)

@semuadmin semuadmin changed the title There's a similar problem when trying to use l-band spartn corrections: KeyError: 'inactivity_timeout' when trying to use l-band spartn corrections: Feb 28, 2024
@semuadmin semuadmin added the bug Something isn't working label Feb 28, 2024
@semuadmin semuadmin self-assigned this Feb 28, 2024
semuadmin added a commit that referenced this issue Feb 28, 2024
@semuadmin semuadmin mentioned this issue Feb 28, 2024
12 tasks
@semuadmin semuadmin added this to the 1.4.12 milestone Feb 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant