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

better logging [2] build_data_flow.py #1162

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from
Open

Conversation

azizb-xlnx
Copy link
Collaborator

Targeting build_data_flow.py as a first example of #1161
Only info prints are replaced in this PR, so only verbosity = 1 produces any exciting prints.

testing with a sandbox customer flow.

Default, verbosity = 0

we will see these stray prints until they are taken care of unfortunatly.

Running FINN flow: Streamline me
/group/cdc_ir/members/azizb/tools/miniconda3/envs/finn_sm/lib/python3.10/site-packages/qonnx/transformation/gemm_to_matmul.py:57: UserWarning: The GemmToMatMul transformation only offers explicit support for version 9 of the Gemm node, but the ONNX version of the supplied model is 14. Thus the transformation may fail or return incomplete results.
  warnings.warn(

No prints in the log file.

-v. verbosity = 1

Console:

[INFO  ] 15-08-2024 18:23:04 - Applying FINN steps on onnx/model.onnx
Running FINN flow: Streamline me
[INFO  ] 15-08-2024 18:23:04 - Building dataflow accelerator from onnx/model.onnx
[INFO  ] 15-08-2024 18:23:04 - Running step: step_tidy_up [1/3]
[INFO  ] 15-08-2024 18:23:04 - Running step: step_qonnx_to_finn [2/3]
/group/cdc_ir/members/azizb/tools/miniconda3/envs/finn_sm/lib/python3.10/site-packages/qonnx/transformation/gemm_to_matmul.py:57: UserWarning: The GemmToMatMul transformation only offers explicit support for version 9 of the Gemm node, but the ONNX version of the supplied model is 14. Thus the transformation may fail or return incomplete results.
  warnings.warn(
[INFO  ] 15-08-2024 18:23:04 - Running step: step_tidy_up [3/3]
[INFO  ] 15-08-2024 18:23:04 - Completed successfully

finn.log

[INFO  ] 15-08-2024 18:23:04 - main - Applying FINN steps on onnx/model.onnx [in /group/cdc_ir/members/azizb/workspace/sandbox_finn/finn-flow/sandbox-flow/gen_ip.py:41]
[INFO  ] 15-08-2024 18:23:04 - finn.builder.build_dataflow - Building dataflow accelerator from onnx/model.onnx [in /group/cdc_ir/members/azizb/workspace/brainwave/finn/src/finn/builder/build_dataflow.py:95]
[INFO  ] 15-08-2024 18:23:04 - finn.builder.build_dataflow - Running step: step_tidy_up [1/3] [in /group/cdc_ir/members/azizb/workspace/brainwave/finn/src/finn/builder/build_dataflow.py:115]
[INFO  ] 15-08-2024 18:23:04 - finn.builder.build_dataflow - Running step: step_qonnx_to_finn [2/3] [in /group/cdc_ir/members/azizb/workspace/brainwave/finn/src/finn/builder/build_dataflow.py:115]
[INFO  ] 15-08-2024 18:23:04 - finn.builder.build_dataflow - Running step: step_tidy_up [3/3] [in /group/cdc_ir/members/azizb/workspace/brainwave/finn/src/finn/builder/build_dataflow.py:115]
[INFO  ] 15-08-2024 18:23:04 - finn.builder.build_dataflow - Completed successfully [in /group/cdc_ir/members/azizb/workspace/brainwave/finn/src/finn/builder/build_dataflow.py:142]

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.

1 participant