Skip to content

Commit 75b3581

Browse files
committed
Fix FITS tests with python3.7, reduce memory usage in rand aug tests (#4844)
* Fix FITS tests with python3.7 * Reduce memory consumption in rand aug tests --------- Signed-off-by: Kamil Tokarski <[email protected]>
1 parent b262c11 commit 75b3581

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed

dali/test/python/auto_aug/test_rand_augment.py

+4-3
Original file line numberDiff line numberDiff line change
@@ -36,9 +36,10 @@
3636
itertools.product(("cpu", "gpu"), (True, False), (True, False), (None, 0), (True, False)))))
3737
def test_run_rand_aug(i, args):
3838
dev, uniformly_resized, use_shape, fill_value, specify_translation_bounds = args
39-
batch_sizes = [1, 8, 7, 64, 13, 64, 128]
40-
ns = [1, 2, 3, 4]
41-
ms = [0, 15, 30]
39+
# Keep batch_sizes ns and ms length co-prime
40+
batch_sizes = [1, 8, 7, 13, 31, 64, 128]
41+
ns = [1, 2, 3]
42+
ms = [0, 12, 15, 30]
4243
batch_size = batch_sizes[i % len(batch_sizes)]
4344
n = ns[i % len(ns)]
4445
m = ms[i % len(ms)]

dali/test/python/reader/test_fits.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ def get_astropy_dtypes(compression):
6060
# The astropy is not actively developed for Python3.6 and the last available
6161
# version does not support some dtypes
6262
vi = sys.version_info
63-
if vi.major < 3 or (vi.major == 3 and vi.minor <= 6):
63+
if vi.major < 3 or (vi.major == 3 and vi.minor <= 7):
6464
excluded |= {np.int8}
6565
# Astropy doesn't support writing those types to compressed image
6666
if compression:

0 commit comments

Comments
 (0)