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.

Sign up for GitHub

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

Jump to bottom

[ARM plugin] Fix ARM tests (case bf16) #13149

Merged
merged 1 commit into from
Sep 21, 2022
Merged

[ARM plugin] Fix ARM tests (case bf16) #13149

merged 1 commit into from
Sep 21, 2022

Conversation

allnes
Copy link
Contributor

@allnes allnes commented Sep 21, 2022

@allnes allnes requested review from a team as code owners September 21, 2022 14:14
@allnes allnes changed the title [ARM plugin]Fix ARM tests (case bf16) [ARM plugin] Fix ARM tests (case bf16) Sep 21, 2022
@allnes allnes mentioned this pull request Sep 21, 2022
Merged
@allnes
Fix arm tests (bf16)
e196ef3 
Fix arm tests bf16
ilya-lavrenov
ilya-lavrenov reviewed Sep 21, 2022
View reviewed changes
src/tests/functional/shared_test_classes/src/base/ov_subgraph.cpp
}
#if !(defined(__arm__) || defined(_M_ARM) || defined(__aarch64__) || defined(_M_ARM64))
if (targetDevice == CommonTestUtils::DEVICE_CPU && !configuration.count(InferenceEngine::PluginConfigParams::KEY_ENFORCE_BF16)) {
configuration.insert({InferenceEngine::PluginConfigParams::KEY_ENFORCE_BF16, InferenceEngine::PluginConfigParams::NO});
Copy link
Contributor

Choose a reason for hiding this comment

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

@iefode do we have a better solution for CPU plugin to add their extra keys? I don't want to have it in a common place..

Copy link
Contributor

Choose a reason for hiding this comment

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

Looks like wa from CPU side. I will check the code, author and reason

@ilya-lavrenov ilya-lavrenov enabled auto-merge (squash) September 21, 2022 16:14
@ilya-lavrenov ilya-lavrenov added this to the 2022.3 milestone Sep 21, 2022
@ilya-lavrenov ilya-lavrenov merged commit ce9751a into openvinotoolkit:master Sep 21, 2022
yury-intel pushed a commit to yury-intel/openvino that referenced this pull request Sep 23, 2022
@allnes @yury-intel
Fix arm tests (bf16) (openvinotoolkit#13149)
d422ef6 
Fix arm tests bf16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@iefode iefode iefode left review comments

@ilya-lavrenov ilya-lavrenov ilya-lavrenov approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
2022.3
Development

Successfully merging this pull request may close these issues.
3 participants
@allnes @ilya-lavrenov @iefode