未验证 提交 87ff40b7 编写于 作者: Y Yulv-git 提交者: GitHub

[CodeStyle] Add CI for self.assertTrue(np.allclose(...)) (#45126)

* Add CI for assert-allclose.

* Update CI script.

* Update check_approval.

* Specify the destination path for the git diff.

* Add test samples.

* Add CI for assert-allclose with \n.

* Update test samples.

* Update ALL_ADDED_LINES_IN_TARGET_PATH.

* update GitHub username to userid, test=document_fix

* add rfc as a specification, test=document_fix

* try to integrate single and multiple rows together, test=document_fix

* remove duplicate dirs, test=document_fix

* add anchor `#background`, test=document_fix

* remove original scripts, test=document_fix

* remove test files, test=document_fix
Co-authored-by: NSigureMo <sigure.qaq@gmail.com>
上级 c17e6af8
...@@ -259,6 +259,12 @@ if [ "${EMPTY_GRAD_OP_REGISTERED}" != "" ] && [ "${GIT_PT_ID}" != "" ]; then ...@@ -259,6 +259,12 @@ if [ "${EMPTY_GRAD_OP_REGISTERED}" != "" ] && [ "${GIT_PT_ID}" != "" ]; then
check_approval 1 43953930 46782768 22165420 22361972 check_approval 1 43953930 46782768 22165420 22361972
fi fi
INVALID_UNITTEST_ASSERT_CHECK=`echo "$ALL_ADDED_LINES" | grep -zoE '\+\s+self\.assert(True|Equal)\((\s*\+\s*)?(np|numpy)\.(allclose|array_equal)[^+]*' || true`
if [ "${INVALID_UNITTEST_ASSERT_CHECK}" != "" ] && [ "${GIT_PR_ID}" != "" ]; then
echo_line="It is recommended to use 'np.testing.assert_allclose' and 'np.testing.array_equal' instead of 'self.assertTrue(np.allclose(...))' and 'self.assertTrue(np.array_equal(...))'.\nPlease modify the code below. If anything is unclear, please read the specification [ https://github.com/PaddlePaddle/community/blob/master/rfcs/CodeStyle/20220805_code_style_improvement_for_unittest.md#background ]. If it is a mismatch, please request qili93 (Recommend) or luotao1 review and approve.\nThe code that do not meet the specification are as follows:\n${INVALID_UNITTEST_ASSERT_CHECK}\n"
check_approval 1 16605440 6836917
fi
HAS_MODIFIED_PHI_FILES=`git diff --name-only upstream/$BRANCH | grep "paddle/phi/" || true` HAS_MODIFIED_PHI_FILES=`git diff --name-only upstream/$BRANCH | grep "paddle/phi/" || true`
PHI_INCLUDE_FLUID_FILES="" PHI_INCLUDE_FLUID_FILES=""
for CHANGE_FILE in ${HAS_MODIFIED_PHI_FILES}; do for CHANGE_FILE in ${HAS_MODIFIED_PHI_FILES}; do
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册