Skip to content

Conversation

@BBuf
Copy link
Contributor

@BBuf BBuf commented Jan 10, 2022

Add pip dependency of OneFlow in docker/docker.ci_cpu and docker/docker.ci_gpu

This pull request will modify TVM ci environment to support UT of OneFlow frontend, the related pull request is here #8790

@jcf94

@masahi
Copy link
Member

masahi commented Jan 10, 2022

Note that updating the docker file alone won't change the CI environment. See for example #9713. I can help upgrade the CI image after this gets in.

Also please remove the cpu docker change unless you have a good reason to use oneflow in the cpu image. Even PyTorch and ONNX are installed only on the gpu image.

@masahi masahi self-assigned this Jan 10, 2022
@masahi
Copy link
Member

masahi commented Jan 10, 2022

@BBuf Please update. I want to do the CI upgrade together with the onnx one in #9882

@BBuf
Copy link
Contributor Author

BBuf commented Jan 11, 2022

@BBuf Please update. I want to do the CI upgrade together with the onnx one in #9882

ok,thanks。

@masahi masahi merged commit 6f9c806 into apache:main Jan 11, 2022
Raghav-Chakravarthy pushed a commit to Raghav-Chakravarthy/tvm that referenced this pull request Jan 28, 2022
* add oneflow ci depend

* fix comment
ylc pushed a commit to ylc/tvm that referenced this pull request Feb 16, 2022
* add oneflow ci depend

* fix comment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants