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

Fix np uplugin log usage #19

Closed
wants to merge 31 commits into from
Closed
Changes from 1 commit
Commits
Show all changes
31 commits
Select commit Hold shift + click to select a range
dc72eca
fix defualt log config value
DanLiu2Intel Jun 23, 2024
c0d7b00
update Logger::global() and Plugin::_logger in getCompiler
DanLiu2Intel Jun 23, 2024
5dd6f4d
keep the max log level in localConfig and update set_peoperty global log
DanLiu2Intel Jul 1, 2024
8cefc36
fix build and remove update_log_level()
DanLiu2Intel Jul 2, 2024
615a45d
fix bug and update logger
DanLiu2Intel Jul 5, 2024
6658171
update logger.setlevel
DanLiu2Intel Jul 6, 2024
0fbd002
using Logger::global to replace Plugin::_logger
DanLiu2Intel Jul 6, 2024
8cc2fec
remove NPUBackend::steup due to not useful when update its log level
DanLiu2Intel Jul 6, 2024
bb7d963
update print loglevel
DanLiu2Intel Jul 6, 2024
07160ef
Update plugin.cpp
DanLiu2Intel Jul 8, 2024
f7cfe87
Update backends.hpp
DanLiu2Intel Jul 8, 2024
acea859
revert config remove code
DanLiu2Intel Jul 10, 2024
1a6eb99
add log member in Config anclass and OptionsDesc class
DanLiu2Intel Jul 10, 2024
7ee0f73
update fix when log set by set_property (global and pass log_None )
DanLiu2Intel Jul 10, 2024
c9be2d9
Merge branch 'master' into fix_NPUplugin_log_usage
DanLiu2Intel Jul 11, 2024
bea8f81
fix comments and resume the use of update_log_level func
DanLiu2Intel Jul 15, 2024
1e170f4
fix build
DanLiu2Intel Jul 16, 2024
db9829d
use global log in config part to avoid more change
DanLiu2Intel Jul 16, 2024
6edeaee
Merge branch 'master' into fix_NPUplugin_log_usage
DanLiu2Intel Jul 16, 2024
e6a84b7
resume log.trace to log.debug
DanLiu2Intel Jul 17, 2024
cc7a070
add log for blob reading and writing
DanLiu2Intel Jul 17, 2024
28b4caa
add log msg for blob and fix format
DanLiu2Intel Jul 17, 2024
d55832a
move ZeroEngineBackend::updateInfo implementation to source file
DanLiu2Intel Jul 17, 2024
d1ea9f2
Merge branch 'master' into fix_NPUplugin_log_usage
DanLiu2Intel Jul 17, 2024
f7acb8d
Merge branch 'master' into fix_NPUplugin_log_usage
DanLiu2Intel Jul 31, 2024
92ed3b6
fix comments
DanLiu2Intel Jul 31, 2024
436b26a
update `log.error` in import_model to `OPENVINO_THROW`
DanLiu2Intel Aug 2, 2024
05cf9e3
Merge branch 'master' into fix_NPUplugin_log_usage
DanLiu2Intel Aug 2, 2024
7811c2e
fix print value
DanLiu2Intel Aug 2, 2024
830b875
Merge branch 'master' into fix_NPUplugin_log_usage
DanLiu2Intel Aug 5, 2024
a96cd21
Merge branch 'master' into fix_NPUplugin_log_usage
DanLiu2Intel Aug 6, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
move ZeroEngineBackend::updateInfo implementation to source file
DanLiu2Intel committed Jul 17, 2024

Verified

This commit was signed with the committer’s verified signature.
webknjaz 🇺🇦 Sviatoslav Sydorenko (Святослав Сидоренко)
commit d55832a662ace30388e7c3641ce82527b7c0ec53
9 changes: 1 addition & 8 deletions src/plugins/intel_npu/src/backend/include/zero_backend.hpp
Original file line number Diff line number Diff line change
@@ -29,14 +29,7 @@ class ZeroEngineBackend final : public IEngineBackend {
bool isWorkloadTypeSupported() const override;

void* getContext() const override;
void updateInfo(const Config& config) override {
_logger.setLevel(config.get<LOG_LEVEL>());
if (_devices.size() > 0) {
for (auto& dev : _devices) {
dev.second->updateInfo(config);
}
}
}
void updateInfo(const Config& config) override;

private:
std::shared_ptr<ZeroInitStructsHolder> _instance;
9 changes: 9 additions & 0 deletions src/plugins/intel_npu/src/backend/src/zero_backend.cpp
Original file line number Diff line number Diff line change
@@ -68,4 +68,13 @@ void* ZeroEngineBackend::getContext() const {
return _instance->getContext();
}

void ZeroEngineBackend::updateInfo(const Config& config) {
_logger.setLevel(config.get<LOG_LEVEL>());
if (_devices.size() > 0) {
for (auto& dev : _devices) {
dev.second->updateInfo(config);
}
}
}

} // namespace intel_npu