In the Sunnylink API, a line was added to reset the last ping time to zero whenever we are trying to register. The purpose of this change is to ensure that registration attempts start with a clean slate, potentially improving the reliability of the registration process.
This code update sets Sunnylink to be disabled in the event that the registration does not go through successfully. This is to ensure that resources aren't wasted trying to access unregistered Sunnylink hardware.
Updated the error logging in sunnylink.py to include the module name for clarity. In addition, adjusted the status display in sidebar.cc to shorten the "REGISTERING" status text for a cleaner user interface.
This change enhances the Sunnylink registration process by including better error checks and exception handling for cases when the public key is in use or known to be not unique. The commit also adjusts the setting of the ping time right after a successful registration. Additionally, cleanup occurred in the UI to streamline some code relating to the display of Sunnylink status. The handling of the last Sunnylink ping time has been removed from the system manager.
* Fix for issue #19776
PyJWT 2.0.0 does not return `bytes` for `encode()` instead returns `str`. So converted the `str` to `bytes` and returned the resulting value
* added check for jwt.encode return type
* Update __init__.py
* Updated with suggested change
* Test to check return type of get_token()
The `get_token()` must return a `str` but for PyJWT version < 2.0.0 the `jwt.encode()` returns `bytes`. This test is to make sure if token returned by `get_token()` is `str`.
* Update test_get_token.py
* Update test_get_token.py
* Delete test_get_token.py
Co-authored-by: Willem Melching <willem.melching@gmail.com>
* use the openpilot/persist directory on PC
* manager runs on mac
* sim runs w/o carla
* fix params location in test
* that rmtree can fail and it's okay
* refactor params clear functionality
* set PARAMS_PATH
DevTekVE