<div dir="ltr">Also sorry I did not review that gerrit topic. Now I notice a number of tests were deleted. That should be a red flag.<br><br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jul 15, 2013 at 4:35 PM, Bill Lorensen <span dir="ltr"><<a href="mailto:bill.lorensen@gmail.com" target="_blank">bill.lorensen@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">I am also sorry I did not check the wiki examples until now. I will set up a nightly examples build against a nightly ITK.<br>
<br></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jul 15, 2013 at 4:33 PM, Johnson, Hans J <span dir="ltr"><<a href="mailto:hans-johnson@uiowa.edu" target="_blank">hans-johnson@uiowa.edu</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I tested backwards compatibility with Slicer, ANTS, BRAINSTools, DTIPrep,<br>
DTIProceses, and ITK tests. All those passed so we thought we were good<br>
to go.<br>
<br>
Sorry about this regression. Ali is looking into it.<br>
<div><br>
Hans<br>
<br>
<br>
-----Original Message-----<br>
From: Matt McCormick <<a href="mailto:matt.mccormick@kitware.com" target="_blank">matt.mccormick@kitware.com</a>><br>
</div><div>Date: Monday, July 15, 2013 3:31 PM<br>
To: Bill Lorensen <<a href="mailto:bill.lorensen@gmail.com" target="_blank">bill.lorensen@gmail.com</a>><br>
</div><div><div>Cc: Hans Johnson <<a href="mailto:hans-johnson@uiowa.edu" target="_blank">hans-johnson@uiowa.edu</a>>, Ali Ghayoor<br>
<<a href="mailto:ali-ghayoor@uiowa.edu" target="_blank">ali-ghayoor@uiowa.edu</a>>, ITK <<a href="mailto:insight-developers@itk.org" target="_blank">insight-developers@itk.org</a>><br>
Subject: Re: [Insight-developers] Recent changes to Transforms break ITK's<br>
API<br>
<br>
On Mon, Jul 15, 2013 at 8:00 PM, Bill Lorensen <<a href="mailto:bill.lorensen@gmail.com" target="_blank">bill.lorensen@gmail.com</a>><br>
wrote:<br>
> Are these API breaking in the 4.4 release? If not, we can wait to fix<br>
>them.<br>
<br>
Right - the 4.4 release is unaffected, but we will want to make sure<br>
4.5.0 is backwards compatible.<br>
<br>
Thanks,<br>
Matt<br>
<br>
<br>
><br>
><br>
> On Mon, Jul 15, 2013 at 2:02 PM, Johnson, Hans J <<a href="mailto:hans-johnson@uiowa.edu" target="_blank">hans-johnson@uiowa.edu</a>><br>
> wrote:<br>
>><br>
>> Adding Ali to the conversation.<br>
>><br>
>> Hans<br>
>><br>
>><br>
>> -----Original Message-----<br>
>> From: Matt McCormick <<a href="mailto:matt.mccormick@kitware.com" target="_blank">matt.mccormick@kitware.com</a>><br>
>> Date: Monday, July 15, 2013 12:53 PM<br>
>> To: Bill Lorensen <<a href="mailto:bill.lorensen@gmail.com" target="_blank">bill.lorensen@gmail.com</a>><br>
>> Cc: ITK <<a href="mailto:insight-developers@itk.org" target="_blank">insight-developers@itk.org</a>><br>
>> Subject: Re: [Insight-developers] Recent changes to Transforms break<br>
>> ITK's API<br>
>><br>
>> Hi Bill,<br>
>><br>
>> I have added your code in this patch:<br>
>><br>
>> <a href="http://review.source.kitware.com/#/c/11961/" target="_blank">http://review.source.kitware.com/#/c/11961/</a><br>
>><br>
>> It reproduces the compilation error.<br>
>><br>
>><br>
>> To overcome this limitation, I propose we:<br>
>><br>
>> 1) SetInput( TransformType * ) -> SetInput( LightObject * )<br>
>> 2) Create a new TransformPrecisionFilter that can convert between<br>
>> transform precisions<br>
>> 3) Internally convert between precisions if required in<br>
>> TransformFileWriter::Update()<br>
>><br>
>> Thanks,<br>
>> Matt<br>
>><br>
>><br>
>> On Sun, Jul 14, 2013 at 3:50 PM, Bill Lorensen <<a href="mailto:bill.lorensen@gmail.com" target="_blank">bill.lorensen@gmail.com</a>><br>
>> wrote:<br>
>> > BTW, if I change the type to double it compiles.<br>
>> ><br>
>> ><br>
>> ><br>
>> > On Sun, Jul 14, 2013 at 11:50 AM, Bill Lorensen<br>
>> > <<a href="mailto:bill.lorensen@gmail.com" target="_blank">bill.lorensen@gmail.com</a>><br>
>> > wrote:<br>
>> >><br>
>> >> I think it works with 4.5.0(~ May 31). Probably fails because of this<br>
>> >> commit:<br>
>> >> <a href="http://review.source.kitware.com/#/c/11396/" target="_blank">http://review.source.kitware.com/#/c/11396/</a><br>
>> >><br>
>> >><br>
>> >><br>
>> >><br>
>> >><br>
>> >> On Sun, Jul 14, 2013 at 11:40 AM, Matt McCormick<br>
>> >> <<a href="mailto:matt.mccormick@kitware.com" target="_blank">matt.mccormick@kitware.com</a>> wrote:<br>
>> >>><br>
>> >>> Hi Bill,<br>
>> >>><br>
>> >>> Did this code with 4.4.0?<br>
>> >>><br>
>> >>> Thanks,<br>
>> >>> Matt<br>
>> >>><br>
>> >>> On Sun, Jul 14, 2013 at 11:31 AM, Bill Lorensen<br>
>> >>><<a href="mailto:bill.lorensen@gmail.com" target="_blank">bill.lorensen@gmail.com</a>><br>
>> >>> wrote:<br>
>> >>> > Folks,<br>
>> >>> ><br>
>> >>> > When I compile the following code I get this compilation error:<br>
>> >>> ><br>
>> >>> ><br>
>> >>> ><br>
>><br>
>> >>><br>
>>>>>/home/lorensen/ProjectsGIT/ITKGerrit/Modules/Remote/WikiExamples/IO/Tr<br>
>>>>>an<br>
>> >>>sformFileWriter.cxx:<br>
>> >>> > In function int main(int, char**)¹:<br>
>> >>> ><br>
>> >>> ><br>
>><br>
>> >>><br>
>>>>>/home/lorensen/ProjectsGIT/ITKGerrit/Modules/Remote/WikiExamples/IO/Tr<br>
>>>>>an<br>
>> >>>sformFileWriter.cxx:20:<br>
>> >>> > error: no matching function for call to<br>
>> >>> ><br>
>> >>> ><br>
>><br>
>> >>><br>
>>>>>itk::TransformFileWriterTemplate<double>::SetInput(itk::SmartPointer<<br>
>>>>>it<br>
>> >>>k::Rigid2DTransform<float><br>
>> >>> >>&)¹<br>
>> >>> ><br>
>> >>> ><br>
>><br>
>> >>><br>
>>>>>/home/lorensen/ProjectsGIT/ITKGerrit/Modules/IO/TransformBase/include/<br>
>>>>>it<br>
>> >>>kTransformFileWriter.hxx:78:<br>
>> >>> > note: candidates are: void<br>
>> >>> > itk::TransformFileWriterTemplate<ScalarType>::SetInput(const<br>
>> >>> > itk::TransformBaseTemplate<TScalarType>*) [with ScalarType =<br>
>>double]<br>
>> >>> ><br>
>> >>> ><br>
>> >>> ><br>
>><br>
>> >>><br>
>>>>>----------------------------------------------------------------------<br>
>>>>>--<br>
>> >>>---------------------------<br>
>> >>> > #include "itkRigid2DTransform.h"<br>
>> >>> > #include "itkTransformFileWriter.h"<br>
>> >>> ><br>
>> >>> > int main(int argc, char *argv[])<br>
>> >>> > {<br>
>> >>> > std::string fileName;<br>
>> >>> > if(argc == 1) // No arguments were provided<br>
>> >>> > {<br>
>> >>> > fileName = "test.tfm";<br>
>> >>> > }<br>
>> >>> > else<br>
>> >>> > {<br>
>> >>> > fileName = argv[1];<br>
>> >>> > }<br>
>> >>> ><br>
>> >>> > typedef itk::Rigid2DTransform< float > TransformType;<br>
>> >>> > TransformType::Pointer transform = TransformType::New();<br>
>> >>> ><br>
>> >>> > itk::TransformFileWriter::Pointer writer =<br>
>> >>> > itk::TransformFileWriter::New();<br>
>> >>> > writer->SetInput(transform);<br>
>> >>> > writer->SetFileName(fileName);<br>
>> >>> > writer->Update();<br>
>> >>> ><br>
>> >>> > return EXIT_SUCCESS;<br>
>> >>> > }<br>
>> >>> ><br>
>> >>> ><br>
>> >>> > _______________________________________________<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://kitware.com/products/protraining.php" target="_blank">http://kitware.com/products/protraining.php</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-developers" target="_blank">http://www.itk.org/mailman/listinfo/insight-developers</a><br>
>> >>> ><br>
>> >><br>
>> >><br>
>> >><br>
>> >><br>
>> >> --<br>
>> >> Unpaid intern in BillsBasement at noware dot com<br>
>> ><br>
>> ><br>
>> ><br>
>> ><br>
>> > --<br>
>> > Unpaid intern in BillsBasement at noware dot com<br>
>> _______________________________________________<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://kitware.com/products/protraining.php" target="_blank">http://kitware.com/products/protraining.php</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-developers" target="_blank">http://www.itk.org/mailman/listinfo/insight-developers</a><br>
>><br>
>><br>
>><br>
>> ________________________________<br>
>> Notice: This UI Health Care e-mail (including attachments) is covered by<br>
>> the Electronic Communications Privacy Act, 18 U.S.C. 2510-2521, is<br>
>> confidential and may be legally privileged. If you are not the intended<br>
>> recipient, you are hereby notified that any retention, dissemination,<br>
>> distribution, or copying of this communication is strictly prohibited.<br>
>> Please reply to the sender that you have received the message in error,<br>
>>then<br>
>> delete it. Thank you.<br>
>> ________________________________<br>
><br>
><br>
><br>
><br>
> --<br>
> Unpaid intern in BillsBasement at noware dot com<br>
<br>
<br>
<br>
________________________________<br>
Notice: This UI Health Care e-mail (including attachments) is covered by the Electronic Communications Privacy Act, 18 U.S.C. 2510-2521, is confidential and may be legally privileged. If you are not the intended recipient, you are hereby notified that any retention, dissemination, distribution, or copying of this communication is strictly prohibited. Please reply to the sender that you have received the message in error, then delete it. Thank you.<br>
________________________________<br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>Unpaid intern in BillsBasement at noware dot com<br>
</div>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>Unpaid intern in BillsBasement at noware dot com<br>
</div>