Skip to content

Conversation

eyurtsev
Copy link
Collaborator

@eyurtsev eyurtsev commented Jul 24, 2025

Further clean up of namespace:

  • Removed prompts (we'll re-add in a separate commit)
  • Remove LocalFileStore until we can review whether all the implementation details are necessary
  • Remove message processing logic from memory (we'll figure out where to expose it)
  • Remove Tool primitive (should be sufficient to use BaseTool for typing purposes)
  • Remove utilities to create kv stores. Unclear if they've had much usage outside MultiparentRetriever

Copy link

vercel bot commented Jul 24, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
langchain ⬜️ Ignored (Inspect) Visit Preview Jul 24, 2025 2:29pm

Copy link

codspeed-hq bot commented Jul 24, 2025

CodSpeed WallTime Performance Report

Merging #32228 will not alter performance

Comparing eugene/clean_up (2d1e1a6) with master (71ad451)

⚠️ 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

✅ 13 untouched benchmarks

@eyurtsev eyurtsev marked this pull request as ready for review July 24, 2025 14:31
@eyurtsev eyurtsev self-assigned this Jul 24, 2025
Copy link

codspeed-hq bot commented Jul 24, 2025

CodSpeed Instrumentation Performance Report

Merging #32228 will not alter performance

Comparing eugene/clean_up (110df1c) with master (71ad451)

Summary

✅ 14 untouched benchmarks

@eyurtsev eyurtsev enabled auto-merge (squash) July 24, 2025 14:38
@eyurtsev eyurtsev merged commit 7995c71 into master Jul 24, 2025
87 checks passed
@eyurtsev eyurtsev deleted the eugene/clean_up branch July 24, 2025 14:41
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