[#910] Guide: Chat bot application example using langchain4j-cdi and RAG#911
Conversation
|
A preview of this PR is available at https://pr-911-wildfly-org.surge.sh |
|
@ehsavoie Could you take a look at this one when you have a chance? thanks! |
|
Hello @ehsavoie any chance to take a look at this guide from your side soon? |
ehsavoie
left a comment
There was a problem hiding this comment.
I think you should ensure consistency on the Chat Bot vs Chatbot wording. Otherwise it looks very good. Thanks a lot for writting that
| include::../../templates/partials/guides/prerequisites.adoc[] | ||
| * Ollama installed and running with the `ollama serve` command. You can follow the instructions at https://docs.ollama.com/linux[Ollama installation guide, window=_blank] to install Ollama on your machine. | ||
|
|
||
| == A Simple Chat Bot example |
There was a problem hiding this comment.
Right, let's choose Chatbot
| The source code of the application developed in this guide is available at https://github.com/wildfly-extras/guides/tree/main/wildfly-car-booking-chat-bot[wildfly-car-booking-chat-bot, window=_blank]. | ||
|
|
||
| include::../../templates/partials/guides/prerequisites.adoc[] | ||
| * Ollama installed and running with the `ollama serve` command. You can follow the instructions at https://docs.ollama.com/linux[Ollama installation guide, window=_blank] to install Ollama on your machine. |
There was a problem hiding this comment.
May add the alternative to use docker to run ollama ?
I think the ollama pull qwen3.5:4b should be written somewhere
There was a problem hiding this comment.
May add the alternative to use docker to run ollama ?
I've added a link to point out the docker alternative
I think the ollama pull qwen3.5:4b should be written somewhere
There is a note in that regards at the "Configure and Run the Application" section
|
Thanks for the review @ehsavoie , I've just addressed your points |
|
@bstansberry Any chance for merging this soon? |
@yersan I can handle this. I will do a final review. In the mean time, when do you want to publish this? ASAP or do you want to schedule it around/after WF 41 announcement? |
The archetype is using WF 40.0.0, maybe does it make sense to wait until WF 41 and change the default version? It doesn't matter at all ... WDYT? |
|
@yersan Given where we are, close to releasing WF 41, I would recommend basing this on it and publishing shortly after the release for better reach. But up to U. |
|
Ok, thanks for the review @rhusar . I will update this guide to latest versions as soon as we have shipped WF41 and ping you back here when done |
rhusar
left a comment
There was a problem hiding this comment.
Few more small changes needed.
…di and RAG Signed-off-by: Yeray Borges <yborgess@redhat.com>
|
I'm waiting for wildfly archetypes based on WF 41 to test this again locally, I'll ping here when done |
|
@yersan They are still syncing as of this morning... maybe I can trigger it manually. |
|
@rhusar I ran the guide locally using WF41, all fine, I didn't find any errors, thanks for reviewing this! |
Fixes #910
Requires sample code available at wildfly-extras/guides#12