Skip to content

Conversation

xadupre
Copy link
Contributor

@xadupre xadupre commented Jun 20, 2024

Description

Fixes #6186, #6185, #6180, #6187.

It also improves the error messages to let users know where something is going wrong.

Signed-off-by: Xavier Dupre <xadupre@microsoft.com>
@xadupre xadupre requested review from a team as code owners June 20, 2024 11:08
Copy link

codecov bot commented Jun 20, 2024

Codecov Report

Attention: Patch coverage is 96.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 57.28%. Comparing base (83194ed) to head (a3eb6c2).
Report is 54 commits behind head on main.

Files Patch % Lines
onnx/test/shape_inference_test.py 96.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6195      +/-   ##
==========================================
+ Coverage   56.95%   57.28%   +0.32%     
==========================================
  Files         506      506              
  Lines       30467    31064     +597     
  Branches     4592     4619      +27     
==========================================
+ Hits        17353    17795     +442     
- Misses      12285    12446     +161     
+ Partials      829      823       -6     

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

Signed-off-by: Xavier Dupre <xadupre@microsoft.com>
xadupre added 2 commits June 20, 2024 13:29
Signed-off-by: Xavier Dupre <xadupre@microsoft.com>
Signed-off-by: Xavier Dupre <xadupre@microsoft.com>
xadupre added 2 commits June 20, 2024 14:48
Signed-off-by: Xavier Dupre <xadupre@microsoft.com>
Signed-off-by: Xavier Dupre <xadupre@microsoft.com>
Signed-off-by: Xavier Dupre <xadupre@microsoft.com>
Copy link
Member

@justinchuby justinchuby left a comment

Choose a reason for hiding this comment

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

Thanks! I would recommend including the actual (invalid) values in all updated error messages applicable. I have found those to be the more helpful information in practice.

@xadupre xadupre enabled auto-merge June 21, 2024 13:46
@xadupre xadupre self-assigned this Jun 21, 2024
Signed-off-by: Xavier Dupre <xadupre@microsoft.com>
@xadupre xadupre added this pull request to the merge queue Jul 3, 2024
Merged via the queue into onnx:main with commit 4bf83b0 Jul 3, 2024
@xadupre xadupre deleted the bugsh branch July 3, 2024 17:06
andife pushed a commit to andife/onnx that referenced this pull request Jul 20, 2024
### Description
Fixes onnx#6186, onnx#6185, onnx#6180, onnx#6187.

It also improves the error messages to let users know where something is
going wrong.

---------

Signed-off-by: Xavier Dupre <xadupre@microsoft.com>
Signed-off-by: Andreas Fehlner <fehlner@arcor.de>
@cjvolzka cjvolzka added this to the 1.16.2 milestone Jul 25, 2024
liqunfu added a commit to microsoft/onnxruntime that referenced this pull request Jul 25, 2024
Signed-off-by: Liqun Fu <liqfu@microsoft.com>
@cjvolzka cjvolzka removed this from the 1.16.2 milestone Jul 25, 2024
snnn pushed a commit to microsoft/onnxruntime that referenced this pull request Jul 27, 2024
…:convPoolShapeInference (#21507)

### Description
onnx 1.16.2 is not available before ort 1.19.0 code freeze. Thus pick
the needed change as patch
linshokaku pushed a commit to linshokaku/onnx that referenced this pull request Oct 2, 2024
### Description
Fixes onnx#6186, onnx#6185, onnx#6180, onnx#6187.

It also improves the error messages to let users know where something is
going wrong.

---------

Signed-off-by: Xavier Dupre <xadupre@microsoft.com>
Signed-off-by: Linsho Kaku <linsho@preferred.jp>
rohan11235813 pushed a commit to quadric-io/onnxruntime that referenced this pull request Aug 19, 2025
…:convPoolShapeInference (#21507)

### Description
onnx 1.16.2 is not available before ort 1.19.0 code freeze. Thus pick
the needed change as patch
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.

onnx.checker crashes on STFT
5 participants