Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update benchmark results after gpuc2 upgrade #316

Merged
merged 1 commit into from
Dec 4, 2024

Conversation

fknorr
Copy link
Contributor

@fknorr fknorr commented Dec 3, 2024

We added more RAM to our benchmark CI runner gpuc2 yesterday, so I figured it's a good time to re-run benchmarks.

There is a noticeable regression in the task-graph group, which is reproducible. Not sure what causes this.

@fknorr fknorr requested review from psalz and PeterTh December 3, 2024 10:02
@fknorr fknorr self-assigned this Dec 3, 2024
Copy link

github-actions bot commented Dec 3, 2024

Check-perf-impact results: (000e2892abb21ddd1ae413a5c86d7d95)

⚠️ Significant slowdown (>1.25x) in some microbenchmark results: generating large task graphs / soup topology

Relative execution time per category: (mean of relative medians)

  • command-graph : 1.03x
  • graph-nodes : 0.98x
  • grid : 1.00x
  • instruction-graph : 1.03x
  • scheduler : 1.02x
  • system : 0.95x
  • task-graph : 1.13x ⚠️

@coveralls
Copy link

Pull Request Test Coverage Report for Build 12137288342

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 94.926%

Totals Coverage Status
Change from base Build 12119181429: 0.0%
Covered Lines: 7136
Relevant Lines: 7254

💛 - Coveralls

@fknorr fknorr merged commit b494ea0 into master Dec 4, 2024
17 checks passed
@fknorr fknorr deleted the gpuc2-upgrade-benchmark branch December 9, 2024 14:32
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.

3 participants