Skip to content

Conversation

AsyaPronina
Copy link
Contributor

Details:

  • Enable three-model pipeline by default

Tickets:

@AsyaPronina AsyaPronina requested review from a team as code owners July 31, 2025 11:06
@github-actions github-actions bot added the category: NPU OpenVINO NPU plugin label Jul 31, 2025
@AsyaPronina AsyaPronina force-pushed the enable_three_model_pp_by_default branch from 72144eb to 346320b Compare July 31, 2025 11:13
@github-actions github-actions bot added the category: NPUW NPUW plugin label Jul 31, 2025
Comment on lines 976 to 980
void update_config(ov::AnyMap& config, const std::pair<std::string, ov::Any>& pair) {
if (config.count(pair.first) == 0) {
config.insert(pair);
}
}
Copy link
Contributor

Choose a reason for hiding this comment

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

What is this about?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed!

split_llm_properties(properties, npuw_llm_props, other_props);
update_config(npuw_llm_props, {"NPUW_LLM_SHARED_HEAD", "YES"});
Copy link
Contributor

Choose a reason for hiding this comment

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

Just flip the property's default value

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done!

@dmatveev dmatveev added this to the 2025.3 milestone Jul 31, 2025
@dmatveev dmatveev self-assigned this Jul 31, 2025
@dmatveev dmatveev changed the title Three-model pipeline becomes default pipeline Enable 3-model pipeline by default Jul 31, 2025
@AsyaPronina AsyaPronina force-pushed the enable_three_model_pp_by_default branch from 346320b to 41bc241 Compare July 31, 2025 13:54
@AsyaPronina AsyaPronina force-pushed the enable_three_model_pp_by_default branch from 41bc241 to 4f8c07f Compare July 31, 2025 13:56
@github-actions github-actions bot removed the category: NPUW NPUW plugin label Jul 31, 2025
@dmatveev dmatveev added the category: NPUW NPUW plugin label Aug 1, 2025
@dmatveev dmatveev added this pull request to the merge queue Aug 1, 2025
Merged via the queue into openvinotoolkit:master with commit 9af49e7 Aug 1, 2025
175 checks passed
@dmatveev dmatveev deleted the enable_three_model_pp_by_default branch August 1, 2025 14:09
github-merge-queue bot pushed a commit that referenced this pull request Aug 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: NPU OpenVINO NPU plugin category: NPUW NPUW plugin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants