Skip to content

Bug Fix: iOS Duplicate Symbols Between MLImage and Vision Task Library#892

Open
priankakariat wants to merge 5 commits into
tensorflow:masterfrom
khanhlvg:mlimage-cocoapods
Open

Bug Fix: iOS Duplicate Symbols Between MLImage and Vision Task Library#892
priankakariat wants to merge 5 commits into
tensorflow:masterfrom
khanhlvg:mlimage-cocoapods

Conversation

@priankakariat

Copy link
Copy Markdown
Contributor

No description provided.

@priankakariat

Copy link
Copy Markdown
Contributor Author

@khanhlvg @lu-wang-g @schmidt-sebastian Raised a PR for fixing the duplicate symbols error between MLImage and Vision Task Library

@khanhlvg

Copy link
Copy Markdown
Member

I discussed with Prianka about this yesterday.

Fundamentally we'll need to depend on MLImage pod rather than bundle MLImage inside TFLiteTaskVision. It'll require us to release a new version of TFLiteTaskVision every time a new version of MLImage is released and newer version of MLKit depends on it. We need to come up with a strategy for that before working on making TFLiteTaskVision to depend on MLImage.

@schmidt-sebastian

Copy link
Copy Markdown
Collaborator

Thanks for the insight.

@schmidt-sebastian schmidt-sebastian left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Need to figure out how to proceed here.

@ewfian

ewfian commented Feb 22, 2023

Copy link
Copy Markdown

related: #885

1 similar comment
@nysamnang

Copy link
Copy Markdown

related: #885

@abdelrahman-muntaser

Copy link
Copy Markdown

Is this gonna work if i used the library from this pull request until they merged it ?

@Karllas

Karllas commented Jan 29, 2024

Copy link
Copy Markdown

When will this be merged? We cannot use GoogleMLKit and TensorFlowLiteTaskVision together

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants