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

Setup dart side of local MLVision text detection #614

Merged
merged 18 commits into from
Jun 15, 2018

Conversation

bparrishMines
Copy link
Contributor

No description provided.

Copy link
Contributor

@kroikie kroikie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to move forward.

return returnValue;
default:
throw new PlatformException(
code: 'Method not implemted: ${methodCall.method}');
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This requires that you handle every channel call. This may be what you want but can get tedious if you are returning some default value for a bunch of them.

@bparrishMines bparrishMines merged commit 1730cad into flutter:master Jun 15, 2018
@bparrishMines bparrishMines deleted the ml_vision_text_dart branch June 15, 2018 18:18
julianscheel pushed a commit to jusst-engineering/plugins that referenced this pull request Mar 11, 2020
* Initial 'fluter create' for Firebase ML Vision plugin

* Formatting

* More Formatting

* Import ordering

* Added android app to firebase

* Added ios app to firebase

* Added ml vision to android and ios

* WIP dart side of ml vision

* finish dart side of ml vision and text detector

* Added TextDetector tests

* Added some comments

* Finished comments

* Removed usused dependency

* Formatting and unused imports

* Changed variable name to restart WIP test

* Removed unnecessary default platformexceptiion
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants