Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

1.16.3 patch release #18491

Merged
merged 7 commits into from
Nov 18, 2023
Merged

1.16.3 patch release #18491

merged 7 commits into from
Nov 18, 2023

Conversation

snnn
Copy link
Member

@snnn snnn commented Nov 17, 2023

No description provided.

@snnn snnn requested review from chilo-ms and jywu-msft November 17, 2023 16:27
justinchuby and others added 3 commits November 17, 2023 08:35
Bump ruff version and remove pylint from the linter list. Fix any new
error detected by ruff.

Ruff covers many of the pylint rules. Since pylint is not enabled in
this repo and runs slow, we remove it from the linters
Bump linter versions and run format.
### Description
Fix a bug in SaveInputOutputNamesToNodeMapping function. The fix was
provided by Scott.

### Motivation and Context
@snnn snnn force-pushed the snnn/rel-1.16.3 branch 2 times, most recently from 4ff14b2 to c6f8abd Compare November 17, 2023 17:01
@snnn snnn linked an issue Nov 17, 2023 that may be closed by this pull request
@snnn snnn requested a review from pranavsharma November 17, 2023 22:16
@jywu-msft
Copy link
Member

the TRT EP change looks good.

why are so many files changed? are the lint changes required for this patch ?

@snnn
Copy link
Member Author

snnn commented Nov 17, 2023

why are so many files changed? are the lint changes required for this patch ?

To get one of the pipelines pass.

@snnn
Copy link
Member Author

snnn commented Nov 17, 2023

The lint changes do not affect the code's behavior. They only added some whitespaces.

@snnn snnn merged commit de0e87e into rel-1.16.3 Nov 18, 2023
97 of 99 checks passed
@snnn snnn deleted the snnn/rel-1.16.3 branch November 18, 2023 01:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Memory leaks when using TRT execution provider
5 participants