Skip to content

Add Compass Labs toolkits to langchain docs #30794

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 31 commits into
base: master
Choose a base branch
from
Open

Add Compass Labs toolkits to langchain docs #30794

wants to merge 31 commits into from
+346 −0

Conversation

scheuclu
Copy link

@scheuclu scheuclu commented Apr 11, 2025 •
edited
Loading

  • Description: Adding documentation notebook for compass-langchain toolkit.
  • Issue: N/a
  • Dependencies: langchain-compass
  • Twitter handle: @labs_compass

@vercel Vercel
Copy link

vercel bot commented Apr 11, 2025 •
edited
Loading

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

Name Status Preview Comments Updated (UTC)
langchain ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 29, 2025 9:03pm

@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Apr 11, 2025
@scheuclu scheuclu changed the title Set up directory. Add Compass Labs toolkits to langchain docs Apr 11, 2025
@dosubot dosubot bot added the community Related to langchain-community label Apr 11, 2025
@scheuclu scheuclu marked this pull request as draft April 11, 2025 16:05
@eyurtsev eyurtsev self-assigned this Apr 11, 2025
@scheuclu scheuclu marked this pull request as ready for review April 16, 2025 16:39
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. 🤖:docs Changes to documentation and examples, like .md, .rst, .ipynb files. Changes to the docs/ folder and removed size:XS This PR changes 0-9 lines, ignoring generated files. labels Apr 16, 2025
@eyurtsev
Copy link
Collaborator

Why do the tool names contain a suffix of "_" ?

# Expected output:

aave_supply_
aave_borrow_
aave_repay_
aave_withdraw_
aave_asset_price_get_
aave_liquidity_change_get_
aave_user_position_summary_get_
...

@scheuclu
Copy link
Author

Why do the tool names contain a suffix of "_" ?

# Expected output:

aave_supply_
aave_borrow_
aave_repay_
aave_withdraw_
aave_asset_price_get_
aave_liquidity_change_get_
aave_user_position_summary_get_
...

Fixed! This was because the tool names where auto-generated from the API endpoints. I adapted the logic and published a new minor release of langchain-compass that fixes this.

scheuclu
scheuclu commented Apr 29, 2025
View reviewed changes
Copy link
Author

@scheuclu scheuclu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Addressed your comment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@eyurtsev eyurtsev

Labels
community Related to langchain-community 🤖:docs Changes to documentation and examples, like .md, .rst, .ipynb files. Changes to the docs/ folder size:L This PR changes 100-499 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@scheuclu @eyurtsev @ccosnett