[Insight-developers] Add ImageSource Module

[Bill Lorensen]

Then maybe ImageSource should require the size, spacing, etc? If is
doesn't, then I'm confused.

I'm making these comments without looking at ImageSource code...

On Fri, Nov 4, 2011 at 1:23 PM, Bradley Lowekamp <blowekamp at mail.nih.gov>wrote:

> I don't like the name GeneratingImageSource either, but I don't think that
> ImageSourceBase is descriptive enough, ImageSource is already a base class
> and we should try to avoid confusion with that. I think that something to
> help describe that this class is for ImageSources that generate an image,
> or that require the size, spacing etc, to be set would be better then base.
>
> Brad
>
> On Nov 4, 2011, at 11:33 AM, Bill Lorensen wrote:
>
> Perhaps the new class should be called ImageSourceBase.
>
>
> On Fri, Nov 4, 2011 at 11:23 AM, Bradley Lowekamp <blowekamp at mail.nih.gov>wrote:
>
>> Hello,
>>
>> I am thinking I should create a base class "GeneratingImageSource", which
>> adds  setting the Size, Origin, Spacing, and Directions in the
>> GenerateOutputInformation.
>>
>> Create a new  ImageSource module.
>>
>> Place my new PhysicalPointImageSource class there.
>>
>> Move GaborImageSource to this location, out of review ( Perhaps this
>> should be derived from ParametricImageSource too like GaussianImageSource?).
>>
>> Look into Moving GaussianImageSource, GridImageSource, and
>> RandomImageSource.
>>
>
> RandomImageSource is in the TestingKernel module, that is likely best left
> there.
>
>
>> Any thought?
>>
>> Thanks,
>> Brad
>>
>>  ========================================================
>> Bradley Lowekamp
>> Medical Science and Computing for
>> Office of High Performance Computing and Communications
>> National Library of Medicine
>> blowekamp at mail.nih.gov
>>
>>
>>
>>
>> _______________________________________________
>> Powered by www.kitware.com
>>
>> Visit other Kitware open-source projects at
>> http://www.kitware.com/opensource/opensource.html
>>
>> Kitware offers ITK Training Courses, for more information visit:
>> http://kitware.com/products/protraining.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
>>
>>
>
>
> --
> Unpaid intern in BillsBasement at noware dot com
>
>
> ========================================================
>
> Bradley Lowekamp
>
> Medical Science and Computing for
>
> Office of High Performance Computing and Communications
>
> National Library of Medicine
>
> blowekamp at mail.nih.gov
>
>
>
>


-- 
Unpaid intern in BillsBasement at noware dot com
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.itk.org/mailman/private/insight-developers/attachments/20111104/4c0a4593/attachment.htm>