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] Simplify parameter handling in Tensorflow frontend #2993

Merged
merged 2 commits into from
Jun 6, 2019

Conversation

alexeyr
Copy link
Contributor

@alexeyr alexeyr commented Apr 9, 2019

On top of #2943.

@alexeyr alexeyr requested review from Huyuwei and Laurawly as code owners April 9, 2019 14:43
@yzhliu
Copy link
Member

yzhliu commented Apr 9, 2019

Please request review from https://github.com/dmlc/tvm/blob/master/CONTRIBUTORS.md

@alexeyr alexeyr force-pushed the tensorflow-params branch 4 times, most recently from 0b58be5 to 1d2d18b Compare April 15, 2019 09:05
@srkreddy1238
Copy link
Contributor

@alexeyr can you rebase ?

@tqchen
Copy link
Member

tqchen commented May 16, 2019

ping @alexeyr

@alexeyr alexeyr force-pushed the tensorflow-params branch 6 times, most recently from f01d7e9 to da3d09a Compare May 17, 2019 11:30
@alexeyr
Copy link
Contributor Author

alexeyr commented May 17, 2019

@srkreddy1238 @Laurawly @Huyuwei Rebased, please review

@tqchen
Copy link
Member

tqchen commented Jun 5, 2019

ping @srkreddy1238

Copy link
Contributor

@srkreddy1238 srkreddy1238 left a comment

Choose a reason for hiding this comment

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

Thanks @alexeyr . LGTM.

@tqchen tqchen merged commit 770ac84 into apache:master Jun 6, 2019
@tqchen
Copy link
Member

tqchen commented Jun 6, 2019

Thanks, @srkreddy1238 @alexeyr , 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.

4 participants