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

[Frontend][TFLite] Add parser support for l2_normalization #4966

Merged
merged 2 commits into from
Feb 29, 2020
Merged

[Frontend][TFLite] Add parser support for l2_normalization #4966

merged 2 commits into from
Feb 29, 2020

Conversation

inadob
Copy link
Contributor

@inadob inadob commented Feb 28, 2020

  • TF doesn't provide uint8 support
  • TFLite does the normalization only if it's over the last dimension
  • TFL uses only the default value for epsilon

@anijain2305 anijain2305 self-assigned this Feb 28, 2020
@inadob
[Frontend][TFLite] Add parser support for l2_normalization
2acb864 
* TF doesn't provide uint8 support
* TFL does the normalization only if it's over the last axis
* TFL uses only the default value for expilon
@anijain2305 anijain2305 merged commit 2355caa into apache:master Feb 29, 2020
@anijain2305
Copy link
Contributor

Thanks @inadob @wyc-ruiker This is merged!

trevor-m pushed a commit to trevor-m/tvm that referenced this pull request Apr 16, 2020
@inadob
[Frontend][TFLite] Add parser support for l2_normalization (apache#4966)
ef16bd8 
* [Frontend][TFLite] Add parser support for l2_normalization

* TF doesn't provide uint8 support
* TFL does the normalization only if it's over the last axis
* TFL uses only the default value for expilon

* Change error message
zhiics pushed a commit to neo-ai/tvm that referenced this pull request Apr 17, 2020
@inadob @zhiics
[Frontend][TFLite] Add parser support for l2_normalization (apache#4966)
d50e6b2 
* [Frontend][TFLite] Add parser support for l2_normalization

* TF doesn't provide uint8 support
* TFL does the normalization only if it's over the last axis
* TFL uses only the default value for expilon

* Change error message
@ZihengJiang ZihengJiang mentioned this pull request Sep 17, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wyc-ruiker wyc-ruiker wyc-ruiker left review comments

@anijain2305 anijain2305 anijain2305 approved these changes

Assignees

@anijain2305 anijain2305

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@inadob @anijain2305 @wyc-ruiker