Skip to content

Fix Enzyme differentiation with constant Hartley inputs - #10

Merged
kazuakiyama merged 1 commit into
mainfrom
fix/enzyme-const-hartley-input
Oct 25, 2026
Merged

kazuakiyama merged 1 commit into
mainfrom
fix/enzyme-const-hartley-input

Conversation

@kazuakiyama

Copy link
Copy Markdown
Member

Summary

  • Add Enzyme forward- and reverse-mode rules for constant Hartley transform inputs.
  • Add Float32 and Float64 regression tests for both AD modes.
  • Enable StationaryRandomFields.jl to remove redundant Enzyme rules.

Testing

  • Pkg.test() passed.

@kazuakiyama
kazuakiyama force-pushed the fix/enzyme-const-hartley-input branch from ddef6d8 to ae496b1 Compare October 25, 2026 06:24
@kazuakiyama

Copy link
Copy Markdown
Member Author

Fix the origin of this issue. EHTJulia/StationaryRandomFields.jl#13 (comment)

@codecov

codecov Bot commented Oct 25, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 68.75000% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 86.12%. Comparing base (0656e3d) to head (ae496b1).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
ext/AbstractFastHartleyTransformsEnzymeCoreExt.jl 68.75% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #10      +/-   ##
==========================================
- Coverage   87.89%   86.12%   -1.78%     
==========================================
  Files           5        5              
  Lines         157      173      +16     
==========================================
+ Hits          138      149      +11     
- Misses         19       24       +5     

☔ View full report in Codecov by Harness.
📢 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.

@kazuakiyama
kazuakiyama merged commit 16883be into main Oct 25, 2026
4 of 6 checks passed
@kazuakiyama
kazuakiyama deleted the fix/enzyme-const-hartley-input branch October 25, 2026 06:41
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