[Insight-developers] Teem : NrrdIO sync with ITK : NrrdIO - now proceeding...

[Michel Audette]

Dear Gordon,

thanks for your kind reply. I'll imbed my reply in yours.

On Sat, Dec 19, 2009 at 1:33 AM, Gordon L. Kindlmann <glk at uchicago.edu>wrote:

>
> Unless you or other ITK developers object, my strategy will be to retain
>> the extra functionality that you have in Teem in the final version, with a
>> view to replacing the size_t's by unsigned ints, and in general keep a
>> superset of the two sets of changes.
>>
>
>
> Can you explain this in a little more detail?   What's an example of a
> size_t that would be changed to an unsigned int?
>
>
I was misinformed on this proposed change to size_t. Based on what I've seen
so far,  much of your NrrdIO source code is either identical to what we have
in ITK, or essentially supercedes it.

After a discussion with Luis, I don't see a reason to revert from size_t to
unsigned int. There may be some modifications in ITK where a cast is used in
the setting of certain variables, which were in response to bugs, so I will
look at your code to see if such as cast is required. The trick will be to
test these changes on at least Linux and Windows at once, and specifically
on 64-bit platforms.


>  I welcome any comments from you and the community that relate to this
>> update.
>>
>
> I've augmented the table here:
>
> http://www.itk.org/Wiki/Proposals:Updating_Nrrd_library_2009#Table
>

Thanks for your comments on this table. I may have to revisit these on an
item-by-item basis with you.


>
> to better document where files in NrrdIO are coming from; whether they're
> copied (via NrrdIO/pre-GNUmakefile) from Teem itself, and if so where.  This
> should help back-propagate the changes.
>
> Is there an *easy* way of linking from the files listed on that table, to a
> ViewVC-type record of the changes that it has seen in ITK?
>

I will look into this and report back. I was not familiar with ViewVC until
your email.


>
> Finally, to make changes in Teem you'll need to be added to the sourceforge
> Teem project; let me know your sourceforge login (or create one if you don't
> have one already) so that I can do this.
>

I will create an account and get back to you.

Best wishes,

Michel

-- 
Michel Audette, Ph.D.
R & D Engineer,
Kitware Inc.,
Chapel Hill, N.C.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.itk.org/mailman/private/insight-developers/attachments/20091221/0671bcea/attachment.htm>