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

Sns explicit tiles #55

Conversation

IvanNovoselov
Copy link
Collaborator

Details:

  • This is a copy of PR 13330, please refer to the original PR for the details

src/plugins/intel_cpu/src/nodes/subgraph.cpp Show resolved Hide resolved
@@ -61,6 +61,14 @@ class SubgraphBaseTest : public CommonTestUtils::TestsCommon {
virtual std::vector<ov::Tensor> get_plugin_outputs();
};

inline std::vector<InputShape> dynamic_shapes_to_test_representation(const std::vector<ov::PartialShape>& shapes) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

That should be approved by tests maintainers.

ov::runtime::Tensor generate(const ov::element::Type& elemType,
const ov::Shape& targetShape,
InputGenerateData inGenData = InputGenerateData(10, 20, 32768, 1)) {
// InputGenerateData inGenData = InputGenerateData(10, 20, 32768, 1)) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Same story? Can it be extracted into separate PR?

Copy link
Collaborator Author

@IvanNovoselov IvanNovoselov Nov 30, 2022

Choose a reason for hiding this comment

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

I contacted Irina Efode to ask her opinion about the changes

Copy link
Collaborator

@dmitry-gorokhov dmitry-gorokhov left a comment

Choose a reason for hiding this comment

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

Leftovers will be fixed in separate PR

@dmitry-gorokhov dmitry-gorokhov merged commit 0075a5f into a-sidorova:feature/snippets/mha Nov 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants