<html>
  <head>

    <meta http-equiv="content-type" content="text/html; charset=UTF-8">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <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,_January_8,_2019#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">Minutes, GlueX Software Meeting, January 8,
              2019</span></h2>
          <div id="bodyContent" class="mw-body-content">
            <div id="mw-content-text" dir="ltr" class="mw-content-ltr"
              lang="en">
              <p>Present:
              </p>
              <ul>
                <li> <b> CMU: </b> Naomi Jarvis</li>
                <li> <b> JLab: </b> Thomas Britton, Ashley Ernst, Mark
                  Ito (chair), Richard Jones, Justin Stevens, Simon
                  Taylor</li>
              </ul>
              <p>There is a <a rel="nofollow" class="external text"
                  href="https://bluejeans.com/s/dIYPe/">recording of
                  this meeting</a> on the BlueJeans site. Use your JLab
                credentials to access it.
              </p>
              <h3><span class="mw-headline" id="Announcements">Announcements</span></h3>
              <ol>
                <li> <b>MCwrapper changes (not a release)</b>. Thomas
                  reported on recent changes.
                  <ul>
                    <li> <b><a rel="nofollow" class="external text"
                          href="https://halldweb.jlab.org/gluex_sim/SubmitSim.html">MCwrapper
                          submission revamp</a></b>.
                      <ul>
                        <li> The CPP experiment has been added to the
                          submission page as an entity separate from
                          GlueX. Other experiments can be added
                          similarly.</li>
                        <li> The "confirm" button will check for similar
                          MC runs and notify the user if they exist.
                          This should lessen duplication of effort.</li>
                      </ul>
                    </li>
                    <li> <b><a rel="nofollow" class="external text"
href="https://halldweb.jlab.org/gluex_sim/SubmitSim.html?prefill=254&mod=1">Self-Service
                          error corrections</a></b>.
                      <ul>
                        <li> If test of submission fails, form is
                          re-presented to user for corrections.
                          Corrections are only allowed if coming from
                          the original user. This feature has been used
                          already by Peter Pauli.</li>
                      </ul>
                    </li>
                  </ul>
                </li>
                <li> <b>hdds updates to eliminate overlaps</b>. Richard
                  has addressed the geometry overlaps as <a
                    rel="nofollow" class="external text"
                    href="https://github.com/JeffersonLab/HDGeant4/issues/60">reported
                    by Maria Patsyuk</a>.
                  <ul>
                    <li> There were 98 occurrences of overlap, many of
                      them of the "rounding" variety (insufficient
                      precision in the geometry parameters).</li>
                    <li> The most serious overlap was one between the
                      BCAL and FDC. A tilt was added to the BCAL to make
                      it consistent with survey data. The axis of
                      rotation turned out to be at the upstream end of
                      the BCAL, causing an overlap with the FDC at the
                      downstream end. Richard changed the rotation to be
                      (effectively) around the center of the BCAL,
                      maintaining the amount of tilt.</li>
                    <li> Action Items:
                      <ol>
                        <li> Richard will submit a pull request to the
                          hdds repository.</li>
                        <li> He will also consult with Sean Dobbs about
                          how to include this change into the
                          CCDB-resident geometry files.</li>
                      </ol>
                    </li>
                  </ul>
                </li>
              </ol>
              <h3><span class="mw-headline"
                  id="Review_of_minutes_from_the_December_11_meeting">Review
                  of minutes from the December 11 meeting</span></h3>
              <p>We went over <a
href="https://halldweb.jlab.org/wiki/index.php/GlueX_Software_Meeting,_December_11,_2018#Minutes"
                  title="GlueX Software Meeting, December 11, 2018">the
                  minutes</a>.
              </p>
              <ul>
                <li> <b>Crashing Monitor Launches?</b>. The cause of
                  the crashes in monitoring launches was traced to a
                  Start Counter plugin by Alex Austregesilo. It was not
                  related to the reconstruction code itself.</li>
                <li> <b>Record of per file event ranges</b>. We need to
                  ask if progress has been made.</li>
              </ul>
              <h3><span class="mw-headline"
                  id="Report_from_the_December_18_HDGeant4_Meeting">Report
                  from the December 18 HDGeant4 Meeting</span></h3>
              <p>We went over <a
href="https://halldweb.jlab.org/wiki/index.php/HDGeant4_Meeting,_December_18,_2018#Minutes"
                  title="HDGeant4 Meeting, December 18, 2018">these
                  minutes</a> as well.
              </p>
              <p>Richard reported that has fixed the long-standing <a
                  rel="nofollow" class="external text"
                  href="https://github.com/JeffersonLab/HDGeant4/issues/66">problem
                  with missing hits along the track</a> in the FDC.
                There was a off-by-one error in the layer labeling. This
                caused layers to be rotated incorrectly in HDGeant4.
              </p>
              <h3><span class="mw-headline"
                  id="Merging_the_DIRC_branches">Merging the DIRC
                  branches</span></h3>
              <p>Justin led us through <a rel="nofollow"
                  class="external text"
