search

AdvancedPhotonSource / tike

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

New ePIE, DM, MLs, API based on the new needed signature in ptychopy. #135

Closed kyuepublic closed 2 years ago

kyuepublic commented 3 years ago

1 Need to add more functionality in ptychopy.c. 2 The new calling sequence would be different from the original one, keeping the original one to keep the ability for doing reconstruction from passing a file path.Ex, ePIE_File(), ePIE_Array(); 3 Two sets of API, one set is for step mode to check the middle results. Another set could be for the whole reconstruction process.

carterbox commented 3 years ago

You don't need two APIs if step takes a parameter like num_step?

kyuepublic commented 3 years ago

The second set is not just taking all the steps, but including CUDA init and post processing. And the first set would have to split to show the middle results of the object and probe value.

On Thu, Mar 11, 2021 at 5:26 PM Daniel Ching @.***> wrote:

You don't need two APIs if step takes a parameter like num_step?

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/tomography/tike/issues/135#issuecomment-797117936, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEMBS2CWENT63CIEFD67F5DTDE7PVANCNFSM4ZBFYVTQ .

-- Best,

Ke Yue