mrdegibbs: update doc to soften warning about partial Fourier#2923
mrdegibbs: update doc to soften warning about partial Fourier#2923
Conversation
|
clang-tidy review says "All clean, LGTM! 👍" |
Lestropie
left a comment
There was a problem hiding this comment.
-
Suggest having
masteras base branch -
Currently a note about filtering is slid in at the last minute on a paragraph about partial Fourier. I would suggest a separate
DESCRIPTIONparagraph on disabling on-scanner filtering (both k-space-based and image-based).
c910e42 to
15e558f
Compare
|
OK, pull request redirected to
|
|
Will require executing |
Co-authored-by: Robert Smith <[email protected]>
|
@Lestropie, feel free to merge if you're OK with these last few changes. |
Resolution of content merged to dev initially omitted from #3061. - #2693 (completely omitted) - #3005 (partial propagation / independent address of some common issues on both branches) - #3001 (completely omitted) - #2908 (completely omitted) - #2955 (completely omitted) - #2600 (completely omitted) - #2962 (completely omitted) - #2935 (completely omitted) - #2923 (completely omitted) - #2910 (completely omitted) - #2638 (completely omitted) - #2698 (completely omitted) - #2721 (completely omitted) - #2794 (completely omitted) - #2768 (completely omitted; required modification to conform to other dev changes) - #2713 (residual compilation errors following adf8fdd, including resolution against changes in #2437 on dev.
Resolution of content merged to dev initially omitted from #3061. - #2693 (completely omitted) - #3005 (partial propagation / independent address of some common issues on both branches) - #3001 (completely omitted) - #2908 (completely omitted) - #2955 (completely omitted) - #2600 (completely omitted) - #2962 (completely omitted) - #2935 (completely omitted) - #2923 (completely omitted) - #2910 (completely omitted) - #2638 (completely omitted) - #2698 (completely omitted) - #2721 (completely omitted) - #2794 (completely omitted) - #2768 (completely omitted; required modification to conform to other dev changes) - #2713 (residual compilation errors following adf8fdd, including resolution against changes in #2437 on dev.
As discussed on forum some time ago:
https://community.mrtrix.org/t/mrdegibbs-in-partial-fourier/2457/2?u=jdtournier