[Halld-cpp] [EXTERNAL] Re: Repositories for CPP software
Simon Taylor
staylor at jlab.org
Thu Nov 11 10:11:23 EST 2021
Hi, Richard.
Regarding the actual use of the geometry XML: yes, for the simulations we are using the GEOMETRY directory in the ccdb. This still needs to be done for this updated geometry.
Regarding the "tree" in the repository links: I did not create this myself -- this appears to be something that is internal to github. My understanding is that these branches are still under the existing repos. The plan is to make pull requests to merge these changes into the master. The idea is to advertise these developments so that others can take a look first.
Simon
________________________________
From: Richard Jones <rjones30 at gmail.com>
Sent: Thursday, November 11, 2021 10:00 AM
To: Simon Taylor <staylor at jlab.org>
Cc: halld-cpp at jlab.org <halld-cpp at jlab.org>
Subject: [EXTERNAL] Re: [Halld-cpp] Repositories for CPP software
Simon and all,
* I have updated the XML specification to include the CTOF: https://github.com/JeffersonLab/hdds/tree/CPPTofV2<https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_JeffersonLab_hdds_tree_CPPTofV2&d=DwMFaQ&c=CJqEzB1piLOyyvZjb8YUQw&r=-OqssvRvRqB76_WgPxaGf00Ynk6tLY-9A8YMYvvMGMI&m=-hTzPHMTZVZ2cT-5bdyvprksH81Cz2FnMnFH66t5qCc7C_5F1VEzaX_u46-Rayex&s=MImRzZUBS3OBtfZ9WxY9p5G0p66IT9tui7piFB0WRp4&e=>. 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.
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.
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.
-Richard Jones
On Thu, Nov 11, 2021 at 9:46 AM Simon Taylor <staylor at jlab.org<mailto:staylor at jlab.org>> wrote:
Hi, all.
There are several branches on github containing CPP-related software updates:
* I have updated the halld_recon CPP branch recently created by David to incorporate the inclusion of the downstream scintillators into the reconstruction: https://github.com/JeffersonLab/halld_recon/tree/CPP<https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_JeffersonLab_halld-5Frecon_tree_CPP&d=DwMFaQ&c=CJqEzB1piLOyyvZjb8YUQw&r=-OqssvRvRqB76_WgPxaGf00Ynk6tLY-9A8YMYvvMGMI&m=-hTzPHMTZVZ2cT-5bdyvprksH81Cz2FnMnFH66t5qCc7C_5F1VEzaX_u46-Rayex&s=NG73o0trt8hF7pHkqmUaz1_4Njsxynohz1WJCYhDna8&e=>
* I have added smearing for the CPP scintillators (CTOF) into mcsmear: https://github.com/JeffersonLab/halld_sim/tree/CPPTofV2<https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_JeffersonLab_halld-5Fsim_tree_CPPTofV2&d=DwMFaQ&c=CJqEzB1piLOyyvZjb8YUQw&r=-OqssvRvRqB76_WgPxaGf00Ynk6tLY-9A8YMYvvMGMI&m=-hTzPHMTZVZ2cT-5bdyvprksH81Cz2FnMnFH66t5qCc7C_5F1VEzaX_u46-Rayex&s=h92bqh09pxK9Zj9kS8HSmA4hZTA5hQ6H19S32-fQrfw&e=>
* I have added the CTOF scintillators as sensitive volumes to HDGeant4: https://github.com/JeffersonLab/HDGeant4/tree/CPPTof<https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_JeffersonLab_HDGeant4_tree_CPPTof&d=DwMFaQ&c=CJqEzB1piLOyyvZjb8YUQw&r=-OqssvRvRqB76_WgPxaGf00Ynk6tLY-9A8YMYvvMGMI&m=-hTzPHMTZVZ2cT-5bdyvprksH81Cz2FnMnFH66t5qCc7C_5F1VEzaX_u46-Rayex&s=Y8mm4nyfxIjeeid2SPX1VzZRr5-OXgrDj3cqPACFUBs&e=>
* I have updated the XML specification to include the CTOF: https://github.com/JeffersonLab/hdds/tree/CPPTofV2<https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_JeffersonLab_hdds_tree_CPPTofV2&d=DwMFaQ&c=CJqEzB1piLOyyvZjb8YUQw&r=-OqssvRvRqB76_WgPxaGf00Ynk6tLY-9A8YMYvvMGMI&m=-hTzPHMTZVZ2cT-5bdyvprksH81Cz2FnMnFH66t5qCc7C_5F1VEzaX_u46-Rayex&s=MImRzZUBS3OBtfZ9WxY9p5G0p66IT9tui7piFB0WRp4&e=>. 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.
*
*
Simon
*
*
*
*
*
*
_______________________________________________
Halld-cpp mailing list
Halld-cpp at jlab.org<mailto:Halld-cpp at jlab.org>
https://mailman.jlab.org/mailman/listinfo/halld-cpp
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.jlab.org/pipermail/halld-cpp/attachments/20211111/3f9c3303/attachment.html>
More information about the Halld-cpp
mailing list