Skip to content

Conversation

yongwww
Copy link
Member

@yongwww yongwww commented Mar 31, 2019

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 @yongwww .
Good to go with minor comments.

@yzhliu yzhliu added the status: need update need update based on feedbacks label Apr 2, 2019
@yongwww
Copy link
Member Author

yongwww commented Apr 3, 2019

@srkreddy1238 incorporated comments. Pls take another look.

@srkreddy1238 srkreddy1238 merged commit eb82e7b into apache:master Apr 3, 2019
@srkreddy1238
Copy link
Contributor

Thanks @zhiics @yongwww @jroesch . This is now merged.

wweic pushed a commit to wweic/tvm that referenced this pull request Apr 7, 2019
* [Relay][Frontend] Support tf.where

* fix comments
wweic pushed a commit to wweic/tvm that referenced this pull request Apr 7, 2019
* [Relay][Frontend] Support tf.where

* fix comments
wweic pushed a commit to wweic/tvm that referenced this pull request Apr 8, 2019
* [Relay][Frontend] Support tf.where

* fix comments
wweic pushed a commit to wweic/tvm that referenced this pull request Apr 10, 2019
* [Relay][Frontend] Support tf.where

* fix comments
wweic pushed a commit to neo-ai/tvm that referenced this pull request Apr 11, 2019
* [Relay][Frontend] Support tf.where

* fix comments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: need update need update based on feedbacks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants