Skip to content

Conversation

@beckermr
Copy link
Collaborator

I am seeing if this has the same performance as the manual loops.

@codspeed-hq
Copy link

codspeed-hq bot commented Jan 30, 2025

CodSpeed Performance Report

Merging #141 will not alter performance

Comparing fori_loop_unroll (2dac00e) with main (3e45fb5)

Summary

✅ 29 untouched benchmarks
🆕 2 new benchmarks

Benchmarks breakdown

Benchmark BASE HEAD Change
🆕 test_benchmark_spergel_calcfluxrad[compile] N/A 15.4 s N/A
🆕 test_benchmark_spergel_calcfluxrad[run] N/A 2.1 ms N/A

@beckermr beckermr marked this pull request as ready for review January 30, 2025 18:54
@beckermr beckermr merged commit 84abf17 into main Jan 31, 2025
5 checks passed
@beckermr beckermr deleted the fori_loop_unroll branch January 31, 2025 22:48
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