Skip to content

Conversation

@ccurme
Copy link
Collaborator

@ccurme ccurme commented Oct 3, 2025

Cherry-pick of #33242

@ccurme ccurme requested a review from eyurtsev as a code owner October 3, 2025 12:25
@github-actions github-actions bot added core Related to the package `langchain-core` feature labels Oct 3, 2025
@codspeed-hq
Copy link

codspeed-hq bot commented Oct 3, 2025

CodSpeed WallTime Performance Report

Merging #33248 will improve performances by 32.65%

Comparing cc/add-message-ids-to-convert-to-openai-messages (7e6af87) with master (99361e6)1

⚠️ Unknown Walltime execution environment detected

Using the Walltime instrument on standard Hosted Runners will lead to inconsistent data.

For the most accurate results, we recommend using CodSpeed Macro Runners: bare-metal machines fine-tuned for performance measurement consistency.

Summary

⚡ 4 improvements
✅ 9 untouched

Benchmarks breakdown

Benchmark BASE HEAD Change
test_async_callbacks_in_sync 25.4 ms 19.1 ms +32.65%
test_import_time[Document] 219.3 ms 197.7 ms +10.97%
test_import_time[HumanMessage] 268.3 ms 204.1 ms +31.42%
test_import_time[InMemoryRateLimiter] 170.8 ms 150.5 ms +13.5%

Footnotes

  1. No successful run was found on v0.3 (c9590ef) during the generation of this report, so master (99361e6) was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

@ccurme ccurme merged commit 5904cbe into v0.3 Oct 3, 2025
74 checks passed
@ccurme ccurme deleted the cc/add-message-ids-to-convert-to-openai-messages branch October 3, 2025 15:37
@eyurtsev eyurtsev mentioned this pull request Nov 19, 2025
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core Related to the package `langchain-core` feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants