Add telemetry functions to BLEClient and Explore classes.#379
Closed
Kieran-Conroy wants to merge 1037 commits into
Closed
Add telemetry functions to BLEClient and Explore classes.#379Kieran-Conroy wants to merge 1037 commits into
Kieran-Conroy wants to merge 1037 commits into
Conversation
* change unstable bt threshold to .5s * add hint for disabled BT
* change unstable bt threshold to .5s * add hint for disabled BT
works for bin2csv at the moment but this might change in future for bt streaming
* change unstable bt threshold to .5s * add hint for disabled BT * save last exg timestamp * count samples lost in seconds * update missing sample count for sudden disconnection * clear bt_duration after interval
Update fletcher bytes
Put False into result queue on connection error
…version pass channel count to bin converter
…4.1.0 Bumpversion to 4.1.0
Release 4.1.0
add mne loader script
MNE data loader
… plotting library
… square plot), added cmap cycling with c key, documentation
…d call to use glfw backend
…time-spectrogram Apis 1286 real time spectrogram
RT spectrogram
Firmware version and battery have been tested only using the mock server. RTT and RSSI have not been tested (no pro device). - Changed bt_mock_client.py slightly to fix some problems I was getting with the mock server. - Implemented asynchronous methods `get_rssi` and `measure_rtt` in BLEClient for retrieving RSSI values and measuring round-trip time. - Updated Explore class to include methods for fetching the latest battery level, firmware version, RSSI, and RTT - Added ENV packet subscription to capture battery level updates. - Added a short telemetry_test.py script to test the functions.
Member
|
Hi Kieran, thank you for the PR. I had a quick test with a Pro device and it seems like some modification is needed for that case. I am gettting the following error:
Happy to merge this when you receive the pro device. Cheers! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Firmware version and battery have been tested only using the mock server. RTT and RSSI have not been tested (no pro device).
get_rssiandmeasure_rttin BLEClient for retrieving RSSI values and measuring round-trip time.