NeutrinosPlatform / cordova-plugin-document-scanner

cordova plugin for document scan
https://www.neutrinos.co/
MIT License
85 stars 60 forks source link

Initial Windows version #62

Open convey-gilbert opened 4 years ago

convey-gilbert commented 4 years ago

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Fixes # (issue)

Type of change

Please delete options that are not relevant.

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

Test Configuration:

Checklist:

convey-gilbert commented 4 years ago

This is a first version supporting Windows 10. Edge detection is based on OpenCV like in the Android version of the plugin. The camera window UI and photo capture code is entirely written in JavaScript using Windows 10 only APIs and works with all Windows 10 versions using the APIs appropriate to the version. If the device supports a separate photo stream, the photo is captured from that, otherwise the video stream is used to capture the photo. The OpenCVBridge wrapper is written C++. OpenCV 3.1. sourc code is included together with the binaries to be able to rebuild the dependent modules. The elswhere available binary OpenVC DLLs won't confirm to Microsoft Store specifications due to wrong compiler and linker flags used. So it's better to include all here in the project. There are some additional options to provide to the scanDoc function. Description in detail is following later. Loading photos from Photo Library isn't supported yet, option will be ignored. This feature will follow later. A Windows 8.1 branch of the plugin, using a Media Transfrom Filter instead of a Media Frame Reader is in work and will be provided with coming pull request later, maybe next week.

ChrisTomAlx commented 4 years ago

Hey @convey-gilbert This is brilliant!! Really appreciate your contribution.

Unfortunately though I have no way of testing this, so could you maybe send me a video of your tests? I will then test and make sure this pr doesn't break any of the other platforms (Technically it shouldn't but I'd rather not risk it) and then merge it.

Cheers and have a nice day :) Chris Neutrinos

convey-gilbert commented 4 years ago

Hello Chris, You can find Videos here (on x64-CPU Desktop PC with Windows 10 v. 1903 and ARM-CPU Lumia 950 with Windows 10 Mobile v. 1709): https://secure.convey.de/publish/leadsuccess/img_0353.mov https://secure.convey.de/publish/leadsuccess/img_0354.mov

We've also tested already with some Tablets Lenovo MIIX and Surface Pro 4 running Windows 10 v.1903 and on x64 CPU Desktop PC running Windows 8.1.

Further tests with x86 CPU (Windows 8.1 and Windows 10) and ARM CPU (Windows 8.1 RT and Windows 10 on ARM) yet missing.