Skip to content

Conversation

sgbihu
Copy link
Contributor

@sgbihu sgbihu commented Sep 8, 2025

Details:

  • Place Slice Operator at the graph
  • Let EliminateSlice pass to optimize out the Slice if needs.

Tickets:

@sgbihu sgbihu requested a review from a team as a code owner September 8, 2025 08:36
@sgbihu sgbihu requested review from Copilot and gkrivor and removed request for a team September 8, 2025 08:36
@github-actions github-actions bot added the category: ONNX FE OpenVINO ONNX FrontEnd label Sep 8, 2025
Copilot

This comment was marked as outdated.

@sgbihu sgbihu requested a review from Copilot September 8, 2025 08:42
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR replaces the direct pass-through behavior of the ONNX Identity operator with a Slice operation that copies data from input to output. The change addresses a specific issue where data may not be properly copied when Identity connects directly from input to result nodes.

  • Replaces direct output forwarding with a Slice operation using full range parameters
  • Adds necessary includes for Slice and Constant operations
  • Includes comments explaining the optimization strategy via EliminateSlice pass

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: ONNX FE OpenVINO ONNX FrontEnd
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant