Skip to content

perf: run ablations for the paper #1408

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

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from
Draft

perf: run ablations for the paper #1408

wants to merge 8 commits into from

Conversation

avik-pal
Copy link
Collaborator

No description provided.

@avik-pal avik-pal changed the base branch from main to ap/common_optimization_options June 18, 2025 03:15
@avik-pal avik-pal force-pushed the ap/paper_benchmarks branch 3 times, most recently from 1c0438a to f06786c Compare June 18, 2025 22:31
Base automatically changed from ap/common_optimization_options to main June 18, 2025 23:47
@avik-pal avik-pal force-pushed the ap/paper_benchmarks branch 2 times, most recently from 14817ba to f2c5e75 Compare June 19, 2025 01:38
@avik-pal
Copy link
Collaborator Author

DeepONet

image

Fourier Neural Operator

image

Need to investigate if the pass pipelines are being correctly set the results seem quite strange honestly

@avik-pal
Copy link
Collaborator Author

FNO case is heavily dominated by FFTs so nothing much we can do there, but the gradients with passes enabled being consistently slower seems off

@wsmoses
Copy link
Member

wsmoses commented Jun 19, 2025

I assume this is all straight line (no control flow like while), and we have no fft specific optimizations yet

in which case this also becomes a question of how well xla’s allocation and other passes may recognize certain patterns

@wsmoses
Copy link
Member

wsmoses commented Jun 19, 2025

But also concurrently we built the passes for the primal and post jax differentiation , so we may be missing optimizations relevant to enzyme generated programs

@avik-pal avik-pal force-pushed the ap/paper_benchmarks branch from 139af10 to 70f90b7 Compare June 19, 2025 15:54
@avik-pal avik-pal changed the base branch from main to ap/more_option_fixes June 19, 2025 15:55
Base automatically changed from ap/more_option_fixes to main June 20, 2025 22:28
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