Skip to content
This repository has been archived by the owner on Sep 22, 2023. It is now read-only.

Add feature -> Edge detection and crop #13

Open
1 of 2 tasks
Aman-zishan opened this issue Oct 4, 2020 · 22 comments
Open
1 of 2 tasks

Add feature -> Edge detection and crop #13

Aman-zishan opened this issue Oct 4, 2020 · 22 comments
Labels
enhancement New feature or request hacktoberfest celebrate opensource! help wanted Extra attention is needed new feature Add a new feature!

Comments

@Aman-zishan
Copy link
Owner

Aman-zishan commented Oct 4, 2020

AS of now the app simply converts the captured image to pdf without scaling or cropping. It would be better to add a crop option and edge detection feature while scanning documents.

  • edge detection after image is captured

  • crop option incase if edge detection fails to get the required outcome

Probably we will need to add a new screen with the preview of image and then processsing the new cropped or scaled image to generate pdf page.

@Aman-zishan Aman-zishan added enhancement New feature or request hacktoberfest celebrate opensource! help wanted Extra attention is needed new feature Add a new feature! labels Oct 4, 2020
@Amitbhave
Copy link

Amitbhave commented Oct 6, 2020

@Aman-zishan do you have anything in mind what should be used for edge detection?
For image crop, I was thinking to use image_cropper plugin.

@Aman-zishan
Copy link
Owner Author

@Aman-zishan do you have anything in mind what should be used for edge detection?
For image crop, I was thinking to use image_cropper plugin.

There is edge detection plugin too, the problem is to split the logic into different processes and implement the idea. As of now the camera takes a pic and goes directly to share and print so we have to create a new preview screen which triggers for this edge detection and crop . This screen should also be triggered when users switch to greyscale or multi image capture/picker mode(which will be implemented later #11 and #10)

@Amitbhave
Copy link

Sure, can I work on this?

@Aman-zishan
Copy link
Owner Author

Aman-zishan commented Oct 6, 2020

Sure, can I work on this?

Are you implementing the same logic? Please do share a Screenshot while opening a PR too. Thank you!

@Amitbhave
Copy link

Will it be okay, if we do edge detection at the time of capturing the image only?

@Aman-zishan
Copy link
Owner Author

Aman-zishan commented Oct 6, 2020

Will it be okay, if we do edge detection at the time of capturing the image only?

As in whenever the camera is used right, umm that would be ok i guess cause if we make use of crop for images thats uploaded from gallery there isnt a requirement for edge detection. Yeah that would do

@Amitbhave
Copy link

There is this issue with edge_detection package.
sawankumarbundelkhandi/edge_detection#20

@Aman-zishan
Copy link
Owner Author

Aman-zishan commented Oct 6, 2020

There is this issue with edge_detection package.
sawankumarbundelkhandi/edge_detection#20

Did you try in a physical device?

Have you made a good amount of progress?. Then i guess we can open a new branch and work on this in that branch?. So we both can try and fix

I have created a new branch maybe you can PR to this branch so that i can look into it.

@Amitbhave
Copy link

Why do you need new branch? I have forked your repo.. While raising PR, I will attach screenshots as well don't worry.

@Aman-zishan
Copy link
Owner Author

Oh cool then..i thought you were stuck at that error so we could work on it in that branch

@Aman-zishan Aman-zishan pinned this issue Oct 7, 2020
@sawankumarbundelkhandi
Copy link

@Aman-zishan @Amitbhave Please try using the plugin on a real device. I have never tested the edge_detection plugin on the emulator.

@Aman-zishan
Copy link
Owner Author

@Aman-zishan @Amitbhave Please try using the plugin on a real device. I have never tested the edge_detection plugin on the emulator.

Yeah saw a lot of issues with emulators while using this package.

@Amitbhave
Copy link

@Aman-zishan so should we be using this package or not as it doesn't work on emulator?

@Aman-zishan
Copy link
Owner Author

@Aman-zishan so should we be using this package or not as it doesn't work on emulator?

I'm actually using a physical device. Can you use one too

@Amitbhave
Copy link

@Aman-zishan you can go ahead with this using actual physical device.

@Amitbhave Amitbhave removed their assignment Oct 15, 2020
@Aman-zishan
Copy link
Owner Author

?? what happened

@Amitbhave
Copy link

Actually I prefer to use emulator only for testing rather than using my own device.

@Aman-zishan
Copy link
Owner Author

No problem

@Prakhar314
Copy link

Can I work on this? I am thinking of using this plugin. It is using opencv using dart:ffi.
The example there has an implementation of edge detection and crop.
Maybe we can add some filters as well, using this.

@Aman-zishan
Copy link
Owner Author

Yes sure

@Aman-zishan
Copy link
Owner Author

Hey @Prakhar314 , we have already implemented crop feature would be better if you work on edge detection

@Prakhar314
Copy link

I'm sorry, I won't be able to work on this any time soon.

@Prakhar314 Prakhar314 removed their assignment Oct 26, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request hacktoberfest celebrate opensource! help wanted Extra attention is needed new feature Add a new feature!
Projects
None yet
Development

No branches or pull requests

4 participants