Skip to content
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.

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

Langchain::Tool::QuicknodeBitcoin tool that wraps Quicknode's Bitcoin API #597

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

acidtib
Copy link
Contributor

@acidtib acidtib commented Apr 28, 2024

No description provided.

@acidtib acidtib changed the title <code>Langchain::Tool::QuicknodeBitcoin</code> tool that wraps Quicknode's Bitcoin API Langchain::Tool::QuicknodeBitcoin tool that wraps Quicknode's Bitcoin API Apr 28, 2024
@acidtib acidtib changed the title Langchain::Tool::QuicknodeBitcoin tool that wraps Quicknode's Bitcoin API Langchain::Tool::QuicknodeBitcoin tool that wraps Quicknode's Bitcoin API Apr 28, 2024
Copy link
Contributor

@mattlindsey mattlindsey left a comment

Choose a reason for hiding this comment

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

I dont know anything about bitcoin, but looks cool.
Noticed a few things:

  • Tool methods should have YARD annotations in comments
  • Add QUICKNODE_BITCOIN_URL to .sample.env
  • Add quicknode_bitcoin to list of Available Tools in README

@acidtib
Copy link
Contributor Author

acidtib commented Apr 29, 2024

hey @mattlindsey thanks for the comments, the latest commits should take care of things, it was a blast adding the tool, heres a little demo using the example https://asciinema.org/a/656460

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.

None yet

2 participants