Skip to content

Conversation

@waliwali777
Copy link
Contributor

@waliwali777 waliwali777 commented Nov 20, 2024

PR types

Function optimization

PR changes

Others

Description

  1. 之前需要在每个 test 开始时候 export_envcd run/path,现在转移到每个 ci_case_*.dy 的入口处
  2. 之前为了兼容 Paddle CI,每个 case 脚本中包含两个函数列表操作函数,例如:llama_case_list_autorestore_llama_case_list_auto_func,现在将两个函数合在一起,通过传参数执行不同的逻辑, Paddle CI 通过 exec_case 参数执行旧逻辑,PaddleNLP CI 通过prepare_case 参数执行新逻辑
  3. CI 统计 test 执行情况的track_case_status 函数存在运行失败 test 检测不到的 bug,有些 test 运行出错后并不会产生类似func_name_FAIL.log的日志文件,因此,在通过日志名字统计失败 test 的方法存在 bug。现在通过全局变量 在 execute_func_list 中记录每个 case 的执行结果,将每个失败测试名字通过数组记录,在 CI 结束的时候汇总输出

@paddle-bot
Copy link

paddle-bot bot commented Nov 20, 2024

Thanks for your contribution!

@waliwali777 waliwali777 changed the title Fix CI Fix auto parallel CI exit -6 Nov 20, 2024
@codecov
Copy link

codecov bot commented Nov 20, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 52.81%. Comparing base (619c1b9) to head (3643c8a).
Report is 229 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #9460      +/-   ##
===========================================
- Coverage    52.81%   52.81%   -0.01%     
===========================================
  Files          677      677              
  Lines       107941   107943       +2     
===========================================
  Hits         57010    57010              
- Misses       50931    50933       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@wawltor wawltor left a comment

Choose a reason for hiding this comment

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

LGTM

@wawltor wawltor merged commit 5ceb930 into PaddlePaddle:develop Nov 20, 2024
9 of 12 checks passed
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.

2 participants