href="https://mailman.jlab.org/pipermail/halld-offline/2019-January/003469.html">his
                  email proposing a scheme for merging the DIRC branches</a>
                onto the master branches. Please see the email for a
                nice description of the scheme, its features, and a
                comment on tests completed.
              </p>
              <p>For HDGeant4 (hdg4), since the geometry is defined at
                run time using files from the CCDB, there is no
                ambiguity about whether the DIRC should be included or
                not. For hdgeant (hdg3) the decision is made at build
                time by pointing to an appropriate build of HDDS, one
                with or without the DIRC as appropriate. We discussed
                whether we want to even support hdg3 with the DIRC; if
                not we could define hdg3 as a DIRC-free program.
              </p>
              <p>Richard proposed a mechanism where inclusion of the
                DIRC is determined by a CCDB-resident constant. All hdg3
                builds going forward would include the DIRC, but during
                initialization, i. e., at run time, if the CCDB says
                there should not be a DIRC, the DIRC geometry will be
                replaced by air. Again, this would only be for hdg3.
                Richard volunteered to code the new initialization step;
                Justin volunteered to create the new item in the CCDB.
              </p>
              <p>Justin mentioned that initially DIRC hits would be put
                into the REST format. This is in contrast to our usual
                practice of storing only reconstructed quantities. The
                intention is to do this only until the DIRC
                reconstruction code matures. For now having the hits in
                the REST events allows for efficient development of the
                reconstruction. Justin will look into how much this will
                cost in terms of event size.
              </p>
              <p>No objections to the plan were voiced. Justin plans to
                submit the pull requests tomorrow (Wednesday).
              </p>
              <h3><span class="mw-headline"
                  id="Developer-friendly.2C_container-based_builds">Developer-friendly,
                  container-based builds</span></h3>
              <p>Mark filled in some details in <a rel="nofollow"
                  class="external text"
href="https://mailman.jlab.org/pipermail/halld-offline/2018-December/003465.html">his
                  announcement</a> of Oasis-resident builds that can be
                used for code development. Please see <a rel="nofollow"
                  class="external text"
href="https://docs.google.com/presentation/d/1tFgT4Q3qWg72LycfC_72LCffta7KNvQTxHRwtga7qp0/edit?usp=sharing">his
                  slides</a> for the details. They are also <a
                  rel="nofollow" class="external text"
href="https://halldweb.jlab.org/doc-private/DocDB/ShowDocument?docid=3855">available
                  as PDF from GlueX Doc 3855</a>.
              </p>
              <h3><span class="mw-headline"
                  id="Debug_and_optimized_builds_of_halld_recon_and_halld_sim">Debug
                  and optimized builds of halld_recon and halld_sim</span></h3>
              <p>Mark gave further explanation of <a rel="nofollow"
                  class="external text"
href="https://mailman.jlab.org/pipermail/halld-offline/2019-January/003468.html">his
                  announcement of build_scripts 1.44</a>. The most
                significant change is the introduction of "debug
                levels"; new combinations of optimization levels and
                inclusion/exclusion of debugging symbols. See <a
                  rel="nofollow" class="external text"
href="https://docs.google.com/presentation/d/1DpaLMUnviTgtJ_SRT6iZcaORcKeKNOdAxzic-LlFLPY/edit?usp=sharing">his
                  slides</a> for the details. They are also <a
                  rel="nofollow" class="external text"
href="https://halldweb.jlab.org/doc-private/DocDB/ShowDocument?docid=3856">available
                  as PDF from GlueX Doc 3856</a>)
              </p>
              <p>The idea is to (a) save disk space (b) tailor builds to
                the task (i. e., debugging, production) via additional
                choices of build parameters.
              </p>
              <h3><span class="mw-headline"
                  id="Review_of_recent_issues_and_pull_requests">Review
                  of recent issues and pull requests</span></h3>
              <p>We decided to forego this section of the agenda.
              </p>
            </div>
            <div class="printfooter">
              Retrieved from "<a dir="ltr"
href="https://halldweb.jlab.org/wiki/index.php?title=GlueX_Software_Meeting,_January_8,_2019&oldid=90784">https://halldweb.jlab.org/wiki/index.php?title=GlueX_Software_Meeting,_January_8,_2019&oldid=90784</a>"</div>
          </div>
        </div>
      </div>
      <div id="footer" role="contentinfo">
        <ul id="f-list">
          <li id="lastmod"> This page was last modified on 9 January
            2019, at 21:04.</li>
        </ul>
      </div>
    </div>
  </body>
</html>