quarkiverse / quarkus-langchain4j

Quarkus Langchain4j extension
https://docs.quarkiverse.io/quarkus-langchain4j/dev/index.html
Apache License 2.0
120 stars 65 forks source link

Introduce global logging properties #536

Closed geoand closed 2 months ago

geoand commented 2 months ago

This allows us to use quarkus.langchain4j.log-requests=true instead of specifying needing to specify the provider as well

jmartisk commented 2 months ago

Great idea! There's a failure in a watsonx test that seems related

geoand commented 2 months ago

Right, I'll fix it soon

jmartisk commented 2 months ago

I think it would also be nice to mention it on the Getting started page in the docs

geoand commented 2 months ago

Having it in the docs definitely makes sense, but I am not sure which page you are referring to.

jmartisk commented 2 months ago

the landing page https://docs.quarkiverse.io/quarkus-langchain4j/dev/index.html :)

geoand commented 2 months ago

Would you like to push a commit with that change?

jmartisk commented 2 months ago

Sure

geoand commented 2 months ago

🙏🏼

maxandersen commented 2 months ago

Bedt feature ever ! (At least this morning thus far).