Skip to content

ENH: In GHA display ccache configuration and statistics#5924

Merged
hjmjohnson merged 1 commit intoInsightSoftwareConsortium:mainfrom
blowekamp:ccache_stats
Mar 11, 2026
Merged

ENH: In GHA display ccache configuration and statistics#5924
hjmjohnson merged 1 commit intoInsightSoftwareConsortium:mainfrom
blowekamp:ccache_stats

Conversation

@blowekamp
Copy link
Copy Markdown
Member

PR Checklist

  • No API changes were made (or the changes have been approved)
  • No major design changes were made (or the changes have been approved)
  • Added test (or behavior not changed)
  • Updated API documentation (or API not changed)
  • Added license to new files (if any)
  • Added Python wrapping to new files (if any) as described in ITK Software Guide Section 9.5
  • Added ITK examples for all new major features (if any)

Refer to the ITK Software Guide for
further development details if necessary.

@blowekamp blowekamp requested a review from hjmjohnson March 11, 2026 18:06
@github-actions github-actions Bot added type:Infrastructure Infrastructure/ecosystem related changes, such as CMake or buildbots type:Enhancement Improvement of existing methods or implementation labels Mar 11, 2026
Copy link
Copy Markdown
Member

@dzenanz dzenanz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No objections from me.

@dzenanz
Copy link
Copy Markdown
Member

dzenanz commented Mar 11, 2026

In here: Cacheable calls: 48 / 4231 ( 1.13%). This seems worrying.

@dzenanz
Copy link
Copy Markdown
Member

dzenanz commented Mar 11, 2026

In the Ubuntu workflow: Hits: 0 / 4258 (0.00 %)

Copy link
Copy Markdown
Member

@dzenanz dzenanz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes look good. Are the cache hit rates expected to be this low?

@blowekamp blowekamp marked this pull request as ready for review March 11, 2026 21:58
@hjmjohnson hjmjohnson merged commit 680450e into InsightSoftwareConsortium:main Mar 11, 2026
8 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:Enhancement Improvement of existing methods or implementation type:Infrastructure Infrastructure/ecosystem related changes, such as CMake or buildbots

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants