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

Commit

Permalink
CD Fixes (#16127)
Browse files Browse the repository at this point in the history
* Ignore load lib test in CD jobs

* Removes cu80 and adds cu101 support to CD builds

* Disable cython in CD python tests

* Updates CD documentation to reflect variant changes
  • Loading branch information
perdasilva authored and zachgk committed Sep 12, 2019
1 parent b4b7bfb commit c5383f7
Show file tree
Hide file tree
Showing 5 changed files with 12 additions and 4 deletions.
2 changes: 1 addition & 1 deletion cd/Jenkinsfile_cd_pipeline
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ pipeline {

parameters {
// Release parameters
string(defaultValue: "cpu,mkl,cu80,cu80mkl,cu90,cu90mkl,cu92,cu92mkl,cu100,cu100mkl", description: "Comma separated list of variants", name: "MXNET_VARIANTS")
string(defaultValue: "cpu,mkl,cu90,cu90mkl,cu92,cu92mkl,cu100,cu100mkl,cu101,cu101mkl", description: "Comma separated list of variants", name: "MXNET_VARIANTS")
booleanParam(defaultValue: false, description: 'Whether this is a release build or not', name: "RELEASE_BUILD")
}

Expand Down
4 changes: 2 additions & 2 deletions cd/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -31,14 +31,14 @@ Currently, 10 variants are supported:

* *cpu*: CPU
* *mkl*: CPU w/ MKL
* *cu80*: CUDA 8.0
* *cu80mkl*: CUDA 8.0 w/ MKL-DNN
* *cu90*: CUDA 9.0
* *cu90mkl*: CUDA 9.0 w/ MKL-DNN
* *cu92*: CUDA 9.2
* *cu92mkl*: CUDA 9.2 w/ MKL-DNN
* *cu100*: CUDA 10
* *cu100mkl*: CUDA 10 w/ MKL-DNN
* *cu101*: CUDA 10
* *cu101mkl*: CUDA 10.1 w/ MKL-DNN

*For more on variants, see [here](https://github.com/apache/incubator-mxnet/issues/8671)*

Expand Down
2 changes: 2 additions & 0 deletions ci/docker/runtime_functions.sh
Original file line number Diff line number Diff line change
Expand Up @@ -834,6 +834,8 @@ cd_unittest_ubuntu() {
export PYTHONPATH=./python/
export MXNET_MKLDNN_DEBUG=1 # Ignored if not present
export MXNET_STORAGE_FALLBACK_LOG_VERBOSE=0
export MXNET_ENABLE_CYTHON=0
export CD_JOB=1 # signal this is a CD run so any unecessary tests can be skipped

local mxnet_variant=${1:?"This function requires a mxnet variant as the first argument"}
local python_cmd=${2:?"This function requires a python command as the first argument"}
Expand Down
5 changes: 5 additions & 0 deletions python/mxnet/test_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -2220,3 +2220,8 @@ def collapse_sum_like(a, shape):
assert s == 1
axes.append(i+ndim_diff)
return np.sum(a, axis=tuple(axes)).reshape(shape)


def is_cd_run():
"""Checks if the test is running as part of a Continuous Delivery run"""
return os.environ.get("CD_JOB", 0) == "1"
3 changes: 2 additions & 1 deletion tests/python/unittest/test_library_loading.py
Original file line number Diff line number Diff line change
Expand Up @@ -22,12 +22,13 @@
import unittest
import mxnet as mx
from mxnet.base import MXNetError
from mxnet.test_utils import download
from mxnet.test_utils import download, is_cd_run

def check_platform():
return platform.machine() not in ['x86_64', 'AMD64']

@unittest.skipIf(check_platform(), "not all machine types supported")
@unittest.skipIf(is_cd_run(), "continuous delivery run - ignoring test")
def test_library_loading():
if (os.name=='posix'):
lib = 'libsample_lib.so'
Expand Down

0 comments on commit c5383f7

Please sign in to comment.