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

[Squeeze] Introduce Squeeze and Unsqueeze hardware operators #11

Merged
merged 7 commits into from
Feb 6, 2025

Conversation

iksnagreb
Copy link

@iksnagreb iksnagreb commented Jan 20, 2025

This includes HWCustomOp and HLSBackend specializations of the operators aiming for full ONNX compliance. Adds infrastructure for converting the standard ONNX version of the operators to the FINN dialect, which mostly means transplanting the node into the FINN domain and setting a few type and shape attributes. Adds unit tests in Python, C++ and RTL simulation as well as a simple integration test starting from PyTorch model export.

See Xilinx#1153 for details and discussion

  • Add AXI-Stream passthrough provided by Thomas
  • Generalize to more/all special cases of reshape

iksnagreb and others added 3 commits August 7, 2024 15:21
This includes HWCustomOp and HLSBackend specializations of the operators
aiming for full ONNX compliance. Adds infrastructure for converting the
standard ONNX version of the operators to the FINN dialect, which mostly
means transplanting the node into the FINN domain and setting a few type
and shape attributes. Adds unit tests in Python, C++ and RTL simulation
as well as a simple integration test starting from PyTorch model export.
@iksnagreb iksnagreb self-assigned this Jan 28, 2025
@iksnagreb iksnagreb requested a review from fpjentzsch February 6, 2025 09:09
@iksnagreb iksnagreb marked this pull request as ready for review February 6, 2025 09:10
@fpjentzsch fpjentzsch merged commit 957b5c1 into dev Feb 6, 2025
1 check passed
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.

3 participants