Skip to content

feat: Report per-ShaderGroup complexity statistics leaderboard#2120

Open
lgritz wants to merge 1 commit into
AcademySoftwareFoundation:mainfrom
lgritz:lg-compstats
Open

feat: Report per-ShaderGroup complexity statistics leaderboard#2120
lgritz wants to merge 1 commit into
AcademySoftwareFoundation:mainfrom
lgritz:lg-compstats

Conversation

@lgritz
Copy link
Copy Markdown
Collaborator

@lgritz lgritz commented May 25, 2026

Store the new compile-time complexity metrics that were added by PR #2085 (which printed the the stats in the log as each group compiles) as fields on ShaderGroup so they survive after optimization.

Expose them via getattribute(group, "stat:compiled_active_layers") etc., and extend getstats() to emit a "Shader compilation stats, post-optimized" section listing min/max/median and a leaderboard with a ranked top-N list for each metric (controlled by the existing "stat:rank_groups" option).

Tests: testsuite/compstats/ test covering both the getstats output and the getattribute keys. Added a testshade --print-group-stats flag for querying per-group stat keys.

Assisted-by: Claude Code / claude-sonnet-4-6

Store the new compile-time complexity metrics that were added by PR
2085 (but only printed the the log as each group compiles) as fields
on ShaderGroup so they survive after optimization.

Expose them via getattribute(group, "stat:compiled_active_layers")
etc., and extend getstats() to emit a "Shader compilation stats,
post-optimized" section listing min/max/median and a leaderboard with
a ranked top-N list for each metric (controlled by the existing
"stat:rank_groups" option).

Tests: testsuite/compstats/ test covering both the getstats output and
the getattribute keys. Added a testshade --print-group-stats flag for
querying per-group stat keys.

Assisted-by: Claude Code / claude-sonnet-4-6

Signed-off-by: Larry Gritz <lg@larrygritz.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant