Skip to content
This repository has been archived by the owner on Aug 30, 2024. It is now read-only.

update profling log when NS_PROFILING is OFF #102

Merged
merged 2 commits into from
Feb 1, 2024
Merged

Conversation

zhenwei-intel
Copy link
Contributor

@zhenwei-intel zhenwei-intel commented Jan 31, 2024

Type of Change

feature or bug fix or documentation or others
API changed or not

Description

when NS_PROFILING is OFF and NEURAL_SPEED_VERBOSE=0/2, will print warning instead of 0ms logs
[Warning] To collect profiling data, please recompile with NS_PROFILING=ON.

Expected Behavior & Potential Risk

the expected behavior that triggered by this PR

How has this PR been tested?

how to reproduce the test (including hardware information)

Dependency Change?

any library dependency introduced or removed

Copy link
Contributor

@luoyu-intel luoyu-intel left a comment

Choose a reason for hiding this comment

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

OK

Copy link
Contributor

@DDEle DDEle left a comment

Choose a reason for hiding this comment

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

LGTM

setup.py Outdated Show resolved Hide resolved
Signed-off-by: zhenwei-intel <[email protected]>
@VincyZhang VincyZhang merged commit 6f85518 into main Feb 1, 2024
10 checks passed
@zhenwei-intel zhenwei-intel deleted the lzw/update_log branch February 29, 2024 01:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants