Add telemetry #7
Open
Conversation
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
* add gcr lookup table * refactor, add skeleton state implementation * add telem variables * return bool from dshot_cmd decode * move all code related to dshot_cmd inside the state add telem state add switching to telem state * add telem annotations * store dshot kbaud make dshot period lookup dynamic * add rough received bit handling no error handling etc, no limit on # of bits (crashes) * tidyup * transition back to cmd state after 21 bits * remove previously remove var * fix high bit * use bitwise operators for bit add skeleton functions * default to bidir * add annotation types * add start/end for telem packets move debugging to annotation * add force edt option * fix gcr table * update annotations * initial attempt at xoring and gcr * checksums match, send it * tidyup * tidyup * add crc handling * cleanup * cleanup
* initial - back to motor command bits barely recognized * motor bits displayed correctly, checksum passing failing on telem * tidyup, move telem data * telem bits rxed again, no reset * Split modules off into package for unit testing
* initial unit testing * seperate and test xor and gcr * add packet test and fix regressions of telem handling * fix regressions of telem handling * add custom exception * fix erpm unit tests * tidyup * move add_bit to dshot_common * reimplement telem data display fix state handling add dshot cmd state * add test for telem zero packet * raise error on CRC fail * add .idea to .gitignore
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
TODO
Nice to haves/for next release: