Skip to content

fix: merge session-only model analytics rows - #45582

Merged
teknium1 merged 1 commit into
mainfrom
hermes/hermes-c6397904
Jun 13, 2026
Merged

fix: merge session-only model analytics rows#45582
teknium1 merged 1 commit into
mainfrom
hermes/hermes-c6397904

Conversation

@teknium1

Copy link
Copy Markdown
Contributor

Summary

Models analytics now folds unaccounted session-only rows into the matching accounted provider row, so direct-provider OpenRouter usage does not render as a duplicate zero-token model card.

Changes

  • /api/analytics/models: merge zero-usage, no-provider rows into the sole accounted provider row for the same model when unambiguous.
  • Tests: cover the DeepSeek/OpenRouter duplicate-card shape with one session-only row plus one accounted row.

Validation

Check Result
python3 -m py_compile hermes_cli/web_server.py tests/hermes_cli/test_web_server.py pass via /home/teknium/.hermes/hermes-agent/.venv/bin/python
pytest tests/hermes_cli/test_web_server.py -q -k 'models_analytics_merges_session_only_duplicate or analytics_usage' 3 passed, 271 deselected

Note: the worktree has no local .venv; verification used the main checkout venv at /home/teknium/.hermes/hermes-agent/.venv/bin/python, which has the dashboard deps installed.

Infographic

Model Analytics Cleanup

@github-actions

Copy link
Copy Markdown
Contributor

🔎 Lint report: hermes/hermes-c6397904 vs origin/main

ruff

Total: 0 on HEAD, 0 on base (➖ 0)

🆕 New issues: none

✅ Fixed issues: none

Unchanged: 0 pre-existing issues carried over.

ty (type checker)

Total: 10889 on HEAD, 10889 on base (➖ 0)

🆕 New issues: none

✅ Fixed issues: none

Unchanged: 5711 pre-existing issues carried over.

Diagnostics are surfaced as warnings — this check never fails the build.

@alt-glitch alt-glitch added type/bug Something isn't working P3 Low — cosmetic, nice to have comp/cli CLI entry point, hermes_cli/, setup wizard labels Jun 13, 2026
@teknium1
teknium1 merged commit 0333a99 into main Jun 13, 2026
28 checks passed
@teknium1
teknium1 deleted the hermes/hermes-c6397904 branch June 13, 2026 12:52
AIalliAI pushed a commit to AIalliAI/Hermes that referenced this pull request Jun 14, 2026
T02200059 pushed a commit to T02200059/hermes-agent that referenced this pull request Jun 18, 2026
xyshanren pushed a commit to xyshanren/hermes-agent-cn that referenced this pull request Jun 25, 2026
waefrebeorn pushed a commit to waefrebeorn/slermes that referenced this pull request Jul 2, 2026
habarmc1223-sudo pushed a commit to habarmc1223-sudo/hermes-agent-fluxmem that referenced this pull request Jul 8, 2026
santhreal pushed a commit to santhreal/hermes-agent that referenced this pull request Jul 13, 2026
donbowman pushed a commit to donbowman/hermes-agent that referenced this pull request Jul 13, 2026
Gravezzz pushed a commit to Gravezzz/hermes-agent that referenced this pull request Jul 21, 2026
leewenjie pushed a commit to leewenjie/hermes-agent that referenced this pull request Aug 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/cli CLI entry point, hermes_cli/, setup wizard P3 Low — cosmetic, nice to have type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants