Skip to content

Benchmark using the cache for a few misses#199

Merged
seddonym merged 1 commit intomasterfrom
add-benchmark-for-mostly-cached
Apr 4, 2025
Merged

Benchmark using the cache for a few misses#199
seddonym merged 1 commit intomasterfrom
add-benchmark-for-mostly-cached

Conversation

@seddonym
Copy link
Collaborator

@seddonym seddonym commented Apr 4, 2025

It's useful to distinguish this scenario from the others, as it's a common situation which we should make sure we optimize for.

It's useful to distinguish this scenario from the others, as it's a
common situation which we should make sure we optimize for.
@codspeed-hq
Copy link

codspeed-hq bot commented Apr 4, 2025

CodSpeed Performance Report

Merging #199 will not alter performance

Comparing add-benchmark-for-mostly-cached (b4504ce) with master (a311e6c)

Summary

✅ 18 untouched benchmarks
🆕 3 new benchmarks
⁉️ 1 (👁 1) dropped benchmarks

Benchmarks breakdown

Benchmark BASE HEAD Change
👁 test_build_django_from_cache 136.2 ms N/A N/A
🆕 test_build_django_from_cache_a_few_misses[15] N/A 136.4 ms N/A
🆕 test_build_django_from_cache_a_few_misses[2] N/A 135.5 ms N/A
🆕 test_build_django_from_cache_no_misses N/A 135.4 ms N/A

@seddonym seddonym merged commit a318714 into master Apr 4, 2025
18 checks passed
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