[CodeStyle][Ruff][BUAA][A-[1-4]] Fix ruff Q003 diagnostic for 4 fil…

…es in `paddle/fluid/eager/auto_code_generator/` (PaddlePaddle#67300) * Add lerp CINN. * update * Refine lerp op CINN. * Refine code. * Refine code. * Refine code. * Refine A-1-4 * Apply suggestions from code review --------- Co-authored-by: Nyakku Shigure <[email protected]>
Jeff114514 · Aug 14, 2024 · c6c78cf · c6c78cf
1 parent 702272a
commit c6c78cf
Show file tree

Hide file tree

Showing 4 changed files with 65 additions and 63 deletions.
diff --git a/paddle/fluid/eager/auto_code_generator/generate_file_structures.py b/paddle/fluid/eager/auto_code_generator/generate_file_structures.py
@@ -125,15 +125,15 @@ def GenerateFileStructureForIntermediateDygraph(eager_dir, split_count):
         f.write("add_custom_target(\n")
         f.write("  copy_dygraph_node\n")
         f.write(
-            "  COMMAND ${CMAKE_COMMAND} -E copy_if_different \"${PADDLE_SOURCE_DIR}/paddle/fluid/eager/api/generated/fluid_generated/nodes/nodes.tmp.h\" \"${PADDLE_SOURCE_DIR}/paddle/fluid/eager/api/generated/fluid_generated/nodes/nodes.h\"\n"
+            '  COMMAND ${CMAKE_COMMAND} -E copy_if_different "${PADDLE_SOURCE_DIR}/paddle/fluid/eager/api/generated/fluid_generated/nodes/nodes.tmp.h" "${PADDLE_SOURCE_DIR}/paddle/fluid/eager/api/generated/fluid_generated/nodes/nodes.h"\n'
         )
         for i in range(split_count):
             f.write(
-                "  COMMAND ${CMAKE_COMMAND} -E copy_if_different \"${PADDLE_SOURCE_DIR}/paddle/fluid/eager/api/generated/fluid_generated/nodes/nodes"
+                '  COMMAND ${CMAKE_COMMAND} -E copy_if_different "${PADDLE_SOURCE_DIR}/paddle/fluid/eager/api/generated/fluid_generated/nodes/nodes'
                 + str(i + 1)
-                + ".tmp.cc\" \"${PADDLE_SOURCE_DIR}/paddle/fluid/eager/api/generated/fluid_generated/nodes/nodes"
+                + '.tmp.cc" "${PADDLE_SOURCE_DIR}/paddle/fluid/eager/api/generated/fluid_generated/nodes/nodes'
                 + str(i + 1)
-                + ".cc\"\n"
+                + '.cc"\n'
             )
 
         f.write("  DEPENDS legacy_eager_codegen\n")
@@ -151,24 +151,24 @@ def GenerateFileStructureForIntermediateDygraph(eager_dir, split_count):
         f.write("add_custom_target(\n")
         f.write("  copy_dygraph_forward_functions\n")
         f.write(
-            "  COMMAND ${CMAKE_COMMAND} -E copy_if_different \"${PADDLE_SOURCE_DIR}/paddle/fluid/eager/api/generated/fluid_generated/dygraph_forward_api.tmp.h\" \"${PADDLE_SOURCE_DIR}/paddle/fluid/eager/api/generated/fluid_generated/dygraph_forward_api.h\"\n"
+            '  COMMAND ${CMAKE_COMMAND} -E copy_if_different "${PADDLE_SOURCE_DIR}/paddle/fluid/eager/api/generated/fluid_generated/dygraph_forward_api.tmp.h" "${PADDLE_SOURCE_DIR}/paddle/fluid/eager/api/generated/fluid_generated/dygraph_forward_api.h"\n'
         )
         for i in range(split_count):
             f.write(
-                "  COMMAND ${CMAKE_COMMAND} -E copy_if_different \"${PADDLE_SOURCE_DIR}/paddle/fluid/eager/api/generated/fluid_generated/forwards/dygraph_forward_functions"
+                '  COMMAND ${CMAKE_COMMAND} -E copy_if_different "${PADDLE_SOURCE_DIR}/paddle/fluid/eager/api/generated/fluid_generated/forwards/dygraph_forward_functions'
                 + str(i + 1)
-                + ".tmp.cc\" \"${PADDLE_SOURCE_DIR}/paddle/fluid/eager/api/generated/fluid_generated/forwards/dygraph_forward_functions"
+                + '.tmp.cc" "${PADDLE_SOURCE_DIR}/paddle/fluid/eager/api/generated/fluid_generated/forwards/dygraph_forward_functions'
                 + str(i + 1)
-                + ".cc\"\n"
+                + '.cc"\n'
             )
         f.write(
-            "  COMMAND ${CMAKE_COMMAND} -E copy_if_different \"${PADDLE_SOURCE_DIR}/paddle/fluid/eager/api/generated/fluid_generated/forwards/dygraph_forward_functions_args_info.tmp.cc\" \"${PADDLE_SOURCE_DIR}/paddle/fluid/eager/api/generated/fluid_generated/forwards/dygraph_forward_functions_args_info.cc\"\n"
+            '  COMMAND ${CMAKE_COMMAND} -E copy_if_different "${PADDLE_SOURCE_DIR}/paddle/fluid/eager/api/generated/fluid_generated/forwards/dygraph_forward_functions_args_info.tmp.cc" "${PADDLE_SOURCE_DIR}/paddle/fluid/eager/api/generated/fluid_generated/forwards/dygraph_forward_functions_args_info.cc"\n'
         )
         f.write(
-            "  COMMAND ${CMAKE_COMMAND} -E copy_if_different \"${PADDLE_SOURCE_DIR}/paddle/fluid/eager/api/generated/fluid_generated/forwards/dygraph_forward_functions_args_type_info.tmp.cc\" \"${PADDLE_SOURCE_DIR}/paddle/fluid/eager/api/generated/fluid_generated/forwards/dygraph_forward_functions_args_type_info.cc\"\n"
+            '  COMMAND ${CMAKE_COMMAND} -E copy_if_different "${PADDLE_SOURCE_DIR}/paddle/fluid/eager/api/generated/fluid_generated/forwards/dygraph_forward_functions_args_type_info.tmp.cc" "${PADDLE_SOURCE_DIR}/paddle/fluid/eager/api/generated/fluid_generated/forwards/dygraph_forward_functions_args_type_info.cc"\n'
         )
         f.write(
-            "  COMMAND ${CMAKE_COMMAND} -E copy_if_different \"${PADDLE_SOURCE_DIR}/paddle/fluid/eager/api/generated/fluid_generated/forwards/dygraph_forward_functions_returns_info.tmp.cc\" \"${PADDLE_SOURCE_DIR}/paddle/fluid/eager/api/generated/fluid_generated/forwards/dygraph_forward_functions_returns_info.cc\"\n"
+            '  COMMAND ${CMAKE_COMMAND} -E copy_if_different "${PADDLE_SOURCE_DIR}/paddle/fluid/eager/api/generated/fluid_generated/forwards/dygraph_forward_functions_returns_info.tmp.cc" "${PADDLE_SOURCE_DIR}/paddle/fluid/eager/api/generated/fluid_generated/forwards/dygraph_forward_functions_returns_info.cc"\n'
         )
         f.write("  DEPENDS legacy_eager_codegen\n")
         f.write("  VERBATIM)\n")

diff --git a/paddle/fluid/eager/auto_code_generator/generator/codegen_utils.py b/paddle/fluid/eager/auto_code_generator/generator/codegen_utils.py
@@ -549,13 +549,13 @@ def CollectOriginalForwardInfo(self):
 
         assert (
             'op' in forward_api_contents.keys()
-        ), "Unable to find \"op\" in forward_api_contents keys"
+        ), 'Unable to find "op" in forward_api_contents keys'
         assert (
             'args' in forward_api_contents.keys()
-        ), "Unable to find \"args\" in forward_api_contents keys"
+        ), 'Unable to find "args" in forward_api_contents keys'
         assert (
             'output' in forward_api_contents.keys()
-        ), "Unable to find \"output\" in forward_api_contents keys"
+        ), 'Unable to find "output" in forward_api_contents keys'
 
         # Collect Original Forward Inputs/Outputs and then perform validation checks
         (