-
Notifications
You must be signed in to change notification settings - Fork 2.6k
[LPT] Plugin tests cleanup #30700
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
base: master
Are you sure you want to change the base?
[LPT] Plugin tests cleanup #30700
Conversation
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.
This legacy test is removed since it duplicates more actual mat_mul_transformation.cpp test
@@ -48,16 +48,13 @@ const std::vector<MultiplyToGroupConvolutionTransformationParam> params = { | |||
"U8", | |||
true}}; | |||
|
|||
//Comment out the tests because of the transformation is disabled by another WR |
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.
These tests are skipped in skip_tests_config.cpp as it is a more preferrable way to skip the instances
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.
This legacy test is removed since it duplicates more actual ConcatWithNeighborsGraphTransformation
test. The same is actual for output_layers_concat_multi_channel.cpp
9016efd
to
83a1e39
Compare
Details:
LayerTransformation::Params
is removed from plugin tests parameters, since this param doesn't affect plugin tests behavior at allLayerTransformation
base class cleanup: unused helpers/structures are removedTickets: