Skip to content

Path dir llm again#226

Merged
armgilles merged 2 commits intodevfrom
path_dir_llm_again
Apr 14, 2025
Merged

Path dir llm again#226
armgilles merged 2 commits intodevfrom
path_dir_llm_again

Conversation

@armgilles
Copy link
Owner

Continue to fix #224

Signed-off-by: Armand <arm.gilles@gmail.com>
Signed-off-by: Armand <arm.gilles@gmail.com>
@armgilles armgilles added the LLM label Apr 14, 2025
@armgilles armgilles added this to the V1.5.1 milestone Apr 14, 2025
@armgilles armgilles self-assigned this Apr 14, 2025
@codspeed-hq
Copy link

codspeed-hq bot commented Apr 14, 2025

CodSpeed Performance Report

Merging #226 will degrade performances by 14.92%

Comparing path_dir_llm_again (2cff6ae) with master (dd2be42)

Summary

⚡ 1 improvements
❌ 1 (👁 1) regressions
✅ 11 untouched benchmarks

Benchmarks breakdown

Benchmark BASE HEAD Change
test_benchmark_get_consecutive_no_transactions_out 2.4 ms 2.1 ms +14.54%
👁 test_benchmark_get_transaction_all 1.6 ms 1.9 ms -14.92%

@armgilles armgilles marked this pull request as ready for review April 14, 2025 12:42
@armgilles armgilles merged commit 0c7cf6d into dev Apr 14, 2025
5 of 6 checks passed
@armgilles armgilles deleted the path_dir_llm_again branch April 14, 2025 12:54
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