Skip to content

fix: install tesseract-ocr and cleanup#596

Merged
aatmanvaidya merged 4 commits intotattle-made:developmentfrom
omkar-334:fix-tesseract-ocr
Apr 23, 2025
Merged

fix: install tesseract-ocr and cleanup#596
aatmanvaidya merged 4 commits intotattle-made:developmentfrom
omkar-334:fix-tesseract-ocr

Conversation

@omkar-334
Copy link
Copy Markdown
Contributor

Resolves #594

@aatmanvaidya aatmanvaidya self-requested a review April 22, 2025 17:55
@aatmanvaidya
Copy link
Copy Markdown
Collaborator

hi @omkar-334 thank you for the fix, feel free to ignore the ruff lint error that is coming, we will fix it later, its not for this PR

@omkar-334
Copy link
Copy Markdown
Contributor Author

I added the comment fix in the detect_lewd_images operator to trigger the operator tests

Comment thread .github/workflows/operator-test.yml Outdated
@omkar-334
Copy link
Copy Markdown
Contributor Author

The operator tests are passing now.

Comment thread operators/detect_lewd_images/detect_lewd_images.py
@aatmanvaidya
Copy link
Copy Markdown
Collaborator

The operator tests are passing now.

only the operator you change, the test for it run's, you will have to make a commit change in the tessearct operator, then only its test will trigger

@omkar-334
Copy link
Copy Markdown
Contributor Author

The operator tests are passing now.

only the operator you change, the test for it run's, you will have to make a commit change in the tessearct operator, then only its test will trigger

Oh, yeah, i checked the test log and only detect_lewd_images operator tests ran

@omkar-334
Copy link
Copy Markdown
Contributor Author

Okay, now the detect_text_in_image_tesseract tests are passing.

Comment thread .github/workflows/operator-test.yml Outdated
@aatmanvaidya
Copy link
Copy Markdown
Collaborator

Okay, now the detect_text_in_image_tesseract tests are passing.

perfect thank you, can you also undo the small change in the file?

@aatmanvaidya aatmanvaidya merged commit 335c980 into tattle-made:development Apr 23, 2025
3 of 4 checks 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.

Fix modified operator tests github action to support tesseract operator

2 participants