SyneRBI / SIRF

Main repository for the CCP SynerBI software
http://www.ccpsynerbi.ac.uk
Other
58 stars 29 forks source link

sirf.Reg only wraps non symetrical version of f3d #684

Open AnderBiguri opened 4 years ago

AnderBiguri commented 4 years ago

This is due to a bug in NiftyReg. Would need them to fix it to wrap both.

More info in #680

AnderBiguri commented 4 years ago

@rijobro btw, the algorithm is still called sirf.Reg.NiftyF3dSym. Should that maybe be different now?

rijobro commented 4 years ago

I'd rather keep it that way so that:

  1. it doesn't break backwards compatibility; and
  2. once the bug is fixed, it can be symmetric with the option to not be symmetric.