Skip to content

Conversation

justinchuby
Copy link
Member

Remove the deprecated mappings as scheduled.

Fixes #4554

@justinchuby justinchuby requested a review from a team as a code owner April 21, 2025 04:09
@github-project-automation github-project-automation bot moved this to In progress in PR Tracker Apr 21, 2025
@justinchuby justinchuby requested a review from andife April 21, 2025 04:10
@justinchuby justinchuby added this to the 1.19 milestone Apr 21, 2025
@justinchuby justinchuby enabled auto-merge April 21, 2025 04:11
Copy link

codecov bot commented Apr 21, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 56.31%. Comparing base (d2e2d2c) to head (57d28a6).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6914      +/-   ##
==========================================
- Coverage   56.31%   56.31%   -0.01%     
==========================================
  Files         509      509              
  Lines       32607    32590      -17     
  Branches     3099     3097       -2     
==========================================
- Hits        18364    18354      -10     
+ Misses      13385    13378       -7     
  Partials      858      858              

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

@justinchuby justinchuby force-pushed the justinchu/remove-mapping branch from 6f2018c to 0c9a95a Compare April 23, 2025 05:07
justinchuby and others added 6 commits April 22, 2025 22:08
Signed-off-by: Justin Chu <justinchuby@users.noreply.github.com>
Signed-off-by: Justin Chu <justinchu@microsoft.com>
Signed-off-by: Justin Chu <justinchuby@users.noreply.github.com>
Signed-off-by: Justin Chu <justinchu@microsoft.com>
Signed-off-by: Justin Chu <justinchuby@users.noreply.github.com>
Signed-off-by: Justin Chu <justinchu@microsoft.com>
Signed-off-by: Justin Chu <justinchuby@users.noreply.github.com>
Signed-off-by: Justin Chu <justinchu@microsoft.com>
Signed-off-by: Justin Chu <justinchu@microsoft.com>
Signed-off-by: Justin Chu <justinchu@microsoft.com>
@justinchuby justinchuby force-pushed the justinchu/remove-mapping branch from 0c9a95a to 6e75258 Compare April 23, 2025 05:08
@andife
Copy link
Member

andife commented Apr 23, 2025

@justinchuby: Where was it "scheduled"?

@github-project-automation github-project-automation bot moved this from In progress to Reviewer approved in PR Tracker Apr 23, 2025
@andife andife requested a review from xadupre April 23, 2025 05:28
@justinchuby justinchuby added this pull request to the merge queue Apr 23, 2025
Merged via the queue into main with commit 4ebc37b Apr 23, 2025
39 checks passed
@justinchuby justinchuby deleted the justinchu/remove-mapping branch April 23, 2025 06:25
@github-project-automation github-project-automation bot moved this from Reviewer approved to Done in PR Tracker Apr 23, 2025
@justinchuby
Copy link
Member Author

@andife its documented here #4554, the code was deprecated in 1.15 and should be removed according to the tracking issue.

gramalingam pushed a commit to gramalingam/onnx that referenced this pull request May 19, 2025
Remove the deprecated mappings as scheduled.

Fixes onnx#4554

---------

Signed-off-by: Justin Chu <justinchuby@users.noreply.github.com>
Signed-off-by: Justin Chu <justinchu@microsoft.com>
Co-authored-by: Andreas Fehlner <fehlner@arcor.de>
Signed-off-by: gramalingam <grama@microsoft.com>
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.

[Tracking] Move all variables in mapping.py into _mapping.py
2 participants