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(powerboxesrs): stop using useless epsilon value #56

Merged
merged 1 commit into from
May 4, 2024

Conversation

Smirkey
Copy link
Owner

@Smirkey Smirkey commented May 4, 2024

No description provided.

Copy link

codspeed-hq bot commented May 4, 2024

CodSpeed Performance Report

Merging #56 will improve performances by 29.28%

Comparing remove/utils/eps (cbf16d8) with main (3d615a6)

🎉 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

⚡ 5 improvements
✅ 144 untouched benchmarks

Benchmarks breakdown

Benchmark main remove/utils/eps Change
test_box_convert_cxcywh_xywh[int64] 64.8 µs 50.9 µs +27.33%
test_box_convert_xywh_xyxy[float64] 63.2 µs 49.3 µs +28.39%
test_box_convert_xywh_xyxy[int64] 62.4 µs 48.3 µs +29.28%
test_box_convert_xyxy_cxcywh[float64] 65.7 µs 51.5 µs +27.62%
test_tiou_distance[int64] 1.4 ms 1.2 ms +20.04%

@Smirkey Smirkey merged commit f9e08fd into main May 4, 2024
23 checks passed
@Smirkey Smirkey deleted the remove/utils/eps branch May 4, 2024 16:51
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.

None yet

1 participant