Skip to content
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

chore(bench): update CodSpeed/action to v2 #54

Merged
merged 1 commit into from
Apr 27, 2024

Conversation

adriencaccia
Copy link
Contributor

Upgrading to the v2 of https://github.com/CodSpeedHQ/action will bring a better base run selection algorithm, better logging, and continued support.

Upgrading to the v2 of https://github.com/CodSpeedHQ/action will bring a better base run selection algorithm, better logging, and continued support.
Copy link
Owner

@Smirkey Smirkey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks !

Copy link

codspeed-hq bot commented Apr 27, 2024

CodSpeed Performance Report

Merging #54 will degrade performances by 60.38%

Comparing adriencaccia:patch-1 (01ca671) with main (db09f08)

🎉 Hooray! codspeed-rust just leveled up to 2.3.3!

A heads-up, this is a breaking change and it might affect your current performance baseline a bit. But here's the exciting part - it's packed with new, cool features and promises improved result stability 🥳!
Curious about what's new? Visit our releases page to delve into all the awesome details about this new version.

Summary

⚡ 1 improvements
❌ 9 regressions
✅ 139 untouched benchmarks

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Benchmarks breakdown

Benchmark main adriencaccia:patch-1 Change
test_rtree_nms[float32] 315.3 µs 531.3 µs -40.65%
test_rtree_nms[float64] 302.3 µs 521.7 µs -42.06%
test_rtree_nms[int16] 298.5 µs 504 µs -40.77%
test_rtree_nms[int32] 305.8 µs 508.8 µs -39.9%
test_rtree_nms[int64] 312.5 µs 510 µs -38.72%
test_rtree_nms_many_boxes[10000] 39.9 ms 95.3 ms -58.09%
test_rtree_nms_many_boxes[1000] 3.2 ms 7.2 ms -56.12%
test_rtree_nms_many_boxes[20000] 84.3 ms 212.7 ms -60.38%
test_rtree_nms_many_boxes[5000] 18.3 ms 45 ms -59.46%
parallel giou distance benchmark 2.6 ms 2 ms +28.34%

@Smirkey Smirkey merged commit 08c63a5 into Smirkey:main Apr 27, 2024
22 of 23 checks passed
@adriencaccia adriencaccia deleted the patch-1 branch April 29, 2024 02:09
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