UI - #537
Conversation
Greptile SummaryThe PR adds a LingBot browser UI and example-scene catalog while extending live prompt handling and WebRTC integration.
Confidence Score: 3/5The PR is not yet safe to merge because prompt submissions remain excluded from canonicalization and clearing a prompt leaves stale model conditioning active. The current source capability still cannot feed the changed text-event converter, while the clear path continues selecting the active prompt instead of restoring the base prompt; these previously reported failures remain reachable in the browser prompt flow. Files Needing Attention: integrations/lingbot/lingbot/input_mapping.py, integrations/lingbot/lingbot/webrtc/session.py Important Files Changed
Sequence DiagramsequenceDiagram
participant B as Browser UI
participant W as WebRTC input source
participant C as Canonicalizer
participant M as LingBot input mapping
participant S as Model session
B->>W: Camera or text event
W->>C: Timed user-input window
C->>M: Canonical controls
M->>S: Camera trajectory and prompt conditioning
S-->>B: Generated video chunk
Reviews (15): Last reviewed commit: "run" | Re-trigger Greptile |
| else: | ||
| # No event_id and no dynamic prompt - keep active or restore base | ||
| prompt = self._active_prompt or self._base_prompt |
There was a problem hiding this comment.
Clear retains stale conditioning
When a user activates a catalog or dynamic prompt and then clears it, this branch prefers _active_prompt, which still contains the selected prompt, instead of restoring _base_prompt. Subsequent chunks therefore continue using the scene conditioning the user attempted to clear.
Knowledge Base Used: Interactive world-model integrations
|
Want your agent to iterate on Greptile's feedback? Try greploops. |
No description provided.