Skip to content

Feat/implement fusion for irfft#4736

Merged
Sublime12 merged 12 commits intomainfrom
feat/implement-fusion-for-irfft
Apr 7, 2026
Merged

Feat/implement fusion for irfft#4736
Sublime12 merged 12 commits intomainfrom
feat/implement-fusion-for-irfft

Conversation

@Sublime12
Copy link
Copy Markdown
Contributor

@Sublime12 Sublime12 commented Apr 6, 2026

Checklist

  • Confirmed that cargo run-checks command has been executed.
  • Made sure the book is up to date with changes in this PR.

Changes

Add the fusion implementation for irfft

Testing

# activate the tests in cubecl.rs and:
cargo test-wgpu fft

Copy link
Copy Markdown
Member

@louisfd louisfd left a comment

Choose a reason for hiding this comment

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

LGTM

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 6, 2026

Codecov Report

❌ Patch coverage is 0% with 54 lines in your changes missing coverage. Please review.
✅ Project coverage is 63.33%. Comparing base (c3917da) to head (eaceb04).
⚠️ Report is 6 commits behind head on main.

Files with missing lines Patch % Lines
crates/burn-fusion/src/ops/module.rs 0.00% 21 Missing ⚠️
crates/burn-ir/src/operation.rs 0.00% 20 Missing ⚠️
crates/burn-router/src/runner.rs 0.00% 7 Missing ⚠️
crates/burn-fusion/src/stream/context.rs 0.00% 6 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4736      +/-   ##
==========================================
- Coverage   63.37%   63.33%   -0.05%     
==========================================
  Files        1111     1111              
  Lines      146015   146120     +105     
==========================================
+ Hits        92542    92546       +4     
- Misses      53473    53574     +101     

☔ 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.

Base automatically changed from feat/implement-fusion-for-rfft to main April 7, 2026 13:11
Copy link
Copy Markdown
Member

@laggui laggui left a comment

Choose a reason for hiding this comment

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

Just have to resolve the conflicting changes from #4735 which was just merged

@Sublime12 Sublime12 merged commit 0eb82b5 into main Apr 7, 2026
11 checks passed
@Sublime12 Sublime12 deleted the feat/implement-fusion-for-irfft branch April 7, 2026 17:17
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