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

add mkl multi-thread test cases in PR-CI-INFERENCE #34946

Merged
merged 3 commits into from
Aug 17, 2021

Conversation

OliverLPH
Copy link
Contributor

@OliverLPH OliverLPH commented Aug 16, 2021

PR types

Others

PR changes

Others

Describe

  1. 新增 ernie text cls 模型(无trt匹配的子图),gpu测试,mkldnn多线程测试(旧版本有bug)
  2. test_suite.h 增加按数据类型copy from cpu
  3. det_mv3_db模型新增mkldnn多线程测试

@paddle-bot-old
Copy link

Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@OliverLPH OliverLPH requested a review from jiweibo August 17, 2021 02:43
std::map<std::string, paddle::test::Record> infer_output_data,
truth_output_data;
// prepare groudtruth config
paddle_infer::Config config, config_no_ir;
Copy link
Contributor

Choose a reason for hiding this comment

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

以后可以把每个测试case的config.Summary()打印出来,develop和今后的2.2支持

Copy link
Contributor Author

Choose a reason for hiding this comment

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

收到,下个PR就把 config summary打印出来

Copy link
Contributor

@jiweibo jiweibo left a comment

Choose a reason for hiding this comment

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

LGTM

@OliverLPH OliverLPH merged commit 9d4f00b into PaddlePaddle:develop Aug 17, 2021
@OliverLPH OliverLPH deleted the add_mkl_multi_thread branch August 17, 2021 06:35
std::vector<int64_t> segment_data(input_num, 0);
srand((unsigned)time(NULL));
for (int x = 0; x < input_data.size(); x++) {
input_data[x] = cRandom<int>(1, 100);
Copy link
Contributor

Choose a reason for hiding this comment

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

This cRandom method seems to always return 1 https://ideone.com/Al9qCY

Copy link
Contributor

Choose a reason for hiding this comment

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

I got a build error on CI so I looked at this
image

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