Skip to content

Fix Pybind memory leak#23105

Merged
yuslepukhin merged 1 commit intomainfrom
yuslepukhin/py_sequence_leak
Dec 16, 2024
Merged

Fix Pybind memory leak#23105
yuslepukhin merged 1 commit intomainfrom
yuslepukhin/py_sequence_leak

Conversation

@yuslepukhin
Copy link
Member

Description

Array GETITEM returns new reference which is a leak

Motivation and Context

Address #22271

@yuslepukhin yuslepukhin changed the title Fix the Python memory leak Fix Pybind memory leak Dec 13, 2024
@yuslepukhin yuslepukhin merged commit ae97068 into main Dec 16, 2024
@yuslepukhin yuslepukhin deleted the yuslepukhin/py_sequence_leak branch December 16, 2024 18:38
guschmue pushed a commit that referenced this pull request Dec 16, 2024
### Description
<!-- Describe your changes. -->
Array GETITEM returns new reference which is a leak


### Motivation and Context
Address  #22271
guschmue pushed a commit that referenced this pull request Dec 20, 2024
### Description
<!-- Describe your changes. -->
Array GETITEM returns new reference which is a leak


### Motivation and Context
Address  #22271
tarekziade pushed a commit to tarekziade/onnxruntime that referenced this pull request Jan 10, 2025
### Description
<!-- Describe your changes. -->
Array GETITEM returns new reference which is a leak


### Motivation and Context
Address  microsoft#22271
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.

3 participants