Skip to content

Conversation

@hariharans29
Copy link
Member

@hariharans29 hariharans29 commented Dec 11, 2025

Reverts #26104

It seems like this fix although is correct and necessary, it needs some patches in some other places (like release pipelines and onnxruntime inference examples). We will try and address the actual issue the PR was addressing in a subsequent release. I will re-open all the GH issues that the fix PR closed out so that the actual issue is still tracked.

Will reopen:

#24003
#26186
#23642
#25279
#25242

@hariharans29
Copy link
Member Author

The optional Lint failure can be ignored - IIRC the original PR had the same issue.

@hariharans29 hariharans29 merged commit b3780dc into main Dec 15, 2025
90 of 91 checks passed
@hariharans29 hariharans29 deleted the revert-26104-main branch December 15, 2025 16:43
Sumit2318 pushed a commit that referenced this pull request Jan 6, 2026
)

Reverts #26104

It seems like this fix although is correct and necessary, it needs some
patches in some other places (like release pipelines and onnxruntime
inference examples). We will try and address the actual issue the PR was
addressing in a subsequent release. I will re-open all the GH issues
that the fix PR closed out so that the actual issue is still tracked.

Will reopen:

#24003
#26186
#23642
#25279
#25242
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