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

Feat: Adding ResizeConvolutionToDeconvolution transformation #102

Merged

Conversation

i-colbert
Copy link
Contributor

@i-colbert i-colbert commented Feb 20, 2024

AddingResizeConvolutionToDeconvolution transformation using the "weight convolution" algorithm introduced in An Energy-Efficient Edge Computing Paradigm for Convolution-Based Image Upsampling. Also adding unit tests to demonstrate usage and verify that it maintains functionality of the original graph when necessary.

@i-colbert i-colbert changed the title Adding ResizeConvolutionToDeconvolution transformation Feat: Adding ResizeConvolutionToDeconvolution transformation Feb 20, 2024
@i-colbert i-colbert force-pushed the icolbert/nn-resize-to-deconv branch from e8d0c22 to 19c73fe Compare February 27, 2024 04:28
Copy link
Collaborator

@maltanar maltanar left a comment

Choose a reason for hiding this comment

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

Thanks for the PR! Looks good in general, but I have a few small comments, would be great if you could address them.

src/qonnx/transformation/resize_conv_to_deconv.py Outdated Show resolved Hide resolved
src/qonnx/transformation/resize_conv_to_deconv.py Outdated Show resolved Hide resolved
src/qonnx/transformation/resize_conv_to_deconv.py Outdated Show resolved Hide resolved
@i-colbert i-colbert requested a review from maltanar March 11, 2024 21:00
@maltanar maltanar merged commit c5bd87f into fastmachinelearning:main Mar 12, 2024
5 checks passed
@i-colbert i-colbert deleted the icolbert/nn-resize-to-deconv branch March 12, 2024 22:12
@maltanar maltanar added this to the v0.4.0 milestone Dec 20, 2024
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