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

Add STORE_LOGGER_LOG #27627

Conversation

WenjingKangIntel
Copy link
Contributor

Details:

  • item1
  • ...

Tickets:

  • 154386

Signed-off-by: Kang Wenjing <[email protected]>
@WenjingKangIntel WenjingKangIntel requested review from a team as code owners November 20, 2024 02:03
@github-actions github-actions bot added the category: NPU OpenVINO NPU plugin label Nov 20, 2024
@WenjingKangIntel WenjingKangIntel marked this pull request as draft November 20, 2024 06:31
Signed-off-by: Kang Wenjing <[email protected]>
Signed-off-by: Kang Wenjing <[email protected]>
* Type: boolean, default is false.
* This option allows to store the log during compilation
*/
static constexpr ov::Property<bool> store_logger_log{"NPU_STORE_LOGGER_LOG"};
Copy link
Contributor

Choose a reason for hiding this comment

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

Why we actually need a new property for that?

Copy link
Contributor

Choose a reason for hiding this comment

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

Nor sure if we need to have a switch to turn off this feature on plugin side, if we always keep the feature on, i think can remove the option

Copy link
Contributor

This PR will be closed in a week because of 2 weeks of no activity.

@github-actions github-actions bot added the Stale label Dec 23, 2024
Copy link
Contributor

This PR was closed because it has been stalled for 2 week with no activity.

@github-actions github-actions bot closed this Dec 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: NPU OpenVINO NPU plugin Stale
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants