Skip to content

Conversation

xadupre
Copy link
Contributor

@xadupre xadupre commented Apr 2, 2024

Description

Intermediate results can only be printed right now. With this PR, they can be returned as well.

Motivation and Context

See #6025.

Signed-off-by: Xavier Dupre <xadupre@microsoft.com>
@xadupre xadupre requested a review from a team as a code owner April 2, 2024 15:18
Copy link

codecov bot commented Apr 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 56.97%. Comparing base (fa0b899) to head (08cdf5f).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6066      +/-   ##
==========================================
+ Coverage   56.95%   56.97%   +0.02%     
==========================================
  Files         506      506              
  Lines       30467    30483      +16     
  Branches     4592     4593       +1     
==========================================
+ Hits        17353    17369      +16     
  Misses      12285    12285              
  Partials      829      829              

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

@justinchuby
Copy link
Member

lgtm

xadupre added 3 commits April 4, 2024 10:27
Signed-off-by: Xavier Dupre <xadupre@microsoft.com>
Signed-off-by: Xavier Dupre <xadupre@microsoft.com>
@xadupre xadupre enabled auto-merge April 4, 2024 16:01
@xadupre xadupre added this pull request to the merge queue Apr 4, 2024
Merged via the queue into onnx:main with commit 8f987ee Apr 4, 2024
@xadupre xadupre deleted the intermediate branch April 4, 2024 17:32
gramalingam pushed a commit to gramalingam/onnx that referenced this pull request Apr 12, 2024
### Description

Intermediate results can only be printed right now. With this PR, they
can be returned as well.

### Motivation and Context
See onnx#6025.

---------

Signed-off-by: Xavier Dupre <xadupre@microsoft.com>
Signed-off-by: Ganesan Ramalingam <grama@microsoft.com>
andife pushed a commit to andife/onnx that referenced this pull request Jul 20, 2024
### Description

Intermediate results can only be printed right now. With this PR, they
can be returned as well.

### Motivation and Context
See onnx#6025.

---------

Signed-off-by: Xavier Dupre <xadupre@microsoft.com>
Signed-off-by: Andreas Fehlner <fehlner@arcor.de>
linshokaku pushed a commit to linshokaku/onnx that referenced this pull request Oct 2, 2024
### Description

Intermediate results can only be printed right now. With this PR, they
can be returned as well.

### Motivation and Context
See onnx#6025.

---------

Signed-off-by: Xavier Dupre <xadupre@microsoft.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