Skip to content

🍻 Debug benchmark, compare old code vs new without coverage interferring in overhead #14971

Closed
tiangolo wants to merge 4 commits intomasterfrom
no-coverage
Closed

🍻 Debug benchmark, compare old code vs new without coverage interferring in overhead #14971
tiangolo wants to merge 4 commits intomasterfrom
no-coverage

Conversation

@tiangolo
Copy link
Member

🍻 Debug benchmark, compare old code vs new without coverage interferring in overhead

@codspeed-hq
Copy link

codspeed-hq bot commented Feb 23, 2026

Merging this PR will degrade performance by 62.14%

❌ 4 regressed benchmarks
✅ 16 untouched benchmarks

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

Performance Changes

Benchmark BASE HEAD Efficiency
test_async_return_large_dict_with_response_model 3.8 ms 9.9 ms -62.07%
test_sync_return_large_dict_with_response_model 4.1 ms 10.3 ms -59.79%
test_async_return_large_model_with_response_model 3.8 ms 10 ms -62.14%
test_sync_return_large_model_with_response_model 4.2 ms 10.4 ms -60.09%

Comparing no-coverage (a0a2d84) with master (2826124)

Open in CodSpeed

@tiangolo
Copy link
Member Author

It seems the previous performance tests were accurate enough too. ☕

@tiangolo tiangolo closed this Feb 23, 2026
@tiangolo tiangolo deleted the no-coverage branch February 23, 2026 11:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant