Skip to content

chore(registry): mirror deprecated flag on the two retired yaw metrics (DS-1163) - #11

Merged
monazhu merged 1 commit into
mainfrom
fix/ds-1163-registry-yaw-deprecated
Sep 10, 2026
Merged

monazhu merged 1 commit into
mainfrom
fix/ds-1163-registry-yaw-deprecated

Conversation

@monazhu

@monazhu monazhu commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Resolves DS-1163. Follows cvr-cortex PR #25 (DS-1160).

Change

R/registry.R: add c3d.metrics.boundary_relative_yaw and c3d.metrics.virtual_yaw to deprecated_registry_keys (sorted position), bump the "Last synced" header to 2026-09-10.

Both are retired session circular-mean yaw metrics. No producer since July 2025. Neither is in compact_columns, so compact output is unchanged.

Verification

tests/testthat/test-compact-columns.R with SLICER_YAML pointing at cortex main:

  • with this change: 5 passed
  • without it: fails at line 82, "R/registry.R deprecated_registry_keys is stale — resync it."

cognitive3dr is not in cortex's registry-dispatch.yml fan-out, so this mirror is updated by hand. Whether to add it to the fan-out is noted on the ticket.

🤖 Generated with Claude Code

…s (DS-1163)

cvr-cortex marked c3d.metrics.boundary_relative_yaw and c3d.metrics.virtual_yaw
deprecated (DS-1160): session circular-mean yaw metrics, no producer since
July 2025. test-compact-columns cross-checks this list against the registry
and was one resync behind.
@monazhu
monazhu marked this pull request as ready for review September 10, 2026 21:49
@monazhu monazhu self-assigned this Sep 10, 2026

@matt-manuel matt-manuel left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@monazhu
monazhu merged commit 1309b88 into main Sep 10, 2026
6 checks passed
@monazhu
monazhu deleted the fix/ds-1163-registry-yaw-deprecated branch September 10, 2026 21:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants