test=document_fix coverage failed

Author: swgu98

.github/workflows/_Coverage.yml

@@ -218,7 +218,7 @@ jobs:
     needs: build
     if: needs.build.outputs.can-skip != 'true'
     runs-on:
-      group: BD_BJ_v100
+      group: BD_BJ-V100
     steps:
       - name: Download paddle.tar.gz and update test branch
         run: |
@@ -315,15 +315,13 @@ jobs:
           fi
           '
 
-      - name: Check coverage data
+      - name: Generate coverage information
         run: |
           docker exec -t ${{ env.container_name }} /bin/bash -c '
           source ~/.bashrc
           unset GREP_OPTIONS
           source ${{ github.workspace }}/../../../proxy
           source ${ci_scripts}/utils.sh; check_coverage
-          COVERAGE_EXIT_CODE=$?
-          echo "COVERAGE_EXIT_CODE=${COVERAGE_EXIT_CODE}" >> ${{ github.env }}
           '
 
       - name: Upload coverage report
@@ -333,12 +331,20 @@ jobs:
 
       - name: Determine whether the coverage rate reaches 90%
         run: |
+          docker exec -t ${{ env.container_name }} /bin/bash -c '
+          source ~/.bashrc
+          unset GREP_OPTIONS
+          source ${{ github.workspace }}/../../../proxy
+          source ${ci_scripts}/utils.sh
+          bash $ci_scripts/coverage_judge.sh
+          COVERAGE_EXIT_CODE=$?
           if [ "${{ env.COVERAGE_EXIT_CODE }}" -ne 0 ]; then
-            echo "Coverage check failed, unit test has passed, please do not rerun, check whether the code is fully executed and recommit."
+            echo "Coverage check failed, unit tests have all passed, please do not rerun, check whether the code has been fully executed and recommit."
             exit ${{ env.COVERAGE_EXIT_CODE }}
           else
             echo "Coverage passed"
           fi
+          '
 
       - name: Terminate and delete the container
         if: always()

ci/coverage_info.sh

@@ -218,29 +218,3 @@ python ${PADDLE_ROOT}/ci/coverage_diff.py python-coverage-diff.info python-git-d
 mv -f python-coverage-diff.tmp python-coverage-diff.info
 
 cp python-coverage-diff.info coverage_files
-
-# assert coverage lines
-
-echo "Assert Diff Coverage"
-
-python ${PADDLE_ROOT}/tools/coverage/coverage_lines.py coverage-diff.info 0.9 || COVERAGE_LINES_ASSERT=1
-
-echo "Assert Python Diff Coverage"
-
-if [ ${WITH_XPU:-OFF} == "ON" ]; then
-    echo "XPU has no python coverage!"
-else
-    if [[ "${NO_PYTHON_COVERAGE_DATA}" != "1" ]];then
-        python ${PADDLE_ROOT}/tools/coverage/coverage_lines.py python-coverage-diff.info 0.9 || PYTHON_COVERAGE_LINES_ASSERT=1
-    fi
-fi
-
-if [ "$COVERAGE_LINES_ASSERT" = "1" ] || [ "$PYTHON_COVERAGE_LINES_ASSERT" = "1" ]; then
-    echo "exit 9" > /tmp/paddle_coverage.result
-    python ${PADDLE_ROOT}/tools/get_pr_title.py skip_coverage_check && NOT_CHECK_COVERAGE_PR=1
-    if [[ "${NOT_CHECK_COVERAGE_PR}" = "1" ]];then
-        echo "Skip coverage check in the PR-CI-Coverage pipeline."
-        exit 0
-    fi
-    exit 9
-fi

ci/coverage_judge.sh

@@ -0,0 +1,40 @@
+# Copyright (c) 2025 PaddlePaddle Authors. All Rights Reserved.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+# assert coverage lines
+
+echo "Assert Diff Coverage"
+cd ${PADDLE_ROOT}/build
+
+python ${PADDLE_ROOT}/tools/coverage/coverage_lines.py coverage-diff.info 0.9 || COVERAGE_LINES_ASSERT=1
+
+echo "Assert Python Diff Coverage"
+
+if [ ${WITH_XPU:-OFF} == "ON" ]; then
+    echo "XPU has no python coverage!"
+else
+    if [[ "${NO_PYTHON_COVERAGE_DATA}" != "1" ]];then
+        python ${PADDLE_ROOT}/tools/coverage/coverage_lines.py python-coverage-diff.info 0.9 || PYTHON_COVERAGE_LINES_ASSERT=1
+    fi
+fi
+
+if [ "$COVERAGE_LINES_ASSERT" = "1" ] || [ "$PYTHON_COVERAGE_LINES_ASSERT" = "1" ]; then
+    echo "exit 9" > /tmp/paddle_coverage.result
+    python ${PADDLE_ROOT}/tools/get_pr_title.py skip_coverage_check && NOT_CHECK_COVERAGE_PR=1
+    if [[ "${NOT_CHECK_COVERAGE_PR}" = "1" ]];then
+        echo "Skip coverage check in the PR-CI-Coverage pipeline."
+        exit 0
+    fi
+    exit 9
+fi

paddle/fluid/framework/attribute.cc

@@ -318,4 +318,32 @@ void CanonicalizeScalarAttrs(const proto::OpProto& op_proto,
     }
   }
 }
+
+void CanonicalizeScalarAttrs_copy(const proto::OpProto& op_proto,
+                                  AttributeMap* attrs) {
+  PADDLE_ENFORCE_NOT_NULL(
+      attrs, common::errors::InvalidArgument("attrs can not be nullptr"));
+  for (auto& attr : op_proto.attrs()) {
+    proto::AttrType attr_type = attr.type();
+    const std::string& attr_name = attr.name();
+    auto it = attrs->find(attr_name);
+    if (it == attrs->end()) {
+      continue;
+    }
+    proto::AttrType actual_attr_type = AttrTypeID(it->second);
+    if (actual_attr_type == attr_type) {
+      continue;
+    }
+    if (actual_attr_type == proto::AttrType::VAR ||
+        actual_attr_type == proto::AttrType::VARS) {
+      continue;  // VAR& VARS are not proper attribute
+    }
+    if (attr_type == proto::AttrType::SCALAR) {
+      it->second = Attribute(MakeScalarFromAttribute(it->second));
+    } else if (attr_type == proto::AttrType::SCALARS) {
+      it->second = Attribute(MakeScalarsFromAttribute(it->second));
+    }
+  }
+}
+
 }  // namespace paddle::framework

paddle/fluid/framework/attribute.h

@@ -357,5 +357,7 @@ TEST_API std::vector<paddle::experimental::Scalar> MakeScalarsFromAttribute(
     const Attribute& v);
 void CanonicalizeScalarAttrs(const proto::OpProto& op_proto,
                              AttributeMap* attrs);
+void CanonicalizeScalarAttrs_copy(const proto::OpProto& op_proto,
+                                  AttributeMap* attrs);
 }  // namespace framework
 }  // namespace paddle