-
Notifications
You must be signed in to change notification settings - Fork 171
[Update] Firebase for iOS v8.10.0 & MLKitVision #501
Conversation
|
@dmariogatto Thank you for keeping these PRs coming. |
|
@mayhammf Unfortunately I do not. My own speculation is these binding libraries are not a priority for Microsoft, and from what I gather the teams that work on Xamarin/MAUI are short-staffed & don't have the time/resources to maintain these projects. Which isn't ideal as I'd regard binding important ecosystem libraries, such as Firebase, as critical to the long-term success of the platform. For now, I'll continue to update the bindings with new versions & hopefully it will make the team's job a little easier whenever they get time to look at it. |
…g][ObjectDetectionAndTracking][DigitalInkRecognition] initial commit
|
@dmariogatto Thank you for this amazing effort and amazing work! 🎉🎉🎉 I'm planning to start reviewing this ASAP, do you have any WIP that I should wait to be merge?? |
|
Might as well go with the latest 😀, I've just pushed a new branch firebase-891 (including Maps 5.2, Places 5.1, Cast 4.7 & AdMob 8.13). Shall I create a new PR, or merge into this branch? Additionally, I just recently created bindings & a sample for the below GoogleML libraries.
firebase-891-mlkitvision - this can be merged along with the Firebase update to 8.9.1, or separately later. |
|
@dmariogatto Feel free to merge it here. Please, let me know when ready. Thank you! |
|
@SotoiGhost Done 🚀 |
|
You should also be aware that some of the new archive files are deeply nested and run into max path length errors on Windows when extracted by XBD. I have already made a PR, xamarin/XamarinComponents#1296, addressing the issue. |
|
@SotoiGhost I've bumped Firebase to 8.10.0, as well as, Google Maps/Places to 6.0.0. Additionally, as per my comment above XBD has been updated to 0.11.0. |
Previous PRs: #495, #494, #489, #485, #479
Includes
Firebase
Firebase.MLKitFirebase.MLKit.CommonFirebase.MLKit.ModelInterpreterFirebase.MLKit.NaturalLanguageFirebase.MLKit.VisionFirebase.AppDistribution) & App Check (Firebase.AppCheck)Google
Google.AppIndexingGoogle.CoreGoogle.InstanceIDFirebase.InstanceIDGoogle MLKit
Added a new component category
MLKitwith new bindings for the below,MLKit.Core5.0.0MLKit.TextRecognition1.4.0-beta3MLKit.Vision3.0.0MLKit.TextRecognition1.4.0-beta3MLKit.TextRecognition.Latin1.0.0-beta3MLKit.TextRecognition.Chinese1.0.0-beta3MLKit.TextRecognition.Devanagari1.0.0-beta3MLKit.TextRecognition.Japanese1.0.0-beta3MLKit.TextRecognition.Korean1.0.0-beta3MLKit.FaceDetection1.5.0MLKit.BarcodeScanning1.6.0MLKit.ImageLabeling1.5.0MLKit.ObjectDetection1.5.0MLKit.DigitalInkRecognition1.5.0Notes
Removed the
Wrapmethods/properties that where marked asObsolete(seems like a major version bump is the time to do it 😅).Added a new build property that can be used to link Firebase with/without Ad ID support.
There were quite a few binding changes, so would definitely appreciate someone to look them over. All the samples do run & link as far as I can tell.
UMP has been separated out from AdMob (
Google.UserMessagingPlatform).Any issues let me know!