ICODE-ADC/lammps
4
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix logic bug with allowing quick_max to run without requiring quick to be True

Browse Source
This commit is contained in:
Trung Nguyen
2024-10-14 10:45:53 -05:00
parent 3e7642bc8c
commit b0f2484441
1 changed files with 7 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
tools/regression-tests/run_tests.py
View File

@ -1123,7 +1123,7 @@ if __name__ == "__main__":
help="Determine which test inputs have commands changed between a branch and the head")
parser.add_argument("--quick-branch", dest="quick_branch", default=quick_branch,
help="Branch to which compare the current head to for changed styles")
parser.add_argument("--quick-max", dest="quick_max", default=50,
parser.add_argument("--quick-max", dest="quick_max", default=0,
help="Maximum number of inputs to randomly select")
parser.add_argument("--quick-reference", dest="quick_reference", default=quick_reference,
help="Reference YAML file with progress data from full regression test run")
@ -1370,13 +1370,12 @@ if __name__ == "__main__":
example_inputs.append(input)
num_inputscripts += 1
# allow to select randomly some input scripts at this point if quick is set to be True
if quick:
if len(example_inputs) > quick_max:
example_inputs = random.sample(example_inputs, quick_max)
msg = "\nTesting " + str(quick_max) + " randomly selected inputs"
print(msg)
logger.info(msg)
# allow to select randomly some input scripts at this point if quick_max is set
if quick_max > 0 and len(example_inputs) > quick_max:
example_inputs = random.sample(example_inputs, quick_max)
msg = "\nTesting " + str(quick_max) + " randomly selected inputs"
print(msg)
logger.info(msg)
example_subfolders = folder_list
msg = f"\nThere are {num_inputscripts} input scripts listed in {list_input}."