Skip to content

Conversation

yanxi0830
Copy link
Contributor

@yanxi0830 yanxi0830 commented Feb 21, 2025

What does this PR do?

Test Plan

  • test in following PR

@yanxi0830 yanxi0830 marked this pull request as ready for review February 21, 2025 03:57
@yanxi0830 yanxi0830 changed the title feat: agents resume turn (Sync updates from stainless branch: yanxi0830/dev) feat (1/n): agents resume turn (Sync updates from stainless branch: yanxi0830/dev) Feb 21, 2025
yanxi0830 added a commit to llamastack/llama-stack that referenced this pull request Feb 21, 2025
# What does this PR do?
- #1178
- #1187
- #1194

**client changes**
- llamastack/llama-stack-client-python#157
- llamastack/llama-stack-client-python#158

## Test Plan
```
LLAMA_STACK_BASE_URL=http://localhost:8321 pytest -v tests/client-sdk/agents/test_agents.py --inference-model meta-llama/Llama-3.1-8B-Instruct
```

```
LLAMA_STACK_CONFIG=fireworks pytest -v tests/client-sdk/agents/test_agents.py --inference-model meta-llama/Llama-3.1-8B-Instruct
```

<img width="1080" alt="image"
src="https://github.com/user-attachments/assets/6c48e062-5d00-4fed-98de-ecca627e5195"
/>

**llama-stack-apps**
```
python -m examples.agents.react_agent localhost 8321
```

- Test with script:
https://gist.github.com/yanxi0830/f2e407527f468998a700cd29fd271b15

**Output Before**: we have 2 `turn_id` with 2 turns
- https://gist.github.com/yanxi0830/9fbd7a80fcddc784a28c59d4a9c1d943

**Output After**: we have 1 `turn_id`, the final turn have all 3 steps
- https://gist.github.com/yanxi0830/17754d56d08ccbeaec419b693137500c

<img width="622" alt="image"
src="https://github.com/user-attachments/assets/ed7f42ca-41c8-4ee9-b6fa-2299901cafb4"
/>

**Telemetry**
<img width="1389" alt="image"
src="https://github.com/user-attachments/assets/031bf6ab-959c-46a7-aa85-70a511cba296"
/>


[//]: # (## Documentation)
# What does this PR do?
- #157

- Server change: llamastack/llama-stack#1194

## Test Plan
- See llamastack/llama-stack#1194

<img width="1080" alt="image"
src="https://github.com/user-attachments/assets/fb4cf70d-1c3d-423d-ac75-432c2a3505d7"
/>


[//]: # (## Documentation)
[//]: # (- [ ] Added a Changelog entry if the change is significant)
@yanxi0830 yanxi0830 merged commit 3cf8ac8 into main Feb 21, 2025
2 checks passed
@yanxi0830 yanxi0830 deleted the agents_resume branch February 21, 2025 21:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants