Skip to content

macOS support + NT results-only publishing#12

Merged
TheTripleV merged 1 commit into
mainfrom
claude/macos-networktables-cleanup-wmAQP
Mar 17, 2026
Merged

macOS support + NT results-only publishing#12
TheTripleV merged 1 commit into
mainfrom
claude/macos-networktables-cleanup-wmAQP

Conversation

@TheTripleV
Copy link
Copy Markdown
Contributor

  • Mark macOS as a supported platform in Platform.java
  • Remove the platform support check in Main.java that exited on macOS
  • Remove all camera settings from NT publishing (pipeline index, driver mode, fps limit, camera intrinsics/distortion)
  • Remove the apriltag field layout subscriber from NT
  • NTDataPublisher now only publishes vision results to NT
  • PhotonCameraSim publishes intrinsics/distortion directly instead of through NTTopicSet

https://claude.ai/code/session_01EfivnKEs8qbAeEBHL6BoNn

Description

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

Include fun testing screenshots or a cool video, to collect test evidence in a place where we can later reference it. Including proof this change was tested makes reviewing easier, helps us make sure we tested all our edge cases, and helps provide context for the future.

Any issues this pull request closes or pull requests this supersedes should be linked with Closes #issuenumber.

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

- Mark macOS as a supported platform in Platform.java
- Remove the platform support check in Main.java that exited on macOS
- Remove all camera settings from NT publishing (pipeline index,
  driver mode, fps limit, camera intrinsics/distortion)
- Remove the apriltag field layout subscriber from NT
- NTDataPublisher now only publishes vision results to NT
- PhotonCameraSim publishes intrinsics/distortion directly instead
  of through NTTopicSet

https://claude.ai/code/session_01EfivnKEs8qbAeEBHL6BoNn
@TheTripleV TheTripleV merged commit 8c60a04 into main Mar 17, 2026
66 of 69 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants