-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
[CI] Tensorflow 2.3.1 upgrade blocker #6754
Comments
The failures are resolved in #6774 |
@tqchen I think this can be closed now |
Sorry, too early to claim it is completely fixed :(. I just checked again and there was another side effect, this time on Keras testing. When using pure
I see this looks like tensorflow/models#6177 and tensorflow/tensorflow#38589. Apparently it is not fixed yet. @ANSHUMAN87 are you able to reproduce this issues when running frontend tests? |
@leandron : Sure i will look into it. Can you please confirm whether keras version in ur env is 2.3.1 ? |
@ANSHUMAN87 - we just use the same Docker image as upstream ci_cpu,so it would be the version as in ubuntu_install_tensorflow.sh . So I think it's keras 2.3.1 install/ubuntu_install_tensorflow.sh:pip3 install tensorflow==2.3.1 keras==2.3.1 h5py |
Yes, I confirm as per https://github.com/apache/incubator-tvm/blob/main/docker/install/ubuntu_install_tensorflow.sh and also by checking the installation logs. |
Okay, i checked the failures, we have to upgrade the keras version too, along with it. Will raise one PR for it now. |
I confirm now all the issues are gone. |
Thanks @leandron for confirmation! |
seems was due to #6808 changes to keras 2.3.1 |
Sorry, I reviewed it and it was something I changed wrong. New PR coming in a few seconds. |
Followup of #6706 Some updates in TF and TFLite are needed to enable TF 2.3.1 support
https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/ci-docker-staging/28/pipeline/320
The text was updated successfully, but these errors were encountered: