Skip to content

Conversation

daquexian
Copy link
Member

Description

#5628 missed some getNextUnique() and causes name conflict.

Signed-off-by: daquexian <daquexian566@gmail.com>
@daquexian daquexian requested a review from a team as a code owner March 3, 2024 07:50
Copy link

codecov bot commented Mar 3, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 56.81%. Comparing base (238f2b9) to head (415a42f).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5992   +/-   ##
=======================================
  Coverage   56.81%   56.81%           
=======================================
  Files         506      506           
  Lines       30357    30357           
  Branches     4589     4589           
=======================================
  Hits        17246    17246           
  Misses      12283    12283           
  Partials      828      828           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@justinchuby justinchuby changed the title fix unique_name Fix unique_name Mar 4, 2024
@justinchuby
Copy link
Member

@cjvolzka @gramalingam should we include this in 1.16?

@justinchuby justinchuby added this to the 1.16 milestone Mar 4, 2024
@justinchuby justinchuby requested a review from gramalingam March 4, 2024 17:56
@gramalingam
Copy link
Contributor

@cjvolzka @gramalingam should we include this in 1.16?

Yes, that would be useful if possible, thanks!

@gramalingam gramalingam added this pull request to the merge queue Mar 4, 2024
Merged via the queue into onnx:main with commit d7e2b81 Mar 4, 2024
cjvolzka pushed a commit that referenced this pull request Mar 4, 2024
### Description
* Cherry-pick #5992 into `rel-1.16.0` branch
* #5628 missed some getNextUnique() and causes name conflict.

Signed-off-by: daquexian <daquexian566@gmail.com>
cjvolzka added a commit that referenced this pull request Mar 4, 2024
### Description
* Cherry-pick #5992 into `rel-1.16.0` branch
* #5628 missed some getNextUnique() and causes name conflict.

Signed-off-by: daquexian <daquexian566@gmail.com>
Co-authored-by: daquexian <daquexian566@gmail.com>
isdanni pushed a commit to isdanni/onnx that referenced this pull request Mar 18, 2024
### Description
onnx#5628 missed some getNextUnique() and causes name conflict.

Signed-off-by: daquexian <daquexian566@gmail.com>
Signed-off-by: isdanni <leedanni@gmail.com>
linshokaku pushed a commit to linshokaku/onnx that referenced this pull request Oct 2, 2024
### Description
onnx#5628 missed some getNextUnique() and causes name conflict.

Signed-off-by: daquexian <daquexian566@gmail.com>
Signed-off-by: Linsho Kaku <linsho@preferred.jp>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants