jacg / petalo

PET scanner image reconstruction
1 stars 4 forks source link

Foms #13

Closed arasskin closed 2 years ago

arasskin commented 2 years ago

Created a new combined version of the nema7 and jaszczak foms python wrapper and called it foms.py.

jacg commented 2 years ago

The original jaszczak_foms.py and nema7foms.py need to be removed, too.

jacg commented 2 years ago

As discussed, we also want to add an option for plotting 6 separate graphs (one for each sphere) vs. all 6 spheres on a single graph.

On the graphs, we decided to

jacg commented 2 years ago

The point of the TODO about adding ability to read foms from generated fom files, was to avoid recalculating the foms if you want to show the plot. As it stands, if I calculate the foms (and the file is written), and then decide I want to look at the graphs, and add the --show-plot=True option, it will calculate all the figures all over again, wasting a lot of time.

So:

jacg commented 2 years ago

The lower limit on the y-axis needs to be fixed at zero.

jacg commented 2 years ago

I've rebased this branch on top of the latest master.