Skip to content

Conversation

ndrego
Copy link
Contributor

@ndrego ndrego commented Nov 14, 2023

Description

Changes setup of subgraph input feeds to utilize input ordering while also ensuring that the name matches. Removes /duplicated from the input name, if present. This can happen when the custom op is created before quantization.

Motivation and Context

@ndrego ndrego force-pushed the nigel-handle-duplicated-input-names branch from 516dbaf to 8ffea84 Compare November 14, 2023 00:24
@ndrego ndrego force-pushed the nigel-handle-duplicated-input-names branch from 8ffea84 to d9b3a8e Compare November 14, 2023 00:52
Copy link

@Adhyyan1252 Adhyyan1252 left a comment

Choose a reason for hiding this comment

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

Actually now that I read this again, the diff looks fine to me. I was a bit confused yesterday but I think thats because I thought we were iterating over something else.

@ndrego ndrego merged commit 7845828 into main Nov 14, 2023
@ndrego ndrego deleted the nigel-handle-duplicated-input-names branch November 14, 2023 21:19
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