Skip to content

Refactor load analyzer: extract report formatter, improve token/select-release parsing and worker-type handling#12

Merged
mouxinqq merged 2 commits intodevelopfrom
codex/update-skill-creator-for-user-input
Apr 12, 2026
Merged

Refactor load analyzer: extract report formatter, improve token/select-release parsing and worker-type handling#12
mouxinqq merged 2 commits intodevelopfrom
codex/update-skill-creator-for-user-input

Conversation

@mouxinqq
Copy link
Copy Markdown
Owner

Motivation

  • Improve accuracy and readability of the Load Analyzer by fixing token/select-release parsing bugs and separating report formatting from analysis logic.
  • Make select/release matching more robust by excluding token-only releases from request lifecycle matching and by normalizing worker types (prefill/decode/mixed).
  • Provide both a short summary and a detailed report file for load analysis to be saved alongside other diagnostic details.
  • Update skill prompts in SKILL.md to make AskUserQuestion options explicit.

Description

  • Added analyzers/load_report.py which contains format_load_report(result) that returns (summary_text, detail_text) and renders tables/sparklines using chart utilities.
  • Updated analyzers/load.py to import format_load_report, removed the embedded formatter, fixed regexes for token events, corrected regex group indexing, added _normalize_worker_type, and adjusted internal token parsing and _grep_lines pattern for token releases.
  • Enhanced log_parser.py token/select regexes and match_select_release logic to normalize worker types, treat token releases separately, only use counter releases for matching request lifecycles, and produce per_worker and type_summary with token select/release counts and inferred token selects for mixed workers.
  • Updated troubleshoot.py to accept the (summary, detail) output from format_load_report, include the detailed load report in the details dict, and save it as detail/load_select_release.md when present.
  • Minor documentation tweaks in SKILL.md to clarify discrete AskUserQuestion options and defaults.

Testing

  • No automated tests were added or run as part of this change.

Codex Task

@mouxinqq mouxinqq merged commit a75419e into develop Apr 12, 2026
20 of 25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant