Skip to content

Consolidate the unreflected axis baseline test - #120

Merged
kou merged 1 commit into
enactic:mainfrom
stevenaya:codex/consolidate-axis-tests
Nov 4, 2026
Merged

kou merged 1 commit into
enactic:mainfrom
stevenaya:codex/consolidate-axis-tests

Conversation

@stevenaya

Copy link
Copy Markdown
Contributor

The revolute mirror test now checks its unreflected input axis before comparing reflected motion. This incorporates the existing standalone baseline assertion and avoids one extra Xacro expansion.

Validation

  • 27 tests pass on ROS 2 Jazzy.
  • The oblique-component/limit test and the prismatic and principal-axis cases retain their existing assertions.

Copilot AI balanced review requested due to automatic review settings November 4, 2026 09:16

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.

An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.

@kou
kou merged commit bc45157 into enactic:main Nov 4, 2026
10 checks passed

This branch was successfully deployed

1 active deployment
release — 40754e73 Deployed Nov 4, 2026 by stevenaya via Publish #337
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.

3 participants