AdvancedPhotonSource / tike

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

API: Convert EPIE to RPIE #185

Closed carterbox closed 2 years ago

carterbox commented 2 years ago

Purpose

Add RPIE implementation.

Closes #142

Approach

Converts the EPIE implementation into an RPIE implementation because EPIE is a special case of RPIE. Also adds RPIE style regularization to the ADAM updates.

Worked with A. Tripathi to develop this RPIE update for the case when there are multiple illuminations:

Where p is the number of illumination, j is the number of scan posiiotns.

Pre-Merge Checklists

Submitter

Reviewer

pep8speaks commented 2 years ago

Hello @carterbox! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 192:17: E124 closing bracket does not match visual indentation Line 222:19: E124 closing bracket does not match visual indentation

Comment last updated at 2022-01-27 21:20:50 UTC