Skip to content

realsense stuff - #3449

Open
jeff-hykin wants to merge 5 commits into
mainfrom
jeff/feat/stereo_prep
Open

realsense stuff#3449
jeff-hykin wants to merge 5 commits into
mainfrom
jeff/feat/stereo_prep

Conversation

@jeff-hykin

@jeff-hykin jeff-hykin commented Aug 13, 2026

Copy link
Copy Markdown
Member

realsense stuff for stereo vslam

…ntrols

The camera publishes its own mount tree read off the device (factory
extrinsics) instead of xacro-transcribed constants; frames are named after
the camera model actually plugged in. Every sensor is pinned to the host
clock, the IMU runs at one negotiated rate reported back, and color
auto-exposure priority is exposed. Extracted from jeff/feat/cuvslam.
@codecov

codecov Bot commented Aug 13, 2026

Copy link
Copy Markdown

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
4259 1 4258 69
View the top 1 failed test(s) by shortest run time
dimos.e2e_tests.test_manipulation_planning_groups::test_single_arm_plans_and_executes_through_control_coordinator
Stack Traces | 136s run time
lcm_spy = <dimos.e2e_tests.lcm_spy.LcmSpy object at 0x714875801160>
start_blueprint = <function start_blueprint.<locals>.set_name_and_start at 0x714770d20400>

    def test_single_arm_plans_and_executes_through_control_coordinator(
        lcm_spy: LcmSpy,
        start_blueprint: Callable[..., DimosCliCall],
    ) -> None:
        """Plan with one arm and execute through its trajectory task."""
        _start_openarm_mock_planner(start_blueprint, lcm_spy)
    
        client = RPCClient(None, ManipulationModule)
        coordinator_client = RPCClient(None, ControlCoordinator)
        try:
>           [left] = _wait_for_groups(client, 1)

client     = <dimos.core.rpc_client.RPCClient object at 0x714772eaeba0>
coordinator_client = <dimos.core.rpc_client.RPCClient object at 0x71477114af60>
lcm_spy    = <dimos.e2e_tests.lcm_spy.LcmSpy object at 0x714875801160>
start_blueprint = <function start_blueprint.<locals>.set_name_and_start at 0x714770d20400>

dimos/e2e_tests/test_manipulation_planning_groups.py:158: 
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 

client = <dimos.core.rpc_client.RPCClient object at 0x714772eaeba0>, count = 1

    def _wait_for_groups(
        client: RPCClient,
        count: int,
        *,
        timeout: float = 120.0,
    ) -> tuple[PlanningGroupInfo, ...]:
        deadline = time.time() + timeout
        last_error: BaseException | None = None
        while time.time() < deadline:
            try:
                groups = client.list_planning_groups()
                if len(groups) == count:
                    return groups
            except Exception as exc:
                last_error = exc
            time.sleep(0.5)
>       raise TimeoutError(f"Timed out waiting for {count} planning groups") from last_error
E       TimeoutError: Timed out waiting for 1 planning groups

client     = <dimos.core.rpc_client.RPCClient object at 0x714772eaeba0>
count      = 1
deadline   = 1786682525.2814548
groups     = (PlanningGroupInfo('left_arm/manipulator', joints=('left_arm/openarm_left_joint1', 'left_arm/openarm_left_joint2', 'le...right_joint6', 'right_arm/openarm_right_joint7'), base='openarm_body_link0', tip='openarm_right_link7', gripper=False))
last_error = None
timeout    = 120.0

dimos/e2e_tests/test_manipulation_planning_groups.py:64: TimeoutError

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

@github-actions github-actions Bot added the ready-to-merge Required CI checks have passed on this PR label Aug 13, 2026
@jeff-hykin jeff-hykin closed this Aug 13, 2026
auto-merge was automatically disabled August 13, 2026 01:00

Pull request was closed

@jeff-hykin jeff-hykin reopened this Aug 14, 2026
@github-actions github-actions Bot removed the ready-to-merge Required CI checks have passed on this PR label Aug 14, 2026
@github-actions github-actions Bot added the ready-to-merge Required CI checks have passed on this PR label Aug 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PlzReview ready-to-merge Required CI checks have passed on this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant