Skip to content

[Transformation][CPU]Strided_slice branches concatination fused to single gather#36811

Open
chenhu-wang wants to merge 5 commits into
openvinotoolkit:masterfrom
chenhu-wang:chenhu/strided_slice_reshape_concat_fusion_transformation
Open

[Transformation][CPU]Strided_slice branches concatination fused to single gather#36811
chenhu-wang wants to merge 5 commits into
openvinotoolkit:masterfrom
chenhu-wang:chenhu/strided_slice_reshape_concat_fusion_transformation

Conversation

@chenhu-wang

@chenhu-wang chenhu-wang commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Details:

  • There is single source consumed by multiple strided_slice nodes, the sliced output then are concatenated. This pattern could be replaced by single gather node with index collected from slice. This could dramatically reduce node number and improve compile time/tput and reduce used memory

Tickets:

AI Assistance:

  • AI assistance used: no / yes
  • If yes, summarize how AI was used and what human validation was performed (build/tests/manual checks).

@chenhu-wang
chenhu-wang requested review from a team as code owners July 10, 2026 03:43
@github-actions github-actions Bot added category: CPU OpenVINO CPU plugin category: transformations OpenVINO Runtime library - Transformations labels Jul 10, 2026
@chenhu-wang
chenhu-wang force-pushed the chenhu/strided_slice_reshape_concat_fusion_transformation branch from 0577feb to 779496f Compare July 10, 2026 06:31
@chenhu-wang
chenhu-wang requested a review from maxnick July 10, 2026 08:18
@maxnick maxnick added this to the 2026.4 milestone Jul 13, 2026
@chenhu-wang
chenhu-wang force-pushed the chenhu/strided_slice_reshape_concat_fusion_transformation branch from 1e44d5c to 19aa7cd Compare July 14, 2026 08:51
@maxnick
maxnick requested a review from CuriousPanCake July 14, 2026 08:52

@CuriousPanCake CuriousPanCake left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please, review transformations-related comments

@CuriousPanCake
CuriousPanCake dismissed their stale review July 14, 2026 11:04

Dismissing my own review request

@CuriousPanCake

Copy link
Copy Markdown
Contributor

Please, updated the title and the description

@chenhu-wang
chenhu-wang force-pushed the chenhu/strided_slice_reshape_concat_fusion_transformation branch from 19aa7cd to 1705e0b Compare July 15, 2026 04:47
@chenhu-wang chenhu-wang changed the title [CPU]strided_slice_reshape_concat_fusion_to_single_gather [Transformation][CPU]Strided_slice branches concatanation fused to single gather Jul 15, 2026
@chenhu-wang chenhu-wang changed the title [Transformation][CPU]Strided_slice branches concatanation fused to single gather [Transformation][CPU]Strided_slice branches concatination fused to single gather Jul 15, 2026
@chenhu-wang

Copy link
Copy Markdown
Contributor Author

Please, updated the title and the description

updated. thanks

@maxnick
maxnick requested a review from CuriousPanCake July 15, 2026 07:56
@maxnick maxnick modified the milestones: 2026.4, 2026.3 Jul 15, 2026
@maxnick
maxnick added this pull request to the merge queue Jul 17, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 17, 2026
@github-actions

Copy link
Copy Markdown
Contributor

CI Doctor — Merge Queue failure on this PR

Pipeline: Windows (VS 2022, Python 3.11, Release)
Failure: JS API E2E Electron cleanup fails on Windows - Permission denied removing locked electron.exe/dll
Automatic restart: ✅ Re-run of failed jobs requested (reason: Flaky Windows test cleanup - electron.exe file locks prevent rm -rf; a plain restart is likely to succeed)

Possible remedy

  1. Re-run is already triggered — this failure is unrelated to your PR changes (CPU plugin transformation).
  2. If it fails again, the JS API team should replace rm -rf with fs.rm({ recursive: true, force: true, maxRetries: 5 }) in the mocha 'after all' cleanup hook.
  3. Alternatively, add an explicit taskkill /F /IM electron.exe step before cleanup on Windows runners.
  4. A dedicated tracking issue should be filed — this pattern has recurred 11 times since 2026-07-09.

What happened

The JS API E2E test's cleanup hook ran rm -rf demo-electron-app-project after a successful Electron integration test, but Windows file locks on electron.exe, d3dcompiler_47.dll, libEGL.dll, and libGLESv2.dll caused Permission denied errors. This is a known flaky infrastructure issue in the Windows CI that has been seen 3 times in the last 12 hours alone (runs 29564867992, 29568028549, 29573015584).

Generated by CI Failure Doctor — Merge Queue · 146.1 AIC · ⌖ 6.23 AIC · ⊞ 20K ·

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: CPU OpenVINO CPU plugin category: transformations OpenVINO Runtime library - Transformations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants