openvino icon indicating copy to clipboard operation
openvino copied to clipboard

Add FP64 precision support in intel_npu

Open CailiWuIntel opened this issue 1 year ago • 5 comments

Details:

Add FP64 precision pipeline support in intel_npu。

Tickets:

  • E-121878

CailiWuIntel avatar Apr 29 '24 06:04 CailiWuIntel

RUN_CI_ALL

CailiWuIntel avatar Apr 29 '24 06:04 CailiWuIntel

There are no triggers like RUN_CI_ALL for OpenVINO repo.

Also can you please update the description to avoid using a direct link? We normally post just number like this E-121878

ArtemySkrebkov avatar Apr 29 '24 10:04 ArtemySkrebkov

There are no triggers like RUN_CI_ALL for OpenVINO repo.

Also can you please update the description to avoid using a direct link? We normally post just number like this E-121878

Thanks for your kindly reminder, the related ticket description is updated.

CailiWuIntel avatar Apr 30 '24 02:04 CailiWuIntel

Can you please enable the relevant functional tests (for infer_request) as well in vpux-plugin repository?

PatrikStepan avatar May 08 '24 06:05 PatrikStepan

Can you please enable the relevant functional tests (for infer_request) as well in vpux-plugin repository?

That's OK, I will add the test case for fp64 in this PR, but it should be added after this PR merged, because this test pass need fp64 supporting in OV, and I cannot merge my PR8666 without test pass. I've tried to add the test case locally, and it passes, detail shows in comment https://github.com/intel-innersource/applications.ai.vpu-accelerators.vpux-plugin/pull/8666#issuecomment-2071605773

CailiWuIntel avatar May 08 '24 13:05 CailiWuIntel

Hi @ArtemySkrebkov @StefaniaHergane, could you help to start CI, or tell how can I start it? Also, could you help to merge this PR after CI pass, or tell me what else should I do?

CailiWuIntel avatar May 29 '24 14:05 CailiWuIntel

Started the CI checks.

ArtemySkrebkov avatar May 30 '24 09:05 ArtemySkrebkov

Have started OV Jenkins CI pipeline

iefode avatar May 31 '24 11:05 iefode

The relevant NPU functional tests are now part of OpenVINO repository as well: https://github.com/openvinotoolkit/openvino/tree/master/src/plugins/intel_npu/tests/functional/shared_tests_instances/behavior/ov_infer_request Can we enable BehaviorTests/OVInferRequestCheckTensorPrecision*type=f64 now?

PatrikStepan avatar Jun 05 '24 13:06 PatrikStepan

The relevant NPU functional tests are now part of OpenVINO repository as well: https://github.com/openvinotoolkit/openvino/tree/master/src/plugins/intel_npu/tests/functional/shared_tests_instances/behavior/ov_infer_request Can we enable BehaviorTests/OVInferRequestCheckTensorPrecision*type=f64 now?

I've checked https://github.com/openvinotoolkit/openvino/tree/master/src/plugins/intel_npu/tests/functional/shared_tests_instances/behavior/ov_infer_request/io_tensor.cpp and https://github.com/openvinotoolkit/openvino/tree/master/src/plugins/intel_npu/tests/functional/shared_tests_instances/skip_tests_config.cpp, now the f64 precison test has been added and do not be skipped image

image So that, it's no need to add any test in this PR.

CailiWuIntel avatar Jun 11 '24 15:06 CailiWuIntel

Hi, @iefode @PatrikStepan, could you help to check and start the CI.

CailiWuIntel avatar Jun 17 '24 06:06 CailiWuIntel

Hi @ArtemySkrebkov, could you help to start CI.

CailiWuIntel avatar Jun 17 '24 07:06 CailiWuIntel

Sure, started the CI checks.

ArtemySkrebkov avatar Jun 17 '24 15:06 ArtemySkrebkov

build_jenkins

ArtemySkrebkov avatar Jun 18 '24 14:06 ArtemySkrebkov