[SyneRBI/SIRF-Exercises] Mcir cil (PR #204)
ckolbPTB
I made sure the scripts run again. PDHG in g_non_cartesian_reconstruction is not yet working.
@KrisThielemans I uploaded the required test data to zenodo and adapted the download script such that it is automatically downloaded. Not sure if this is ok or if we want a separate download script for this very specific data.
You can view, comment on, or merge this pull request online at:
https://github.com/SyneRBI/SIRF-Exercises/pull/204
Commit Summary
- 85a94a1 MCIL with CIL
- cbe377a updated after bug
- 0404f7a updates to notebooks on MCIR
- d4df5ad Merge branch 'master' into MCIR_CIL
- 0c8b5ce grpe reconstruction + fista
- 1cb79a1 Merge branch 'master' into MCIR_CIL
- 3c6f8fe visualisation improved
- 0daf2c8 mcir comments started
- 198900b MCIR finished
- e655291 PDHG added but not working
- 105ae44 Merge branch 'master' into MCIR_CIL
- fdea931 GRPE download added
- da5602c update to new CIL interface
File Changes
(7 files)
- M notebooks/MR/README.md (6)
- A notebooks/MR/g_non_cartesian_reconstruction.ipynb (541)
- A notebooks/MR/h_mr_mcir_grpe.ipynb (703)
- A notebooks/PET/MCIR_CIL.ipynb (402)
- A notebooks/PET/MemoryTest.ipynb (427)
- A notebooks/PET/ViewResults.ipynb (156)
- M scripts/download_data.sh (13)
Patch Links:
- https://github.com/SyneRBI/SIRF-Exercises/pull/204.patch
- https://github.com/SyneRBI/SIRF-Exercises/pull/204.diff
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
ckolbPTB
@ckolbPTB pushed 2 commits.
—
View it on GitHub or unsubscribe.
You are receiving this because you are subscribed to this thread.
Kris Thielemans
@ckolbPTB what is the status here? Doable for 3.6? (soon)
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
Edoardo Pasca
@paskino pushed 1 commit.
- c6fca9f updated MR notebooks to SIRF 3.9+ and CIL 25
—
View it on GitHub or unsubscribe.
You are receiving this because you are subscribed to this thread.
Edoardo Pasca
@paskino pushed 1 commit.
- 5c2d57f add to readme
—
View it on GitHub or unsubscribe.
You are receiving this because you are subscribed to this thread.
Edoardo Pasca
@paskino pushed 1 commit.
- db8b686 Merge remote-tracking branch 'dorigin/master' into MCIR_CIL
—
View it on GitHub or unsubscribe.
You are receiving this because you are subscribed to this thread.
Edoardo Pasca
@paskino approved this pull request.
Tested with @ckolbPTB on a VM running SIRF 3.9+ and CIL 25
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
Edoardo Pasca
@paskino pushed 1 commit.
- d9855da added header and intro to PET notebook
—
View it on GitHub or unsubscribe.
You are receiving this because you are subscribed to this thread.
Kris Thielemans
@KrisThielemans commented on this pull request.
Hi, I cannot possibly review this now, but thanks for the invitation. In any case, if you both reviewed it, I trust you! Feel free to merge
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
Edoardo Pasca
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.