Add Onnx2NeoML executable #507
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Boris Zimka <boris.zimka@abbyy.com>
Signed-off-by: Boris Zimka <boris.zimka@abbyy.com>
Signed-off-by: Boris Zimka <boris.zimka@abbyy.com>
Signed-off-by: Boris Zimka <boris.zimka@abbyy.com>
Signed-off-by: Boris Zimka <boris.zimka@abbyy.com>
Signed-off-by: Boris Zimka <boris.zimka@abbyy.com>
FedyuninV
reviewed
Dec 20, 2021
NeoML/CMakeLists.txt
Outdated
| @@ -82,7 +82,7 @@ endif() | |||
| if(NeoML_INSTALL) | |||
| set(INSTALLED_TARGETS NeoML NeoMathEngine) | |||
| if(NeoOnnx_BUILD) | |||
| list(APPEND INSTALLED_TARGETS NeoOnnx) | |||
| list(APPEND INSTALLED_TARGETS NeoOnnx ) | |||
There was a problem hiding this comment.
- (мелочь) надо убрать пробел
- (не мелочь) надо добавить эту утилиту в install. Как пример:
if(TARGET Onnx2NeoML)
list(APPEND INSTALLED_TARGETS Onnx2NeoML)
endif()
или NOT IOS NOT ANDROID
FedyuninV
reviewed
Dec 20, 2021
NeoOnnx/src/CMakeLists.txt
Outdated
| @@ -88,6 +88,9 @@ target_sources( ${PROJECT_NAME} PRIVATE | |||
| TensorLayout.h | |||
| TensorUtils.h | |||
| ) | |||
| if(NOT ANDROID AND NOT IOS) | |||
| add_subdirectory(Onnx2NeoML) | |||
There was a problem hiding this comment.
(мелочь) Я бы перенес либо в выше (где-нибудь сразу после add_library) либо в самый низ. Иначе как-то неочевидно, что посередине CMake который описывает сборку NeoOnnx внезапно затесалась доп. утилита.
There was a problem hiding this comment.
Ой, вообще лучше всю утилиту вынести в отдельную поддиректорию на уровень выше! А этот if перенести NeoML/NeoOnnx/CMakeLists.txt
FedyuninV
requested changes
Dec 20, 2021
Signed-off-by: Boris Zimka <boris.zimka@abbyy.com>
|
Перенес утилиту на уровень выше (в NeoOnnx из NeoOnnx/src), добавил в install кроме мобилок |
FedyuninV
approved these changes
Dec 22, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
В PR добавляется простейшая исполняемая обертка вокруг NeoOnnx::LoadFromOnnx.
Сейчас NeoOnnx предоставляет программное api для конвертации Onnx в CDnn (NeoOnnx::LoadFromOnnx).
Т.к. NeoOnnx нельзя собрать вместе с FineObjects, использовать это api в проектах на основе FineObj сложно. Для конвертации onnx-сети приходится создавать новое временное приложение, что не очень удобно.
С другой стороны, в процессе конвертации нет никаких параметров/флагов/настроек, и, например, в tf2onnx можно конвертировать и через cli, и через программный api.
Здесь по аналогии добавлен executable, который выполняет конвертацию.
Протестировано на нейронке с одним Dense слоем.