feat(bench): emit compact workload category metadata - #7663
Merged
Merged
Conversation
Co-authored-by: Derek Cofausper <256792747+decofe@users.noreply.github.com>
Co-authored-by: Derek Cofausper <256792747+decofe@users.noreply.github.com>
Co-authored-by: Derek Cofausper <256792747+decofe@users.noreply.github.com>
Co-authored-by: Derek Cofausper <256792747+decofe@users.noreply.github.com>
onbjerg
approved these changes
Sep 16, 2026
onbjerg
enabled auto-merge
September 16, 2026 11:58
shekhirin
approved these changes
Sep 16, 2026
sds
added a commit
that referenced
this pull request
Sep 17, 2026
…ows-rollout * origin/main: feat(consensus): ensure certificate tip epoch matches height index (#7671) fix(ci): restore Docker PR build notifications (#7667) fix(consensus): enforce network identity baked into binary/chainspec (#7630) fix(xtask): simplify DKG outcome arguments (#7381) feat(ci): trigger Docker image builds from PR comments (#7655) feat(bench): emit compact workload category metadata (#7663) feat: allow expiring nonce discriminators (#7474) perf(consensus): avoid rehashing locally built proposals (#7650) # Conflicts: # .github/workflows/docker-profiling.yml # .github/workflows/docker.yml
This was referenced Sep 17, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Emit
workload_mix_version=1andworkload_mix_weightsonly forpublic-mix, extracting and aggregating.mixdirectly from its prepared file with jq-compatible yq (pinned via uv; requires jq). Remove the recursive YAML metadata reader so every other workload, includingtip20:nonce=2d, retains its existing metadata path without parsing extra YAML or invoking yq.Previously validated locally (test file removed as requested): public-mix, 20,002 expanded entries, large YAML integers, ignored includes, input preservation, and non-public-mix bypasses; the query also exactly matches both saved live benchmark reports from run 35087133011. No new live benchmark was run for this simplification.
Companions: perf reader #166 (deploy first), multi-region sender #409. Historical migration remains unperformed.
Prompted by: @onbjerg