You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mxnet.apache.org by GitBox <gi...@apache.org> on 2020/10/09 03:58:56 UTC
[GitHub] [incubator-mxnet] waytrue17 opened a new pull request #19319: Update cuda arch for cu110 dynamic build
waytrue17 opened a new pull request #19319:
URL: https://github.com/apache/incubator-mxnet/pull/19319
## Description ##
Currently the mxnet dynamic build pipeline uses `Auto` flag to specify cuda arch: https://github.com/apache/incubator-mxnet/blob/9f8af7bebd8b9354fb162b5ec07499b27ec1badf/CMakeLists.txt#L43
This includes cuda arch 3.0 into cu110 build and causes build error (cu110 does not support arch 3.0).
This PR aims to correct the cuda arch for cu110 dynamic build.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-mxnet] leezu commented on pull request #19319: Update cuda arch for cu110 dynamic build
Posted by GitBox <gi...@apache.org>.
leezu commented on pull request #19319:
URL: https://github.com/apache/incubator-mxnet/pull/19319#issuecomment-706314776
The dynamic build on CD is actually not very meaningful. Both libtype=static and libtype=dynamic build the dynamic libmxnet, the only difference is the number of dependencies statically linked. But in fact libtype=dynamic artifacts are not used anywhere. libtype=static is fully sufficient and we can reduce complexity and costs while achieving the same goal by removing unused libtype=dynamic. The goal here is to keep things simple and avoid unnecessary complexity.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-mxnet] waytrue17 commented on pull request #19319: Update cuda arch for cu110 dynamic build
Posted by GitBox <gi...@apache.org>.
waytrue17 commented on pull request #19319:
URL: https://github.com/apache/incubator-mxnet/pull/19319#issuecomment-705955268
@mseth10 Can you please review? Thanks
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-mxnet] waytrue17 closed pull request #19319: Update cuda arch for cu110 dynamic build
Posted by GitBox <gi...@apache.org>.
waytrue17 closed pull request #19319:
URL: https://github.com/apache/incubator-mxnet/pull/19319
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-mxnet] waytrue17 commented on pull request #19319: Update cuda arch for cu110 dynamic build
Posted by GitBox <gi...@apache.org>.
waytrue17 commented on pull request #19319:
URL: https://github.com/apache/incubator-mxnet/pull/19319#issuecomment-706416055
We will work on removing the libtype=dynamic pipeline altogether in a separate PR
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #19319: Update cuda arch for cu110 dynamic build
Posted by GitBox <gi...@apache.org>.
mxnet-bot commented on pull request #19319:
URL: https://github.com/apache/incubator-mxnet/pull/19319#issuecomment-705954076
Hey @waytrue17 , Thanks for submitting the PR
All tests are already queued to run once. If tests fail, you can trigger one or more tests again with the following commands:
- To trigger all jobs: @mxnet-bot run ci [all]
- To trigger specific jobs: @mxnet-bot run ci [job1, job2]
***
**CI supported jobs**: [centos-cpu, windows-cpu, edge, unix-cpu, windows-gpu, miscellaneous, website, unix-gpu, clang, centos-gpu, sanity]
***
_Note_:
Only following 3 categories can trigger CI :PR Author, MXNet Committer, Jenkins Admin.
All CI tests must pass before the PR can be merged.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #19319: Update cuda arch for cu110 dynamic build
Posted by GitBox <gi...@apache.org>.
mxnet-bot commented on pull request #19319:
URL: https://github.com/apache/incubator-mxnet/pull/19319#issuecomment-705954076
Hey @waytrue17 , Thanks for submitting the PR
All tests are already queued to run once. If tests fail, you can trigger one or more tests again with the following commands:
- To trigger all jobs: @mxnet-bot run ci [all]
- To trigger specific jobs: @mxnet-bot run ci [job1, job2]
***
**CI supported jobs**: [centos-cpu, windows-cpu, edge, unix-cpu, windows-gpu, miscellaneous, website, unix-gpu, clang, centos-gpu, sanity]
***
_Note_:
Only following 3 categories can trigger CI :PR Author, MXNet Committer, Jenkins Admin.
All CI tests must pass before the PR can be merged.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-mxnet] waytrue17 commented on pull request #19319: Update cuda arch for cu110 dynamic build
Posted by GitBox <gi...@apache.org>.
waytrue17 commented on pull request #19319:
URL: https://github.com/apache/incubator-mxnet/pull/19319#issuecomment-705955268
@mseth10 Can you please review? Thanks
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org