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

Some tests for double-packed convs failing #10

Open
maltanar opened this issue Nov 4, 2021 · 0 comments
Open

Some tests for double-packed convs failing #10

maltanar opened this issue Nov 4, 2021 · 0 comments

Comments

@maltanar
Copy link
Collaborator

maltanar commented Nov 4, 2021

While testing the new PR #9 I noticed that the following tests are failing (seemingly not related to the PR):

FAILED tests/fpgadataflow/test_fpgadataflow_conv_dsp.py::test_fpgadataflow_packed_dsp[cppsim-32-odt0-tdt0-idt0-wdt0-3-2-5-64-3] - AssertionError: cppsim failed
FAILED tests/fpgadataflow/test_fpgadataflow_conv_dsp.py::test_fpgadataflow_packed_dsp[cppsim-32-odt0-tdt0-idt0-wdt1-3-2-5-64-3] - AssertionError: cppsim failed
FAILED tests/fpgadataflow/test_fpgadataflow_conv_dsp.py::test_fpgadataflow_packed_dsp[cppsim-224-odt0-tdt0-idt0-wdt0-3-2-5-64-3] - FileNotFoundError: [Errno 2] No such file or direct...
FAILED tests/fpgadataflow/test_fpgadataflow_conv_dsp.py::test_fpgadataflow_packed_dsp[cppsim-224-odt0-tdt0-idt0-wdt1-3-2-5-64-3]

I haven't looked at this at any depth but it seems the combination of kernel size (5) and stride (2) with other params is causing this failure, as the tests for other tests (including the case that corresponds to the ResNet-50 first layer) pass.

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

No branches or pull requests

1 participant