Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cinn_coverage #61384

Merged
merged 8 commits into from
Feb 5, 2024
Merged

cinn_coverage #61384

merged 8 commits into from
Feb 5, 2024

Conversation

Galaxy1458
Copy link
Contributor

@Galaxy1458 Galaxy1458 commented Jan 31, 2024

PR types

Others

PR changes

Others

Description

add cinn_coverage script

card-67007

risemeup1
risemeup1 previously approved these changes Feb 4, 2024
risemeup1
risemeup1 previously approved these changes Feb 4, 2024
@@ -4158,6 +4158,9 @@ function main() {
;;
test)
parallel_test
if [ "${WITH_CINN}" == "ON" ] ; then
check_coverage
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里应该有tab间隔

@@ -222,5 +252,6 @@ fi

if [ "$COVERAGE_LINES_ASSERT" = "1" ] || [ "$PYTHON_COVERAGE_LINES_ASSERT" = "1" ]; then
echo "exit 9" > /tmp/paddle_coverage.result
echo "You must one RD(liuhongyu or lanxiang or zhenghuihuang or tianchao zhangliujie)to approval this PR."
exit 9
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tianchao or zhangliujie?更好点儿?

risemeup1
risemeup1 previously approved these changes Feb 5, 2024
@Galaxy1458 Galaxy1458 merged commit 4f1bffe into PaddlePaddle:develop Feb 5, 2024
31 of 32 checks passed
risemeup1 added a commit that referenced this pull request Feb 5, 2024
risemeup1 added a commit that referenced this pull request Feb 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants