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

make building tests optional (on by default) #1887

Merged
merged 1 commit into from
Jun 27, 2023
Merged

Conversation

apwojcik
Copy link
Collaborator

The PR from the series to enable MIGraphX on Windows.

@codecov
Copy link

codecov bot commented Jun 27, 2023

Codecov Report

Merging #1887 (808a4f9) into develop (84a8f45) will not change coverage.
The diff coverage is n/a.

❗ Current head 808a4f9 differs from pull request most recent head 6f82413. Consider uploading reports for the commit 6f82413 to get more accurate results

@@           Coverage Diff            @@
##           develop    #1887   +/-   ##
========================================
  Coverage    91.44%   91.44%           
========================================
  Files          419      419           
  Lines        15516    15516           
========================================
  Hits         14188    14188           
  Misses        1328     1328           

@migraphx-bot
Copy link
Collaborator

Test Batch Rate new
6f8241
Rate old
84a8f4
Diff Compare
torchvision-resnet50 64 895.49 896.26 -0.09%
torchvision-resnet50_fp16 64 5,313.32 5,309.71 0.07%
torchvision-densenet121 32 1,126.07 1,126.51 -0.04%
torchvision-densenet121_fp16 32 3,278.40 3,276.51 0.06%
torchvision-inceptionv3 32 594.11 593.21 0.15%
torchvision-inceptionv3_fp16 32 2,517.05 2,514.58 0.10%
cadene-inceptionv4 16 329.04 329.95 -0.28%
cadene-resnext64x4 16 393.76 396.36 -0.66%
slim-mobilenet 64 7,137.72 7,134.15 0.05%
slim-nasnetalarge 64 160.07 159.96 0.07%
slim-resnet50v2 64 1,090.43 1,089.90 0.05%
bert-mrpc-onnx 8 719.28 719.15 0.02%
bert-mrpc-tf 1 372.42 371.96 0.12%
pytorch-examples-wlang-gru 1 308.85 301.75 2.35%
pytorch-examples-wlang-lstm 1 309.86 311.74 -0.60%
torchvision-resnet50_1 1 91.58 91.98 -0.44%
torchvision-inceptionv3_1 1 129.15 130.00 -0.65%
cadene-dpn92_1 1 336.46 335.33 0.34%
cadene-resnext101_1 1 236.87 236.90 -0.01%
slim-vgg16_1 1 53.39 53.39 -0.00%
slim-mobilenet_1 1 1,507.00 1,517.98 -0.72%
slim-inceptionv4_1 1 101.54 101.30 0.24%
onnx-taau-downsample 1 316.33 317.23 -0.29%
dlrm-criteoterabyte 1 21.45 21.45 0.00%
dlrm-criteoterabyte_fp16 1 39.94 39.97 -0.06%
agentmodel 1 5,789.05 5,933.63 -2.44%
unet_fp16 2 52.79 52.82 -0.05%

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

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

🔴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

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

    :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

@apwojcik apwojcik removed the request for review from varunsh-xilinx June 27, 2023 20:54
@causten causten merged commit 632d69f into develop Jun 27, 2023
12 checks passed
@causten causten deleted the build_testing branch June 27, 2023 21:20
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.

5 participants