Cecilia/upstream/pdpd frontend#3
Cecilia/upstream/pdpd frontend#3ceciliapeng2011 wants to merge 62 commits intonosovmik:pdpd_frontendfrom
Conversation
| return res; | ||
| } | ||
|
|
||
| INSTANTIATE_TEST_CASE_P(FrontendOpTest, FrontendOpTest, |
There was a problem hiding this comment.
I'd recommend to use PDPDOpTest as first here. Because when we'll have multiple frontends with shared common tests, each generated test shall be unique.
E.g. user will be able to do "./unit-test --gtest_filter=PDPDOpTest"
.gitignore
Outdated
| ngraph/src/protobuf/ | ||
| ngraph/src/src/ | ||
| ngraph/src/test/ | ||
| ngraph/test/files/paddlepaddle/models/* |
There was a problem hiding this comment.
Actually unit tests are not intended to generate models on the fly, we should put it to source control (under git lfs) to make unit tests runnable.
So, this path shall not be in '.gitignore', also .pdmodel, .pdiparams shall be pushed to repository.
Generate scripts are basically just for reference for developers, so that everyone will know how existing test models were generated
|
|
||
| //Inputs inputs; | ||
| // data (1, 3, 4, 4) input tensor | ||
| res.inputs.emplace_back(test::NDArray<float, 4>{{{{{0.f, 1.f, 2.f, 3.f, 4.f}, |
There was a problem hiding this comment.
I'm getting exception here, looks like input needs 1x3x4x4=48 elements, but only 35 is given
|
|
||
| std::string FrontendOpTest::getTestCaseName(const testing::TestParamInfo<FrontendOpTestParam> &obj) { | ||
| std::string res = obj.param.m_frontEndName + "_" + obj.param.m_modelName; | ||
| //res += "I" + joinStrings(obj.param.m_oldInputs) + joinStrings(obj.param.m_newInputs); |
There was a problem hiding this comment.
it's fine to have it this way for now, just note that generated 'testCaseName' must be unique for test system.
Combining it only with frontendName and modelName, you'll not be able to run FrontendOptest with same model, but with different inputs
|
|
||
| // convert | ||
| std::shared_ptr<ngraph::Function> function; | ||
| function = m_frontEnd->convert(m_inputModel); |
|
|
||
| INSTANTIATE_TEST_CASE_P(FrontendOpTest, pool2dTest, | ||
| ::testing::Values( | ||
| avgPool(), |
There was a problem hiding this comment.
I'm getting "Unsupported pooling type" here, is it expected?
…cilite developer.
…eciliapeng2011/openvino into bym/pdpd_frontend/op X
… an unified test case.
…penvino into bym/pdpd_frontend/op
fix bug in matmul support group conv
add ops: hard_sigmoid hard_swish leaky_relu pow relu6 sigmoid slice squeeze unsqueeze2
Yi3/merge to internal
Bym/pdpd frontend/op add relu & softmax
|
close it as it is obsolete. |
* [FrontEnd]enable 16 ops conversion * [FrontEnd]fix typo in interpolate * param support tensor (#3) * Update hard_sigmoid.cpp (#4) * Yi3/fix paddle part2 * fix paddle conversion * ops forward declaration * use tile instead of broadcast * Add validation check in dropout (#8) * fix conflict * fix code style * fix transpose2 * use perm size in transpose2 * remove check in transpose2 Co-authored-by: Luo Cheng <cheng.luo@intel.com> Co-authored-by: Mang Guo <mang.guo@intel.com>
…toolkit#9691) * [Transformations] Added interchangeable reshape elimination * Applied comments #2 * returned Reshape in condition * applied comments #3 * applied comments #4 * added comment in plugin with reason about transformation
Details:
Tickets: