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

feat(community): Prepare implementation IBM WatsonxAI to langchain community package. #6916

Merged
merged 9 commits into from
Oct 21, 2024

Conversation

FilipZmijewski
Copy link
Contributor

@FilipZmijewski FilipZmijewski commented Oct 1, 2024

Prepare implementation IBM WatsonxAI to langchain community package.

Provide solutions for:

  • LLMs
  • embeddings
  • tests

Thank you for contributing to LangChain.js!

  • PR title: "package: description"

Where "package" is whichever of langchain, community, core, experimental, etc. is being modified. Use "docs: ..." for purely docs changes, "templates: ..." for template changes, "infra: ..." for CI changes.
Example: "community: add foobar LLM"

  • PR message:

Description: The aim of these changes is to create first Langchain.js by IBM implementation. Solutions applied consist of LLMs and embeddings.
fyi @LukaszCmielowski

  • Add tests and docs:

Expected tests for both of implemented features were added as well as docs and examples

  • Lint and test: Run yarn format, yarn lint, yarn test and yarn test:int from the root of the package(s) you've modified.

Copy link

vercel bot commented Oct 1, 2024

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

Name Status Preview Comments Updated (UTC)
langchainjs-docs ✅ Ready (Inspect) Visit Preview Oct 8, 2024 4:01pm
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
langchainjs-api-refs ⬜️ Ignored (Inspect) Oct 8, 2024 4:01pm

@FilipZmijewski FilipZmijewski marked this pull request as ready for review October 1, 2024 13:49
@dosubot dosubot bot added size:XXL This PR changes 1000+ lines, ignoring generated files. auto:enhancement A large net-new component, integration, or chain. Use sparingly. The largest features labels Oct 1, 2024
@FilipZmijewski
Copy link
Contributor Author

Hey @jacoblee93.
I have created a PR to community as discussed. Would you be possible to have someone review it? We will be very happy and thankful about that!
Have a great day.

Provide solutions for:

- LLMs
- embeddings
- tests
Provide solutions for:

- LLMs
- embeddings
- tests

* Fix docs, dependencies and minor changes to code
@langchain-ai langchain-ai locked as spam and limited conversation to collaborators Oct 9, 2024
* Applied fixes according to review
Provide solutions for:
- chat
- add tests for chat
};
}

model = "mistralai/mistral-large";
Copy link
Collaborator

Choose a reason for hiding this comment

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

Generally we suggest not using a default (models change frequently), but up to you


projectId?: string;

truncate_input_tokens?: number;
Copy link
Collaborator

Choose a reason for hiding this comment

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

nit: prefer not mixing camel and snake case


version = "2024-05-31";

max_new_tokens = 100;
Copy link
Collaborator

Choose a reason for hiding this comment

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

nit: avoid mixing camel and snake case

@dosubot dosubot bot added the lgtm PRs that are ready to be merged as-is label Oct 21, 2024
@jacoblee93 jacoblee93 changed the title community: Prepare implementation IBM WatsonxAI to langchain community package. feat(community): Prepare implementation IBM WatsonxAI to langchain community package. Oct 21, 2024
@jacoblee93
Copy link
Collaborator

jacoblee93 commented Oct 21, 2024

Thanks for your patience! Will ship later today.

@jacoblee93 jacoblee93 merged commit bd9ce75 into langchain-ai:main Oct 21, 2024
35 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
auto:enhancement A large net-new component, integration, or chain. Use sparingly. The largest features lgtm PRs that are ready to be merged as-is size:XXL This PR changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants