Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

CoreML maps tensorflow's pad op's symmetric mode to reflect mode. #2280

Open
philloooo opened this issue Jul 18, 2024 · 0 comments
Open

CoreML maps tensorflow's pad op's symmetric mode to reflect mode. #2280

philloooo opened this issue Jul 18, 2024 · 0 comments
Labels
bug Unexpected behaviour that should be corrected (type)

Comments

@philloooo
Copy link

See here

coremltools/coremltools/converters/mil/frontend/tensorflow/ops.py

Line 1897 in 0e292a0

if mode == "symmetric":

The pad symmetric mode is mapped to reflect mode, but according to tensorflow documentation the behavior is different - symmetric mode includes the edge values.
So it should report as an error that symmetric mode is not supported by CoreML?
@philloooo philloooo added the bug Unexpected behaviour that should be corrected (type) label Jul 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Unexpected behaviour that should be corrected (type)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@philloooo