Skip to content

chore[docs]: Recommend ethernet tether over port forwarding for VH-109#1847

Merged
Gold856 merged 1 commit intoPhotonVision:mainfrom
samfreund:patch-1
Apr 11, 2025
Merged

chore[docs]: Recommend ethernet tether over port forwarding for VH-109#1847
Gold856 merged 1 commit intoPhotonVision:mainfrom
samfreund:patch-1

Conversation

@samfreund
Copy link
Member

@samfreund samfreund commented Mar 27, 2025

closes #1825

The new radio has four ethernet ports, and one dedicated port for the DS. There's no reason for teams to tether to the roborio via USB-B, and tethering via radio will provide more realistic match conditions. Therefore, recommend tethering via radio.

Merge checklist:

  • Pull Request title is short, imperative summary of proposed changes
  • The description documents the what and why
  • 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 settings back to v2024.3.1
  • If this PR addresses a bug, a regression test for it is added

@samfreund samfreund requested a review from a team as a code owner March 27, 2025 03:36
@samfreund samfreund changed the title chore[docs]: Recommend ethernet tether for new radios chore[docs]: Recommend ethernet tether over port forwarding for VH-109 Mar 27, 2025
mcm001
mcm001 previously requested changes Mar 27, 2025
@samfreund
Copy link
Member Author

Not sure why the linting is failing, I'll run the linger when I make it home.

@samfreund samfreund added the documentation Anything relating to https://docs.photonvision.org label Apr 9, 2025
@samfreund samfreund dismissed mcm001’s stale review April 11, 2025 04:33

We discussed it in the discord

@Gold856 Gold856 dismissed their stale review April 11, 2025 04:52

Fixed

@Gold856 Gold856 enabled auto-merge (squash) April 11, 2025 04:53
@Gold856 Gold856 merged commit a93be9c into PhotonVision:main Apr 11, 2025
38 of 39 checks passed
@samfreund samfreund deleted the patch-1 branch April 11, 2025 18:54
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.

Recommend tethering via Ethernet instead of USB+Port forwarding

4 participants