Skip to content

Add clarity to networking for multiple coprocs#2383

Merged
samfreund merged 5 commits intoPhotonVision:mainfrom
samfreund:clarify-multicoproc-static-ip
Mar 9, 2026
Merged

Add clarity to networking for multiple coprocs#2383
samfreund merged 5 commits intoPhotonVision:mainfrom
samfreund:clarify-multicoproc-static-ip

Conversation

@samfreund
Copy link
Member

Description

What changed? Why? (the code + comments should speak for itself on the "how")

We've had a couple people who didn't find it clear that multiple coprocessors required different static IP addresses, so this PR aims to clarify that point in the docs.

Meta

Merge checklist:

  • Pull Request title is short, imperative summary of proposed changes
  • The description documents the what and why, including events that led to this PR
  • If this PR changes behavior or adds a feature, user documentation is updated
  • If this PR touches photon-serde, all messages have been regenerated and hashes have not changed unexpectedly
  • If this PR touches configuration, this is backwards compatible with all settings going back to the previous seasons's last release (seasons end after champs ends)
  • If this PR touches pipeline settings or anything related to data exchange, the frontend typing is updated
  • If this PR addresses a bug, a regression test for it is added
  • If this PR adds a dependency, the license has been checked for compatibility and steps taken to follow it

@samfreund samfreund requested a review from a team as a code owner March 4, 2026 01:19
@github-actions github-actions bot added the documentation Anything relating to https://docs.photonvision.org label Mar 4, 2026
Copy link
Member

@Gold856 Gold856 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The link actually doesn't point to any particular section. Seems to have changed as part of the VH radio change. We likely should also specifically call out the .6-.19 range, since I think those are reserved for static IPs and are never assigned via DHCP.

@samfreund samfreund requested a review from Gold856 March 5, 2026 16:21
@samfreund samfreund force-pushed the clarify-multicoproc-static-ip branch from 74c3709 to 2315338 Compare March 5, 2026 16:23
@samfreund samfreund enabled auto-merge (squash) March 9, 2026 17:02
@samfreund samfreund merged commit cd3d9b0 into PhotonVision:main Mar 9, 2026
59 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Anything relating to https://docs.photonvision.org

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants