diff --git a/scripts/errors/coll/ParamMatchingType.py b/scripts/errors/coll/ParamMatchingType.py index 9014e55f5485acf239bdc93e51ed5df727173400..54274e808b07fea39c2a72027bfa97142b457bb4 100644 --- a/scripts/errors/coll/ParamMatchingType.py +++ b/scripts/errors/coll/ParamMatchingType.py @@ -33,7 +33,6 @@ class InvalidComErrorColl(ErrorGenerator): def generate(self, generate_level, real_world_score_table): - for func_to_use in self.func_one_type_arg: checked_types = set() @@ -65,7 +64,8 @@ class InvalidComErrorColl(ErrorGenerator): ## local missmatch - tm.set_description("ParamMatching-Type-" + func_to_use, "Wrong datatype matching: %s vs %s" % (type_1,type_2)) + tm.set_description("ParamMatching-Type-" + func_to_use, + "Wrong datatype matching: %s vs %s" % (type_1, type_2)) for call in tm.get_instruction("MPICALL", return_list=True): call.set_rank_executing(0) @@ -88,13 +88,12 @@ class InvalidComErrorColl(ErrorGenerator): if generate_level == REAL_WORLD_TEST_LEVEL: if (not is_combination_important(real_world_score_table, func_to_use, datatype=type_1.lower(), - communicator=comm)) : + communicator=comm)): # not relevant in real world continue tm = get_collective_template(func_to_use) - type_var_1, buf_name_1, type_var_2, buf_name_2 = get_type_buffers(tm, type_1, type_2,1,1) - + type_var_1, buf_name_1, type_var_2, buf_name_2 = get_type_buffers(tm, type_1, type_2, 1, 1) tm.set_description("ParamMatching-Type-" + func_to_use, "Wrong datatype matching: %s vs %s" % (type_1, type_2)) @@ -102,7 +101,7 @@ class InvalidComErrorColl(ErrorGenerator): for call in tm.get_instruction("MPICALL", return_list=True): call.set_rank_executing(0) call.set_arg("datatype", type_var_1) - call.set_arg("count",1) + call.set_arg("count", 1) call.set_has_error() if call.has_arg("recvbuf"): call.set_arg("recvbuf", buf_name_1) @@ -113,7 +112,7 @@ class InvalidComErrorColl(ErrorGenerator): c = CorrectMPICallFactory.get(func_to_use) c.set_rank_executing('not0') c.set_arg("datatype", type_var_2) - call.set_arg("count", 1) + c.set_arg("count", 1) c.set_has_error() if c.has_arg("recvbuf"): c.set_arg("recvbuf", buf_name_2) @@ -128,9 +127,11 @@ class InvalidComErrorColl(ErrorGenerator): # missmatch with matching sizes tm = get_collective_template(func_to_use) - type_var_1, buf_name_1, type_var_2, buf_name_2 = get_type_buffers(tm, type_1, type_2, get_bytes_size_for_type( - type_2), get_bytes_size_for_type( - type_1)) + type_var_1, buf_name_1, type_var_2, buf_name_2 = get_type_buffers(tm, type_1, type_2, + get_bytes_size_for_type( + type_2), + get_bytes_size_for_type( + type_1)) tm.set_description("ParamMatching-Type-" + func_to_use, "Wrong datatype matching: %s vs %s" % (type_1, type_2)) @@ -138,8 +139,7 @@ class InvalidComErrorColl(ErrorGenerator): for call in tm.get_instruction("MPICALL", return_list=True): call.set_rank_executing(0) call.set_arg("datatype", type_var_1) - call.set_arg("count", get_bytes_size_for_type( - type_2)) + call.set_arg("count", get_bytes_size_for_type(type_2)) call.set_has_error() if call.has_arg("recvbuf"): call.set_arg("recvbuf", buf_name_1) @@ -150,8 +150,7 @@ class InvalidComErrorColl(ErrorGenerator): c = CorrectMPICallFactory.get(func_to_use) c.set_rank_executing('not0') c.set_arg("datatype", type_var_2) - call.set_arg("count", get_bytes_size_for_type( - type_1)) + call.set_arg("count", get_bytes_size_for_type(type_1)) c.set_has_error() if c.has_arg("recvbuf"): c.set_arg("recvbuf", buf_name_2) @@ -162,5 +161,3 @@ class InvalidComErrorColl(ErrorGenerator): tm.insert_instruction(c, after_instruction=call) yield tm - -