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 2020/04/07 21:02:18 UTC
[GitHub] [incubator-tvm] icemelon9 opened a new pull request #5270: [CI]
Change MxNet from MKL verion to regular CPU version
icemelon9 opened a new pull request #5270: [CI] Change MxNet from MKL verion to regular CPU version
URL: https://github.com/apache/incubator-tvm/pull/5270
Because mxnet-mkl doesn't support AMD cpu. see #5240
cc @tqchen @anijain2305 @shoubhik
----------------------------------------------------------------
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
With regards,
Apache Git Services
[GitHub] [incubator-tvm] icemelon9 edited a comment on issue #5270: [CI]
Change MxNet from MKL verion to regular CPU version
Posted by GitBox <gi...@apache.org>.
icemelon9 edited a comment on issue #5270: [CI] Change MxNet from MKL verion to regular CPU version
URL: https://github.com/apache/incubator-tvm/pull/5270#issuecomment-610673485
Thanks @leezu for the clarification. According to the link @leezu shares, mxnet-mkl is able to run on AMD CPU. So should we just keep mxnet-mkl so that @shoubhik can use the quantization feature for test cases?
@tqchen
----------------------------------------------------------------
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
With regards,
Apache Git Services
[GitHub] [incubator-tvm] icemelon9 commented on issue #5270: [CI] Change
MxNet from MKL verion to regular CPU version
Posted by GitBox <gi...@apache.org>.
icemelon9 commented on issue #5270: [CI] Change MxNet from MKL verion to regular CPU version
URL: https://github.com/apache/incubator-tvm/pull/5270#issuecomment-610673485
According to the link @leezu shared, mxnet-mkl is able to run on AMD CPU. So should we just keep mxnet-mkl so that @shoubhik can use the quantization feature for test cases?
@tqchen
----------------------------------------------------------------
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
With regards,
Apache Git Services
[GitHub] [incubator-tvm] leezu commented on issue #5270: [CI] Change MxNet
from MKL verion to regular CPU version
Posted by GitBox <gi...@apache.org>.
leezu commented on issue #5270: [CI] Change MxNet from MKL verion to regular CPU version
URL: https://github.com/apache/incubator-tvm/pull/5270#issuecomment-610664463
mxnet-mkl is not build with mkl but only mkldnn. The name is confusing.
----------------------------------------------------------------
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
With regards,
Apache Git Services
[GitHub] [incubator-tvm] tqchen commented on issue #5270: [CI] Change MxNet
from MKL verion to regular CPU version
Posted by GitBox <gi...@apache.org>.
tqchen commented on issue #5270: [CI] Change MxNet from MKL verion to regular CPU version
URL: https://github.com/apache/incubator-tvm/pull/5270#issuecomment-611226008
OK, we will try to update the CI to mxnet-mkl and see how it goes, closing this for now, will update the other thread in a few days
----------------------------------------------------------------
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
With regards,
Apache Git Services
[GitHub] [incubator-tvm] icemelon9 commented on issue #5270: [CI] Change
MxNet from MKL verion to regular CPU version
Posted by GitBox <gi...@apache.org>.
icemelon9 commented on issue #5270: [CI] Change MxNet from MKL verion to regular CPU version
URL: https://github.com/apache/incubator-tvm/pull/5270#issuecomment-610683044
@anijain2305 What do you mean ARM CPUs? mxnet doesn't have pip wheel for ARM.
----------------------------------------------------------------
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
With regards,
Apache Git Services
[GitHub] [incubator-tvm] anijain2305 edited a comment on issue #5270: [CI]
Change MxNet from MKL verion to regular CPU version
Posted by GitBox <gi...@apache.org>.
anijain2305 edited a comment on issue #5270: [CI] Change MxNet from MKL verion to regular CPU version
URL: https://github.com/apache/incubator-tvm/pull/5270#issuecomment-610685005
> @anijain2305 What do you mean ARM CPUs? mxnet doesn't have pip wheel for ARM.
Nvm. I got confused. I was thinking that if we add ARM CPUs to CI in future, will it cause issues (as mxnet-mkl is not supported on ARM)?
----------------------------------------------------------------
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
With regards,
Apache Git Services
[GitHub] [incubator-tvm] anijain2305 commented on issue #5270: [CI] Change
MxNet from MKL verion to regular CPU version
Posted by GitBox <gi...@apache.org>.
anijain2305 commented on issue #5270: [CI] Change MxNet from MKL verion to regular CPU version
URL: https://github.com/apache/incubator-tvm/pull/5270#issuecomment-610685005
> @anijain2305 What do you mean ARM CPUs? mxnet doesn't have pip wheel for ARM.
Nvm. I got confused. I was thinking that if we add ARM CPUs to CI in future, will it cause issues (as mxnet-mkl is not supported on ARM).
----------------------------------------------------------------
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
With regards,
Apache Git Services
[GitHub] [incubator-tvm] shoubhik commented on issue #5270: [CI] Change
MxNet from MKL verion to regular CPU version
Posted by GitBox <gi...@apache.org>.
shoubhik commented on issue #5270: [CI] Change MxNet from MKL verion to regular CPU version
URL: https://github.com/apache/incubator-tvm/pull/5270#issuecomment-610684437
If mxnet-mkl does not break any existing functionality we should probably take it as a dependency. That gives us enough wiggle room and infra in the future to experiment on how we can add tests for mxnet Qnn. What do you guys think?
----------------------------------------------------------------
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
With regards,
Apache Git Services
[GitHub] [incubator-tvm] leezu commented on issue #5270: [CI] Change MxNet
from MKL verion to regular CPU version
Posted by GitBox <gi...@apache.org>.
leezu commented on issue #5270: [CI] Change MxNet from MKL verion to regular CPU version
URL: https://github.com/apache/incubator-tvm/pull/5270#issuecomment-610664896
Any release after mxnet 1.6 will build with mkldnn by default. If you don't want to use mkldnn, you need to use mxnet-native package in the future. See https://lists.apache.org/thread.html/1a22dbd79098adab6d02d16e8d607bae2acc908c0bb1b085d28a51ba@%3Cdev.mxnet.apache.org%3E
----------------------------------------------------------------
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
With regards,
Apache Git Services
[GitHub] [incubator-tvm] anijain2305 commented on issue #5270: [CI] Change
MxNet from MKL verion to regular CPU version
Posted by GitBox <gi...@apache.org>.
anijain2305 commented on issue #5270: [CI] Change MxNet from MKL verion to regular CPU version
URL: https://github.com/apache/incubator-tvm/pull/5270#issuecomment-610675721
How about ARM CPUs?
----------------------------------------------------------------
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
With regards,
Apache Git Services
[GitHub] [incubator-tvm] tqchen closed pull request #5270: [CI] Change MxNet
from MKL verion to regular CPU version
Posted by GitBox <gi...@apache.org>.
tqchen closed pull request #5270: [CI] Change MxNet from MKL verion to regular CPU version
URL: https://github.com/apache/incubator-tvm/pull/5270
----------------------------------------------------------------
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
With regards,
Apache Git Services