Skip to content

RPC Timeout implementation#550

Closed
leshy wants to merge 1682 commits intodevfrom
rpcstress
Closed

RPC Timeout implementation#550
leshy wants to merge 1682 commits intodevfrom
rpcstress

Conversation

@leshy
Copy link
Contributor

@leshy leshy commented Aug 12, 2025

No description provided.

@leshy leshy changed the base branch from main to dev August 12, 2025 23:11
@leshy leshy changed the title Rpcstress RPC Timeout implementation Aug 14, 2025
alexlin2 and others added 27 commits December 9, 2025 01:44
Former-commit-id: 9bfb1c4 [formerly 4f5bbf5]
Former-commit-id: 4680c0c
Former-commit-id: 90694ff [formerly e54306e]
Former-commit-id: ebc000d
Former-commit-id: 3a66364 [formerly 653bc35]
Former-commit-id: f2beb1c
Former-commit-id: 1b0c6ca [formerly 0599c19]
Former-commit-id: ea92ef3
Former-commit-id: f7f939a [formerly d1b96c1]
Former-commit-id: 965c098
Former-commit-id: 3f8758d [formerly 4d64e55]
Former-commit-id: 44db38a
Former-commit-id: ed5829b [formerly c4493f2]
Former-commit-id: 9999b74
Former-commit-id: 1847ee1 [formerly 6ac4da1]
Former-commit-id: 83dee96
Former-commit-id: 6b50a28 [formerly 6a760fa]
Former-commit-id: c9a765d
Former-commit-id: 2d7a457 [formerly fa41361]
Former-commit-id: 02b1969
Former-commit-id: 76f9cc2 [formerly 0f5a214]
Former-commit-id: af709c4
Former-commit-id: 6508f3b [formerly 9c73136]
Former-commit-id: 2012821
Former-commit-id: 2150c91 [formerly bb32bb4]
Former-commit-id: b2caefb
Former-commit-id: 3e4ea11 [formerly 8a54752]
Former-commit-id: 2bff5e1
Former-commit-id: fb44ed1 [formerly 31cb901]
Former-commit-id: ca449ac
Former-commit-id: 0f3a0e6 [formerly 39ba624]
Former-commit-id: d57b64c
Former-commit-id: fe988e8 [formerly cc0bf7b]
Former-commit-id: bc79646
Former-commit-id: 2cf56c3 [formerly cebc383]
Former-commit-id: e1746f7
Former-commit-id: 216c056 [formerly 2b9a02e]
Former-commit-id: dc6f153
Former-commit-id: a3500a8 [formerly eb733fc]
Former-commit-id: 669a9a7
Former-commit-id: 4d98704 [formerly cb9fc5f]
Former-commit-id: 206783d
Former-commit-id: cbd1adf [formerly 3e0cd6c]
Former-commit-id: 984d923
Former-commit-id: 0838e49 [formerly 5441bf1]
Former-commit-id: 4f3263a
Former-commit-id: 6bf5144 [formerly bebc9fa]
Former-commit-id: 3b5a9c2
Former-commit-id: df93de7 [formerly 1e8da3f]
Former-commit-id: bac1db4
Former-commit-id: 8d0fe31 [formerly b1804a3]
Former-commit-id: a324a64
alexlin2 and others added 16 commits December 12, 2025 09:15
Former-commit-id: 0329e52 [formerly 1ba7361]
Former-commit-id: ed70d16
Former-commit-id: 77e020e [formerly 80191fa]
Former-commit-id: 72703f3
Former-commit-id: 679ceb9 [formerly 5e3ae8e]
Former-commit-id: 6e815fa
Former-commit-id: be5783f [formerly 41c8657]
Former-commit-id: d3e987d
Former-commit-id: 2c7b55a [formerly 2679359]
Former-commit-id: 2a01f30
Former-commit-id: b1f041f [formerly 18cda83]
Former-commit-id: affc323
Former-commit-id: 84c15fa [formerly df12393]
Former-commit-id: e11bac1
…haul

# Perception and Navigation Architecture Overhaul

Major refactoring to improve modularity and organization of perception and navigation systems.

## Key Changes

### Core
- Added timeout to RPC calls

### Navigation
- Added recovery server
- Added stuck checking
- Added orientation to Path                                                                                                                                                                                                                                 

### 👁️ Perception Improvements
- Enhanced object tracking with improved spatial perception
- Updated manipulation modules to use standardized DimOS types
- Removed deprecated visual servoing code

### ⚙️ Core Infrastructure
- Enhanced LCM pub/sub with observable operators
- Improved message types (OccupancyGrid, Image, Header)
- Added comprehensive transform utilities and testing
- New camera module for monocular depth estimation

### 🤖 skill refactor
- All navigation skills now work again

## Impact
This refactoring establishes a cleaner separation of concerns between robot-specific code and reusable navigation/perception modules, setting up the foundation for more scalable development.

Former-commit-id: f25afaf [formerly 001cda9]
Former-commit-id: 56bf4eb
Former-commit-id: 8c91f77 [formerly fb55748]
Former-commit-id: 7bb183d
…execution and processing

Skill coordinator

```py

    # we register a hook to call an agent with state of skills update (when something finished evaluating, when voice command has been received etc)
    agentInterface = AgentInterface(agent_callback=print)
    agentInterface.start()

    testContainer = TestContainer()

    # we can register a skill container (even if deployed remotely as a module)
    agentInterface.register_skills(testContainer)

    # we can investigate availabe skills (to provide to an agent)
    print(agentInterface.skills())

    # we can execute a skill
    agentInterface.execute_skill("delayadd", 1, 2)

    # while skill is executing, we can introspect the state
    # (we see that the skill is running)
    time.sleep(0.25)
    print(agentInterface)
    time.sleep(0.75)
    # skill has finished, we can check the messages we received from it and it's state
    print(agentInterface)
```

adds `agentspy` cli

<img width="1526" height="1260" alt="2025-08-06_00-46" src="https://github.com/user-attachments/assets/2a2723da-3d3b-448f-9342-e37b801125c4" />



Former-commit-id: 25db809 [formerly 61ead46]
Former-commit-id: ee579c0
Former-commit-id: 2f653d3 [formerly 2deb8c4]
Former-commit-id: 16b4412
Former-commit-id: 914a6d4 [formerly 8a1cd03]
Former-commit-id: 2a772c0
Former-commit-id: 00f0e8d [formerly 8e1cc73]
Former-commit-id: 7a36e07
Former-commit-id: b66ba72 [formerly 08ea604]
Former-commit-id: b40db9b
@spomichter spomichter requested a review from a team January 8, 2026 13:59
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Jan 8, 2026

Too many files changed for review.

1 similar comment
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Jan 8, 2026

Too many files changed for review.

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Jan 8, 2026

Too many files changed for review.

2 similar comments
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Jan 8, 2026

Too many files changed for review.

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Jan 8, 2026

Too many files changed for review.

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Jan 8, 2026

Too many files changed for review.

@paul-nechifor
Copy link
Contributor

Closed because it's old and it's before the rebase.

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.

5 participants