Skip to content

chore: dnm: estimate overhead of custom resolver#7

Closed
bdraco wants to merge 3 commits intomainfrom
estimate_overhead_of_constructors
Closed

chore: dnm: estimate overhead of custom resolver#7
bdraco wants to merge 3 commits intomainfrom
estimate_overhead_of_constructors

Conversation

@bdraco
Copy link
Member

@bdraco bdraco commented Feb 21, 2025

No description provided.

@codspeed-hq
Copy link

codspeed-hq bot commented Feb 21, 2025

CodSpeed Performance Report

Merging #7 will improve performances by 9.48%

Comparing estimate_overhead_of_constructors (602e435) with main (b7e7ad5)

Summary

⚡ 2 improvements

Benchmarks breakdown

Benchmark BASE HEAD Change
test_large_parse_yaml 4.7 s 4.3 s +10.69%
test_simple_parse_yaml 32.3 ms 29.5 ms +9.48%

@bdraco
Copy link
Member Author

bdraco commented Mar 17, 2025

implementation in #16

@bdraco bdraco closed this Mar 17, 2025
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.

1 participant

Comments