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

[ARM CPU] Add ACL deconvolution operation #18655

Merged
merged 45 commits into from
Aug 2, 2023

Conversation

allnes
Copy link
Contributor

@allnes allnes commented Jul 19, 2023

No description provided.

@allnes allnes added category: CPU OpenVINO CPU plugin platform: arm OpenVINO on ARM / ARM64 labels Jul 19, 2023
@allnes allnes added this to the 2023.1 milestone Jul 19, 2023
@allnes allnes requested a review from a team as a code owner July 19, 2023 19:31
@allnes allnes requested a review from a team as a code owner July 19, 2023 19:31
src/plugins/intel_cpu/src/nodes/deconv.cpp Show resolved Hide resolved
src/plugins/intel_cpu/src/nodes/deconv.cpp Outdated Show resolved Hide resolved
src/plugins/intel_cpu/src/nodes/deconv.cpp Outdated Show resolved Hide resolved
src/plugins/intel_cpu/src/nodes/deconv.cpp Show resolved Hide resolved
src/plugins/intel_cpu/src/nodes/deconv.cpp Outdated Show resolved Hide resolved
src/plugins/intel_cpu/src/nodes/executors/deconv_list.hpp Outdated Show resolved Hide resolved
static bool customIsSupported(const DeconvAttrs& deconvAttrs,
const std::vector<MemoryDescPtr>& srcDescs,
const std::vector<MemoryDescPtr>& dstDescs) {
if (srcDescs[0]->getShape().getDims().size() != 4 ||
Copy link
Contributor

Choose a reason for hiding this comment

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

ACL says input could be 3D or 4D and output precision should be exact as input one.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@alvoron done

}

private:
static bool validate_deconvolution_output_dimensions(unsigned int in_width, unsigned int in_height,
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 this method is copied from ACL.
Shouldn't we rely on NEDeconvolutionLayer::validate? I assume it makes the same checks as well.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@alvoron done

@allnes allnes requested a review from alvoron July 27, 2023 16:29
src/plugins/intel_cpu/src/nodes/executors/deconv.hpp Outdated Show resolved Hide resolved
src/plugins/intel_cpu/src/nodes/executors/deconv_list.cpp Outdated Show resolved Hide resolved
src/plugins/intel_cpu/src/nodes/executors/deconv.hpp Outdated Show resolved Hide resolved
src/plugins/intel_cpu/src/nodes/executors/deconv.hpp Outdated Show resolved Hide resolved
src/plugins/intel_cpu/src/nodes/deconv.cpp Show resolved Hide resolved
@allnes allnes requested a review from dmitry-gorokhov August 1, 2023 16:36
@dmitry-gorokhov dmitry-gorokhov enabled auto-merge (squash) August 2, 2023 15:58
@dmitry-gorokhov dmitry-gorokhov merged commit f3bafef into openvinotoolkit:master Aug 2, 2023
@allnes allnes deleted the an/exp_deconv_1 branch August 3, 2023 22:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: CPU OpenVINO CPU plugin platform: arm OpenVINO on ARM / ARM64
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants