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.

Sign up for GitHub

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

Jump to bottom

Merged ONNX editor and ONNX importer to the one folder #6672

Merged
merged 1 commit into from
Jul 16, 2021
Merged

Merged ONNX editor and ONNX importer to the one folder #6672

merged 1 commit into from
Jul 16, 2021

Conversation

ilyachur
Copy link
Contributor

Details:

  • Merged ONNX importer and ONNX editor

Tickets:

  • NA

@ilyachur ilyachur added this to the 2022.1 milestone Jul 16, 2021
@openvino-pushbot openvino-pushbot added the category: Core OpenVINO Core (aka ngraph) label Jul 16, 2021
@ilyachur
Copy link
Contributor Author

@tomdol @tsocha @mbencer @postrational please have a look

tomdol
tomdol approved these changes Jul 16, 2021
View reviewed changes
Copy link
Contributor

@tomdol tomdol left a comment

Choose a reason for hiding this comment

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

Looks good overall but I suggest to wait for #6615 first.

@ilyachur
Copy link
Contributor Author

Looks good overall but I suggest to wait for #6615 first.

@tomdol As I see #6615 IE already have some merge conflicts. Maybe if we merge this PR before in the PR #6615 we will be able to use one ONNX importer inside the ONNX frontend.

@azhogov azhogov merged commit 6c22af8 into openvinotoolkit:master Jul 16, 2021
@ilyachur ilyachur deleted the merged_editor_importer branch July 16, 2021 12:14
rnugmanx pushed a commit to rnugmanx/openvino that referenced this pull request Aug 26, 2021
@ilyachur
Merged ONNX editor and ONNX importer to the one folder (openvinotoolk…
b1b5f93 
…it#6672)
andrei-cv pushed a commit to andrei-cv/openvino that referenced this pull request Aug 30, 2021
@ilyachur @andrei-cv
Merged ONNX editor and ONNX importer to the one folder (openvinotoolk…
5fcb554 
…it#6672)
akuporos pushed a commit to akuporos/openvino that referenced this pull request Sep 29, 2021
@ilyachur @akuporos
Merged ONNX editor and ONNX importer to the one folder (openvinotoolk…
07fd336 
…it#6672)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tsocha tsocha tsocha approved these changes

@tomdol tomdol tomdol approved these changes

@azhogov azhogov azhogov approved these changes

@mbencer mbencer mbencer approved these changes

@ilya-lavrenov ilya-lavrenov ilya-lavrenov approved these changes

@postrational postrational Awaiting requested review from postrational

Assignees
No one assigned
Labels
category: Core OpenVINO Core (aka ngraph) category: ONNX FE OpenVINO ONNX FrontEnd
Projects
None yet
Milestone
2022.1
Development

Successfully merging this pull request may close these issues.
7 participants
@ilyachur @ilya-lavrenov @mbencer @azhogov @tomdol @tsocha @openvino-pushbot