Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

Fix MXNet R package build #13952

Merged
merged 12 commits into from
Jan 31, 2019
Merged

Fix MXNet R package build #13952

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
6db3f6b
fix mxnet r package build
Jan 22, 2019
7f82402
add ci
Jan 23, 2019
c9c7bdd
remove mkldnn-gpu test for R
Jan 24, 2019
dc0b42e
Merge remote-tracking branch 'origin' into r_build
Jan 24, 2019
3c33417
Merge remote-tracking branch 'origin' into r_build
Jan 24, 2019
9e0ec3a
add minimal test for MKLDNN-R
Jan 29, 2019
cd36fb1
Merge remote-tracking branch 'origin' into r_build
Jan 29, 2019
8ae9a57
Merge remote-tracking branch 'origin' into r_build
Jan 30, 2019
e55d00a
Merge remote-tracking branch 'origin' into r_build
Jan 30, 2019
1a0c2e6
Merge remote-tracking branch 'origin' into r_build
Jan 30, 2019
4f65e2c
pick mlp as minimal R test
Jan 31, 2019
f941133
Merge remote-tracking branch 'origin' into r_build
Jan 31, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -599,6 +599,13 @@ rpkg:
mkdir -p R-package/inst/libs
cp src/io/image_recordio.h R-package/src
cp -rf lib/libmxnet.so R-package/inst/libs

if [ -e "lib/libmkldnn.so.0" ]; then \
cp -rf lib/libmkldnn.so.0 R-package/inst/libs; \
cp -rf lib/libiomp5.so R-package/inst/libs; \
cp -rf lib/libmklml_intel.so R-package/inst/libs; \
fi

mkdir -p R-package/inst/include
cp -rf include/* R-package/inst/include
rm R-package/inst/include/dmlc
Expand Down
28 changes: 28 additions & 0 deletions ci/jenkins/Jenkins_steps.groovy
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -901,6 +901,20 @@ def test_unix_r_cpu() {
}]
}

def test_unix_r_mkldnn_cpu() {
return ['R: MKLDNN-CPU': {
node(NODE_LINUX_CPU) {
ws('workspace/ut-r-mkldnn-cpu') {
timeout(time: max_time, unit: 'MINUTES') {
utils.unpack_and_init('mkldnn_cpu', mx_mkldnn_lib, true)
utils.docker_run('ubuntu_cpu', 'unittest_ubuntu_cpu_R', false)
utils.publish_test_coverage()
}
}
}
}]
}

def test_unix_perl_cpu() {
return ['Perl: CPU': {
node(NODE_LINUX_CPU) {
Expand Down Expand Up @@ -985,6 +999,20 @@ def test_unix_r_gpu() {
}]
}

def test_unix_r_mkldnn_gpu() {
return ['R: MKLDNN-GPU': {
node(NODE_LINUX_GPU) {
ws('workspace/ut-r-mkldnn-gpu') {
timeout(time: max_time, unit: 'MINUTES') {
utils.unpack_and_init('mkldnn_gpu', mx_mkldnn_lib, true)
utils.docker_run('ubuntu_gpu', 'unittest_ubuntu_gpu_R', true)
utils.publish_test_coverage()
}
}
}
}]
}

def test_unix_julia07_cpu() {
return ['Julia 0.7: CPU': {
node(NODE_LINUX_CPU) {
Expand Down
1 change: 1 addition & 0 deletions ci/jenkins/Jenkinsfile_unix_cpu
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,7 @@ core_logic: {
custom_steps.test_unix_clojure_cpu(),
custom_steps.test_unix_perl_cpu(),
custom_steps.test_unix_r_cpu(),
custom_steps.test_unix_r_mkldnn_cpu(),
custom_steps.test_unix_julia07_cpu(),
custom_steps.test_unix_julia10_cpu(),
custom_steps.test_unix_onnx_cpu(),
Expand Down
1 change: 1 addition & 0 deletions ci/jenkins/Jenkinsfile_unix_gpu
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,7 @@ core_logic: {
custom_steps.test_unix_python3_tensorrt_gpu(),
custom_steps.test_unix_perl_gpu(),
custom_steps.test_unix_r_gpu(),
custom_steps.test_unix_r_mkldnn_gpu(),
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not sure whether we need GPU+MKLDNN for R since we already test it in python. Could you elaborate what concerns you are specifically having?

custom_steps.test_unix_cpp_gpu(),
custom_steps.test_unix_cpp_mkldnn_gpu(),
custom_steps.test_unix_python3_integration_gpu(),
Expand Down