Skip to content

Conversation

@snnn
Copy link
Contributor

@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
Contributor 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
Contributor 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
@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

6 participants