search

django / django-asv

Benchmarks for Django using asv
MIT License
40 stars 12 forks source link

Added Truncator Benchmarks #70

Closed smithdc1 closed 1 year ago

smithdc1 commented 1 year ago

Added benchmarks for ticket #30686

@carltongibson -- Can I add this to your list? Aim is then to try and use the benchmark label on django/django

smithdc1 commented 1 year ago

May be a bit much to add this permanently to the benchmark suite 🤷

carltongibson commented 1 year ago

Super. Thanks @smithdc1 — I will take a look next week! 👍

smithdc1 commented 1 year ago

Just a short note to say I expect these benchmarks to show my current patch is much much slower (20x?!) for short truncations.

I've got a fix which will make it faster (2x?), but it needs more work to refine it.

So while I'd like it considering if these are suitable benchmarks. I need a bit more time to refine my proposed patch.

Something like asv continuous main [my prs commit hash] should show the diff.