-
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] Update ci-gpu image #15836
[CI] Update ci-gpu image #15836
Conversation
The ci-gpu is updated to make changes in apache#15819 reflect in CI. Change-Id: I5370f80fa82839a8e2f97cf3de9bd1ede7e2d350
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! Thanks @lhutton1 😄
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, please take a look at the ci errors
Change-Id: I4e6caca8d9e3aa93b67b006a2f03b391ec708d53
797b82d
to
9bef182
Compare
Hit a
on the cortexm build, retriggering... |
Between the previous ci_gpu image and the one in use now, the version of coreml has changed since it is unpinned (6.3.0 -> 7.0.0). This caused a test failure in the coreml frontend tests. The problem is that coreml now uses a new default format for models. This commit forces coreml to use the old model format so the test passes. More information can be found here: https://apple.github.io/coremltools/docs-guides/source/convert-learning-models.html Change-Id: Id2273ccac0ae771f10dda6590e9f4cdd657a5d25
9bef182
to
6d3269a
Compare
closes: #15754 |
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 - thanks @lhutton1 :)
The ci-gpu is updated to make changes in #15819 reflect in CI.
cc @ashutosh-arm