Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Sample] new sample added(crop image preprocessor) #110

Closed
wants to merge 3 commits into from

Conversation

RohanHBTU
Copy link

@RohanHBTU RohanHBTU commented Dec 14, 2022

This PR is related to issue #40.

@Udayraj123
Copy link
Owner

@RohanHBTU please share screenshots of output image with save_image_level = 4

@Udayraj123
Copy link
Owner

Would appreciate if you add one more image of the same OMR template.
Also you need to put this one into the community folder with your username/nickname(without spaces).

Sample7 etc are reserved for new functionalities.

@Udayraj123
Copy link
Owner

And see if you can make use of a new omr_marker.jpg that contains those square pointers on all 4 sides. It may need change in the ratio

@RohanHBTU
Copy link
Author

@RohanHBTU please share screenshots of output image with save_image_level = 4

Screenshot (79)

@RohanHBTU
Copy link
Author

Please review my PR as I have done the changes as said.

@Udayraj123
Copy link
Owner

@RohanHBTU The name omr_marker.jpg is not for OMR scans. It's a special name for those 4 pointers on corners. You need to keep the names like omr-1.jpg, omr-2.jpg and so on. And lastly add another image omr_marker.jpg which contains one of the square boxes from the corner

@Udayraj123
Copy link
Owner

@RohanHBTU any updates?

@Udayraj123 Udayraj123 changed the title new sample added(crop image preprocessor) [Sample] new sample added(crop image preprocessor) Jan 22, 2023
@Udayraj123 Udayraj123 closed this Jun 17, 2023
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.

2 participants