<html xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<meta name="Title" content="">
<meta name="Keywords" content="">
<meta name="Generator" content="Microsoft Word 15 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
{font-family:"Courier New";
panose-1:2 7 3 9 2 2 5 2 4 4;}
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
{font-family:"Avenir Next";
panose-1:2 11 5 3 2 2 2 2 2 4;}
@font-face
{font-family:PMingLiU;
panose-1:2 2 5 0 0 0 0 0 0 0;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0in;
margin-bottom:.0001pt;
font-size:12.0pt;
font-family:"Times New Roman";}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
{mso-style-priority:99;
color:purple;
text-decoration:underline;}
pre
{mso-style-priority:99;
mso-style-link:"HTML Preformatted Char";
margin:0in;
margin-bottom:.0001pt;
font-size:10.0pt;
font-family:"Courier New";}
span.HTMLPreformattedChar
{mso-style-name:"HTML Preformatted Char";
mso-style-priority:99;
mso-style-link:"HTML Preformatted";
font-family:Courier;}
span.EmailStyle19
{mso-style-type:personal-reply;
font-family:"Avenir Next";
color:windowtext;
font-weight:normal;
font-style:normal;
text-decoration:none none;}
span.msoIns
{mso-style-type:export-only;
mso-style-name:"";
text-decoration:underline;
color:teal;}
.MsoChpDefault
{mso-style-type:export-only;
font-size:10.0pt;}
@page WordSection1
{size:595.0pt 842.0pt;
margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
{page:WordSection1;}
--></style>
</head>
<body bgcolor="white" lang="EN-US" link="blue" vlink="purple">
<div class="WordSection1">
<p class="MsoNormal"><span style="font-family:"Avenir Next"">Thanks guys - I’ve pushed a patch to Gerrit and added you as reviewers.<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Avenir Next""><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Avenir Next"">Toby<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Avenir Next""><o:p> </o:p></span></p>
<div>
<div>
<p class="MsoNormal" style="margin-left:.5in">On 20/09/2017, 14:25, "Jon Haitz Legarreta" <<a href="mailto:jhlegarreta@vicomtech.org">jhlegarreta@vicomtech.org</a>> wrote:<o:p></o:p></p>
</div>
</div>
<div>
<p class="MsoNormal" style="margin-left:.5in"><o:p> </o:p></p>
</div>
<div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:0in;margin-right:0in;margin-bottom:12.0pt;margin-left:.5in">
+1 for Matt's proposal.<o:p></o:p></p>
</div>
<p class="MsoNormal" style="mso-margin-top-alt:0in;margin-right:0in;margin-bottom:12.0pt;margin-left:.5in">
JON HAITZ<o:p></o:p></p>
<div>
<div>
<div>
<p class="MsoNormal" style="margin-left:.5in"><br clear="all">
<o:p></o:p></p>
<div>
<div>
<div>
<div>
<div>
<div>
<div>
<div>
<div>
<div>
<div align="center">
<table class="MsoNormalTable" border="0" cellspacing="0" cellpadding="0">
<tbody>
<tr>
<td style="padding:0in 0in 0in 0in"></td>
</tr>
</tbody>
</table>
</div>
<pre style="mso-margin-top-alt:0in;margin-right:0in;margin-bottom:12.0pt;margin-left:.5in;word-wrap:break-word;white-space:pre-wrap"><span style="color:black">--<o:p></o:p></span></pre>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
<p class="MsoNormal" style="margin-left:.5in"><o:p> </o:p></p>
<div>
<p class="MsoNormal" style="margin-left:.5in">On 20 September 2017 at 15:16, Matt McCormick <<a href="mailto:matt.mccormick@kitware.com" target="_blank">matt.mccormick@kitware.com</a>> wrote:<o:p></o:p></p>
<blockquote style="border:none;border-left:solid #CCCCCC 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-right:0in">
<p class="MsoNormal" style="margin-left:.5in">Hi Toby,<br>
<br>
Thanks again for the contribution.<br>
<br>
Yes, a small follow-up patch in UseITK.cmake will do the trick. For a<br>
large addition like this, some follow-up effort is expected.<br>
<br>
In my opinion, "2dseq" is obscure enough that the addition of "Bruker"<br>
helps identify the class.<br>
<br>
Thanks,<br>
Matt<o:p></o:p></p>
<div>
<div>
<p class="MsoNormal" style="margin-left:.5in"><br>
On Wed, Sep 20, 2017 at 8:43 AM, Wood, Tobias <<a href="mailto:tobias.wood@kcl.ac.uk">tobias.wood@kcl.ac.uk</a>> wrote:<br>
> Hello,<br>
><br>
> I updated my local copy of ITK to master to test my Bruker patch that was merged a couple of weeks ago. Unfortunately my patch did not change the UseITK.cmake to follow the new name of the module (it was suggested to me that the module name be shortened in
case of anymore Bruker formats), so the Bruker reader is not correctly registered.<br>
><br>
> I think I have worked out the required changes to UseITK.cmake, so can submit a patch. However, this raises the question of whether the naming scheme could be rationalized further. Before, both module and class were called “Bruker2DSEQ”. The module is now
“Bruker”, and the class is “Bruker2dseq”, which is the correct capitalisation for the format. Should the class/format be shortened to just “2dseq”?<span style="font-family:PMingLiU"><br>
</span>><span style="font-family:PMingLiU"><br>
</span>> I’m sorry not to catch this before my original patch was merged. Changing the name was one of the last things I did, and because the tests explicitly register the ImageIOFactory, they would not pick this up.<br>
><br>
> Best wishes,<br>
> Toby<br>
><br>
> _______________________________________________<br>
> Community mailing list<br>
> <a href="mailto:Community@itk.org">Community@itk.org</a><br>
> <a href="https://emea01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fpublic.kitware.com%2Fmailman%2Flistinfo%2Fcommunity&data=01%7C01%7Ctobias.wood%40kcl.ac.uk%7C94f0e797bf884af63bd708d5002b2c49%7C8370cf1416f34c16b83c724071654356%7C0&sdata=t%2BXOpQcFsw5U20AZmmu8%2B%2B5dgCtavYRRj7%2BBM5nGYIs%3D&reserved=0" target="_blank">
http://public.kitware.com/mailman/listinfo/community</a><br>
_______________________________________________<br>
Community mailing list<br>
<a href="mailto:Community@itk.org">Community@itk.org</a><br>
<a href="https://emea01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fpublic.kitware.com%2Fmailman%2Flistinfo%2Fcommunity&data=01%7C01%7Ctobias.wood%40kcl.ac.uk%7C94f0e797bf884af63bd708d5002b2c49%7C8370cf1416f34c16b83c724071654356%7C0&sdata=t%2BXOpQcFsw5U20AZmmu8%2B%2B5dgCtavYRRj7%2BBM5nGYIs%3D&reserved=0" target="_blank">http://public.kitware.com/mailman/listinfo/community</a><o:p></o:p></p>
</div>
</div>
</blockquote>
</div>
<p class="MsoNormal" style="margin-left:.5in"><o:p> </o:p></p>
</div>
</div>
</div>
</div>
</div>
</body>
</html>