<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">
<div class=""><br class="">
</div>
The reviewer though will likely never need to download the data and run the analysis whereas someone elese will need to compile and run the code.
<div class="">Just a thought.<br class="">
<div class="">
<div dir="auto" style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0); letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">
Regards,<br class="">
-David<br class="">
<br class="">
-------------------------------------------------------------<br class="">
David Lawrence Ph.D.<br class="">
Staff Scientist, Thomas Jefferson National Accelerator Facility<br class="">
Newport News, VA<br class="">
<a href="mailto:davidl@jlab.org" class="">davidl@jlab.org</a><br class="">
(757) 269-5567 W<br class="">
(757) 746-6697 C<br class="">
<br class="">
</div>
</div>
<div><br class="">
<blockquote type="cite" class="">
<div class="">On Apr 29, 2020, at 10:32 AM, Richard Jones <<a href="mailto:richard.t.jones@uconn.edu" class="">richard.t.jones@uconn.edu</a>> wrote:</div>
<br class="Apple-interchange-newline">
<div class="">
<div dir="ltr" style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class="">
Or how about just emailing the author and asking, have you done this? Or can you do this one more test, and I will approve it. Something like the way peer review of articles works. Usually the reviewer does not download the data and carry out independent checks
of the results.
<div class=""><br class="">
</div>
<div class="">-rtj</div>
</div>
<br style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class="">
<div class="gmail_quote" style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;">
<div dir="ltr" class="gmail_attr">On Wed, Apr 29, 2020 at 9:37 AM Naomi Jarvis <<a href="mailto:nsj@cmu.edu" class="">nsj@cmu.edu</a>> wrote:<br class="">
</div>
<blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-style: solid; border-left-color: rgb(204, 204, 204); padding-left: 1ex;">
<div class="">
<div class=""><br class="webkit-block-placeholder">
</div>
<div style="background-color: rgb(255, 235, 156); width: 734.71875px; border: 1pt none rgb(250, 235, 204); padding: 10pt; font-size: 11pt; line-height: 12pt; font-family: Calibri; text-align: left;" class="">
<span style="color: rgb(156, 101, 0);" class=""></span>*Message sent from a system outside of UConn.*</div>
<br class="">
<div class=""><br class="webkit-block-placeholder">
</div>
<div class="">
<div dir="ltr" class="">I thought the review process would be something like download the new branch, make sure it compiles, make sure it runs, make sure it does whatever it was supposed to do, then go to the website and click the merge button (and presumably
one more now). That's what I do before approving PRs (& why I don't approve many).
<div class=""><br class="">
</div>
<div class="">Naomi. </div>
</div>
<br class="">
<div class="gmail_quote">
<div dir="ltr" class="gmail_attr">On Wed, Apr 29, 2020 at 9:00 AM Sean Dobbs <<a href="mailto:sdobbs@fsu.edu" target="_blank" class="">sdobbs@fsu.edu</a>> wrote:<br class="">
</div>
<blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-style: solid; border-left-color: rgb(204, 204, 204); padding-left: 1ex;">
<div class="">
<div dir="auto" class="">We could just start with a short comment like “looks good” for now, and then revisit the question next software meeting when more people are familiar with what functionality is available. </div>
</div>
<div dir="auto" class=""><br class="">
</div>
<div dir="auto" class="">Cheers</div>
<div dir="auto" class="">Sean</div>
<div class=""><br class="">
<div class="gmail_quote">
<div dir="ltr" class="gmail_attr">On Wed, Apr 29, 2020 at 8:46 AM Mark Ito <<a href="mailto:marki@jlab.org" target="_blank" class="">marki@jlab.org</a>> wrote:<br class="">
</div>
<blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-style: solid; border-left-color: rgb(204, 204, 204); padding-left: 1ex;">
Mark,<br class="">
<br class="">
That is a good point, well taken. Anyone who was at the meeting<span class="Apple-converted-space"> </span><br class="">
yesterday care to take this one?<br class="">
<br class="">
-- Mark<br class="">
<br class="">
On 4/29/20 8:08 AM, Mark-Macrae Dalton wrote:<br class="">
> It might be useful to spell out more clearly what is expected from a review of a pull request, other than clicking the button.<br class="">
><br class="">
> Mark<br class="">
><br class="">
>> On Apr 28, 2020, at 8:06 PM, Mark Ito <<a href="mailto:marki@jlab.org" target="_blank" class="">marki@jlab.org</a>> wrote:<br class="">
>><br class="">
>> Cool. Good to know.<br class="">
>><br class="">
>> On 4/28/20 7:59 PM, Justin Stevens wrote:<br class="">
>>> FYI, just successfully executed the review -> merge PR sequence. There’s basically just one more button to push to “Approve” the review, and then you can merge as usual.<br class="">
>>> -Justin<br class="">
>>><br class="">
>>>> On Apr 28, 2020, at 5:04 PM, Mark Ito <<a href="mailto:marki@jlab.org" target="_blank" class="">marki@jlab.org</a>> wrote:<br class="">
>>>><br class="">
>>>> Folks,<br class="">
>>>><br class="">
>>>> At the software meeting just now, we decided to require "reviews" of pull requests before they can be merged onto the master branch. For now this is only for halld_sim and halld_recon. The webpages for the individual pull requests now reflect this. The
idea is to prevent inadvertent changes to the master branches.<br class="">
>>>><br class="">
>>>> -- Mark<br class="">
>>>><br class="">
>>>><br class="">
>>>> _______________________________________________<br class="">
>>>> Halld-offline mailing list<br class="">
>>>><span class="Apple-converted-space"> </span><a href="mailto:Halld-offline@jlab.org" target="_blank" class="">Halld-offline@jlab.org</a><br class="">
>>>><span class="Apple-converted-space"> </span><a href="https://urldefense.proofpoint.com/v2/url?u=https-3A__nam10.safelinks.protection.outlook.com_-3Furl-3Dhttps-253A-252F-252Fmailman.jlab.org-252Fmailman-252Flistinfo-252Fhalld-2Doffline-26data-3D02-257C01-257Crichard.t.jones-2540uconn.edu-257C27fd9227d73c4da2c00a08d7ec41b3d3-257C17f1a87e2a254eaab9df9d439034b080-257C0-257C0-257C637237639251846251-26sdata-3DgkJIGEsJ703IQhrHeWKZ-252FMkMnKATp4BMnhUAJVqvY2A-253D-26reserved-3D0&d=DwMFaQ&c=CJqEzB1piLOyyvZjb8YUQw&r=Te_hCR4EUlJ6iCDYLJ8Viv2aDOR7D9ZZMoBAvf2H0M4&m=s1rydy2wtTm4oNwi0LnpZYCnXrxxs7xi6nYvhktvqrw&s=Uo92_uGUaoF8XYALdVBVoyrm-crKFl56IQgWHhHVHVU&e=" rel="noreferrer" target="_blank" class="">https://mailman.jlab.org/mailman/listinfo/halld-offline</a><br class="">
>> _______________________________________________<br class="">
>> Halld-offline mailing list<br class="">
>><span class="Apple-converted-space"> </span><a href="mailto:Halld-offline@jlab.org" target="_blank" class="">Halld-offline@jlab.org</a><br class="">
>><span class="Apple-converted-space"> </span><a href="https://urldefense.proofpoint.com/v2/url?u=https-3A__nam10.safelinks.protection.outlook.com_-3Furl-3Dhttps-253A-252F-252Fmailman.jlab.org-252Fmailman-252Flistinfo-252Fhalld-2Doffline-26data-3D02-257C01-257Crichard.t.jones-2540uconn.edu-257C27fd9227d73c4da2c00a08d7ec41b3d3-257C17f1a87e2a254eaab9df9d439034b080-257C0-257C0-257C637237639251846251-26sdata-3DgkJIGEsJ703IQhrHeWKZ-252FMkMnKATp4BMnhUAJVqvY2A-253D-26reserved-3D0&d=DwMFaQ&c=CJqEzB1piLOyyvZjb8YUQw&r=Te_hCR4EUlJ6iCDYLJ8Viv2aDOR7D9ZZMoBAvf2H0M4&m=s1rydy2wtTm4oNwi0LnpZYCnXrxxs7xi6nYvhktvqrw&s=Uo92_uGUaoF8XYALdVBVoyrm-crKFl56IQgWHhHVHVU&e=" rel="noreferrer" target="_blank" class="">https://mailman.jlab.org/mailman/listinfo/halld-offline</a><br class="">
_______________________________________________<br class="">
Halld-offline mailing list<br class="">
<a href="mailto:Halld-offline@jlab.org" target="_blank" class="">Halld-offline@jlab.org</a><br class="">
<a href="https://urldefense.proofpoint.com/v2/url?u=https-3A__nam10.safelinks.protection.outlook.com_-3Furl-3Dhttps-253A-252F-252Fmailman.jlab.org-252Fmailman-252Flistinfo-252Fhalld-2Doffline-26data-3D02-257C01-257Crichard.t.jones-2540uconn.edu-257C27fd9227d73c4da2c00a08d7ec41b3d3-257C17f1a87e2a254eaab9df9d439034b080-257C0-257C0-257C637237639251856246-26sdata-3D0acpsNjdYzHIez-252BvBUDisiD6Sj8p6oM78XUE685C98I-253D-26reserved-3D0&d=DwMFaQ&c=CJqEzB1piLOyyvZjb8YUQw&r=Te_hCR4EUlJ6iCDYLJ8Viv2aDOR7D9ZZMoBAvf2H0M4&m=s1rydy2wtTm4oNwi0LnpZYCnXrxxs7xi6nYvhktvqrw&s=uVv3yNkBzGPBbTo82Q8EhRnOB6xhe0NLTA_aK_-6WSM&e=" rel="noreferrer" target="_blank" class="">https://mailman.jlab.org/mailman/listinfo/halld-offline</a></blockquote>
</div>
</div>
_______________________________________________<br class="">
Halld-offline mailing list<br class="">
<a href="mailto:Halld-offline@jlab.org" target="_blank" class="">Halld-offline@jlab.org</a><br class="">
<a href="https://urldefense.proofpoint.com/v2/url?u=https-3A__nam10.safelinks.protection.outlook.com_-3Furl-3Dhttps-253A-252F-252Fmailman.jlab.org-252Fmailman-252Flistinfo-252Fhalld-2Doffline-26data-3D02-257C01-257Crichard.t.jones-2540uconn.edu-257C27fd9227d73c4da2c00a08d7ec41b3d3-257C17f1a87e2a254eaab9df9d439034b080-257C0-257C0-257C637237639251856246-26sdata-3D0acpsNjdYzHIez-252BvBUDisiD6Sj8p6oM78XUE685C98I-253D-26reserved-3D0&d=DwMFaQ&c=CJqEzB1piLOyyvZjb8YUQw&r=Te_hCR4EUlJ6iCDYLJ8Viv2aDOR7D9ZZMoBAvf2H0M4&m=s1rydy2wtTm4oNwi0LnpZYCnXrxxs7xi6nYvhktvqrw&s=uVv3yNkBzGPBbTo82Q8EhRnOB6xhe0NLTA_aK_-6WSM&e=" rel="noreferrer" target="_blank" class="">https://mailman.jlab.org/mailman/listinfo/halld-offline</a></blockquote>
</div>
</div>
</div>
_______________________________________________<br class="">
Halld-offline mailing list<br class="">
<a href="mailto:Halld-offline@jlab.org" target="_blank" class="">Halld-offline@jlab.org</a><br class="">
<a href="https://urldefense.proofpoint.com/v2/url?u=https-3A__nam10.safelinks.protection.outlook.com_-3Furl-3Dhttps-253A-252F-252Fmailman.jlab.org-252Fmailman-252Flistinfo-252Fhalld-2Doffline-26amp-3Bdata-3D02-257C01-257Crichard.t.jones-2540uconn.edu-257C27fd9227d73c4da2c00a08d7ec41b3d3-257C17f1a87e2a254eaab9df9d439034b080-257C0-257C0-257C637237639251886232-26amp-3Bsdata-3DdoNOCLzRPLj59h9l4LLjZ04WFccf2s7KPmLa0vXRHEs-253D-26amp-3Breserved-3D0&d=DwMFaQ&c=CJqEzB1piLOyyvZjb8YUQw&r=Te_hCR4EUlJ6iCDYLJ8Viv2aDOR7D9ZZMoBAvf2H0M4&m=s1rydy2wtTm4oNwi0LnpZYCnXrxxs7xi6nYvhktvqrw&s=6e6-OfuqOOge3abxyj5Vicwdfy_vI-t762D8-aTxNxY&e=" rel="noreferrer" target="_blank" class="">https://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fmailman.jlab.org%2Fmailman%2Flistinfo%2Fhalld-offline&data=02%7C01%7Crichard.t.jones%40uconn.edu%7C27fd9227d73c4da2c00a08d7ec41b3d3%7C17f1a87e2a254eaab9df9d439034b080%7C0%7C0%7C637237639251886232&sdata=doNOCLzRPLj59h9l4LLjZ04WFccf2s7KPmLa0vXRHEs%3D&reserved=0</a></blockquote>
</div>
<span style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; float: none; display: inline !important;" class="">_______________________________________________</span><br style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class="">
<span style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; float: none; display: inline !important;" class="">Halld-offline
mailing list</span><br style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class="">
<a href="mailto:Halld-offline@jlab.org" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px;" class="">Halld-offline@jlab.org</a><br style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class="">
<a href="https://mailman.jlab.org/mailman/listinfo/halld-offline" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px;" class="">https://mailman.jlab.org/mailman/listinfo/halld-offline</a></div>
</blockquote>
</div>
<br class="">
</div>
</body>
</html>