chat-with-your-data-solution-accelerator icon indicating copy to clipboard operation
chat-with-your-data-solution-accelerator copied to clipboard

feat: Set limit for advanced image processing images

Open cecheta opened this issue 1 year ago • 1 comments

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

cecheta avatar May 24 '24 15:05 cecheta

Coverage

Coverage Report •
FileStmtsMissCoverMissing
code/backend/batch/utilities/helpers
   env_helper.py140695%240–242, 261–263
code/backend/batch/utilities/tools
   question_answer_tool.py700100% 
TOTAL250356177% 

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

github-actions[bot] avatar May 24 '24 15:05 github-actions[bot]

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.

cecheta avatar May 28 '24 08:05 cecheta

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

The release is available on GitHub release

Your semantic-release bot :package::rocket:

github-actions[bot] avatar May 28 '24 10:05 github-actions[bot]