Skip to content

feat(examples/avatar): added animation triggers and trimmed personas#5898

Closed
brycedli wants to merge 1 commit into
livekit:mainfrom
brycedli:main
Closed

feat(examples/avatar): added animation triggers and trimmed personas#5898
brycedli wants to merge 1 commit into
livekit:mainfrom
brycedli:main

Conversation

@brycedli
Copy link
Copy Markdown

Summary

  • Add wave/dance/turn tools for Leila, Jess, and Mr Fox via LemonSlice pose triggers (actions.py)
  • Auto-wave on session start and after switching to a motion-capable persona
  • Trim persona list from 15 to 9; add Leila and Jess, remove underperforming personas; default to Leila
  • Update playground dropdown and README

@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

devin-ai-integration[bot]

This comment was marked as resolved.

@brycedli brycedli marked this pull request as draft May 30, 2026 01:59
@brycedli brycedli closed this May 30, 2026
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

Devin Review found 1 new potential issue.

View 6 additional findings in Devin Review.

Open in Devin Review

Comment thread examples/avatar/README.md
Comment on lines +17 to +19
- **Hero motions** β€” for **Leila** only, the LLM can trigger wave, dance,
or turn via tool calls (one motion at a time, ~6 seconds each). She
waves automatically when the session starts.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟑 README claims motions are "for Leila only" but code enables them for Leila, Jess, and Mr Fox

The newly added README text states "Hero motions β€” for Leila only" but the code at examples/avatar/actions.py:17 defines ACTION_PERSONAS = frozenset({"leila", "jess", "mr_fox"}), and both Jess and Mr Fox personas have action-describing system prompts in examples/avatar/personas.py:80-83 and examples/avatar/personas.py:169-172. Users reading the README would not know that Jess and Mr Fox also support wave/dance/turn motions.

Suggested change
- **Hero motions** β€” for **Leila** only, the LLM can trigger wave, dance,
or turn via tool calls (one motion at a time, ~6 seconds each). She
waves automatically when the session starts.
- **Hero motions** β€” for **Leila**, **Jess**, and **Mr Fox**, the LLM can trigger wave, dance,
or turn via tool calls (one motion at a time, ~6 seconds each). They
wave automatically when the session starts.
Open in Devin Review

Was this helpful? React with πŸ‘ or πŸ‘Ž to provide feedback.

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