New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

refactor: remove fluff config from places #732

Merged

benfdking merged 1 commit into main from refactoring_out_fluff_config

Sep 13, 2024

Collaborator

benfdking commented Sep 12, 2024

No description provided.

benfdking force-pushed the refactoring_out_fluff_config branch from 47ae98f to 56c5c8e Compare

September 12, 2024 22:55

github-actions bot commented Sep 12, 2024

Benchmark for `0a27039`

Click to view benchmark

Test	Base	PR	%
DepthMap::from_parent	41.3±0.21µs	41.5±0.22µs	+0.48%
fix_complex_query	11.2±0.19ms	11.2±0.13ms	0.00%
parse_complex_query	2.8±0.04µs	2.8±0.05µs	0.00%
parse_expression_recursion	5.0±0.04µs	5.1±0.07µs	+2.00%
parse_simple_query	822.9±12.26ns	811.0±48.44ns	-1.45%


          refactor: remove fluff config from places

338e9d2

benfdking force-pushed the refactoring_out_fluff_config branch from 56c5c8e to 338e9d2 Compare

September 13, 2024 08:53

github-actions bot commented Sep 13, 2024

Benchmark for `d222191`

Click to view benchmark

Test	Base	PR	%
DepthMap::from_parent	40.7±0.55µs	56.3±0.18µs	+38.33%
fix_complex_query	11.4±0.29ms	11.3±0.19ms	-0.88%
parse_complex_query	2.8±0.02µs	2.8±0.06µs	0.00%
parse_expression_recursion	5.2±0.06µs	5.0±0.06µs	-3.85%
parse_simple_query	813.0±7.81ns	812.6±32.39ns	-0.05%

benfdking merged commit 65a542b into main

14 checks passed

benfdking deleted the refactoring_out_fluff_config branch

September 13, 2024 09:44

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet