jacg / petalo

PET scanner image reconstruction
1 stars 4 forks source link

Outstanding items from foms PR #14

Open arasskin opened 2 years ago

arasskin commented 2 years ago

13 was merged but there is still work to be done

jacg commented 2 years ago

The vertical OSEM-iterator separation (and OSEM x-axis labels) are done. I"m not entirely satisfied with the axis labels, as they look a bit messy.

Displaying background variabilities on the same graph as CRCs will be tricky, as they are approximately proportional to eachother, so they naturally interfere with eachother if placed on the same graph. SNR has the opposite trend to CRC, so they avoid eachother for the most part, and where they do interfere, they cross almost orthogonally, so there is little confusion.

jacg commented 2 years ago

For the CLI options issue: I'm not convinced by docopt. Perhaps click is more pleasant to use?