-
Notifications
You must be signed in to change notification settings - Fork 12
Feature/adding blip to caf #155
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
Changes from 44 commits
7f7a897
18d8269
9751aa3
1387b2a
f5c315a
2ce06f9
2d37c3f
55d9f70
343d755
99a3c01
bcbb2aa
5bfaf2c
90a5dc2
a535dc9
bbd96f7
8b3cb45
47f8630
d4991e5
91eba8c
83efc10
3d4c322
4e06944
5ae8d14
1fd419a
913d94b
ec53ab8
1595e6c
6194c03
41e66f4
3736ddd
cc24452
dfd4d1d
4c9288e
d460853
bdd98cb
c16b5be
f4872e1
e718325
6930443
5b3feee
db31ab5
55f7c5d
f7b4b84
9524c9e
8cd0ca7
b63c719
0f56d1c
f7110fa
4ba0fb5
b6d0793
ee45ae3
38e4c1c
0e27710
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| #include "sbnobj/SBND/Blip/BlipDataTypes.h" |
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| @@ -0,0 +1,206 @@ | ||||||||||||||||||||||||||||||||||||||||||
| #ifndef BLIPDATATYPE | ||||||||||||||||||||||||||||||||||||||||||
| #define BLIPDATATYPE | ||||||||||||||||||||||||||||||||||||||||||
| #include "lardataobj/RecoBase/Hit.h" | ||||||||||||||||||||||||||||||||||||||||||
| #include "nusimdata/SimulationBase/MCParticle.h" | ||||||||||||||||||||||||||||||||||||||||||
| #include "larcoreobj/SimpleTypesAndConstants/geo_vectors.h" | ||||||||||||||||||||||||||||||||||||||||||
| #include <vector> | ||||||||||||||||||||||||||||||||||||||||||
| #include <map> | ||||||||||||||||||||||||||||||||||||||||||
| #include <set> | ||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||
| namespace blip { | ||||||||||||||||||||||||||||||||||||||||||
| const int kNplanes = 3; ///< Wire planes in a given tpc in this detector | ||||||||||||||||||||||||||||||||||||||||||
| const int kNTPCs = 2; ///< TPC in this detector | ||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||
| /** Extra struct used in blip reco for grabbing key recob::hit information | ||||||||||||||||||||||||||||||||||||||||||
| * Most attributes are grabbed directly from the recob::hit, but some, like charge, involve extra processing. | ||||||||||||||||||||||||||||||||||||||||||
| */ | ||||||||||||||||||||||||||||||||||||||||||
| struct HitInfo { | ||||||||||||||||||||||||||||||||||||||||||
| int hitid = -9; ///< Index of hit in recob::hit vector | ||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||
| int hitid = -9; ///< Index of hit in recob::hit vector | |
| int hitid = -9; ///< Index of hit in recob::Hit vector |
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should enable Doxygen to make a cross-reference:
| int chan = -9; ///< Channel ID containing hit. From recob::Hit->Channel() | |
| int chan = -9; ///< Channel ID containing hit. From recob::Hit::Channel() |
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| int shwrid = -9; ///< Depreciated | |
| int shwrid = -9; ///< Deprecated |
It's about praying (depreco, I pray it to go away), not price (depretio, I lower the cost).
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| float integralADC = -999; ///< Integral area from the recob::hit [ADCs-ticks] | |
| float integralADC = -999; ///< Integral area from the recob::Hit [ADCs-ticks] |
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
More Doxygen-y:
| float charge = -999; ///< Total charge as estimated from hit integral [e-] | |
| /*! | |
| Charge is reconstructed from calo::CalorimetryAlg ( "sbnd_calorimetryalgmc" )-> ElectronsFromADCArea function | |
| Configuration is in sbndcode/sbndcode/LArSoftConfigurations/calorimetry_sbnd.fcl | |
| */ | |
| /** | |
| * @brief Total charge as estimated from hit [e-] | |
| * | |
| * Charge is reconstructed from calo::CalorimetryAlg::ElectronsFromADCArea(). | |
| * . | |
| * SBND configuration is in `sbnd_calorimetryalgmc` in | |
| * `sbndcode/LArSoftConfigurations/calorimetry_sbnd.fcl` | |
| */ | |
| float charge = -999; |
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| float gof = -9; ///< GoodnessOfFit from recob::hit | |
| float gof = -9; ///< from recob::Hit::GoodnessOfFit() |
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is it not bool then?
| int isPrimary = -9; ///< Bool value of simbMCParticle is Primary | |
| int isPrimary = -9; ///< Bool value of simb::MCParticle is Primary |
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| int numTrajPts = -9; ///< part.NumberTrajectoryPoints() from mcparticle | |
| int numTrajPts = -9; ///< From simb::MCParticle::NumberTrajectoryPoints() |
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| double numElectrons = -9; ///< Depreciated | |
| double numElectrons = -9; ///< Deprecated |
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| double pathLength = -9; ///< Sum of distances between interaction vertecies within the active volume [cm] | |
| double pathLength = -9; ///< Sum of distances between interaction vertices within the active volume [cm] |
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am not sure about the need for "used to be TVector3" since this is the first commit of this class.
| geo::Point_t startPoint; ///< Starting Position of particle [cm] //Used to be TVector3 | |
| geo::Point_t endPoint; ///< End position of particle [cm] //Used to be TVector3 | |
| geo::Point_t Position; ///< Central position of the trajectory [cm] //Used to be TVector3 | |
| geo::Point_t startPoint; ///< Starting Position of particle [cm] | |
| geo::Point_t endPoint; ///< End position of particle [cm] | |
| geo::Point_t Position; ///< Central position of the trajectory [cm] |
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| int DeadWireSep = 99; ///< Separation between the extreme ends of the hitcluster and the nearest dead wire. | |
| /*! | |
| DeadWireSep can be between 0 and 5 and valid. Larger separations are filled in as 99. | |
| */ | |
| /** | |
| * @brief Separation between the extreme ends of the hitcluster and the nearest dead wire. | |
| * | |
| * `DeadWireSep` can be between `0` and `5` and valid. Larger separations are filled in as `99`. | |
| */ | |
| int DeadWireSep = 99; ///< |
PetrilloAtWork marked this conversation as resolved.
Show resolved
Hide resolved
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| float StartHitTime = -999; ///< Depreciated | |
| float EndHitTime = -999; ///< Depreciated | |
| float StartHitTime = -999; ///< Deprecated | |
| float EndHitTime = -999; ///< Deprecated |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Similar argument to SBNSoftware/sbnanaobj#173 (comment)
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| std::set<int> HitIDs; ///< Index of the recob::hit objects making up this cluster | |
| std::set<int> Wires; ///< Set of geo::wireIDs contributing hits to this cluster | |
| std::set<int> HitIDs; ///< Index of the recob::Hit objects making up this cluster | |
| std::set<int> Wires; ///< Set of geo::WireID contributing hits to this cluster |
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same comment as above:
| std::map<int,geo::Point_t> IntersectLocations; ///< Internal reconstruction variable for recording where hit-clusters on different planes would overlap //used to be TVector3 | |
| std::map<int,geo::Point_t> IntersectLocations; ///< Internal reconstruction variable for recording where hit-clusters on different planes would overlap |
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
More Doxygen-y:
| float TimeTick = -999; ///< Average time of hitclusters making up blips. [tpc tick] | |
| /*! | |
| Hit cluster time is the charge-weighted average of the hit-peak times for hits in the cluster | |
| */ | |
| float Time = -999; ///< Average time of hitclusters making up blips. [us] | |
| /*! | |
| Hit cluster time is the charge-weighted average of the hit-peak times for hits in the cluster | |
| */ | |
| /** | |
| * @brief Average time of hitclusters making up blips. [tpc tick] | |
| * | |
| * Hit cluster time is the charge-weighted average of the hit-peak times for hits in the cluster. | |
| */ | |
| float TimeTick = -999; | |
| /** | |
| * @brief Average time of hitclusters making up blips. [us] | |
| * | |
| * Hit cluster time is the charge-weighted average of the hit-peak times for hits in the cluster. | |
| */ | |
| float Time = -999; |
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| float ProxTrkDist = -9; ///< 3-D distance to closest track, assuming the blip was concident with event trigger [cm] | |
| /*! | |
| for properly flash matched out-of-time tracks this distance will be wrong! The blips have no such flash matching ability as of yet | |
| */ | |
| /** | |
| * @brief 3-D distance to closest track, assuming the blip was coincident with event trigger [cm] | |
| * | |
| * For properly flash matched out-of-time tracks this distance will be wrong! | |
| * The blips have no such flash matching ability as of yet. | |
| */ | |
| float ProxTrkDist = -9; |
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| /*! | |
| please note the blip X position is unreliable, so these distance and 3-d position derived variables may be incorrect | |
| */ | |
| geo::Point_t Position; ///< 3D position vector. Reconstructed with wrong t0! [cm] | |
| /** | |
| * @brief 3D position vector. Reconstructed with wrong t0! [cm] | |
| * | |
| * Please note the blip X position is unreliable, | |
| * so these distance and 3-d position derived variables may be incorrect. | |
| */ | |
| geo::Point_t Position; |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,11 @@ | ||
| cet_make_library( | ||
| SOURCE | ||
| BlipDataTypes.cc | ||
| LIBRARIES | ||
| lardataobj::RecoBase | ||
| nusimdata::SimulationBase | ||
| ) | ||
| art_dictionary(DICTIONARY_LIBRARIES sbnobj::SBND_Blip) | ||
| install_headers() | ||
| install_source() | ||
| install_fhicl() |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,32 @@ | ||
| // | ||
| // Build a dictionary. | ||
| // | ||
| // $Id: classes.h,v 1.8 2010/04/12 18:12:28 Exp $ | ||
| // $Author: $ | ||
| // $Date: 2010/04/12 18:12:28 $ | ||
| // | ||
| // Original author Rob Kutschke, modified by wes | ||
| // | ||
|
|
||
| #include "canvas/Persistency/Common/Wrapper.h" | ||
|
|
||
| // data-products | ||
| // lardataobj | ||
| #include "canvas/Persistency/Common/Assns.h" | ||
| #include "lardataobj/RecoBase/Hit.h" | ||
| #include "sbnobj/SBND/Blip/BlipDataTypes.h" | ||
| #include "lardataobj/RecoBase/SpacePoint.h" | ||
|
|
||
| // | ||
| // Only include objects that we would like to be able to put into the event. | ||
| // Do not include the objects they contain internally. | ||
| // | ||
|
|
||
| template class art::Assns<recob::Hit,blip::Blip>; | ||
| template class art::Wrapper<art::Assns<recob::Hit,blip::Blip> >; | ||
| template class std::vector<blip::Blip>; | ||
| template class art::Wrapper<std::vector<blip::Blip> >; | ||
| template class art::Assns<blip::Blip,recob::Hit>; | ||
| template class art::Wrapper<art::Assns<blip::Blip,recob::Hit> >; | ||
| template class art::Assns<blip::Blip,recob::SpacePoint>; | ||
| template class art::Wrapper<art::Assns<blip::Blip,recob::SpacePoint> >; |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,22 @@ | ||
| <lcgdict> | ||
| <class name="art::Wrapper<std::vector<blip::Blip> >"/> | ||
| <class name="blip::Blip" ClassVersion="10" > | ||
| <version ClassVersion="10" checksum="2428317905"/> | ||
| </class> | ||
| <class name="std::vector<blip::Blip>"/> | ||
| <class name="blip::HitClust" ClassVersion="10" > | ||
| <version ClassVersion="10" checksum="1162717057"/> | ||
| </class> | ||
| <class name="vector<blip::HitClust>"/> | ||
| <class name="art::Wrapper<std::vector<blip::HitClust> >"/> | ||
| <class name="blip::TrueBlip" ClassVersion="10" > | ||
| <version ClassVersion="10" checksum="3344816662"/> | ||
| </class> | ||
| <class name="art::Assns<blip::Blip,recob::Hit,void>"/> | ||
| <class name="std::map<int,geo::Point_t>"/> | ||
| <class name="art::Assns<recob::Hit, blip::Blip,void>"/> | ||
| <class name="art::Assns<blip::Blip,recob::SpacePoint,void>"/> | ||
| <class name="art::Wrapper<art::Assns<blip::Blip,recob::Hit,void> >"/> | ||
| <class name="art::Wrapper<art::Assns<recob::Hit,blip::Blip,void> >"/> | ||
| <class name="art::Wrapper<art::Assns<blip::Blip,recob::SpacePoint,void> >"/> | ||
PetrilloAtWork marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| </lcgdict> | ||
Uh oh!
There was an error while loading. Please reload this page.