-
Notifications
You must be signed in to change notification settings - Fork 620
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix nose
imports
#5698
Fix nose
imports
#5698
Changes from 2 commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -279,7 +279,7 @@ def check_fancy_upsampling_body(batch_size, img_type, device): | |
@params(1, 8) | ||
def test_fancy_upsampling(batch_size): | ||
if nvidia.dali.backend.GetNvjpegVersion() < 12001: | ||
from nose import SkipTest | ||
from nose_utils import SkipTest | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I would move this to the beginning of the file. |
||
|
||
raise SkipTest("nvJPEG doesn't support fancy upsampling in this version") | ||
data_path = os.path.join(test_data_root, good_path, "jpeg") | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -29,7 +29,7 @@ | |
from test_utils import get_arch | ||
from test_utils import dump_as_core_artifacts | ||
from nose2.tools import params | ||
from nose import SkipTest | ||
from nose_utils import SkipTest | ||
|
||
|
||
def get_img_files(data_path, subdir="*", ext=None): | ||
|
@@ -280,7 +280,7 @@ def check_fancy_upsampling_body(batch_size, img_type, device): | |
@params(1, 8) | ||
def test_fancy_upsampling(batch_size): | ||
if nvidia.dali.backend.GetNvjpegVersion() < 12001: | ||
from nose import SkipTest | ||
from nose_utils import SkipTest | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think it can be removed here as is duplicates L32. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. True. |
||
|
||
raise SkipTest("nvJPEG doesn't support fancy upsampling in this version") | ||
data_path = os.path.join(test_data_root, good_path, "jpeg") | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -19,7 +19,7 @@ | |
from nvidia.dali import pipeline_def | ||
|
||
from nose_utils import assert_raises | ||
from nose.plugins.attrib import attr | ||
from nose_utils import attr | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Maybe you can merge it with the above. |
||
from nose2.tools import cartesian_params | ||
|
||
|
||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -23,8 +23,8 @@ | |
from test_utils import get_dali_extra_path, is_mulit_gpu, skip_if_m60 | ||
from nvidia.dali.backend import TensorListGPU | ||
from nose2.tools import params | ||
from nose import SkipTest | ||
from nose.plugins.attrib import attr | ||
from nose_utils import SkipTest | ||
from nose_utils import attr | ||
from nose_utils import assert_raises | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Maybe you can merge it with the above. |
||
|
||
|
||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -22,7 +22,7 @@ | |
from scipy.ndimage import convolve1d | ||
import os | ||
from nose_utils import assert_raises, raises | ||
from nose.plugins.attrib import attr | ||
from nose_utils import attr | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Maybe you can merge it with the above. |
||
|
||
from sequences_test_utils import video_suite_helper, ArgCb | ||
from test_utils import ( | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -21,7 +21,7 @@ | |
from scipy.ndimage import convolve1d, filters as sp_filters | ||
import os | ||
from nose_utils import assert_raises | ||
from nose.plugins.attrib import attr | ||
from nose_utils import attr | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Maybe you can merge it with the above. |
||
|
||
from sequences_test_utils import video_suite_helper, ArgCb | ||
from test_utils import get_dali_extra_path, check_batch, RandomlyShapedDataIterator | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -28,7 +28,7 @@ | |
) | ||
from nvidia.dali.plugin.numba.fn.experimental import numba_function | ||
from numba import cuda | ||
from nose.plugins.attrib import attr | ||
from nose_utils import attr | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Maybe you can merge it with L22. |
||
|
||
test_data_root = get_dali_extra_path() | ||
lmdb_folder = os.path.join(test_data_root, "db", "lmdb") | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -23,8 +23,7 @@ | |
import scipy.ndimage.measurements | ||
import random | ||
from test_utils import check_batch, np_type_to_dali | ||
from nose.tools import nottest | ||
from nose.plugins.attrib import attr | ||
from nose_utils import attr, nottest | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Can be merged with L17 |
||
|
||
|
||
np.random.seed(1234) | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -23,7 +23,7 @@ | |
from test_utils import compare_pipelines, get_dali_extra_path | ||
from nose_utils import assert_raises, raises | ||
from nose2.tools import cartesian_params | ||
from nose import SkipTest | ||
from nose_utils import SkipTest | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Can be merged with L24 |
||
|
||
|
||
def skip_second(src, dst): | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -23,7 +23,7 @@ | |
import random | ||
import tempfile | ||
from nose_utils import assert_raises | ||
from nose import SkipTest | ||
from nose_utils import SkipTest | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Maybe you can merge it with the above. |
||
from nose2.tools import params, cartesian_params | ||
from test_utils import compare_pipelines, to_array | ||
|
||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -23,7 +23,7 @@ | |
import nvidia.dali as dali | ||
|
||
from nose_utils import assert_raises | ||
from nose import SkipTest | ||
from nose_utils import SkipTest | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Maybe you can merge it with the above and below. |
||
from test_utils import dali_type_to_np, py_buffer_from_address, get_device_memory_info | ||
|
||
|
||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -18,7 +18,7 @@ | |
import test_dali_tf_dataset_mnist as mnist | ||
from test_utils_tensorflow import skip_for_incompatible_tf, available_gpus | ||
from nose_utils import raises | ||
from nose import SkipTest | ||
from nose_utils import SkipTest | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Maybe you can merge it with the above. |
||
from packaging.version import Version | ||
|
||
tf.compat.v1.enable_eager_execution() | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -16,7 +16,7 @@ | |
from nvidia.dali import tensors, pipeline_def | ||
import nvidia.dali.fn as fn | ||
from nose_utils import raises, assert_equals | ||
from nose.plugins.attrib import attr | ||
from nose_utils import attr | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Maybe you can merge it with the above. |
||
import numpy as np | ||
|
||
|
||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -20,7 +20,7 @@ | |
from nose_utils import assert_raises | ||
import sys | ||
import inspect | ||
from nose.plugins.attrib import attr | ||
from nose_utils import attr | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Can be merged with L20 |
||
import nose | ||
|
||
|
||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -21,7 +21,7 @@ | |
from nvidia.dali import fn, types | ||
from test_utils import get_dali_extra_path, get_arch | ||
from nose_utils import raises, assert_raises | ||
from nose import SkipTest | ||
from nose_utils import SkipTest | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Maybe you can merge it with the above. |
||
|
||
test_data_root = get_dali_extra_path() | ||
images_dir = os.path.join(test_data_root, "db", "imgproc") | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -14,7 +14,7 @@ | |
|
||
import numpy as np | ||
import os | ||
from nose.plugins.attrib import attr | ||
from nose_utils import attr | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Can be merged with L23 and L25 |
||
|
||
from nvidia.dali import fn | ||
from nvidia.dali import tensors | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -15,7 +15,7 @@ | |
|
||
import numpy as np | ||
import nvidia.dali.fn as fn | ||
from nose.plugins.attrib import attr | ||
from nose_utils import attr | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Can be merged with L24 |
||
from nose2.tools import params | ||
from numpy.random import default_rng | ||
from nvidia.dali import pipeline_def | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -23,7 +23,7 @@ | |
import math | ||
import random | ||
import os | ||
from nose.tools import nottest | ||
from nose_utils import nottest | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Can be moved to L15 |
||
|
||
# Filtering librispeech samples | ||
audio_files = get_files("db/audio/wav", "wav") | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe you can merge it with the above.