Skip to content

bench: check for empty text result to prevent panic#540

Merged
trekhopton merged 1 commit intomainfrom
text-panic-fix
Apr 23, 2025
Merged

bench: check for empty text result to prevent panic#540
trekhopton merged 1 commit intomainfrom
text-panic-fix

Conversation

@trekhopton
Copy link
Collaborator

@trekhopton trekhopton commented Apr 16, 2025

Addresses and closes #539

Copy link
Member

@ao-david ao-david left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice find

@ao-david
Copy link
Member

I just edited the overview, I don't think " closes issue #539 " links correctly to auto close the issue, but " closes #539 " does

@trekhopton trekhopton merged commit 83a7b6c into main Apr 23, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bench: panic in getText due to empty media slice from model.GetText

2 participants