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

[Bill Lorensen]

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
>>  ======================================================================
>>
>>
> _______________________________________________
> Powered by www.kitware.com
>
> Visit other Kitware open-source projects at
> http://www.kitware.com/opensource/opensource.html
>
> Please keep messages on-topic and check the ITK FAQ at:
> http://www.itk.org/Wiki/ITK_FAQ
>
> Follow this link to subscribe/unsubscribe:
> http://www.itk.org/mailman/listinfo/insight-developers
>