Support dynamic disabling of Structured Output Native#5412
Support dynamic disabling of Structured Output Native#5412filiphr wants to merge 1 commit intospring-projects:mainfrom
Conversation
filiphr
force-pushed
the
support-dynamic-structure-output-disable
branch
from
7791917 to
2b02ab6
Compare
Signed-off-by: Filip Hrisafov <filip.hrisafov@gmail.com>
filiphr
force-pushed
the
support-dynamic-structure-output-disable
branch
from
2b02ab6 to
a9d24cd
Compare
|
Did someone from the Spring AI team manage to have a look at this? Is this something that is acceptable for the team? My use case is to enable structured outputs globally, but use an advisor and / or disable it in some particular cases. Additionally, I have a commit ready on top of this that would allow any custom |
|
@ilayaperumalg, @tzolov, @sdeleuze, @ericbottard, Congrats on the 2.0.0-M3 release. I wanted to check with you whether or not this PR is something that is acceptable for Spring AI, or I should look for an alternative on our side? |
|
@filiphr Sorry about the delay in reviewing this PR. Thanks for the PR and inputs! I will discuss with the team and provide feedback soon. |
|
Thank you very much @ilayaperumalg. Let me know about the outcome. I'm happy to rebase this on top of main and adjust it if needed |
| <version>${mockk-jvm.version}</version> | ||
| <scope>test</scope> | ||
| </dependency> | ||
| <dependency> |
There was a problem hiding this comment.
This needs to be test scope. I will fix when merging.
|
That's amazing. Thanks a lot @ilayaperumalg and @tzolov |
3 participants
With this PR it allows users to dynamically disable Structured Output Native, even if it is globally enabled.