DuskSystems / wayfind

A speedy, flexible router for Rust.
Apache License 2.0
10 stars 0 forks source link

Move regex to feature #56

Closed CathalMullan closed 3 months ago

CathalMullan commented 3 months ago

Not sure how to handle benchmarks/CI for multiple features.

Also, magic cache is causing issues with rate-limiting to often for my liking.

codspeed-hq[bot] commented 3 months ago

CodSpeed Performance Report

Merging #56 will degrade performances by 2.25%

Comparing 55-move-regex-support-to-an-optional-feature (61923a4) with main (d0f3b01)

Summary

⚡ 2 improvements ❌ 1 (👁 1) regressions ✅ 17 untouched benchmarks

Benchmarks breakdown

Benchmark main 55-move-regex-support-to-an-optional-feature Change
👁 path-tree 7.8 µs 8 µs -2.25%
wayfind 7.7 µs 7.3 µs +4.8%
wayfind 44.8 µs 43.4 µs +3.29%