-
Notifications
You must be signed in to change notification settings - Fork 9.8k
Vision cloud label detection #695
Vision cloud label detection #695
Conversation
The issues are noted. |
Fixing a copy/paste mistake brought from FIRVisionLabel
Hey, @bparrishMines I see that you've been writing most of the ml-vision plugin. I tried to add support for cloud labeling of images. But I'm a bit stumped by the failing build here. It builds and runs fine for me locally and I can't make out what action I could take to fix the breaking build. Any ideas or pointers? |
Hi @DennisAlund, Could you do a Also, this error may be the culprit for the build issue:
|
Thanks for spotting that @bparrishMines, I coded the ios part a bit in the dark on my pc. |
Any other ideas on the current build errors @bparrishMines? I don't have XCode to test on myself, so I am a bit lost on troubleshooting the iOS parts. I'd be happy if there's someone in the community that could give some insights. |
Yea, if you click on |
Hi @DennisAlund, are you still working on this PR? |
Hey, I didn't have time to look into it yet. Would need to try it out with xcode since it is the objectivec code that is causing issues. But I haven't had a chance to do that. Would appreciate input from the community if there's someone who's got xcode and a bit of time. |
@bparrishMines the build issues were resolved with the most recent merge of |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @DennisAlund,
Thanks for your contribution! Everything looks good! Just a few small things before I merge it.
packages/firebase_ml_vision/lib/src/vision_cloud_detector_options.dart
Outdated
Show resolved
Hide resolved
packages/firebase_ml_vision/lib/src/vision_cloud_detector_options.dart
Outdated
Show resolved
Hide resolved
Thanks! I have a branch that will bump the version and this will get published with it. |
Adding cloud label detection support
This reverts commit 2fd2e9a.
Adding cloud label detection support
This is an implementation of the cloud based API in ML Kit for Firebase
I found that the cloud based APIs were missing and wrote one straight off due to personal need in a planned workshop. I haven't reported this in any feature request, but I figured that it would be part of the roadmap.