<html><head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body>
<p>Folks,</p>
<p>Please find the minutes <a moz-do-not-send="true" href="https://halldweb.jlab.org/wiki/index.php/GlueX_Software_Meeting,_October_11,_2021#Minutes">here</a>
and below.</p>
<p> -- Mark</p>
<p> ___________________________________</p>
<p>
</p>
<div id="globalWrapper">
<div id="column-content">
<div id="content" class="mw-body" role="main">
<h2 id="firstHeading" class="firstHeading" lang="en"><span dir="auto">GlueX Software Meeting, October 11, 2021, </span><span class="mw-headline" id="Minutes">Minutes</span></h2>
<div id="bodyContent" class="mw-body-content">
<div id="mw-content-text" dir="ltr" class="mw-content-ltr" lang="en">
<p>Present: Alex Austregesilo, Thomas Britton, Sean Dobbs,
Mark Ito (chair), Richard Jones, Simon Taylor, Beni
Zihlmann
</p>
<p>There is a <a rel="nofollow" class="external text" href="https://bluejeans.com/s/m0Hbo268UaO/">recording
of this meeting</a>. Log into the <a rel="nofollow" class="external text" href="https://jlab.bluejeans.com">BlueJeans site</a>
first to gain access (use your JLab credentials).
</p>
<h3><span class="mw-headline" id="Announcements">Announcements</span></h3>
<ol>
<li> <a rel="nofollow" class="external text" href="https://mailman.jlab.org/pipermail/halld-offline/2021-October/008654.html">Doxygen
sites for halld_recon and halld_sim</a>. Mark
announce the working Doxygen sites for halld_recon and
halld_sim. We discussed how we might improve them.
<ul>
<li> We will try educate ourselves on what is
possible to do with Doxygen. Beni and Alex posted
link which Mark will collect into a wiki page.</li>
<li> Richard suggested asking people in the High
Energy community if they have developed best
practices for configuring Doxygen, or indeed
whether they are using Doxygen as a code
documentation tool still.</li>
</ul>
</li>
<li> <a rel="nofollow" class="external text" href="https://mailman.jlab.org/pipermail/halld-offline/2021-October/008655.html">Advanced
support versions, bug fixes, and a new version set:
5.0.1</a> We are now back with advanced versions of
ROOT and Geant4 as the default.</li>
</ol>
<h3><span class="mw-headline" id="Review_of_Minutes_from_the_Last_Software_Meeting">Review
of Minutes from the Last Software Meeting</span></h3>
<p>We went over the <a href="https://halldweb.jlab.org/wiki/index.php/GlueX_Software_Meeting,_September_27,_2021#Minutes" title="GlueX Software Meeting, September 27, 2021">Minutes
from the September 27th meeting</a>. Alex pointed out
that we need a new set of recon-launch compatible builds
to go with the current master branch of halld_sim. There
are fixes to the beam properties class, used by some of
our generators, that is not captured in the last tagged
version.
</p>
<h3><span class="mw-headline" id="Review_of_Minutes_from_the_Last_HDGeant4_Meeting">Review
of Minutes from the Last HDGeant4 Meeting</span></h3>
<p>We went over the <a href="https://halldweb.jlab.org/wiki/index.php/HDGeant4_Meeting,_October_4,_2021#Minutes" title="HDGeant4 Meeting, October 4, 2021">minutes from
the meeting on October 4th</a>. We need to verify that
the current version of Geant4, 10.04, fixes <a rel="nofollow" class="external text" href="https://github.com/JeffersonLab/HDGeant4/issues/195">HDGeant4
Issue #195: Crash adding daughters on RHEL/CentOS 8</a>.
</p>
<h3><span class="mw-headline" id="FAQ_of_the_Fortnight:_How_do_I_restart_the_automatic_pull-request_test.3F">FAQ
of the Fortnight: How do I restart the automatic
pull-request test?</span></h3>
<p>We went over the <a href="https://halldweb.jlab.org/wiki/index.php/GlueX_Offline_FAQ#How_do_I_restart_the_automatic_pull-request_test.3F" title="GlueX Offline FAQ">FAQ</a> that describes the
procedure. This action is appropriate when it is known
that a pull-request test failed due to external
circumstances, those unrelated to the content of the
code in the request, and the issue with those
circumstances have been resolved. Note that the restart
requires admin privileges on the repository in question
(halld_recon or halld_sim). Only members of the <a rel="nofollow" class="external text" href="https://github.com/orgs/JeffersonLab/teams/gluex-admin/members">gluex_admin
team</a> have this privilege. Collaborators without
admin privilege can trigger a re-test by pushing an
additional change (including changes which do not affect
the operation of the code) to the branch on GitHub which
is the subject of the pull request.
</p>
<h3><span class="mw-headline" id="Review_of_recent_issues_and_pull_requests">Review
of recent issues and pull requests</span></h3>
<p>Alex reported that he is still seeing crashes described
in halld_recon <a rel="nofollow" class="external text" href="https://github.com/JeffersonLab/halld_recon/issues/81">Issue #81,
DL1MCTrigger Crashes With mysql rcdb/ccdb</a>, first
reported in January 2019 by Jon Zarling. The odd thing
about this one is that failures only occur on RCDB
access to the MySQL server and not when using an SQLite
file. Alex will put together a simple test case and
Richard will have a look at it.
</p>
<h3><span class="mw-headline" id="Action_Item_Review">Action
Item Review</span></h3>
<ol>
<li> Create a Wiki page of Doxygen links. (Mark)</li>
<li> Ask the HEP Software Foundation about code
documentation best practices.</li>
<li> Create new tagged version of halld_sim and
corresponding recon-launch compatible version sets.
(Mark)</li>
<li> Add Alex to the gluex_admin team on GitHub. (Mark)</li>
<li> Add non-privileged pull-request-re-test procedure
to the FAQ.</li>
</ol>
</div>
<br>
</div>
</div>
</div>
</div>
</body>
</html>