You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2020/10/18 14:52:09 UTC

[GitHub] [incubator-tvm] ANSHUMAN87 opened a new pull request #6706: [Relay][Frontend] Tensorflow version support upgrade from 2.1.0 to 2.3.1

ANSHUMAN87 opened a new pull request #6706:
URL: https://github.com/apache/incubator-tvm/pull/6706


   Change summary:
   
   - Version bumped latest up-to-date
   - Few dependent failures resolved.
   
   Background:
     Please refer #6685 .
   To support recent advances in Sparse area in Tensorflow community, the version need to be upgraded.
   
   cc @siju-samuel , @tkonolige , @tqchen , @jroesch , @junrushao1994 , @FrozenGene !


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-tvm] ANSHUMAN87 commented on pull request #6706: [CI] Tensorflow version support upgrade from 2.1.0 to 2.3.1

Posted by GitBox <gi...@apache.org>.
ANSHUMAN87 commented on pull request #6706:
URL: https://github.com/apache/incubator-tvm/pull/6706#issuecomment-717094780


   > FYI, we will need to resolve #6754 in order to upgrade the TF images to 2.3.1
   
   Thanks @tqchen, I will look into it. 


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-tvm] tqchen edited a comment on pull request #6706: [CI] Tensorflow version support upgrade from 2.1.0 to 2.3.1

Posted by GitBox <gi...@apache.org>.
tqchen edited a comment on pull request #6706:
URL: https://github.com/apache/incubator-tvm/pull/6706#issuecomment-716176115


   FYI, we will need to resolve  https://github.com/apache/incubator-tvm/issues/6754 in order to upgrade the TF images to 2.3.1


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-tvm] tkonolige commented on pull request #6706: [Relay][Frontend] Tensorflow version support upgrade from 2.1.0 to 2.3.1

Posted by GitBox <gi...@apache.org>.
tkonolige commented on pull request #6706:
URL: https://github.com/apache/incubator-tvm/pull/6706#issuecomment-712263195


   Does this mean that users with tensorflow version <2.3.0 will not be able to import their models?


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-tvm] ANSHUMAN87 commented on pull request #6706: [Relay][Frontend] Tensorflow version support upgrade from 2.1.0 to 2.3.1

Posted by GitBox <gi...@apache.org>.
ANSHUMAN87 commented on pull request #6706:
URL: https://github.com/apache/incubator-tvm/pull/6706#issuecomment-713341409


   @tqchen : If no further concern here! Would you please help to get this PR merged with latest docker images. So that our forth coming Sparse PRs will be smooth to handle. TIA!


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-tvm] tqchen commented on pull request #6706: [CI] Tensorflow version support upgrade from 2.1.0 to 2.3.1

Posted by GitBox <gi...@apache.org>.
tqchen commented on pull request #6706:
URL: https://github.com/apache/incubator-tvm/pull/6706#issuecomment-717329368


   @leandron  feel free to send a PR to revert the version


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-tvm] tqchen merged pull request #6706: [CI] Tensorflow version support upgrade from 2.1.0 to 2.3.1

Posted by GitBox <gi...@apache.org>.
tqchen merged pull request #6706:
URL: https://github.com/apache/incubator-tvm/pull/6706


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-tvm] ANSHUMAN87 commented on pull request #6706: [Relay][Frontend] Tensorflow version support upgrade from 2.1.0 to 2.3.1

Posted by GitBox <gi...@apache.org>.
ANSHUMAN87 commented on pull request #6706:
URL: https://github.com/apache/incubator-tvm/pull/6706#issuecomment-712621795


   > Does this mean that users with tensorflow version <2.3.0 will not be able to import their models?
   
   I am not too sure if i understood your query clearly. Please let me know in case my response is upto. 
   If user has tensorflow version <2.3.0 , they might get error while executing our test scripts, but importing models if it is based on older version they should be able to do it.Thanks!


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-tvm] leandron commented on pull request #6706: [CI] Tensorflow version support upgrade from 2.1.0 to 2.3.1

Posted by GitBox <gi...@apache.org>.
leandron commented on pull request #6706:
URL: https://github.com/apache/incubator-tvm/pull/6706#issuecomment-717310619


   We are facing issues (reported broken TFlite tests #6754) when using the latest Docker files to run tests in our downstream CI.
   
   Can we temporarily revert this change here, so that it is merged again when the rest of the sources are prepared to support TF 2.3.x?


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-tvm] tqchen commented on pull request #6706: [CI] Tensorflow version support upgrade from 2.1.0 to 2.3.1

Posted by GitBox <gi...@apache.org>.
tqchen commented on pull request #6706:
URL: https://github.com/apache/incubator-tvm/pull/6706#issuecomment-716176115


   FYI https://github.com/apache/incubator-tvm/issues/6754


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-tvm] leandron edited a comment on pull request #6706: [CI] Tensorflow version support upgrade from 2.1.0 to 2.3.1

Posted by GitBox <gi...@apache.org>.
leandron edited a comment on pull request #6706:
URL: https://github.com/apache/incubator-tvm/pull/6706#issuecomment-717310619


   We are facing issues (reported broken TFlite tests #6754) when using the latest Docker files to run tests in our downstream CI.
   
   Can we temporarily revert this change here, so that it is merged again when the rest of the sources are prepared to support TF 2.3.x?
   
   cc @u99127 


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-tvm] ANSHUMAN87 commented on pull request #6706: [CI] Tensorflow version support upgrade from 2.1.0 to 2.3.1

Posted by GitBox <gi...@apache.org>.
ANSHUMAN87 commented on pull request #6706:
URL: https://github.com/apache/incubator-tvm/pull/6706#issuecomment-717373007


   > We are facing issues (reported broken TFlite tests #6754) when using the latest Docker files to run tests in our downstream CI.
   > 
   > Can we temporarily revert this change here, so that it is merged again when the rest of the sources are prepared to support TF 2.3.x?
   > 
   > cc @u99127
   
   I am working on the fix currently. Thanks!


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org