-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
[intel-npu] Adding NPU_DYNAMIC_QUANTIZATION property #28316
base: master
Are you sure you want to change the base?
Conversation
* Set or verify state of dynamic quantization in the NPU compiler | ||
* @ingroup ov_runtime_npu_prop_cpp_api | ||
*/ | ||
static constexpr ov::Property<bool> compiler_dynamic_quantization{"NPU_COMPILER_DYNAMIC_QUANTIZATION"}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please, add Python API
std::ostringstream dqstr; | ||
dqstr << ov::intel_npu::compiler_dynamic_quantization.name() << KEY_VALUE_SEPARATOR << VALUE_DELIMITER << "\\S+" | ||
<< VALUE_DELIMITER; | ||
logger.warning("COMPILER_DYNAMIC_QUANTIZATION property is not suppored by this compiler version. Removing from " |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
suppored typo
https://github.com/intel-innersource/applications.ai.vpu-accelerators.vpux-plugin/pull/15283 PR bumping the minor version |
@@ -538,6 +538,16 @@ std::string DriverCompilerAdapter::serializeConfig(const Config& config, | |||
content = std::regex_replace(content, std::regex(batchstr.str()), ""); | |||
} | |||
|
|||
// COMPILER_DYNAMIC_QUANTIZATION is not supported in versions < 6.1 - need to remove it |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Version in the comment does not match the implementation
Details:
Tickets: