Repository navigation
Conversation
pranayD21
marked this pull request as ready for review
November 6, 2026 03:35
pranayD21
force-pushed
the
pranay/flight-ethernet-daq
branch
from
November 10, 2026 05:44
c4d3611 to
e86bd46
Compare
Ported onto main's src/ layout (#187-#189); replaces the four earlier commits on this branch with one. - src/chips/w5500_udp.c: socket 0 UDP, 2 KiB rings, wrap-safe transfers, SEND_OK/TIMEOUT handling. - src/app/daq_{wire,records,network,task}.c: BOARD_HEARTBEAT and SENSOR_DATA from lib/daq-protocol, SERVER_HEARTBEAT / SENSOR_CONFIG handling, server discovery and pinning, 10 s expiry, UID-derived MAC. - Networking stays disabled (daq_board_config) until #180 is signed off and the DAQ server can route the board. - tests/ethernet now also runs in ./flight_test (26/26 green). DHCP is not implemented: #180's static .81 fallback conflicts with the MAC-derived recovery address every other board moved to in #171. See src/app/ETHERNET-DAQ.md. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
pranayD21
force-pushed
the
pranay/flight-ethernet-daq
branch
from
November 13, 2026 04:30
e86bd46 to
b57102c
Compare
This branch has not been deployed
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
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.
Update (Nov 9): rebased onto main's
src/layout as a single commit../flight_testpasses 26/26 and now includestests/ethernet;pio run -e benchalso builds. Networking is still disabled.Decision needed before DHCP: #180 proposes a static
192.168.2.81fallback, but #171 has since moved every other board to a MAC-derived169.254.x.yrecovery address and takes the board ID from the lease. #513 says to copy the other boards. @inez9, which should the flight computer follow? Details are insrc/app/ETHERNET-DAQ.md.The flight computer could identify its W5500 but had no Ethernet data path. This adds a bounded UDP driver and a separate network task that sends the existing version-0 BOARD_HEARTBEAT and SENSOR_DATA packets. Sensor/GNSS producers copy selected fields into a nonblocking queue while retaining eMMC logging.
Following issue #180, the disabled default profile now proposes board ID 81, board 192.168.2.81:5005, subnet 255.255.255.0, and server 192.168.2.20:5006. This supersedes the earlier local .20:5006 placeholder. A local unicast MAC is derived from the STM32's full 96-bit UID; the physical board's resulting MAC still needs collision checking and registration.
Discovery-enabled operation broadcasts heartbeats until a valid SERVER_HEARTBEAT or supported SENSOR_CONFIG arrives, pins the responding server, then forgets it after ten seconds of silence. Invalid, unmapped and actuator packets cannot claim or refresh that association. Fixed-server filtering remains available. The task drops stale samples, stops telemetry without configuration, and recovers from link/SPI/send failures. It never dispatches actuator or flight-state commands.
Validation:
Deployment remains incomplete; networking stays disabled pending the sign-off requested in #180:
ETHERNET-DAQ.md records the implemented behavior, proposed configuration, and remaining work from #180. This PR does not close #180 or STARProject #513.
🤖 Generated with Claude Code