Skip to content

yolov3 accuracy#7235

Merged
jywu-msft merged 2 commits intomicrosoft:masterfrom
intel:sahar/yolov3_acc
Apr 11, 2021
Merged

yolov3 accuracy#7235
jywu-msft merged 2 commits intomicrosoft:masterfrom
intel:sahar/yolov3_acc

Conversation

@sfatimar
Copy link
Copy Markdown
Contributor

@sfatimar sfatimar commented Apr 5, 2021

This one line change is to ensure myriad configuration is set to achieve maximum yolov3 accuracy.

Motivation and Context

  • We have found that the reason is that in this case preprocessing is not a part of the model and is performed separately (while the converted IRs usually have the preprocessing inside the model) which in turn affects MYX heuristic algorithms. To avoid the degradation we added an additional option, which should be used in this case.
  • If it fixes an open issue, please link to the issue here.

@sfatimar sfatimar requested a review from a team as a code owner April 5, 2021 09:08
@jywu-msft
Copy link
Copy Markdown
Member

/azp run Windows GPU CI Pipeline, Windows GPU TensorRT CI Pipeline, centos7_cpu, centos7_cpu (linux_centos_ci Debug), centos7_cpu (linux_centos_ci Release), orttraining-linux-ci-pipeline, orttraining-linux-gpu-ci-pipeline, orttraining-amd-gpu-ci-pipeline, Linux Nuphar CI Pipeline, orttraining-distributed

@jywu-msft
Copy link
Copy Markdown
Member

/azp run Linux CPU CI Pipeline, Linux CPU Minimal Build E2E CI Pipeline, Linux CPU x64 NoContribops CI Pipeline, Linux GPU TensorRT CI Pipeline, Linux OpenVINO CI Pipeline, MacOS CI Pipeline, MacOS NoContribops CI Pipeline, Windows CPU CI Pipeline, orttraining-ortmodule, orttraining-ortmodule-distributed

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 6 pipeline(s).

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 10 pipeline(s).

@jywu-msft
Copy link
Copy Markdown
Member

/azp run Linux GPU CI Pipeline

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

jywu-msft
jywu-msft previously approved these changes Apr 8, 2021
@jywu-msft
Copy link
Copy Markdown
Member

looks like there is a merge conflict. can you please resolve?

@sfatimar
Copy link
Copy Markdown
Contributor Author

sfatimar commented Apr 9, 2021

looks like there is a merge conflict. can you please resolve?

Resolved

@jywu-msft
Copy link
Copy Markdown
Member

/azp run Windows GPU CI Pipeline, Windows GPU TensorRT CI Pipeline, centos7_cpu, centos7_cpu (linux_centos_ci Debug), centos7_cpu (linux_centos_ci Release), orttraining-linux-ci-pipeline, orttraining-linux-gpu-ci-pipeline, orttraining-amd-gpu-ci-pipeline, Linux Nuphar CI Pipeline, orttraining-distributed

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 6 pipeline(s).

@jywu-msft
Copy link
Copy Markdown
Member

/azp run Linux CPU CI Pipeline, Linux CPU Minimal Build E2E CI Pipeline, Linux CPU x64 NoContribops CI Pipeline, Linux GPU TensorRT CI Pipeline, Linux OpenVINO CI Pipeline, MacOS CI Pipeline, MacOS NoContribops CI Pipeline, Windows CPU CI Pipeline, orttraining-ortmodule, orttraining-ortmodule-distributed, Linux GPU CI Pipeline

@azure-pipelines
Copy link
Copy Markdown

You have several pipelines (over 10) configured to build pull requests in this repository. Specify which pipelines you would like to run by using /azp run [pipelines] command. You can specify multiple pipelines using a comma separated list.

@jywu-msft
Copy link
Copy Markdown
Member

/azp run Linux CPU CI Pipeline, Linux CPU Minimal Build E2E CI Pipeline, Linux CPU x64 NoContribops CI Pipeline, Linux GPU TensorRT CI Pipeline, Linux OpenVINO CI Pipeline, MacOS CI Pipeline, MacOS NoContribops CI Pipeline, Windows CPU CI Pipeline, orttraining-ortmodule, orttraining-ortmodule-distributed

@jywu-msft
Copy link
Copy Markdown
Member

/azp run Linux GPU CI Pipeline

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 10 pipeline(s).

@jywu-msft jywu-msft merged commit 21c282e into microsoft:master Apr 11, 2021
@sfatimar sfatimar deleted the sahar/yolov3_acc branch October 23, 2024 16:11
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