Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@justinchuby
Copy link
Member

@justinchuby justinchuby commented Oct 30, 2024

bfloat16 was mistakenly added to GlobalLpPool in opset2. It should have been done in opset 22. This PR fixes the types.

Fixes #6502

Signed-off-by: Justin Chu <[email protected]>
Signed-off-by: Justin Chu <[email protected]>
@justinchuby justinchuby requested a review from a team as a code owner October 30, 2024 01:42
@justinchuby justinchuby added the topic: operator Issues related to ONNX operators label Oct 30, 2024
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

You can commit the suggested changes from lintrunner.

@codecov
Copy link

codecov bot commented Oct 30, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 57.36%. Comparing base (9f4b58a) to head (da8a904).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #6503   +/-   ##
=======================================
  Coverage   57.36%   57.36%           
=======================================
  Files         507      507           
  Lines       31502    31502           
  Branches     3544     3544           
=======================================
  Hits        18072    18072           
  Misses      12581    12581           
  Partials      849      849           

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

@justinchuby justinchuby added the auto update doc Generate md/proto files automatically using the CI pipeline label Oct 30, 2024
@justinchuby justinchuby reopened this Oct 30, 2024
Signed-off-by: Justin Chu <[email protected]>
Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@justinchuby justinchuby removed the auto update doc Generate md/proto files automatically using the CI pipeline label Oct 30, 2024
@justinchuby justinchuby reopened this Oct 30, 2024
@justinchuby justinchuby added this pull request to the merge queue Oct 31, 2024
@justinchuby justinchuby added the release notes Important changes to call out in release notes label Oct 31, 2024
@justinchuby justinchuby added this to the 1.18 milestone Oct 31, 2024
Merged via the queue into main with commit 2dd1870 Oct 31, 2024
39 checks passed
@justinchuby justinchuby deleted the justinchu/global-lp-pool branch October 31, 2024 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release notes Important changes to call out in release notes topic: operator Issues related to ONNX operators

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

bfloat16 was mistakenly added to GlobalLpPool in opset2

2 participants