Lunderberg commented on issue #8177:
URL: https://github.com/apache/tvm/issues/8177#issuecomment-872394349


   @mbrookhart  Whoops, I was mistaken.  I thought that `keras` was a 
dependency of TF, because it was in the `ubuntu_install_tensorflow.sh` script, 
and that the removal of the pinned version of keras was part of the same issue 
that required the pinned h5py version.  I was wrong, and the keras installation 
was separate from the installation of TF, but depends on TF.
   
   It looks like the standalone-keras is only used if we're importing a keras 
model that isn't a tensorflow model.  Based on [this 
announcement](https://github.com/keras-team/keras/releases/tag/2.3.0), the 
support for non-tensorflow models is being dropped in keras 2.4, so I think we 
should pin it to `keras<2.4.0`.  @leandron , were the issues you ran into in 
#6810 based on using keras 2.3.1 alongside cuda 10.2, and do you know if they 
would still be present with cuda 11.0?


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to