Matthew Jennings
Matthew Jennings
>GitHub has a PR checklist option. @Matthew-Jennings, would you be open to setting that up and including a checkbox in the template for "added relevant changes to the changelog"? >...
https://github.com/pymedphys/pymedphys/blob/cab692f879843af05728225276da6136c09c7b6d/CHANGELOG.md?plain=1#L15
See list of merged PRs since last release [here](https://github.com/pymedphys/pymedphys/pulls?q=is%3Apr+is%3Amerged). I'm not sure about how/whether to include some of them in the CHANGELOG: - @mguerrajordao, could you please update the dummy...
Stuck on "Waiting for kernel..." 
Hi all, A perennial issue that faces PyMedPhys is that we still in various places incorrectly equate *coordinates/axes* (i.e., `x`, `y`, and `z`) with *ijk* matrix indices (i.e., `slices`, `rows`,...
I agree with @theanega's comments regarding documentation. I suggest: 1. Adding your Statement of Need to the welcome page. 2. Adding near the top of the README some prose (outside...
1. FWIW, my investigations show ~75% test coverage. This isn't too bad, but it might be worth dedicating some spare time to lifting it a little, especially for the files...
Hi @alexzwanenburg. Great stuff putting this project together! Apologies for the delay in review, I hope you find the feedback useful. I'll separate my review items into individual issues. If...