Skip to content

Parallel CO-BRAVO PoC#107

Draft
Shnatsel wants to merge 5 commits intomainfrom
co-bravo-parallel
Draft

Parallel CO-BRAVO PoC#107
Shnatsel wants to merge 5 commits intomainfrom
co-bravo-parallel

Conversation

@Shnatsel
Copy link
Copy Markdown
Collaborator

@Shnatsel Shnatsel commented Apr 10, 2026

Same approach as #97 but applies to CO-BRAVO tiles instead of the inner BRAVO

While the first two commits are 100% safe, the third introduced unsafe just to see what the performance ceiling is without any overhead. Still slower than sequential on 2 threads on my machine with dual-channel memory; needs measurement on servers with more memory channels.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 10, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.91%. Comparing base (3aa61ad) to head (2a16026).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #107      +/-   ##
==========================================
- Coverage   98.87%   97.91%   -0.97%     
==========================================
  Files          10       10              
  Lines        2227     2253      +26     
==========================================
+ Hits         2202     2206       +4     
- Misses         25       47      +22     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Shnatsel Shnatsel mentioned this pull request Apr 11, 2026
@Shnatsel
Copy link
Copy Markdown
Collaborator Author

No change on machines with dual-channel memory (presumably 2 threads already saturate memory bandwidth) but on an 8-channel server this gives a 30% improvement end-to-end, although the safe version only achieves a 10% improvement. We need to bring down the overhead of tracking disjoint regions, and add heuristics for selecting the parallel impl over the default one.

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.

2 participants