Skip to content

Conversation

@ShreyasN707
Copy link

@ShreyasN707 ShreyasN707 commented Dec 3, 2025

Details:

  • Fixed an issue where ReduceMax operations were producing incorrect output shapes (e.g., [1, 960] instead of [1]) on both CPU and GPU plugins.
  • CPU Plugin: Enabled ConvertReduceMaxToPooling transformation in src/plugins/intel_cpu/src/transformations/transformation_pipeline.cpp. This forces ReduceMax to be converted to - MaxPool, which correctly handles shape inference and keep_dims, avoiding the issue where the operation was dropped or mishandled.
  • GPU Plugin: Disabled ConvertReduceToReshape transformation in src/plugins/intel_gpu/src/plugin/transformations_pipeline.cpp. This prevents the plugin from incorrectly optimizing "redundant" reductions into Reshapes, ensuring the ReduceMax operation is preserved.

Tickets:
N/A
@ShreyasN707 Can you please update as much as possbile per following template?

Description of the issue(symptom, root-cause, how it was resolved)

  • A
  • B

The code and line that caused this issue (if it is not changed directly)

  • intel_gpu/src/… (or func name)

Reproduction step and snapshot (if applicable. Do not attach for customer model)

  • $ benchmark_app …
  • ...

Problematic graph

  • If it is related to graph, please describe the graph
  • If you need to show captured graph, attach on the ticket

Checklist

  • Is it a proper fix? (not a workaround)
  • Did you include test case for this fix, if necessary?
  • Did you review existing test that can be extended to cover this scenario? Which test did you review?

Tickets:

  • N/A

@ShreyasN707 ShreyasN707 requested review from a team as code owners December 3, 2025 19:37
@ShreyasN707 ShreyasN707 requested review from mlukasze and removed request for a team December 3, 2025 19:37
@github-actions github-actions bot added category: GPU OpenVINO GPU plugin category: CPU OpenVINO CPU plugin no-match-files labels Dec 3, 2025
@sys-openvino-ci sys-openvino-ci added the ExternalPR External contributor label Dec 3, 2025
@mlukasze
Copy link
Contributor

mlukasze commented Dec 4, 2025

do not modify .gitignore of the project with your local files, please.
to be corrected

@mlukasze mlukasze requested review from isanghao and peterchen-intel and removed request for mlukasze December 4, 2025 05:46
@ShreyasN707
Copy link
Author

do not modify .gitignore of the project with your local files, please. to be corrected

Ohh i byemistakely pushed that too. Will fix it!

@ShreyasN707
Copy link
Author

@isanghao @peterchen-intel please review my PR.

Comment on lines +69 to +71

# Local CI/CD Check Script for OpenVINO

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
# Local CI/CD Check Script for OpenVINO

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: GPU OpenVINO GPU plugin ExternalPR External contributor no-match-files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants