Hello, On Fri, Oct 22, 2010 at 12:43:03PM -0400, Paul Novotny wrote: > On Sun, 2010-10-17 at 23:04 -0500, Steve M. Robbins wrote: > > I'm considering switching to the new refactored Statistics classes in > > ITK. This is an API-breaking move so I'd like to hear of any concerns > > before implementing this. > > > > The change would be done for version 3.20 of InsightToolkit (currently > > stuck in the NEW queue) and not for version 3.18. So this is not > > targeted for squeeze. > > > > My main motivation for doing this is to enable packaging elastix > > (http://elastix.isi.uu.nl/). Note also that ITK 4.0 will use > > the new statistics so code has to change eventually; this would > > just force the change one version earlier as 3.20 is the last > > release destined for ITK 3.x. I've started on adding Review/Statistics. Then I learned that elastix requires other pieces of Review, so I ended up adding all of Review. > Adding Review means you can also add WrapITK. Something I think is a > good idea, but others may not. I took your suggestion and am now building with USE_WRAP_ITK=ON. > Although, one issue that comes up is what dimensions and datatypes to > wrap. I use a lot of datatypes in my work, plus 4D data, so the package > gets over 100MB, and much larger when installed. It might be nice to > have multiple packages to pick from depending on your needs? Yeah, I see what you mean. My initial trial just sets "USE_WRAP_ITK=ON" and the default set of wrappings which includes 2 and 3 dimensions. I gather you add dimension=4. What other datatypes do you add? Thanks, -Steve
Attachment:
signature.asc
Description: Digital signature