<html xmlns:v="urn:schemas-microsoft-com:vml" 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="Generator" content="Microsoft Word 15 (filtered medium)">
<style><!--
/* Font Definitions */
@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;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0in;
font-size:11.0pt;
font-family:"Calibri",sans-serif;}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
span.EmailStyle20
{mso-style-type:personal-reply;
font-family:"Calibri",sans-serif;
color:windowtext;}
.MsoChpDefault
{mso-style-type:export-only;
font-size:10.0pt;}
@page WordSection1
{size:8.5in 11.0in;
margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
{page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang="EN-US" link="blue" vlink="purple">
<div class="WordSection1">
<p class="MsoNormal">Hi Bill and others,<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">I just want reiterate what this is and what this is not.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Aidan is looking to create a group where interested people who would like to show their code, ask for help, and share ideas can do so. This is not mandatory, and it is not necessarily part of the PSWS project. Maybe “Code Review” is the
wrong title for this group, and “Code Sharing, Discussion, and Help” might be better. I see this as an especially good opportunity for students who are learning to code or do data analysis to be able to talk to each other and help each other out on any HamSCI-related
coding they might be doing. This might also be good for a non-student who has wanted to take up a new language, like Python, and be able to talk to others about what they are doing.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">I hope that there are people who will find this a helpful sharing session.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">73 de Nathaniel W2NAF<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal"><b>From:</b> TangerineSDR <tangerinesdr-bounces@lists.tapr.org>
<b>On Behalf Of </b>Engelke, Bill via TangerineSDR<br>
<b>Sent:</b> Sunday, August 30, 2020 3:04 PM<br>
<b>To:</b> TAPR TangerineSDR Modular Software Defined Radio <tangerinesdr@lists.tapr.org><br>
<b>Cc:</b> Engelke, Bill <bill.engelke@ua.edu><br>
<b>Subject:</b> Re: [TangerineSDR] [EXTERNAL] Re: Code Review<o:p></o:p></p>
</div>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Aiden – I don’t wish to be uncooperative, but this could be a significant increase in scope; so I will not be participating in any code reviews until directed to do so by the UA Principal Investigator (Dr. Travis Atkinson). If you want
to pursue this, please go thru the chain of command, i.e., ask Nathaniel (chief PI) to pass this new requirement to our team thru Dr. Atkinson. -73- Bill AB4EJ<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal"><b>From:</b> TangerineSDR <<a href="mailto:tangerinesdr-bounces@lists.tapr.org">tangerinesdr-bounces@lists.tapr.org</a>>
<b>On Behalf Of </b>Aidan Montare via TangerineSDR<br>
<b>Sent:</b> Sunday, August 30, 2020 11:03 AM<br>
<b>To:</b> <a href="mailto:hamsci@googlegroups.com">hamsci@googlegroups.com</a>; TAPR TangerineSDR Modular Software Defined Radio <<a href="mailto:tangerinesdr@lists.tapr.org">tangerinesdr@lists.tapr.org</a>><br>
<b>Cc:</b> Aidan Montare <<a href="mailto:aam141@case.edu">aam141@case.edu</a>><br>
<b>Subject:</b> [EXTERNAL] Re: [TangerineSDR] Code Review<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<div>
<div>
<p class="MsoNormal">Dear all,<o:p></o:p></p>
<div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Thanks everyone for sharing your thoughts on my proposal for hamsci code review sessions! I really appreciate the feedback.<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Please fill out the form (<a href="https://forms.gle/uiEHCqMG9v1BaQiW7" target="_blank">https://forms.gle/uiEHCqMG9v1BaQiW7</a>)
<b>by this Thursday</b> if you haven't already so that I can schedule a time. My college semester just began this week, so I mostly know my own schedule and can pick a time.<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Also, I wanted to provide some comments that will hopefully clarify a few things:<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">The code review sessions are intended to be pretty informal. I want a space for people to talk about programming in our projects, get help on topics they would like to discuss, and polish the aspects of their work they'd like to polish.<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">As Kristina pointed out, "code review" is a bit of a reserved word in certain groups, and often means a specific thing. I'm not trying to be so formal (perhaps I should have called them by a different name). I don't expect that anyone has
to follow any of the recommendations they get at any of the sessions. And I don't expect that all code written in HamSCI has to go through code review.<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Also, these sessions are half for the benefit of our projects, and half for everyone's individual benefit. Part of my motivation is the desire to personally improve my own abilities, and I hope that others will learn things as well. I appreciate
the sentiment that not all good ideas might be implemented now, but some might be worth remembering for the future. If someone decides that we can't implement an idea now, but learns something that can be applied in their future projects, I'd consider that
a success.<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Finally, I'm a bit of a nerd, and I do personally enjoy learning about things like programming standards. But my intent of code review is not to propose any coding standard or style guide.<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">I hope that makes my thoughts a bit clearer. I continue to appreciate the feedback, and enjoy the discussions. Hearing from everyone’s different experiences is one of my favorite parts of hamsci!<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><br>
-- <o:p></o:p></p>
<div>
<div>
<p class="MsoNormal">Sincerely,<br>
<br>
Aidan Montare<br>
CWRU Class of 2021<o:p></o:p></p>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</body>
</html>