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.

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

[Relay][Frontend] Add a bunch of ops in tf converter #3270

Merged
merged 3 commits into from
Jun 13, 2019

Conversation

yongwww
Copy link
Member

@yongwww yongwww commented May 31, 2019

Add several ops in tf converter, fix issue of "Range", add a few test cases like unpack, op list is as below.

  • - Abs
  • - FloorDiv
  • - GatherNd
  • - LeftShift
  • - LogSoftmax
  • - Max
  • - Min
  • - Mod
  • - RightShift
  • - ZerosLike
  • - TruncateMod

cc: @srkreddy1238 @zhiics @wweic @yzhliu

@yongwww yongwww force-pushed the tf_ops branch 3 times, most recently from f4c5b50 to a5c99f9 Compare June 2, 2019 22:08
@yongwww
Copy link
Member Author

yongwww commented Jun 5, 2019

ping @srkreddy1238 @zhiics

@zhiics
Copy link
Member

zhiics commented Jun 5, 2019

It looks good to me. Please rebase. @srkreddy1238 Can you take a look as well.

@yongwww yongwww force-pushed the tf_ops branch 3 times, most recently from 9423313 to 4c1ed8f Compare June 12, 2019 03:43
@tqchen
Copy link
Member

tqchen commented Jun 13, 2019

@yongwww please check the ci error #3270

@yongwww
Copy link
Member Author

yongwww commented Jun 13, 2019

@tqchen CI issue was fixed.

@tqchen tqchen merged commit 9bb1687 into apache:master Jun 13, 2019
@tqchen
Copy link
Member

tqchen commented Jun 13, 2019

Thanks, @yongwww this PR is now merged.

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

Successfully merging this pull request may close these issues.

3 participants