-
Notifications
You must be signed in to change notification settings - Fork 172
[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.MLKit
Firebase.MLKit.Common
Firebase.MLKit.ModelInterpreter
Firebase.MLKit.NaturalLanguage
Firebase.MLKit.Vision
Firebase.AppDistribution
) & App Check (Firebase.AppCheck
)Google
Google.AppIndexing
Google.Core
Google.InstanceID
Firebase.InstanceID
Google MLKit
Added a new component category
MLKit
with new bindings for the below,MLKit.Core
5.0.0MLKit.TextRecognition
1.4.0-beta3MLKit.Vision
3.0.0MLKit.TextRecognition
1.4.0-beta3MLKit.TextRecognition.Latin
1.0.0-beta3MLKit.TextRecognition.Chinese
1.0.0-beta3MLKit.TextRecognition.Devanagari
1.0.0-beta3MLKit.TextRecognition.Japanese
1.0.0-beta3MLKit.TextRecognition.Korean
1.0.0-beta3MLKit.FaceDetection
1.5.0MLKit.BarcodeScanning
1.6.0MLKit.ImageLabeling
1.5.0MLKit.ObjectDetection
1.5.0MLKit.DigitalInkRecognition
1.5.0Notes
Removed the
Wrap
methods/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!