GalSim-developers / JAX-GalSim

JAX port of GalSim, for parallelized, GPU accelerated, and differentiable galaxy image simulations.
Other
25 stars 3 forks source link

feat: add more benchmarks and clean up code #102

Closed beckermr closed 2 months ago

beckermr commented 2 months ago

This PR adds more benchmarks and cleans up the benchmark code to remove duplication.

codspeed-hq[bot] commented 2 months ago

CodSpeed Performance Report

Merging #102 will create unknown performance changes

Comparing more-bench (1684968) with main (b8348bd)

Summary

🆕 11 new benchmarks ⁉️ 2 (👁 2) dropped benchmarks

Benchmarks breakdown

Benchmark main more-bench Change
🆕 test_benchmarks_interpolated_image[compile] N/A 77 ms N/A
🆕 test_benchmarks_interpolated_image[run] N/A 77.3 ms N/A
👁 test_benchmarks_interpolated_image_jit_compile 1.6 s N/A N/A
👁 test_benchmarks_interpolated_image_jit_run 77.8 ms N/A N/A
🆕 test_benchmarks_lanczos_interp[kval-conserve_dc-compile] N/A 42.4 ms N/A
🆕 test_benchmarks_lanczos_interp[kval-conserve_dc-run] N/A 42.5 ms N/A
🆕 test_benchmarks_lanczos_interp[kval-no_conserve_dc-compile] N/A 3.9 ms N/A
🆕 test_benchmarks_lanczos_interp[kval-no_conserve_dc-run] N/A 4.2 ms N/A
🆕 test_benchmarks_lanczos_interp[xval-conserve_dc-compile] N/A 2.3 ms N/A
🆕 test_benchmarks_lanczos_interp[xval-conserve_dc-run] N/A 2.5 ms N/A
🆕 test_benchmarks_lanczos_interp[xval-no_conserve_dc-compile] N/A 1.7 ms N/A
🆕 test_benchmarks_lanczos_interp[xval-no_conserve_dc-run] N/A 1.7 ms N/A
🆕 test_benchmarks_metacal[run] N/A 1.4 s N/A
beckermr commented 2 months ago

This PR has no functional changes and only adds benchmarks for testing further changes in #101. Thus I am going to merge it. Happy to push changes later.