Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CPU][DEBUG_CAPS] Do not include all the nodes to executable ones #27599

Conversation

EgorDuplensky
Copy link
Contributor

@EgorDuplensky EgorDuplensky commented Nov 18, 2024

There is a case, when a node is a static one and it has 0 dim in one of its input shapes.
In this case we must not execute this node, but performing
additional isExecutable() check in runtime for all the static nodes
would be to expensive

@EgorDuplensky EgorDuplensky requested review from a team as code owners November 18, 2024 15:26
@github-actions github-actions bot added the category: CPU OpenVINO CPU plugin label Nov 18, 2024
@EgorDuplensky
Copy link
Contributor Author

@maxnick Could you please take a look

@EgorDuplensky EgorDuplensky force-pushed the do_not_execute_all_nodes_with_debug_caps branch from 8e66553 to 7a943df Compare November 19, 2024 13:14
There is a case, when a node is a static one and it has 0 dim in
one of the input shapes.
In this case we must not execute this node, but performing
addition isExecutable() check in runtime for all the static nodes
would be to expensive
@EgorDuplensky EgorDuplensky force-pushed the do_not_execute_all_nodes_with_debug_caps branch from 7a943df to 25a48cc Compare November 21, 2024 13:23
@maxnick maxnick added this pull request to the merge queue Nov 22, 2024
Merged via the queue into openvinotoolkit:master with commit 229ed28 Nov 22, 2024
165 checks passed
@maxnick maxnick deleted the do_not_execute_all_nodes_with_debug_caps branch November 22, 2024 16:53
NishantPrabhuFujitsu pushed a commit to NishantPrabhuFujitsu/openvino that referenced this pull request Nov 26, 2024
…envinotoolkit#27599)

There is a case, when a node is a static one and it has 0 dim in one of
its input shapes.
In this case we must not execute this node, but performing
additional isExecutable() check in runtime for all the static nodes
would be to expensive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: CPU OpenVINO CPU plugin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants