AdvancedPhotonSource / tike

Repository for ptychography software
http://tike.readthedocs.io
Other
29 stars 15 forks source link

NEW: Implement Flow operator for CuPy #86

Closed carterbox closed 4 years ago

carterbox commented 4 years ago

Purpose

Allow CuPy backend with align module by implementing the Flow operator.

Approach

Switch to Lanczos interpolation for Flow operator because map_coordinates for CuPy only has 1st order spline interpolation.

Pre-Merge Checklists

Submitter

Reviewer

dgursoy commented 4 years ago

Do you know the performance (speed) difference between Lanczos and the 1st order interpolation? Because we solve those problems in ADMM approximately anyways, and an approximate (but fast) interpolator may work too.

carterbox commented 4 years ago

I don't know. It's probably more expensive because of needing to call sinc() in the filter.