Add feature -> Edge detection and crop about docscanner HOT 22 OPEN

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

from docscanner.

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

from docscanner.

@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.

from docscanner.

@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)

from docscanner.

Sure, can I work on this?

from docscanner.

Sure, can I work on this?

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

from docscanner.

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

from docscanner.

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

from docscanner.

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.

from docscanner.

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

from docscanner.

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

from docscanner.

@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.

from docscanner.

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

from docscanner.

@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

from docscanner.

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

from docscanner.

?? what happened

from docscanner.

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

from docscanner.

No problem

from docscanner.

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.

from docscanner.

Yes sure

from docscanner.

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

from docscanner.

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

from docscanner.