Skip to content

feat: use None as default for trunc to avoid JAX branches on trunc value #191

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.

Sign up for GitHub

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

Closed
beckermr wants to merge 5 commits into from

fix: init galsim moffat objects correctly

fd10c6c
Select commit
Loading
Failed to load commit list.
Closed

feat: use None as default for trunc to avoid JAX branches on trunc value #191

fix: init galsim moffat objects correctly
fd10c6c
Select commit
Loading
Failed to load commit list.
CodSpeed HQ / CodSpeed Performance Analysis succeeded Feb 11, 2026 in 0s

Performance Gate Passed

⚠️ Unknown Walltime execution environment detected

Using the Walltime instrument on standard Hosted Runners will lead to inconsistent data.

For the most accurate results, we recommend using CodSpeed Macro Runners: bare-metal machines fine-tuned for performance measurement consistency.

⚡ 2 improved benchmarks
✅ 34 untouched benchmarks

Performance Changes

Mode Benchmark BASE HEAD Efficiency
⚡ Simulation test_benchmark_moffat_init[run] 491.8 µs 233.7 µs ×2.1
⚡ WallTime test_benchmark_moffat_init[run] 135.9 µs 25.5 µs ×5.3

Comparing moffat-trunc-to-none (fd10c6c) with main (51df286)

Open in CodSpeed
View more details on CodSpeed HQ