feat: Set limit for advanced image processing images

Azure-Samples / chat-with-your-data-solution-accelerator

A Solution Accelerator for the RAG pattern running in Azure, using Azure AI Search for retrieval and Azure OpenAI large language models to power ChatGPT-style and Q&A experiences. This includes most common requirements and best practices.

https://azure.microsoft.com/products/search

MIT License

656 stars 327 forks source link

feat: Set limit for advanced image processing images #978

Closed cecheta closed 1 month ago

cecheta commented 1 month ago

Required for #750

Purpose

Set a limit to the number of images that are passed to the LLM when using advanced image processing. Defaults to 1

Does this introduce a breaking change?

[ ] Yes
[x] No

How to Test

Deploy code, with advanced image processing enabled
Upload similar images
Make query about images
Check number of images passed to LLM

github-actions[bot] commented 1 month ago

Coverage Report •

File	Stmts	Miss	Cover	Missing
code/backend/batch/utilities/helpers
env_helper.py	140	6	95%	240–242, 261–263
code/backend/batch/utilities/tools
question_answer_tool.py	70	0	100%
TOTAL	2503	561	77%

Tests	Skipped	Failures	Errors	Time
278	0 :zzz:	0 :x:	0 :fire:	35.309s :stopwatch:

cecheta commented 1 month ago

LGTM! I assume search returns images in order of closest match?

Yep, this is from the docs:

If no OrderBy is specified, the default sort order is descending by document match score.

github-actions[bot] commented 1 month ago

:tada: This PR is included in version 1.4.0 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket: