Skip to content

Conversation

xadupre
Copy link
Contributor

@xadupre xadupre commented Sep 9, 2024

Description

Fixes a bug in ReferenceEvaluator.

Signed-off-by: xadupre <xadupre@microsoft.com>
@xadupre xadupre requested review from a team as code owners September 9, 2024 14:11
Copy link

codecov bot commented Sep 9, 2024

Codecov Report

Attention: Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Project coverage is 57.23%. Comparing base (3823e44) to head (83951ba).
Report is 354 commits behind head on main.

Files with missing lines Patch % Lines
onnx/reference/custom_element_types.py 0.00% 1 Missing ⚠️
onnx/reference/ops/op_non_max_suppression.py 0.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6354      +/-   ##
==========================================
- Coverage   57.23%   57.23%   -0.01%     
==========================================
  Files         507      507              
  Lines       31407    31406       -1     
  Branches     4690     4690              
==========================================
- Hits        17977    17976       -1     
  Misses      12574    12574              
  Partials      856      856              

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

xadupre and others added 2 commits September 9, 2024 17:32
Signed-off-by: xadupre <xadupre@microsoft.com>
@xadupre xadupre added this pull request to the merge queue Sep 9, 2024
Merged via the queue into onnx:main with commit 976a142 Sep 9, 2024
38 checks passed
@xadupre xadupre deleted the nonmax branch September 9, 2024 18:05
linshokaku pushed a commit to linshokaku/onnx that referenced this pull request Oct 2, 2024
### Description
Fixes a bug in ReferenceEvaluator.

---------

Signed-off-by: xadupre <xadupre@microsoft.com>
Co-authored-by: Andreas Fehlner <fehlner@arcor.de>
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
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants