DeepRegNet / DeepReg

Medical image registration using deep learning
Apache License 2.0
560 stars 76 forks source link

Use separate convolutions in ResizeCPTransform #748

Closed acasamitjana closed 3 years ago

acasamitjana commented 3 years ago

Description

Use separable convolutions to speed up the low pass filtering in ResizeCPTransform

Fixes #722

Type of change

What types of changes does your code introduce to DeepReg?

Please check the boxes that apply after submitting the pull request.

Checklist

Please check the boxes that apply after submitting the pull request.

If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

codecov[bot] commented 3 years ago

Codecov Report

Merging #748 (7813241) into main (e8f7149) will not change coverage. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##              main      #748   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           41        41           
  Lines         2501      2485   -16     
=========================================
- Hits          2501      2485   -16     
Impacted Files Coverage Δ
deepreg/model/layer_util.py 100.00% <ø> (ø)
deepreg/loss/kernel.py 100.00% <100.00%> (ø)
deepreg/model/layer.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update e8f7149...7813241. Read the comment docs.

acasamitjana commented 3 years ago

Hi @mathpluscode @YipengHu can you check that?

stale[bot] commented 3 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.