chroma-core / chroma

the AI-native open-source embedding database
https://www.trychroma.com/
Apache License 2.0
15.45k stars 1.3k forks source link

[DOC]: Update Langchain integration example and fix broken link #2771

Closed ishaksebsib closed 2 months ago

ishaksebsib commented 2 months ago

Description of changes

Test plan

How are these changes tested?

Documentation Changes

Are all docstrings for user-facing APIs updated if required? Do we need to make documentation changes in the docs repository?

github-actions[bot] commented 2 months ago

Reviewer Checklist

Please leverage this checklist to ensure your code review is thorough before approving

Testing, Bugs, Errors, Logs, Documentation

jeffchuber commented 2 months ago

@ishaksebsib thanks for this! we are going to do a wholesale rewrite of the langchain docs this week. happy to still give you credit on the PR for finding this stuff! can we cut the code example entirely for now?

ishaksebsib commented 2 months ago

@ishaksebsib thanks for this! we are going to do a wholesale rewrite of the langchain docs this week. happy to still give you credit on the PR for finding this stuff! can we cut the code example entirely for now?

Hi @jeffchuber,

Thanks for the feedback. I've removed the code example as requested and updated the PR.

Let me know if there's anything else I can assist with. I'm also happy to help with the upcoming documentation rewrite if needed.