mem0 icon indicating copy to clipboard operation
mem0 copied to clipboard

Add OctoAILlm support to Embedchain

Open BlacPharaoh opened this issue 1 year ago • 8 comments

Description

Added functionality for OctoAI LLM via langchain

Fixes #685

Type of change

Please delete options that are not relevant.

  • [ ] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

Please delete options that are not relevant.

  • [ ] Unit Test

Checklist:

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

Maintainer Checklist

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

BlacPharaoh avatar Oct 14 '23 23:10 BlacPharaoh

@BlacPharaoh Also can you please update the title of the PR - Add OctoAILlm support to Embedchain

sidmohanty11 avatar Oct 19 '23 17:10 sidmohanty11

Working on All Comments.

BlacPharaoh avatar Nov 26 '23 11:11 BlacPharaoh

Hey @BlacPharaoh thanks for your contribution. Let us know if you need any help. Planning to make this support live soon.

taranjeet avatar Dec 13 '23 03:12 taranjeet

I picked up on this again. Addressing all comments. I should complete by weekend

BlacPharaoh avatar Feb 22 '24 22:02 BlacPharaoh

Thanks @BlacPharaoh. Looking forward to integrating this soon.

deshraj avatar Feb 22 '24 22:02 deshraj

Hi @BlacPharaoh Can you please take a look at the comments and resolve the conflicts so we can review and merge it?

Dev-Khant avatar Jun 04 '24 10:06 Dev-Khant

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

CLAassistant avatar Jul 26 '24 02:07 CLAassistant

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

CLAassistant avatar Jul 26 '24 02:07 CLAassistant

Closing this PR, feel free to work on it again.

Dev-Khant avatar Aug 01 '24 20:08 Dev-Khant