diff --git a/profiler/msprof_analyze/advisor/advisor_backend/cluster_advice/cluster_pipeline_advice.py b/profiler/msprof_analyze/advisor/advisor_backend/cluster_advice/cluster_pipeline_advice.py index 8db0f6fba4dc1ffc92faf1062f93add29b0dd11b..ce0a867df97150db242450afff016b4ad6033be9 100644 --- a/profiler/msprof_analyze/advisor/advisor_backend/cluster_advice/cluster_pipeline_advice.py +++ b/profiler/msprof_analyze/advisor/advisor_backend/cluster_advice/cluster_pipeline_advice.py @@ -372,7 +372,7 @@ class ClusterPipelineAdvice(ClusterAdviceBase): if rank_id in data_map: rank_prof_dirs[rank_id] = data_map[rank_id] else: - logger.warning('Rank %s not found in %s', str(self.collection_path)) + logger.warning('Rank %s not found in %s', str(rank_id), str(self.collection_path)) return rank_prof_dirs diff --git a/profiler/msprof_analyze/advisor/analyzer/analyzer_controller.py b/profiler/msprof_analyze/advisor/analyzer/analyzer_controller.py index 869c0519cfe8f9716917b7898908aa531a28656e..b9a2357735f7a3f92936b6518c22914f60c35e26 100644 --- a/profiler/msprof_analyze/advisor/analyzer/analyzer_controller.py +++ b/profiler/msprof_analyze/advisor/analyzer/analyzer_controller.py @@ -713,12 +713,11 @@ class AnalyzerController: logger.info( "Skip profiling comparison due to longer processing time due to env 'DISABLE_PROFILING_COMPARISON'") return job_list - + kwargs = {} for index, _kwargs in enumerate(compare_profiling_list): kwargs = copy.deepcopy(self.kwargs) kwargs.update(_kwargs) compare_profiling_list[index] = kwargs - compare_kwargs = { "profiling_path": kwargs.get("profiling_path"), "compare_profiling_list": compare_profiling_list, diff --git a/profiler/msprof_analyze/advisor/analyzer/base_analyzer.py b/profiler/msprof_analyze/advisor/analyzer/base_analyzer.py index ee7835cf539602a7226104f34413f922935906d9..7f558c3871a37c5f13c89b0bd6840de6b073d1f3 100644 --- a/profiler/msprof_analyze/advisor/analyzer/base_analyzer.py +++ b/profiler/msprof_analyze/advisor/analyzer/base_analyzer.py @@ -106,7 +106,9 @@ class BaseAnalyzer(VersionControl, metaclass=ABCMeta): pass def identify_profiling_type(self, profiling_type_list): - profiling_type = None + profiling_type = "" + if not profiling_type_list: + return profiling_type if self.collection_path.endswith(ASCEND_MS): profiling_type = [elem for elem in profiling_type_list if Constant.MINDSPORE in elem][0] elif self.collection_path.endswith(ASCEND_PT):