-
Notifications
You must be signed in to change notification settings - Fork 42
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
after manually crop result not correct. #5
Comments
Manually crop is not working correctly especially with gallery picked image. |
@algorr Could you please provide a step-by-step list of how to reproduce this issue, along with the type of device you're using (Android or iOS)? Additionally, could you share the image that is causing the problem? |
I simply installed your example app. Crop is not working with captured image, it is giving wrong output. Gallery image cropping is correct. I think it is because you are using video for capturing and detecting. Also flash option is not showing, it is a highly required feature. |
I've been dealing with this problem for about two days, finally I found it! The problem stems from this; When reading the image bytes coming from the camera, it is read in a different format, which causes the size of the image to change. (ex: normally my image is 1280x720, but what is read is like 720x1280) I found the solution as follows: These lines in OpenCVPlugin file => I replaced it with this => But I don't know how healthy a change it is. Because not all Android phones may capture images in the same format. |
Thanks @mrasityilmaz for your contribution. I will review it, but I am also adjusting the detection of the image orientation to properly arrange the points for cropping. |
@mrasityilmaz I used your work around but I am still facing the same issue so do you have any other idea about it? @criistian14 Thanks for this package! It's a very important package but this issue we should fix asap. |
You downloaded the package codes, added them locally to your project and made changes, right? |
Yes, Right Man! |
|
This is my version |
Yeah, Now It's working @mrasityilmaz Thank you so much for saving my time. @criistian14 Please update this and release a new version as soon as possible. |
No description provided.
The text was updated successfully, but these errors were encountered: