Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

antsMotionCorr iteration/smooth/shrink command line option names differ from antsRegistration #1427

Open
gdevenyi opened this issue Oct 6, 2022 · 1 comment
Labels
wouldbenice Nice ideas not under development (pull requests welcome)

Comments

@gdevenyi
Copy link
Contributor

gdevenyi commented Oct 6, 2022

antsMotionCorr accepts this as options

     -i, --iterations MxNx0...
          Specify the number of iterations at each level. 

     -s, --smoothingSigmas MxNx0...
          Specify the sigma for smoothing at each level. Smoothing may be specified in mm 
          units or voxels with "AxBxCmm" or "AxBxCvox". No units implies voxels. 

     -f, --shrinkFactors MxNx0...
          Specify the shrink factor for the virtual domain (typically the fixed image) at 
          each level. 

While antsRegistration uses this:

     -c, --convergence MxNxO
                       [MxNxO,<convergenceThreshold=1e-6>,<convergenceWindowSize=10>]
          Convergence is determined from the number of iterations per level and is 
          determined by fitting a line to the normalized energy profile of the last N 
          iterations (where N is specified by the window size) and determining the slope 
          which is then compared with the convergence threshold. 

     -s, --smoothing-sigmas MxNxO...
          Specify the sigma of gaussian smoothing at each level. Units are given in terms 
          of voxels ('vox') or physical spacing ('mm'). Example usage is '4x2x1mm' and 
          '4x2x1vox' where no units implies voxel spacing. 

     -f, --shrink-factors MxNxO...
          Specify the shrink factor for the virtual domain (typically the fixed image) at 
          each level. 

The naming here is inconsistent (making automatic generation of a command line a bit annoying), but also the convergence option doesn't support controlling threshold or window size.

@gdevenyi
Copy link
Contributor Author

gdevenyi commented Nov 9, 2022

And a further followup, the convergence settings in antsMotionCorr are hard-coded as 1e-7,10 which is probably overkill, and would be nice better if it were settable in the convergence command line.

@cookpa cookpa added the wouldbenice Nice ideas not under development (pull requests welcome) label Jan 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wouldbenice Nice ideas not under development (pull requests welcome)
Projects
None yet
Development

No branches or pull requests

2 participants