Skip to content

Move metrics subtable to root PV table#2007

Merged
samfreund merged 1 commit intoPhotonVision:mainfrom
samfreund:nt-cluster-fix
Jul 22, 2025
Merged

Move metrics subtable to root PV table#2007
samfreund merged 1 commit intoPhotonVision:mainfrom
samfreund:nt-cluster-fix

Conversation

@samfreund
Copy link
Member

@samfreund samfreund commented Jul 22, 2025

Description

The metrics subtable is now published under the main PV table, as publishing it under the coproc table means that it's viewed as another MAC address, which it isn't.

Additionally adds a note that the coproc table is meant for MAC address stuff only.

Meta

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 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

@samfreund samfreund requested a review from a team as a code owner July 22, 2025 01:43
@Gold856 Gold856 changed the title Move metrics subtable to root pv table Move metrics subtable to root PV table Jul 22, 2025
@Gold856 Gold856 enabled auto-merge (squash) July 22, 2025 02:06
@Gold856 Gold856 disabled auto-merge July 22, 2025 02:24
@samfreund samfreund merged commit 02e6b6d into PhotonVision:main Jul 22, 2025
42 checks passed
@Gold856 Gold856 added the backend Things relating to photon-core and photon-server label Aug 4, 2025
@samfreund samfreund deleted the nt-cluster-fix branch August 8, 2025 04:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend Things relating to photon-core and photon-server

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants