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

Provide WatsonX/BAM models as langchain.chat_models #216

Closed
wants to merge 2 commits into from
Closed

Provide WatsonX/BAM models as langchain.chat_models #216

wants to merge 2 commits into from

Conversation

dove-young
Copy link


Status

READY

Description

langchain.chat_models does not include WatsonX/BAM models
It is not able to run langchain via WatsonX/BAM models so far.

like
https://python.langchain.com/docs/use_cases/question_answering
and
https://python.langchain.com/docs/use_cases/question_answering/code_understanding

just for examples.

So it is not able to leverage langchain capacities with WatsonX/BAM models.

Impacted Areas in Library

  • genai.extensions.langchain.watsonx
  • genai.extensions.langchain.bam

Which issue(s) does this pull-request fix?

#215

Any special notes for your reviewer?

There are 2 classes in this PR

  • Classes:
    • genai.extensions.langchain.watsonx provides a class WatsonX, act as a LLM to work with langchain.chat_models.
    • genai.extensions.langchain.bam provides a class BAM, act as a client of WatsonX to work with BAM API

Here you can find 2 examples implement langchain use cases using this PR


Checklist

  • Automated tests exist
  • Updated Package Requirements (if required, and with maintainers' approval)
  • Local unit tests performed
  • Documentation exists link
  • Local pre-commit hooks performed
  • Desired commit message set as PR title and description set above
  • Link to relevant GitHub issue provided

@Tomas2D Tomas2D closed this Oct 27, 2023
@dove-young dove-young deleted the yangboh-langchain branch October 30, 2023 01:43
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.

2 participants