Northeastern-Electric-Racing / Northeastern-Electric-Racing/Calypso
Test if non-byte-sized signed values can be successfully read in by Calypso
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 2
- Forks
- 2
- Avg merge
- 6d 20h
- Merged PRs (30d)
- 3
Description
Current Features
Right now in RustSynth.py, (big endian) signed fields are read in using reader.read_signed_in. The bitstream-io docs indicate that read_signed_in should correctly read in signed values, even if the field size isn't 8, 16, or 32.
Desired Additional Features
Test this using the stuff in bitstream.c. Add a signed int to a non-byte-sized field in a bitstream, send it over CAN, and see if Calypso reads it correctly. Probably do this a few more times just to make sure
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reading RustSynth.py and the existing bitstream.c setup to understand how signed fields are encoded and sent over CAN. Add a non-byte-sized signed value to the bitstream, send it through the existing path, and verify that Calypso reads the expected value. Repeat with several field sizes or values so the test covers more than one case.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python, rust
- Domain
- embedded-iot, testing-qa
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100