Skip to content
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

Update Docker CI #8193

Merged
merged 11 commits into from Jul 19, 2021
Merged

Update Docker CI #8193

merged 11 commits into from Jul 19, 2021

Conversation

mbrookhart
Copy link
Contributor

@mbrookhart mbrookhart commented Jun 4, 2021

Adding a lot of updates to CI docker images, including onnx, arm, tensorflow, tflite, etc. Fixes a couple of bugs with the update.

@mbrookhart mbrookhart force-pushed the update_onnx branch 4 times, most recently from b4fb990 to 9371913 Compare June 17, 2021 22:17
@mbrookhart mbrookhart mentioned this pull request Jun 22, 2021
@mbrookhart mbrookhart force-pushed the update_onnx branch 4 times, most recently from 13e27ef to c41a4eb Compare July 4, 2021 02:16
@mbrookhart mbrookhart force-pushed the update_onnx branch 3 times, most recently from 0d1693a to 7f5a959 Compare July 12, 2021 15:35
@mbrookhart mbrookhart changed the title [WIP] Update ONNX versions Update Docker CI Jul 14, 2021
@mehrdadh
Copy link
Member

@mbrookhart I just tested it for both host and mps2 target and it passes. It could be a flaky timeout.
Please re-trigger the test.

Jenkinsfile Outdated Show resolved Hide resolved
Jenkinsfile Outdated Show resolved Hide resolved
Jenkinsfile Outdated Show resolved Hide resolved
Jenkinsfile Outdated Show resolved Hide resolved
Thanks, Andrew!

Co-authored-by: Andrew Reusch <areusch@gmail.com>
Copy link
Contributor

@areusch areusch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks @mbrookhart ! please update the tracking issue when you merge

repeat(1, D),
auto_pad="SAME_UPPER",
)
## TODO(mbrookhart): renable autopad tests when CI ONNX
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

per discussion, this will be a follow-on PR

@mbrookhart mbrookhart merged commit 6d88bdd into apache:main Jul 19, 2021
@mbrookhart mbrookhart deleted the update_onnx branch July 19, 2021 15:18
@mbrookhart
Copy link
Contributor Author

Thanks @areusch @mehrdadh

ylc pushed a commit to ylc/tvm that referenced this pull request Sep 29, 2021
* add failing onnx tets

* point jenkins at new docker

* support convtranspose opset 11 autopadding

* Don't force output shape for conv transpose tests, add 1D and 3D cases

* disable test until CI update complete

* try updating docker images again

* skip a test until update complete

* next try at docker images

* manage TF memory use in TF1 tests

* support explicit padding for NCHW TF padding test

* Update to tagged tlcpack images

Thanks, Andrew!

Co-authored-by: Andrew Reusch <areusch@gmail.com>

Co-authored-by: Andrew Reusch <areusch@gmail.com>
ylc pushed a commit to ylc/tvm that referenced this pull request Jan 13, 2022
* add failing onnx tets

* point jenkins at new docker

* support convtranspose opset 11 autopadding

* Don't force output shape for conv transpose tests, add 1D and 3D cases

* disable test until CI update complete

* try updating docker images again

* skip a test until update complete

* next try at docker images

* manage TF memory use in TF1 tests

* support explicit padding for NCHW TF padding test

* Update to tagged tlcpack images

Thanks, Andrew!

Co-authored-by: Andrew Reusch <areusch@gmail.com>

Co-authored-by: Andrew Reusch <areusch@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants