Initial version of gaus_hit_finder example#5
Initial version of gaus_hit_finder example#5wddgit wants to merge 1 commit intoFramework-R-D:mainfrom
Conversation
|
This was from Beojan on SLACK. What level of review are you looking for here? For instance while the file you're printing has a CSV-style header, it isn't a CSV because it has a newline after each field. Is that to match a file produced by the existing GausHitFinder? and I'd suggest using fmt::print instead of iostreams, which can really clean up that code. |
|
And my reply. Thanks. Beojan. I will change it to .txt and use |
wddgit
force-pushed
the
migrateGausHitFinder
branch
from
9f38888 to
b83228e
Compare
|
I force pushed a modification. The most significant part of implements Beojan's suggestions. It changes from using |
|
FYI @beojan Just to make sure you are receiving comments on this PR. |
1 participant
This is an initial version of the result of the migration of the module defined in GausHitFinder_module.cc in the larreco repository of LArSoft from the existing version designed to run under the
artframework to a version that will run under thephlexframework. The purpose of this code is to serve as an example for future migrations and also to be used in prototype testing. It is not intended to be used for reconstruction of data for use in physics analysis. That code should be placed in some other repository and maintained by other groups.See the README.md (best viewed in GitHub) for more information.