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

Script for model converter #524

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

vineelpratap
Copy link
Contributor

Summary: As per title

Differential Revision: D27574935

Summary: As per title

Differential Revision: D27574935

fbshipit-source-id: fac828f102fec6a2c631762dc0064e0efe6998b2
@facebook-github-bot facebook-github-bot added CLA Signed Do not delete this pull request or issue due to inactivity. fb-exported labels Apr 5, 2021
@facebook-github-bot
Copy link

This pull request was exported from Phabricator. Differential Revision: D27574935

@luiyen
Copy link

luiyen commented Aug 8, 2021

Build fail with the colab script.

CMake Error at flashlight/app/asr/tools/CMakeLists.txt:7 (add_executable):
The target name
"/content/flashlight/flashlight/app/asr/tools/serialization/Compat.cpp" is
reserved or not valid for certain CMake features, such as generator
expressions, and may result in undefined behavior.

@@ -31,3 +31,8 @@ build_tool(
${CMAKE_CURRENT_LIST_DIR}/benchmark/ArchBenchmark.cpp
fl_asr_arch_benchmark
)
build_tool(
${CMAKE_CURRENT_LIST_DIR}/serialization/ModelConverter.cpp
${CMAKE_CURRENT_LIST_DIR}/serialization/Compat.cpp
Copy link

Choose a reason for hiding this comment

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

build_tool function accepts 2 arguments which lead to a build error.

@luiyen
Copy link

luiyen commented Aug 8, 2021

Hi @vineelpratap. Is it possible to export 0.2 AM binary to dump file by using this converter?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed Do not delete this pull request or issue due to inactivity. fb-exported
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants