[Insight-developers] [master] Change Ica45be1d: (ITK) ENH: added an API for setting/getting the mask

Jim Miller millerjv at ge.com
Fri Aug 27 10:30:11 EDT 2010


(This is in reference to kicking the tires on Gerritt, a code review system.  I constructed a small patch and submitted the patch to Gerritt which Dan has now verified and reviewed.)

Okay... so now what do we do?

I am also confused as to how Gerritt is being used.  It looks like people (both Mathieu and Brad L) have submitted multiple patches referencing the same topic branch.  Should each patch be a unique topic branch? Or doesn't it matter? 

Jim

On Aug 27, 2010, at 9:37 AM, Daniel Blezek (Code Review) wrote:

> Comments on Patch Set 1:
> 
> Patch Set 1: Verified; Looks good to me, approved
> 
> OK.
> 
> To respond, visit http://review.source.kitware.com/8
> 
> -- 
> To view visit http://review.source.kitware.com/8
> To unsubscribe, visit http://review.source.kitware.com/settings
> 
> Gerrit-MessageType: comment
> Gerrit-Project: ITK
> Gerrit-Branch: master
> Gerrit-Owner: Jim Miller <millerjv at ge.com>
> Gerrit-Reviewer: Daniel Blezek <blezek.daniel at mayo.edu>

Jim Miller
Senior Scientist
GE Research 
Interventional and Therapy

GE imagination at work

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.itk.org/mailman/private/insight-developers/attachments/20100827/4f7cc891/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 3108 bytes
Desc: not available
URL: <http://www.itk.org/mailman/private/insight-developers/attachments/20100827/4f7cc891/attachment.bin>


More information about the Insight-developers mailing list