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

[ONNX] Change translator registration approach #25166

Merged
merged 25 commits into from
Jul 7, 2024
Merged

[ONNX] Change translator registration approach #25166

merged 25 commits into from
Jul 7, 2024

Conversation

gkrivor
Copy link
Contributor

@gkrivor gkrivor commented Jun 21, 2024 •
edited
Loading

Details:

  • Moved to VersionRange usage
  • Moved a translators registration near a translator code
  • Simplified file structure
  • Implemented support of *.lib build
  • Implemented a supported op customization by removing op-files from the build

Tickets:

  • N/A

@github-actions github-actions bot added category: ONNX FE OpenVINO ONNX FrontEnd category: build OpenVINO cmake script / infra labels Jun 21, 2024
@gkrivor gkrivor marked this pull request as ready for review June 25, 2024 06:17
@gkrivor gkrivor requested a review from a team as a code owner June 25, 2024 06:17
@gkrivor
Copy link
Contributor Author

gkrivor commented Jun 25, 2024

build_jenkins

gkrivor added 22 commits June 26, 2024 16:57
@gkrivor
Added centralized storage of available ONNX translators
f41e6ba
@gkrivor
Manually updated ops
70ab0e9
@gkrivor
Updated most of existing translators
b54e1d2
@gkrivor
Removed pragma once in a cpp
1b6ebf8
@gkrivor
Moved domain declaration to operator_set.hpp
fb3bde6
@gkrivor
Made translators of com.microsoft-domain as a self-registered
e9f63dd
@gkrivor
Made translators of org.openvinotoolkit-domain as a self-registered
fc5752b
@gkrivor
Redone multiple registration approach
8f73d26
@gkrivor
Fixed ns naming
dec1de9
@gkrivor
Renamed namespaces op->ai_onnx, set_X -> opset_X
3a6113a
@gkrivor
Fixed renaming artifacts
009830c
@gkrivor
Fixed codestyle
d61191d
@gkrivor
Fixed missing translators registration
71ee787
@gkrivor
Fixed codestyle
9c336b2
@gkrivor
Fixed some translator's registration calls
a028eb8
@gkrivor
Changed an instantiation approach of supported ops map
65808f5
@gkrivor
Updated a registration process
62555bd
@gkrivor
Experimental code generation for static linkage
28cdda9
@gkrivor
Replaced strings in translators by a shared constants
6cc7ece
@gkrivor
Code style fixes
089282d
@gkrivor
Small code improvement
b74ac6f
@gkrivor
Organized source by using macro
ec70082
@gkrivor gkrivor requested a review from mvafin July 1, 2024 11:00
@gkrivor gkrivor added this pull request to the merge queue Jul 7, 2024
Merged via the queue into openvinotoolkit:master with commit 44e4e5d Jul 7, 2024
120 checks passed
@gkrivor gkrivor deleted the onnx_opreg branch July 7, 2024 06:19
spran180 pushed a commit to spran180/openvino that referenced this pull request Jul 27, 2024
@gkrivor @spran180
[ONNX] Change translator registration approach (openvinotoolkit#25166)
7331a49 
### Details:
 - Moved to VersionRange usage
 - Moved a translators registration near a translator code
 - Simplified file structure
 - Implemented support of *.lib build
- Implemented a supported op customization by removing op-files from the
build

### Tickets:
 - N/A
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mvafin mvafin mvafin approved these changes

Assignees
No one assigned
Labels
category: build OpenVINO cmake script / infra category: ONNX FE OpenVINO ONNX FrontEnd
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gkrivor @mvafin