Skip to content

[WIP] Investigate difference between two robot poses#16

Closed
Copilot wants to merge 1 commit into
mainfrom
copilot/compare-robot-poses-difference
Closed

[WIP] Investigate difference between two robot poses#16
Copilot wants to merge 1 commit into
mainfrom
copilot/compare-robot-poses-difference

Conversation

Copilot AI commented Jan 16, 2026

Copy link
Copy Markdown
  • Investigate the difference between the two Pose imports in robot.py
  • Find where localPose is defined (from . import Pose as localPose)
  • Clean up redundant import - both Pose (from common) and localPose (from . import Pose) are the same class
  • Verify the change doesn't break any functionality
  • Run code review

Findings:

  • Line 11: from common import Pose - imports Pose class from /modules/common/common/__init__.py
  • Line 12: from . import Pose as localPose - imports Pose from the lsp package (via lsp/__init__.pylsp/pose.pycommon.Pose)
  • Both are the SAME class - localPose is just common.Pose re-exported through the lsp package
  • The redundant import can be removed, using Pose.cartesian_distance() instead of localPose.cartesian_distance()
Original prompt

What is the difference in these poses?

from common import Pose

vs
from . import Pose as localPose

Where is the second pose defined---that is imported later?


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@arnob2601 arnob2601 closed this Jan 16, 2026
@arnob2601 arnob2601 deleted the copilot/compare-robot-poses-difference branch January 16, 2026 04:00
Copilot AI requested a review from arnob2601 January 16, 2026 04:00
Copilot stopped work on behalf of arnob2601 due to an error January 16, 2026 04:00
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