You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2022/04/28 07:18:48 UTC

[GitHub] [tvm] t-vi opened a new pull request, #11156: Bump cmake version for GPU build

t-vi opened a new pull request, #11156:
URL: https://github.com/apache/tvm/pull/11156

   The cmake version (3.10) in Ubuntu 18.04 does not cope well with the
   more advanced cmake use in libtorch surrounding the CUDA target.
   We switch to a self-built cmake 3.14 (already used by arm and i386 CI).
   
   The context for this is #10758 .
   
   Thank you @masahi @driazati for your helpful discussion, errors are my own.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [tvm] Mousius commented on pull request #11156: Bump cmake version for GPU build

Posted by GitBox <gi...@apache.org>.
Mousius commented on PR #11156:
URL: https://github.com/apache/tvm/pull/11156#issuecomment-1112015086

   @t-vi / @masahi it looks like https://snapshots.linaro.org/ldcg/python-cache/tensorflow-aarch64/ is back, I've restarted your build in Jenkins to see if we can get this moving 😸 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [tvm] Mousius merged pull request #11156: Bump cmake version for GPU build

Posted by GitBox <gi...@apache.org>.
Mousius merged PR #11156:
URL: https://github.com/apache/tvm/pull/11156


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [tvm] masahi commented on pull request #11156: Bump cmake version for GPU build

Posted by GitBox <gi...@apache.org>.
masahi commented on PR #11156:
URL: https://github.com/apache/tvm/pull/11156#issuecomment-1111952502

   @driazati For a PR like this where only `docker/Dockerfile.ci_gpu`, can we skip building ci image for other nodes? Docker scripts update has been painful because "Docker Image Build" stage in the pipeline can easily fail for strange reasons and especially frustrating when the failed image is for unrelated nodes... 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [tvm] t-vi commented on pull request #11156: Bump cmake version for GPU build

Posted by GitBox <gi...@apache.org>.
t-vi commented on PR #11156:
URL: https://github.com/apache/tvm/pull/11156#issuecomment-1112072140

   Thank you, @Mousius  !


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [tvm] masahi commented on pull request #11156: Bump cmake version for GPU build

Posted by GitBox <gi...@apache.org>.
masahi commented on PR #11156:
URL: https://github.com/apache/tvm/pull/11156#issuecomment-1111894353

   @Mousius Seems TF-aarch64 v2.6.2 we try to install at https://github.com/apache/tvm/blob/main/docker/install/ubuntu_install_tensorflow_aarch64.sh#L29 is no longer available? See https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/PR-11156/1/pipeline/
   
   Maybe we should switch to TF 2.7?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [tvm] Mousius commented on pull request #11156: Bump cmake version for GPU build

Posted by GitBox <gi...@apache.org>.
Mousius commented on PR #11156:
URL: https://github.com/apache/tvm/pull/11156#issuecomment-1111945207

   Looks like https://snapshots.linaro.org/ldcg/python-cache/tensorflow-aarch64/ is down for some reason, I've reached out to find out why. 
   
   The reason for `2.6.2` is to maintain parity with `ci_cpu`:
   https://github.com/apache/tvm/blob/9fd279b40aa77677e9ba698ca34c8062b55cb3e8/docker/install/ubuntu_install_tensorflow.sh#L23-L26
   
   Upgrading to `2.7.0` should really be done in both places?
   
   CC @gromero @leandron 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org