[Insight-developers] Moving PAR/REC from Code/Review to Code/IO: Re: [ITK 0005454]: Support Philips PAR/REC image files in ITK

[Sean McBride]

I agree that's pretty low coverage.  I can only say that the existing
implementation is working to our satisfaction.  We are pretty swamped
these days, but in a few weeks might be able to add more testing.

On 2/7/09 11:30 PM, Bill Lorensen said:

>These readers do not have very high test coverage.
>itkPhilipsPAR.cxx has almost 1000 lines of uncovered code.
>itkPhilipsRECImageIO.cxx is only 65% covered.
>
>Bill
>
>On Sat, Feb 7, 2009 at 4:57 PM, Luis Ibanez <luis.ibanez at kitware.com> wrote:
>> Hi Sean,
>>
>> Thanks for updating this bug entry.
>>
>> It seems that you are confident on that we could move these ImageIO
>> classes out of the Code/Review directory and into the Code/IO directory.
>>
>> Now will be a good moment for doing so. In this way, they will be
>> included in ITK 3.12.
>>
>> Please let us know,
>>
>>
>>    Thanks
>>
>>
>>       Luis
>>
>>
>> -----------------------------
>> Mantis Bug Tracker wrote:
>>>
>>> The following issue has been set as RELATED TO issue 0008476.
>>> ======================================================================
>>> http://public.kitware.com/Bug/view.php?id=5454
>>> ======================================================================
>>> Reported By:                sean
>>> Assigned To:                luis.ibanez
>>> ======================================================================
>>> Project:                    ITK
>>> Issue ID:                   5454
>>> Category:                   Reproducibility:            N/A
>>> Severity:                   minor
>>> Priority:                   normal
>>> Status:                     feedback
>>> ======================================================================
>>> Date Submitted:             2007-08-07 13:59 EDT
>>> Last Modified:              2009-02-06 15:26 EST
>>> ======================================================================
>>> Summary:                    Support Philips PAR/REC image files in ITK
>>> Description: ITK supports a wide variety of medical imaging file formats,
>>> such as DICOM,
>>> Analyse, NifTI, etc.
>>>
>>> I request that it also support Philips PAR/REC image files.
>>>
>>> Don Bigler wrote some classes that do exactly this, available here:
>>> http://cnmrr.hmc.psu.edu/itkCNMRRLibrary/html/
>>>
>>> Integrating them into ITK is perhaps the best approach, though I have not
>>> yet tried his classes.
>>> ======================================================================
>>> Relationships       ID      Summary
>>> ----------------------------------------------------------------------
>>> related to          0008476 itkPhilipsRECImageIO throws an exceptio...
>>> ======================================================================
>>> ----------------------------------------------------------------------
>>>  (0010269) luis.ibanez (manager) - 2008-01-23 16:41
>>>  http://public.kitware.com/Bug/view.php?id=5454#c10269
>>> ---------------------------------------------------------------------- The
>>> best way to proceed will be to suggest Don Bigler to submit his classes
>>> to the Insight Journal, and from them we will follow the standard
>>> procedure
>>> for adding new classes to ITK.
>>> ----------------------------------------------------------------------
>>>  (0012206) dcb210 (developer) - 2008-06-03 00:12
>>>  http://public.kitware.com/Bug/view.php?id=5454#c12206
>>> ----------------------------------------------------------------------
>>> Philips PAR/REC ImageIO classes have been submitted to the Insight
Journal.
>>>  They can be found at the following URL http://hdl.handle.net/1926/1381
>>> ----------------------------------------------------------------------
>>>  (0012332) dcb210 (developer) - 2008-06-12 14:10
>>>  http://public.kitware.com/Bug/view.php?id=5454#c12332
>>> ----------------------------------------------------------------------
>>> Philips PAR/REC ImageIO classes have been submitted to the Insight Journal
>>> and are pending acceptance as new classes to ITK.
>>> ----------------------------------------------------------------------
>>>  (0012390) sean (developer) - 2008-06-17 19:11
>>>  http://public.kitware.com/Bug/view.php?id=5454#c12390
>>> ---------------------------------------------------------------------- I
>>> think it makes more sense to keep this bug open until its actually in ITK
>>> and working well on the various dashboards. With the bug closed, we may
>>> forget... :)
>>> Issue History Date Modified    Username       Field
>>>  Change
>>> ======================================================================
>>> 2007-08-07 13:59 sean           New Issue
>>>  2007-08-14 17:01 mcoursolle     Issue Monitored: mcoursolle
>>>    2007-10-17 10:10 sean           Description Updated
>>>    2008-01-23 16:41 luis.ibanez    Note Added: 0010269
>>>    2008-01-23 16:47 luis.ibanez    Status                   new =>
assigned
>>>     2008-01-23 16:47 luis.ibanez    Assigned To               =>
>luis.ibanez
>>>     2008-06-03 00:12 dcb210         Note Added: 0012206
>>>      2008-06-12 14:10 dcb210         Status                   assigned =>
>>> resolved
>>> 2008-06-12 14:10 dcb210         Resolution               open => fixed
>>>   2008-06-12 14:10 dcb210         Note Added: 0012332
>>>    2008-06-17 19:11 sean           Status                   resolved =>
>>> feedback
>>> 2008-06-17 19:11 sean           Resolution               fixed => reopened
>>>   2008-06-17 19:11 sean           Note Added: 0012390
>>>    2009-02-06 15:26 sean           Relationship added       related to
>>> 0008476
>>>  ======================================================================