Commit
·
262c4bc
1
Parent(s):
a1b9bd7
style: Fix markdown lint (MD031/MD032 blank lines)
Browse files
docs/bugs/P2_ROUND_COUNTER_SEMANTIC_MISMATCH.md
CHANGED
|
@@ -46,6 +46,7 @@ message=f"Round {iteration}/{self._max_rounds} (~{est_display} remaining)"
|
|
| 46 |
### Why It Happens
|
| 47 |
|
| 48 |
In a multi-agent workflow with 4 agents (searcher, hypothesizer, judge, reporter):
|
|
|
|
| 49 |
- Each "round" involves the manager delegating to multiple agents
|
| 50 |
- Each agent completion fires an `ExecutorCompletedEvent`
|
| 51 |
- With 4+ agents, we see 4+ events per workflow round
|
|
@@ -57,6 +58,7 @@ In a multi-agent workflow with 4 agents (searcher, hypothesizer, judge, reporter
|
|
| 57 |
## Evidence From Logs
|
| 58 |
|
| 59 |
The session showed this progression:
|
|
|
|
| 60 |
```text
|
| 61 |
Round 1/5 - First agent completed
|
| 62 |
Round 2/5 - Second agent completed
|
|
@@ -203,6 +205,7 @@ est_seconds = rounds_remaining * 45
|
|
| 203 |
### Issue 4: Time Estimate Always Shows "~0s remaining"
|
| 204 |
|
| 205 |
Since `iteration` quickly exceeds `max_rounds`:
|
|
|
|
| 206 |
```python
|
| 207 |
rounds_remaining = max(self._max_rounds - iteration, 0)
|
| 208 |
# When iteration=11, max_rounds=5: rounds_remaining = max(5-11, 0) = 0
|
|
@@ -239,6 +242,7 @@ The time estimate becomes useless after the first few agent completions.
|
|
| 239 |
- A full round of 4 agents might only take 60s total
|
| 240 |
|
| 241 |
3. **API Discovery - Can Track Actual Rounds**:
|
|
|
|
| 242 |
```python
|
| 243 |
# These constants exist in agent_framework:
|
| 244 |
ORCH_MSG_KIND_INSTRUCTION = 'instruction'
|
|
@@ -272,6 +276,7 @@ The time estimate becomes useless after the first few agent completions.
|
|
| 272 |
- Removed unused `_EST_SECONDS_PER_ROUND` constant
|
| 273 |
|
| 274 |
2. **Added Semantic Agent Mapping** (`_get_agent_semantic_name`):
|
|
|
|
| 275 |
```python
|
| 276 |
def _get_agent_semantic_name(self, agent_id: str) -> str:
|
| 277 |
"""Map internal agent ID to user-facing semantic name."""
|
|
|
|
| 46 |
### Why It Happens
|
| 47 |
|
| 48 |
In a multi-agent workflow with 4 agents (searcher, hypothesizer, judge, reporter):
|
| 49 |
+
|
| 50 |
- Each "round" involves the manager delegating to multiple agents
|
| 51 |
- Each agent completion fires an `ExecutorCompletedEvent`
|
| 52 |
- With 4+ agents, we see 4+ events per workflow round
|
|
|
|
| 58 |
## Evidence From Logs
|
| 59 |
|
| 60 |
The session showed this progression:
|
| 61 |
+
|
| 62 |
```text
|
| 63 |
Round 1/5 - First agent completed
|
| 64 |
Round 2/5 - Second agent completed
|
|
|
|
| 205 |
### Issue 4: Time Estimate Always Shows "~0s remaining"
|
| 206 |
|
| 207 |
Since `iteration` quickly exceeds `max_rounds`:
|
| 208 |
+
|
| 209 |
```python
|
| 210 |
rounds_remaining = max(self._max_rounds - iteration, 0)
|
| 211 |
# When iteration=11, max_rounds=5: rounds_remaining = max(5-11, 0) = 0
|
|
|
|
| 242 |
- A full round of 4 agents might only take 60s total
|
| 243 |
|
| 244 |
3. **API Discovery - Can Track Actual Rounds**:
|
| 245 |
+
|
| 246 |
```python
|
| 247 |
# These constants exist in agent_framework:
|
| 248 |
ORCH_MSG_KIND_INSTRUCTION = 'instruction'
|
|
|
|
| 276 |
- Removed unused `_EST_SECONDS_PER_ROUND` constant
|
| 277 |
|
| 278 |
2. **Added Semantic Agent Mapping** (`_get_agent_semantic_name`):
|
| 279 |
+
|
| 280 |
```python
|
| 281 |
def _get_agent_semantic_name(self, agent_id: str) -> str:
|
| 282 |
"""Map internal agent ID to user-facing semantic name."""
|