Skip to content

Commit 2ac6aed

Browse files
committed
address review
1 parent 69f0708 commit 2ac6aed

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

RLTest/utils.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,7 @@ def split_by_semicolon(s):
9292

9393
def args_list_to_dict(args_list):
9494
def dicty(args):
95-
return dict((seq.split(' ')[0].upper(), seq) for seq in args)
95+
return {seq.split(' ')[0].upper(): seq for seq in args}
9696
return list(map(lambda args: dicty(args), args_list))
9797

9898
def join_lists(lists):
@@ -125,11 +125,11 @@ def fix_modulesArgs(modules, modulesArgs, defaultArgs=None, haveSeqs=True):
125125
# No semicolons found - treat as space-separated key-value pairs
126126
words = parts[0].split()
127127
if len(words) % 2 != 0:
128-
print(Colors.Bred('Error in args: odd number of words in key-value pairs: \'%s\'. '
129-
'Use semicolons to separate args with multiple values (e.g. \'KEY1 V1; KEY2 V2 V3\').' % modulesArgs))
128+
print(Colors.Bred(f"Error in args: odd number of words in key-value pairs: '{modulesArgs}'. "
129+
f"Use semicolons to separate args with multiple values (e.g. 'KEY1 V1; KEY2 V2 V3')."))
130130
sys.exit(1)
131131
if len(words) > 2:
132-
parts = [words[i] + ' ' + words[i + 1] for i in range(0, len(words), 2)]
132+
parts = [f"{words[i]} {words[i + 1]}" for i in range(0, len(words), 2)]
133133
modulesArgs = [parts]
134134
elif type(modulesArgs) == list:
135135
args = []

0 commit comments

Comments
 (0)