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

[Snippets] Added PassConfig and PositionedPasses for PassPipeline #21382

Conversation

a-sidorova
Copy link
Contributor

@a-sidorova a-sidorova commented Nov 30, 2023

Details:

  • Added support of PassConfig to lowered PassPipeline to allow developers to enable/disable transformations
  • Added support of PositionedPasses to lowered PassPipeline to allow developers to insert backend specific transformations in needed parts of pipeline without the common pipeline splitting
  • Change std::string m_pass_name to DiscreteTypeInfo m_pass_type_info in Snippets Manager for data flow transformations

Tickets:

  • 113666, 126214

@a-sidorova a-sidorova requested review from a team as code owners November 30, 2023 06:33
@github-actions github-actions bot added the category: CPU OpenVINO CPU plugin label Nov 30, 2023
@a-sidorova a-sidorova added this to the 2024.0 milestone Nov 30, 2023
@a-sidorova a-sidorova force-pushed the feature/snippets/lowered_pipeline_config_position branch 2 times, most recently from 0aa34c5 to 3825c8c Compare December 1, 2023 08:06
@a-sidorova a-sidorova force-pushed the feature/snippets/lowered_pipeline_config_position branch from 3825c8c to 1753e9e Compare December 6, 2023 12:13
@a-sidorova a-sidorova force-pushed the feature/snippets/lowered_pipeline_config_position branch 2 times, most recently from 36fe905 to b92958b Compare December 13, 2023 05:59
@a-sidorova
Copy link
Contributor Author

@v-Golubev @IvanNovoselov Guys, thanks a lot for the review!

@dmitry-gorokhov could you please take a look as well? Thanks

@a-sidorova a-sidorova force-pushed the feature/snippets/lowered_pipeline_config_position branch 2 times, most recently from 36a089a to c7e5aa6 Compare December 26, 2023 05:52
@a-sidorova a-sidorova force-pushed the feature/snippets/lowered_pipeline_config_position branch from c7e5aa6 to 0cd3b18 Compare December 27, 2023 07:39
@dmitry-gorokhov dmitry-gorokhov merged commit 92cb11e into openvinotoolkit:master Jan 3, 2024
88 checks passed
github-merge-queue bot pushed a commit that referenced this pull request Feb 27, 2024
### Details:
- *Introduced loop handlers which define how to handle first/last loop
iteration*
- *Changed loop markup logic: loops with `increment > work_amount` are
not created anymore: in this case `increment` is set equal to
`work_amount`*

### Tickets:
 - *CVS-119851*

### Prerequisites:
- #19335
- #21382
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.

4 participants