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

Support float64 data type as input of benchmark_app #16435

Merged

Conversation

jade-cho
Copy link
Contributor

No description provided.

@jade-cho jade-cho requested review from a team as code owners March 21, 2023 08:59
@jade-cho jade-cho requested review from ilya-lavrenov, ilyachur and tomdol and removed request for a team March 21, 2023 08:59
@github-actions github-actions bot added the category: samples OpenVINO Runtime Samples label Mar 21, 2023
@jade-cho jade-cho force-pushed the support_fp64_as_benchmark_app_input branch from 6e48ab8 to fcf7277 Compare March 21, 2023 11:45
@jade-cho jade-cho force-pushed the support_fp64_as_benchmark_app_input branch 3 times, most recently from 01b846a to df6b8ad Compare March 22, 2023 01:34
@jade-cho jade-cho requested review from a team as code owners March 22, 2023 01:34
@github-actions github-actions bot added category: CPU OpenVINO CPU plugin category: GPU OpenVINO GPU plugin labels Mar 22, 2023
@jade-cho jade-cho force-pushed the support_fp64_as_benchmark_app_input branch 3 times, most recently from 5e4c5cf to 3720eca Compare March 22, 2023 04:29
@github-actions github-actions bot removed category: CPU OpenVINO CPU plugin category: GPU OpenVINO GPU plugin labels Mar 22, 2023
@@ -40,6 +40,8 @@ void fill_buffer_random(void* inputBuffer,
void fill_buffer(void* inputBuffer, size_t elementsNum, const ov::element::Type& type) {
if (type == ov::element::f32) {
fill_buffer_random<float, float>(inputBuffer, elementsNum);
} else if (type == ov::element::f64) {
fill_buffer_random<double, double>(inputBuffer, elementsNum);
} else if (type == ov::element::f16) {
fill_buffer_random<short, short>(inputBuffer, elementsNum);
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this also should be changed to:

Suggested change
fill_buffer_random<short, short>(inputBuffer, elementsNum);
fill_buffer_random<ov::float16, float>(inputBuffer, elementsNum);

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Changed

@jade-cho jade-cho force-pushed the support_fp64_as_benchmark_app_input branch from 3720eca to b12534d Compare March 22, 2023 09:25
@isanghao isanghao merged commit a20b363 into openvinotoolkit:master Mar 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: samples OpenVINO Runtime Samples
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants