<div dir="ltr"><div><div><div>Thanks Matt,<br><br></div>I've set the Git and Gerrit access. I believe everything's ok. I'll get to it next Tuesday.<br><br></div>Cheers,<br><br></div>Pol<br></div><div class="gmail_extra"><br><div class="gmail_quote">2015-04-02 15:22 GMT+02:00 Matt McCormick <span dir="ltr"><<a href="mailto:matt.mccormick@kitware.com" target="_blank">matt.mccormick@kitware.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Pol,<br>
<br>
Thanks for considering make improvements to this code.  Instructions<br>
to submit a patch can be found here [1].  Please let us know if you<br>
have any questions.<br>
<br>
Thanks,<br>
Matt<br>
<br>
[1] <a href="https://insightsoftwareconsortium.github.io/ITKBarCamp-doc/CommunitySoftwareProcess/SubmitAPatchToGerrit/index.html" target="_blank">https://insightsoftwareconsortium.github.io/ITKBarCamp-doc/CommunitySoftwareProcess/SubmitAPatchToGerrit/index.html</a><br>
<div class="HOEnZb"><div class="h5"><br>
On Thu, Apr 2, 2015 at 8:52 AM, Pol Monsó Purtí <<a href="mailto:lluna.nova@gmail.com">lluna.nova@gmail.com</a>> wrote:<br>
> Hello Gaëtan,<br>
><br>
> Yes I agree, Thinking about it later I also reached the same conclusion that<br>
> a filter that changes the regions internally without warning could be a<br>
> pitfall.<br>
><br>
> The RegionFromReferenceLabelMapFilter might actually work for me given that<br>
> I do have the original ROI region and roi image, but I can implement the<br>
> ComputeBoundingBox() function anyway, it's cleaner if I don't use the<br>
> original ROI.<br>
><br>
> What's the workflow for submitting code? Fork the itk repository, add the<br>
> method and PR or something else?<br>
><br>
> If you'd rather have a filter that does this, I unfortunatelly won't have<br>
> the time to do it this month (and maybe next)...<br>
><br>
> Cheers,<br>
><br>
> Pol<br>
><br>
><br>
><br>
> 2015-04-02 14:32 GMT+02:00 Gaetan Lehmann <<a href="mailto:gaetan.lehmann@gmail.com">gaetan.lehmann@gmail.com</a>>:<br>
>><br>
>> Hi Pol,<br>
>><br>
>> 2015-03-30 23:02 GMT+02:00 Pol Monsó Purtí <<a href="mailto:lluna.nova@gmail.com">lluna.nova@gmail.com</a>>:<br>
>>><br>
>>><br>
>>> Hello all,<br>
>>><br>
>>> I have two label objects, how can I merge the two BoundingBoxes together<br>
>>> so I know the necessary size of the image? (other than creating it myself by<br>
>>> searching max/min of each dimension)<br>
>>><br>
>>> How come LabelMap doesn't resize automatically when objects are added? It<br>
>>> could.<br>
>><br>
>><br>
>> The LabelMap class was designed to work as much as possible as the Image<br>
>> class, that does not have this kind of behavior, and to avoid as much as<br>
>> possible the costly operations.<br>
>><br>
>> So while I don't think this should be done automatically, we should have<br>
>> some methods or filters to make this easy to do.<br>
>><br>
>> There are a few filters to manipulate the regions of a LabelMap, but none<br>
>> of them does exactly what you need:<br>
>><br>
>> * PadLabelMapFilter that can enlarge the LabelMap region, but not based on<br>
>> its content<br>
>> * CropLabelMapFilter that can shrink the LabelMap region, but not based on<br>
>> its content<br>
>> * RegionFromReferenceLabelMapFilter that makes the LabelMap region match<br>
>> the region of a reference input image<br>
>> * AutoCropLabelMapFilter that reduce the region covered by a label map<br>
>> based on its content, but can't enlarge it<br>
>><br>
>> In my opinion, we should have a ComputeBoundingBox() method in LabelMap<br>
>> that compute the region covered by all the label objects - the code can be<br>
>> moved from AutoCropLabelMapFilter. We could then do<br>
>><br>
>>   labelMap->AddLabelObject(labelObject1);<br>
>>   labelMap->AddLabelObject(labelObject2);<br>
>>   labelMap->SetRegions(labelMap->ComputeBoundingBox());<br>
>><br>
>> to update the region.<br>
>><br>
>> Would you agree to implement such a method for the LabelMap class?<br>
>><br>
>> Optionally, we could also think to a filter to do that in a pipeline.<br>
>><br>
>> Regards,<br>
>><br>
>> Gaëtan<br>
>><br>
><br>
><br>
><br>
</div></div><div class="HOEnZb"><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.php" target="_blank">http://www.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://public.kitware.com/mailman/listinfo/insight-users" target="_blank">http://public.kitware.com/mailman/listinfo/insight-users</a><br>
><br>
</div></div></blockquote></div><br></div>