mem0 icon indicating copy to clipboard operation
mem0 copied to clipboard

Remove `tools` from LLMs

Open techcontributor opened this issue 1 year ago • 1 comments

Description

Removed the use of tools from all LLMs, also removed unnecessary LLM test cases.

Fixes #2283

Type of change

Please delete options that are not relevant.

  • [ ] Bug fix (non-breaking change which fixes an issue)
  • [ ] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • [x] Refactor (does not change functionality, e.g. code style improvements, linting)
  • [ ] Documentation update

How Has This Been Tested?

Tested locally with the test cases.

  • [x] Unit Test
  • [ ] Test Script (please provide)

Checklist:

  • [x] My code follows the style guidelines of this project
  • [x] I have performed a self-review of my own code
  • [ ] I have made corresponding changes to the documentation
  • [x] My changes generate no new warnings
  • [ ] I have added tests that prove my fix is effective or that my feature works
  • [x] New and existing unit tests pass locally with my changes
  • [x] Any dependent changes have been merged and published in downstream modules
  • [x] I have checked my code and corrected any misspellings

Maintainer Checklist

  • [ ] closes #xxxx (Replace xxxx with the GitHub issue number)
  • [ ] Made sure Checks passed

techcontributor avatar Mar 11 '25 22:03 techcontributor

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

Name Status Preview Comments Updated (UTC)
multimodal-demo ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 12, 2025 7:57pm

vercel[bot] avatar Mar 11 '25 22:03 vercel[bot]

@techcontributor Thanks for the contribution!

Dev-Khant avatar Mar 14 '25 12:03 Dev-Khant