-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Integrate NIBittensorLLM #2385
Integrate NIBittensorLLM #2385
Conversation
BitVoyagerMan
commented
Aug 23, 2023
- Add NIBittensorLLM endpoint based on power of Bittensor.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
langchain/src/llms/bittensor.ts
Outdated
@@ -0,0 +1,156 @@ | |||
import axios, { AxiosResponse } from "axios"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR introduces a new HTTP request using axios to retrieve API keys and interact with the Neural Internet API. Please review this change to ensure it aligns with the project's requirements and best practices.
@@ -52,6 +52,8 @@ | |||
"typescript": "^5.0.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work on the PR! I've noticed a dependency change in the package.json file, specifically the addition of "@types/axios" as a dev dependency and "axios" as a regular dependency. This comment is to flag the change for maintainers to review.
Thanks for this! It would be better to try to do this without optional deps - it looks like we can probably do this with fetch? |