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

Force Onnxruntime build pipe to use mi100+ and no cdna #2248

Merged
merged 3 commits into from
Sep 28, 2023

Conversation

TedThemistokleous
Copy link
Collaborator

Avoid the vega cards for the ORT build runs.

Avoid the vega cards for the ORT build runs.
@TedThemistokleous TedThemistokleous added bugfix Fixes a bug found in the code. Continous Integration Pull request updates parts of continous integration pipeline labels Sep 26, 2023
@TedThemistokleous TedThemistokleous self-assigned this Sep 26, 2023
Jenkinsfile Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Sep 26, 2023

Codecov Report

Merging #2248 (eff4ba3) into develop (434a06c) will decrease coverage by 0.04%.
Report is 12 commits behind head on develop.
The diff coverage is 87.42%.

❗ Current head eff4ba3 differs from pull request most recent head a96ff08. Consider uploading reports for the commit a96ff08 to get more accurate results

@@             Coverage Diff             @@
##           develop    #2248      +/-   ##
===========================================
- Coverage    91.53%   91.49%   -0.04%     
===========================================
  Files          429      430       +1     
  Lines        16011    16128     +117     
===========================================
+ Hits         14655    14757     +102     
- Misses        1356     1371      +15     
Files Coverage Δ
src/auto_contiguous.cpp 96.00% <ø> (-4.00%) ⬇️
src/include/migraphx/instruction.hpp 100.00% <ø> (ø)
src/include/migraphx/op/reshape.hpp 100.00% <100.00%> (+0.90%) ⬆️
src/rewrite_pooling.cpp 95.00% <100.00%> (+2.40%) ⬆️
src/simplify_reshapes.cpp 98.73% <100.00%> (+0.01%) ⬆️
src/include/migraphx/op/reshape_lazy.hpp 99.09% <99.09%> (ø)
src/verify_args.cpp 0.00% <0.00%> (ø)
src/include/migraphx/verify.hpp 54.38% <54.54%> (-2.03%) ⬇️

@migraphx-bot
Copy link
Collaborator

Test Batch Rate new
e7edd2
Rate old
a761ff
Diff Compare
torchvision-resnet50 64 2,290.07 2,288.19 0.08%
torchvision-resnet50_fp16 64 5,368.23 5,369.17 -0.02%
torchvision-densenet121 32 1,831.46 1,837.68 -0.34%
torchvision-densenet121_fp16 32 3,399.21 3,398.49 0.02%
torchvision-inceptionv3 32 1,345.36 1,345.95 -0.04%
torchvision-inceptionv3_fp16 32 2,583.29 2,590.63 -0.28%
cadene-inceptionv4 16 681.17 681.28 -0.02%
cadene-resnext64x4 16 590.42 590.97 -0.09%
slim-mobilenet 64 7,223.75 7,214.30 0.13%
slim-nasnetalarge 64 237.16 236.93 0.10%
slim-resnet50v2 64 2,530.40 2,528.87 0.06%
bert-mrpc-onnx 8 825.11 825.36 -0.03%
bert-mrpc-tf 1 389.76 391.02 -0.32%
pytorch-examples-wlang-gru 1 302.82 300.34 0.82%
pytorch-examples-wlang-lstm 1 309.05 303.23 1.92%
torchvision-resnet50_1 1 558.42 557.56 0.15%
torchvision-inceptionv3_1 1 307.31 306.04 0.42%
cadene-dpn92_1 1 350.69 346.79 1.12%
cadene-resnext101_1 1 220.64 220.62 0.01%
slim-vgg16_1 1 224.96 224.91 0.02%
slim-mobilenet_1 1 1,476.95 1,461.53 1.05%
slim-inceptionv4_1 1 226.27 221.53 2.14%
onnx-taau-downsample 1 320.15 323.56 -1.05%
dlrm-criteoterabyte 1 21.70 21.71 -0.05%
dlrm-criteoterabyte_fp16 1 40.66 40.65 0.04%
agentmodel 1 5,799.35 5,760.93 0.67%
unet_fp16 2 55.28 55.26 0.03%

This build is OK for merge ✅

@migraphx-bot
Copy link
Collaborator


    :white_check_mark:bert-mrpc-onnx: PASSED: MIGraphX meets tolerance

    :white_check_mark:bert-mrpc-tf: PASSED: MIGraphX meets tolerance

    :white_check_mark:pytorch-examples-wlang-gru: PASSED: MIGraphX meets tolerance

    :white_check_mark:pytorch-examples-wlang-lstm: PASSED: MIGraphX meets tolerance

    :white_check_mark:torchvision-resnet50_1: PASSED: MIGraphX meets tolerance

🔴torchvision-inceptionv3_1: FAILED: MIGraphX is not within tolerance - check verbose output


🔴cadene-dpn92_1: FAILED: MIGraphX is not within tolerance - check verbose output


    :white_check_mark:cadene-resnext101_1: PASSED: MIGraphX meets tolerance

    :white_check_mark:slim-vgg16_1: PASSED: MIGraphX meets tolerance

    :white_check_mark:slim-mobilenet_1: PASSED: MIGraphX meets tolerance

🔴slim-inceptionv4_1: FAILED: MIGraphX is not within tolerance - check verbose output


    :white_check_mark:dlrm-criteoterabyte: PASSED: MIGraphX meets tolerance

    :white_check_mark:agentmodel: PASSED: MIGraphX meets tolerance

    :white_check_mark:unet: PASSED: MIGraphX meets tolerance

    :white_check_mark:resnet50v1: PASSED: MIGraphX meets tolerance

🔴bert_base_cased_fp16: FAILED: MIGraphX is not within tolerance - check verbose output


🔴bert_large_uncased_fp16: FAILED: MIGraphX is not within tolerance - check verbose output


    :white_check_mark:bert_large: PASSED: MIGraphX meets tolerance

🔴distilgpt2_fp16: FAILED: MIGraphX is not within tolerance - check verbose output

@causten causten self-requested a review September 28, 2023 14:58
@causten causten merged commit 28614ab into develop Sep 28, 2023
14 of 15 checks passed
@causten causten deleted the ci_cdna_to_mi100 branch September 28, 2023 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix Fixes a bug found in the code. Continous Integration Pull request updates parts of continous integration pipeline
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants