cabouman / mbircone

BSD 3-Clause "New" or "Revised" License
11 stars 9 forks source link

Lamino crash hotfix #152

Closed dyang37 closed 10 months ago

dyang37 commented 11 months ago

This is a hotfix to the laminography crash issue. This PR provides a hook for a more fundamental fix in the future.

As discussed, we add an additional parameter "lamino_mode" to cone3D.recon(). This parameter is automatically set by recon_lamino(), such that a more conservative parallel update pattern is used for laminography reconstruction.

docstring: Screen Shot 2023-10-26 at 2 29 59 PM

Test

In the future, we might need to make N_G a function of #CPU cores.

cabouman commented 10 months ago

It seems to run fine. So I am going to accept the PR.