-
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
[TFLite Runtime] Add TFLite Runtime dependencies to CI CPU docker build #5437
[TFLite Runtime] Add TFLite Runtime dependencies to CI CPU docker build #5437
Conversation
The check_local() test is flakey, something to do with how we retrieve outputs from the tflite interpreter. Leaving it commented it out for now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
# Build the TFLite static library, necessary for building with TFLite ON. | ||
# The library is built at: | ||
# tensorflow/tensorflow/lite/tools/make/gen/*/lib/libtensorflow-lite.a. | ||
git clone https://github.com/tensorflow/tensorflow --branch=r2.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why not 2.1.0 which is the latest stable version and what we install in the tree. I don’t think it’s worth the version skew.
Ramana
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@u99127 Good catch, thank you. I noticed your PR updating the version yesterday but I forgot I was specifying 2.0 here. Will update :)
This PR contains only the docker build changes needed to re-enable the tests in: #5436.