Thanks for the response Luis. I'm still confused though because none of the other pixel accessors use the Set() method for inverse operations, e.g., <span class="Apple-style-span" style="font-family: arial, sans-serif; font-size: 13px; border-collapse: collapse; ">SqrtPixelAccessor, all of the trigonometric accessors, etc. I wrote some code using SqrtPixelAdaptor</span><span class="Apple-style-span" style="font-family: arial, sans-serif; border-collapse: collapse; "> (AddPixelAccessor will not compile, see below) to test the Set() and Get() methods. </span><div>
<div><font class="Apple-style-span" face="arial, sans-serif"><span class="Apple-style-span" style="border-collapse: collapse;"><br></span></font></div><div><font class="Apple-style-span" face="arial, sans-serif"><span class="Apple-style-span" style="border-collapse: collapse;">For an image set to 100: </span></font></div>
<div><font class="Apple-style-span" face="arial, sans-serif"><span class="Apple-style-span" style="border-collapse: collapse;">- Get() returns 10 as expected. </span></font></div><div><font class="Apple-style-span" face="arial, sans-serif"><span class="Apple-style-span" style="border-collapse: collapse;">- Set(10) sets the pixel to sqrt(10), and followed by Get() returns the sqrt(sqrt(10)), which is what you warned me about. </span></font><span class="Apple-style-span" style="font-family: arial, sans-serif; border-collapse: collapse; ">But is this the intended use of Set()?</span></div>
<div><font class="Apple-style-span" face="arial, sans-serif"><span class="Apple-style-span" style="border-collapse: collapse;"><br></span></font></div><div><font class="Apple-style-span" face="arial, sans-serif"><span class="Apple-style-span" style="border-collapse: collapse;">Back to your example with AddPixelAccessor:</span></font></div>
<div><font class="Apple-style-span" face="arial, sans-serif"><span class="Apple-style-span" style="border-collapse: collapse;">It is my understanding that image adaptors behave like images. So SetPixel would set the pixel value according to the pixel accessor used. So, by your example, SetPixel(12) would ultimately just set the pixel to "7", which is the value of the input image. It allows Get() to be used correctly, but doesn't seem too useful.</span></font></div>
<div><font class="Apple-style-span" face="arial, sans-serif"><span class="Apple-style-span" style="border-collapse: collapse;"><br></span></font></div><div><font class="Apple-style-span" face="arial, sans-serif"><span class="Apple-style-span" style="border-collapse: collapse;">Considering that itkAddPixelAccessor will not compile and there is no test for it, and that its Set() method does not follow the other Accessors, is it possible that the functionality for Set() was changed some time ago and </span></font><span class="Apple-style-span" style="font-family: arial, sans-serif; border-collapse: collapse; ">itkAddPixelAccessor fell through the cracks?</span></div>
<div><span class="Apple-style-span" style="font-family: arial, sans-serif; border-collapse: collapse; "><br></span></div><div><span class="Apple-style-span" style="font-family: arial, sans-serif; border-collapse: collapse; ">Thanks.</span></div>
<div><span class="Apple-style-span" style="font-family: arial, sans-serif; border-collapse: collapse; "><br></span></div><div><font class="Apple-style-span" face="arial, sans-serif"><span class="Apple-style-span" style="border-collapse: collapse;">********</span></font></div>
<div><div><div><font class="Apple-style-span" face="arial, sans-serif"><span class="Apple-style-span" style="border-collapse: collapse;"><div><span class="Apple-style-span" style="border-collapse: separate; font-family: arial; ">I attempted to write test code for AddPixelAccessor, but am getting compile errors:</span></div>
</span></font></div><div>typedef itk::AddPixelAccessor<float> AddAccessorType;</div><div><div>itkAddPixelAccesor.h: No such file or directory </div></div><div><br></div><div>I made a copy of itkAddPixelAccesor.h (itkTestAddPixelAccessor.h), and received the following error:</div>
<div><div>error: expected initializer before'<' token</div><div>which usually means the header wasn't included...</div><div><br></div><div>I moved #include "itkTestAddPixelAccessor.h" to the top of my include list and got the following error:</div>
<div><div>itkTestAddPixelAccessor.h:39: error: invalid function declaration</div></div><div><br></div><div>Consider me perplexed...</div><div><br></div><div>On Fri, Dec 10, 2010 at 5:53 AM, Luis Ibanez <span dir="ltr"><<a href="mailto:luis.ibanez@kitware.com">luis.ibanez@kitware.com</a>></span> wrote:</div>
<div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Hi Robert, David<br>
<br>
<br>
The implementation is correct.<br>
<br>
<br>
You are confusing the role of the Set() method with<br>
the role of the Get() method. One must apply the<br>
inverse operation of the other.<br>
<br>
<br>
An ImageAdaptor is intended to present an image<br>
as if the image had been processed by a filter.<br>
<br>
The PixelAccessor for that ImageAdaptor implements<br>
the trick of transforming the pixel on the fly.<br>
<br>
The AddImageAdaptor must present the image A<br>
as A+k (where "k" is a constant that it is added to<br>
every pixel in A).<br>
<br>
B = A + k<br>
<br>
The addition of K is implemented in the "Get" method.<br>
<br>
So, if your input image has a pixel with value "7",<br>
and "k" has been set to "5",<br>
<br>
then calling "Get" in that pixel should give you "7+5=12"<br>
<br>
To be consistent, the "Set" operation must be the<br>
inverse of the "Get" operation.<br>
<br>
So, if you "Set" a pixel to value "12", the "Set" method<br>
must store in "A" the value "12-5 = 7".<br>
<br>
In that way, if you call "Get" later in that same pixel,<br>
you get your "12" back.<br>
<br>
<br>
Regards,<br>
<br>
<br>
Luis<br>
<br>
<br>
---------------------------------------------<br>
<div class="im">On Thu, Dec 9, 2010 at 4:33 PM, robert tamburo <<a href="mailto:robert.tamburo@gmail.com">robert.tamburo@gmail.com</a>> wrote:<br>
</div><div><div></div><div class="h5">> I'm confused with the set function of AddPixelAccessor. It subtracts a value<br>
> from the input rather than adding it.<br>
> Relevant code:<br>
> inline void Set( InternalType & output, const ExternalType & input ) const<br>
> { output = static_cast<InternalType>( input - m_Value ); }<br>
> Shouldn't it add m_Value to the input? Or is there something else going on?<br>
> -Robert<br>
><br>
</div></div><div><div></div><div class="h5">> _____________________________________<br>
> Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
><br>
> Visit other Kitware open-source projects at<br>
> <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
><br>
> Kitware offers ITK Training Courses, for more information visit:<br>
> <a href="http://www.kitware.com/products/protraining.html" target="_blank">http://www.kitware.com/products/protraining.html</a><br>
><br>
> Please keep messages on-topic and check the ITK FAQ at:<br>
> <a href="http://www.itk.org/Wiki/ITK_FAQ" target="_blank">http://www.itk.org/Wiki/ITK_FAQ</a><br>
><br>
> Follow this link to subscribe/unsubscribe:<br>
> <a href="http://www.itk.org/mailman/listinfo/insight-users" target="_blank">http://www.itk.org/mailman/listinfo/insight-users</a><br>
><br>
><br>
</div></div></blockquote></div><br></div></div></div></div>