-
Notifications
You must be signed in to change notification settings - Fork 382
Update E-Document AI tools to GPT-5.3 chat latest #8544
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
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -93,7 +93,7 @@ codeunit 6231 "E-Document MLLM Handler" implements IStructureReceivedEDocument, | |||||||||||
| Base64Data := Base64Convert.ToBase64(InStream); | ||||||||||||
|
|
||||||||||||
| // Build AOAI call | ||||||||||||
| AzureOpenAI.SetAuthorization(Enum::"AOAI Model Type"::"Chat Completions", AOAIDeployments.GetGPT41MiniPreview()); | ||||||||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Vision model replaced with chat-only modelThe MLLM handler uses Recommendation:
Suggested change
👍 useful · ❤️ especially valuable · 👎 wrong - reply with why |
||||||||||||
| AzureOpenAI.SetAuthorization(Enum::"AOAI Model Type"::"Chat Completions", AOAIDeployments.GetGPT53ChatLatest()); | ||||||||||||
| AzureOpenAI.SetCopilotCapability(Enum::"Copilot Capability"::"E-Document MLLM Analysis"); | ||||||||||||
|
|
||||||||||||
| AOAIChatCompletionParams.SetTemperature(0); | ||||||||||||
|
|
||||||||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cost regression: Mini model replaced by full GPT-5.3
The previous model
GetGPT41MiniPreview()was a deliberately chosen smaller/cheaper model for high-volume document processing. Replacing it with the fullGetGPT53ChatLatest()model will substantially increase token costs per document processed through the MLLM extraction path, without any evidence the task complexity justifies the upgrade.Recommendation:
GetGPT53MiniLatest()), prefer it for the MLLM handler to preserve the cost profile. Reserve the full GPT-5.3 model for the matching and tool-processor codeunits where the reasoning complexity is higher.👍 useful · ❤️ especially valuable · 👎 wrong - reply with why