<div dir="ltr">Simon and all,<div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><ul style="color:rgb(0,0,0);font-family:Calibri,Arial,Helvetica,sans-serif;font-size:16px"><li style="margin-left:15px">I have updated the XML specification to include the CTOF: <a href="https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_JeffersonLab_hdds_tree_CPPTofV2&d=DwMFaQ&c=CJqEzB1piLOyyvZjb8YUQw&r=cpPCbeMAbcjUkVM8na3mXiIQlQp45KwF9GDqF5CC4ms&m=JmbyTHw65t3i_QtFZY2Si1XGNGWSMETkYxvzJAn1DAifYChQBRqs2TRqKdpbk6C0&s=0VUb5nsA30TkTQ3KQrCdEB6AwOpbuTzsz6gHSqNY-zI&e=" id="gmail-m_7249379263313876302LPNoLPOWALinkPreview_3" target="_blank">https://github.com/JeffersonLab/hdds/tree/CPPTofV2</a>. I believe I found an overlap problem between the muon detector and a part of the beam line model that I have currently disabled for the CPP configuration.</li></ul></blockquote><div>It is fine to implement these updates in the hdds repo, but to actually use them for simulations the updated geometry should be in the /GEOMETRY directory of ccdb.<br></div><div><br></div><div>Also what is this "tree" thing that appears in the hdds, halld_sim, and HDGeant4 repos? It is one thing to make changes to existing classes or introduce additional subclasses of existing objects under an existing repo. But if you change the directory structure of the repo and call it a "branch" that is a bit too much. I think if you want to make fundamental changes like this to the overall organization of a project, you should fork the repo and give it a new name.</div><div><br></div><div>-Richard Jones</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Nov 11, 2021 at 9:46 AM Simon Taylor <<a href="mailto:staylor@jlab.org">staylor@jlab.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div dir="ltr">
<div style="font-family:Calibri,Arial,Helvetica,sans-serif;font-size:12pt;color:rgb(0,0,0)">
Hi, all.</div>
<div style="font-family:Calibri,Arial,Helvetica,sans-serif;font-size:12pt;color:rgb(0,0,0)">
There are several branches on github containing CPP-related software updates:</div>
<div style="font-family:Calibri,Arial,Helvetica,sans-serif;font-size:12pt;color:rgb(0,0,0)">
<ul>
<li><span>I have updated the halld_recon CPP branch recently created by David to incorporate the inclusion of the downstream scintillators into the reconstruction:
<a href="https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_JeffersonLab_halld-5Frecon_tree_CPP&d=DwMFaQ&c=CJqEzB1piLOyyvZjb8YUQw&r=cpPCbeMAbcjUkVM8na3mXiIQlQp45KwF9GDqF5CC4ms&m=JmbyTHw65t3i_QtFZY2Si1XGNGWSMETkYxvzJAn1DAifYChQBRqs2TRqKdpbk6C0&s=-eMZwofwZ_ZUh2dQ4J80UMtX1CD9wNLP9lNZHDcMVIo&e=" id="gmail-m_7249379263313876302LPNoLPOWALinkPreview" target="_blank">
https://github.com/JeffersonLab/halld_recon/tree/CPP</a></span></li><li><span>I have added smearing for the CPP scintillators (CTOF) into mcsmear: <a href="https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_JeffersonLab_halld-5Fsim_tree_CPPTofV2&d=DwMFaQ&c=CJqEzB1piLOyyvZjb8YUQw&r=cpPCbeMAbcjUkVM8na3mXiIQlQp45KwF9GDqF5CC4ms&m=JmbyTHw65t3i_QtFZY2Si1XGNGWSMETkYxvzJAn1DAifYChQBRqs2TRqKdpbk6C0&s=drFzGu1W8dBYq9ithAmhD7SMC1YP8rYRjdNJP5eTXMY&e=" id="gmail-m_7249379263313876302LPNoLPOWALinkPreview_1" target="_blank">
https://github.com/JeffersonLab/halld_sim/tree/CPPTofV2</a></span></li><li><span>I have added the CTOF scintillators as sensitive volumes to HDGeant4: <a href="https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_JeffersonLab_HDGeant4_tree_CPPTof&d=DwMFaQ&c=CJqEzB1piLOyyvZjb8YUQw&r=cpPCbeMAbcjUkVM8na3mXiIQlQp45KwF9GDqF5CC4ms&m=JmbyTHw65t3i_QtFZY2Si1XGNGWSMETkYxvzJAn1DAifYChQBRqs2TRqKdpbk6C0&s=QyV6Vx-cgyzb3MKv64ppk9YCjm6zXnTFuZDdmovGbFI&e=" id="gmail-m_7249379263313876302LPNoLPOWALinkPreview_2" target="_blank">
https://github.com/JeffersonLab/HDGeant4/tree/CPPTof</a></span></li><li><span>I have updated the XML specification to include the CTOF: <a href="https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_JeffersonLab_hdds_tree_CPPTofV2&d=DwMFaQ&c=CJqEzB1piLOyyvZjb8YUQw&r=cpPCbeMAbcjUkVM8na3mXiIQlQp45KwF9GDqF5CC4ms&m=JmbyTHw65t3i_QtFZY2Si1XGNGWSMETkYxvzJAn1DAifYChQBRqs2TRqKdpbk6C0&s=0VUb5nsA30TkTQ3KQrCdEB6AwOpbuTzsz6gHSqNY-zI&e=" id="gmail-m_7249379263313876302LPNoLPOWALinkPreview_3" target="_blank">
https://github.com/JeffersonLab/hdds/tree/CPPTofV2</a>. I believe I found an overlap problem between the muon detector and a part of the beam line model that I have currently disabled for the CPP configuration.<br>
</span></li><li style="display:block">
<div></div>
</li><li style="display:block"><br>
</li></ul>
<div>Simon<br>
</div>
<ul>
<li style="display:block">
<div></div>
</li><li style="display:block"><br>
</li><li style="display:block">
<div></div>
</li><li style="display:block"><br>
</li><li style="display:block">
<div></div>
</li><li style="display:block"><br>
</li></ul>
</div>
</div>
_______________________________________________<br>
Halld-cpp mailing list<br>
<a href="mailto:Halld-cpp@jlab.org" target="_blank">Halld-cpp@jlab.org</a><br>
<a href="https://mailman.jlab.org/mailman/listinfo/halld-cpp" rel="noreferrer" target="_blank">https://mailman.jlab.org/mailman/listinfo/halld-cpp</a><br>
</blockquote></div>