Skip to content
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

Network improvements #59

Merged
merged 15 commits into from
Jan 29, 2025
Merged

Network improvements #59

merged 15 commits into from
Jan 29, 2025

Conversation

@2bndy5 2bndy5 added the enhancement New feature or request label Jun 8, 2024

This comment was marked as off-topic.

@2bndy5 2bndy5 force-pushed the network-improvements branch from 63cbb01 to 8818abc Compare June 8, 2024 15:28
@2bndy5
Copy link
Member Author

2bndy5 commented Jun 17, 2024

I think the bug fixed in nRF24/RF24Network#229 is specific to the C++ implementation. Looking at the _write_to_pipe(), I forgot how much code gulf I played to speed things up.

There is no RF24Network.networkFlags here because

  • message fragmentation is done directly in _write_to_pipe() (making FLAG_FAST_FRAG obsolete)
  • FLAG_NO_PULL is implemented as _parenthood, even though there is no RF24Gateway/Ethernet implementation in this CirPy port

Furthermore, the network layers here don''t use a writefast() equivalent. Instead, I use a regular blocking write() (RF24.send()) and call RF24Network._tx_standby() which uses RF24.resend() similarly to how C++ RF24::txStandBy() uses RF24::reUseTx() but in a blocking manner.

@2bndy5 2bndy5 force-pushed the network-improvements branch from 24449d8 to 07668d4 Compare June 20, 2024 21:08
@2bndy5 2bndy5 force-pushed the network-improvements branch from e7a2916 to 685ced7 Compare June 28, 2024 23:27
@2bndy5 2bndy5 force-pushed the network-improvements branch from 685ced7 to cb9b60b Compare June 29, 2024 00:02
@2bndy5 2bndy5 marked this pull request as ready for review January 29, 2025 09:09
@2bndy5 2bndy5 merged commit 5462d59 into master Jan 29, 2025
8 checks passed
@2bndy5 2bndy5 deleted the network-improvements branch January 29, 2025 09:54
adafruit-adabot added a commit to adafruit/CircuitPython_Community_Bundle that referenced this pull request Jan 30, 2025
Updating https://github.com/2bndy5/CircuitPython_nRF24L01 to v2.1.4 from 2.1.3:
  > bump version to v2.1.4
  > Bump pypa/gh-action-pypi-publish from 1.10.3 to 1.12.4 (nRF24/CircuitPython_nRF24L01#65)
  > Bump codecov/codecov-action from 4 to 5 (nRF24/CircuitPython_nRF24L01#63)
  > Network improvements (nRF24/CircuitPython_nRF24L01#59)
  > prepare for trusted publisher releases to pypi
  > update list of linting tools used in CONTIBUTING guideline
  > update pre-commit hooks (nRF24/CircuitPython_nRF24L01#58)
  > Bump codecov/codecov-action from 3 to 4 (nRF24/CircuitPython_nRF24L01#57)
  > Bump actions/cache from 3 to 4 (nRF24/CircuitPython_nRF24L01#56)
  > Bump actions/upload-artifact from 3 to 4 (nRF24/CircuitPython_nRF24L01#55)
  > Bump actions/setup-python from 4 to 5 (nRF24/CircuitPython_nRF24L01#53)
  > Update pre commit hooks (nRF24/CircuitPython_nRF24L01#54)
  > Bump actions/checkout from 3 to 4 (nRF24/CircuitPython_nRF24L01#51)
  > Merge remote-tracking branch 'origin/use-dependabot'

Updating https://github.com/adafruit/CircuitPython_Community_Bundle/circuitpython_community_auto_library_list.md to NA from NA:
  > Added the following libraries: CircuitPython_FS3000
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant