-
Notifications
You must be signed in to change notification settings - Fork 54
Add Subgraph Support for Folding Config #212
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
Conversation
|
I have tested this on the MLO end-to-end flow, and it works well. However, I made a change on the FINN side to ensure the naming of subgraph nodes is correct (Xilinx/finn@bb4ae3d). Without clear differentiation between the naming of subgraph nodes and nodes at the upper level, there’s a risk that attributes might get overwritten. Despite this, I’ve marked the PR as |
|
Converted back to draft, @jsmonson and I had a quick chat offline and we would like to incorporate the subgraph hierarchy in the json. That would require changes to ApplyConfig. Will convert back to |
…ure/config_extract_for_subgraphs
…tmachinelearning/qonnx into feature/config_extract_for_subgraphs
…ode subgraph_hier to avoid aliasing
|
This functionality is being moved into FINN. Xilinx/finn#1486 |
No description provided.