Skip to content

Commit 450ad0a

Browse files
committed
remove check=True to capture returncode
Signed-off-by: Jack Luar <[email protected]>
1 parent 087eca0 commit 450ad0a

File tree

5 files changed

+6
-5
lines changed

5 files changed

+6
-5
lines changed

tools/AutoTuner/src/autotuner/distributed.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -106,6 +106,7 @@
106106
)
107107
# Global variable for args
108108
args = None
109+
sys.exit(16)
109110

110111

111112
class AutoTunerBase(tune.Trainable):

tools/AutoTuner/test/smoke_test_algo_eval.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@ def make_base(self):
7474
f"make -C {orfs_dir} DESIGN_CONFIG=./designs/{self.platform}/{self.design}/config.mk update_metadata_autotuner",
7575
]
7676
for command in commands:
77-
out = subprocess.run(command, shell=True, check=True)
77+
out = subprocess.run(command, shell=True)
7878
self.assertTrue(out.returncode in accepted_rc)
7979

8080
def test_algo_eval(self):
@@ -84,7 +84,7 @@ def test_algo_eval(self):
8484
self.make_base()
8585
for command in self.commands:
8686
print(command)
87-
out = subprocess.run(command, shell=True, check=True)
87+
out = subprocess.run(command, shell=True)
8888
successful = out.returncode in accepted_rc
8989
self.assertTrue(successful)
9090

tools/AutoTuner/test/smoke_test_sample_iteration.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ def test_sample_iteration(self):
6767
raise unittest.SkipTest("Platform and design have to be defined")
6868
for command in self.commands:
6969
print(command)
70-
out = subprocess.run(command, shell=True, check=True)
70+
out = subprocess.run(command, shell=True)
7171
successful = out.returncode in accepted_rc
7272
self.assertTrue(successful)
7373

tools/AutoTuner/test/smoke_test_sweep.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ def setUp(self):
7979
def test_sweep(self):
8080
if not (self.platform and self.design):
8181
raise unittest.SkipTest("Platform and design have to be defined")
82-
out = subprocess.run(self.command, shell=True, check=True)
82+
out = subprocess.run(self.command, shell=True)
8383
successful = out.returncode in accepted_rc
8484
self.assertTrue(successful)
8585

tools/AutoTuner/test/smoke_test_tune.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ def setUp(self):
6363
def test_tune(self):
6464
if not (self.platform and self.design):
6565
raise unittest.SkipTest("Platform and design have to be defined")
66-
out = subprocess.run(self.command, shell=True, check=True)
66+
out = subprocess.run(self.command, shell=True)
6767
successful = out.returncode in accepted_rc
6868
self.assertTrue(successful)
6969

0 commit comments

Comments
 (0)