Populate text_block field during OCR processing#1216
Open
majiayu000 wants to merge 1 commit intollmware-ai:mainfrom
Open
Populate text_block field during OCR processing#1216majiayu000 wants to merge 1 commit intollmware-ai:mainfrom
majiayu000 wants to merge 1 commit intollmware-ai:mainfrom
Conversation
When running OCR on library images, only the text_search field was being populated, leaving text_block (mapped from 'text' in the record) empty. This caused semantic queries to return empty text results since they retrieve content from text_block, not text_search. The fix ensures both 'text' and 'text_search' fields are populated with the OCR-extracted content. Fixes llmware-ai#1123 Signed-off-by: majiayu000 <1835304752@qq.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Root cause
The
ocr_images_in_librarymethod only callednew_block.update({"text_search": text_chunk})but not
new_block.update({"text": text_chunk}). Since 'text' maps to 'text_block' in thedatabase and semantic queries retrieve from 'text_block', the text was not returned.
Test plan
Fixes #1123
Signed-off-by: majiayu000 1835304752@qq.com