Add first version of IBM BAM support

[geoand]

No description provided.

[andreadimaio]

Hi @geoand , I'm looking at the code that you are writing to implement the watsonx module in the quarkus-langchain4j extension, and from the defaults in the Langchain4jWatsonxConfig.java and ChatModelConfig.java I understand that the BAM APIs are your starting point.

This could be a problem because the API endpoint that you are calling in the WatsonxChatModel.java class, the v2/text/chat, is not present in watsonx. So if you change the base URL in application.properties to switch from BAM to watsonx you will get an exception [404].

The correct url to call should be ml/v1-beta/generation/text which is very similar to the text generation API in BAM.

This is what I understood by reading some of the online documentation and trying the free version of the watsonx product.
I hope this could be useful for what you are doing 😃

[andreadimaio]

Another two differences are:

• To call a REST Api you need to create a project, each project has a project_id and this must be passed as a parameter in the request body.

• Authentication requires an intermediate step. From an API Key you need to generate an IAM Token (valid for 60 minutes) which is used to execute the REST API call.

Maybe using the BAM APIs as a starting point is not so good because from my understanding there are differences that don't allow your code to be used for both products.

[cescoffier]

I guess we need to rename in into bam instead of watsonx.

[andreadimaio]

If your idea is to integrate the BAM APIs into the quarkus-langchain4j extension, I could contribute with my code.
The only thing you need to know is that the BAM API is for internal use only, which means that the new module cannot be used by the entire Quarkus community.

[geoand]

Thanks for the input @andreadimaio!

So maybe we should just keep this as for us to be able to experiment with BAM is and not merge it?

[cescoffier]

I would just rename it and keep it there. We are likely going to use this for demo purposes.

…

On Mon 15 Jan 2024 at 07:40, Georgios Andrianakis ***@***.***> wrote: Thanks for the input @andreadimaio <https://github.com/andreadimaio>! So maybe we should just keep this as for us to be able to experiment with BAM is and not merge it? — Reply to this email directly, view it on GitHub <#222 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AADCG7IPPI3CJHS3VRPE7ILYOTFORAVCNFSM6AAAAABBYKKD76VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJRGM4TGNBYGE> . You are receiving this because your review was requested.Message ID: ***@***.***>

[geoand]

PR updated to replace all occurences of watsonx with bam