-
Notifications
You must be signed in to change notification settings - Fork 0
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
implement nRF24L01 driver #1
Draft
2bndy5
wants to merge
55
commits into
main
Choose a base branch
from
impl-rf24
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Conversation
This file contains 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
Member
2bndy5
commented
Jul 9, 2024
•
edited
Loading
edited
- add source code to drive the nRF24L01
- add a build/test/lint CI workflows; no release CI for crates.io (yet)
- add unit tests using embedded-hal-mock (and activate codecov)
- add (Linux only) FFI bindings (consider FFI bindings #2)
- python binding
- node binding
- add examples
- python examples
- node.js (typescript) examples
- rust examples
2bndy5
added
documentation
Improvements or additions to documentation
enhancement
New feature or request
labels
Jul 9, 2024
includes - CI workflow - funding info - dependabot config
- prefer to `use rf24_rs::radio::prelude::*` in user code to pull in all the required radio traits. - support LNA_CUR bit in non-plus/clones - prefer to `use rf24_rs::radio::{RF24 Nrf24Error}` in user code because namespaces will be very important in the future
Welcome to Codecov 🎉Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests. Thanks for integrating Codecov - We've got you covered ☂️ |
- add badges - tick first milestone in roadmap
rust is so cool! This new macro generates rust code at compile time and then compiles everything with the generated code in place of the macro calls.
removes the need to construct a mutable byte
This comment was marked as off-topic.
This comment was marked as off-topic.
2bndy5
force-pushed
the
impl-rf24
branch
2 times, most recently
from
October 8, 2024 06:07
aec4db1
to
6031361
Compare
introduce supplemental docs (using mkdocs) and WIP examples (incomplete at this time) migrate to using [just](https://just.systems) as a task runner and utilize it in CI jobs.
3 tasks
2bndy5
force-pushed
the
impl-rf24
branch
5 times, most recently
from
October 29, 2024 09:33
6b84c93
to
d747b35
Compare
|
considering we might be distributing multiple rust crates from 1 repo. also review rust docs - added API layout to landing page - filled in some missing info in functions' docs review `print_details()` output - show pipe information in `print_details()` - show TX address in `print_details()` - fix IRQ settings' output
2bndy5
force-pushed
the
impl-rf24
branch
3 times, most recently
from
October 30, 2024 07:13
47f9a35
to
a959097
Compare
Only the node binding's `StatusFlags` and `WriteConfig` interfaces must use explicit boolean values. To implicitly cast those interfaces' values to boolean would - incur much more overhead - lose precise typing information See napi-rs docs about Env and Context.
2bndy5
force-pushed
the
impl-rf24
branch
4 times, most recently
from
November 10, 2024 13:10
aec0850
to
9a70482
Compare
and add colors to examples/node/ts/scanner.ts
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
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.