-
Notifications
You must be signed in to change notification settings - Fork 26
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
FIRST solarcluster PRODUCER UPLOAD #136
base: develop
Are you sure you want to change the base?
Conversation
…eys_solarcluster
trigger build |
✔️ CI build for LArSoft Succeeded on slf7 for e26:prof -- details available through the CI dashboard |
✔️ CI build for LArSoft Succeeded on slf7 for c14:prof -- details available through the CI dashboard |
❌ CI build for DUNE Failed at phase ci_tests DUNE on slf7 for c14:prof - ignored warnings for build -- details available through the CI dashboard 🚨 For more details about the failed phase, check the ci_tests DUNE phase logs parent CI build details are available through the CI dashboard |
🚨 For more details about the warning phase, check the ci_tests DUNE phase logs parent CI build details are available through the CI dashboard |
Hi @aolivier23 thanks for taking care of this. I would say the more reviews, the better; so feel free to take a look, but I would also like @lpaulucc to validate this. There is no need to merge this this week; it can wait. Thanks! |
This is a SolarCluster producer module. It also includes the definition of a new class for low energy analysis "LowECluster". A LowECluster is a 3D reconstructed cluster associated with matched recob::Clusters which have been produced "per-plane" following a standard clustering approach. The LowEClusters object is the minimum expression of reconstruction for a lowe analysis and will be the triggering component of a future LowEEvent which is a class combining a set of LowEClusters (main + adjacent), PDS Flash (SolarOpFlash) and possible associated lowe tracks and pandora objects.