- Member AcceptFindNeighbors::isNear (const BoundingBox &b) const
- activating these should throw a warning or something
- Member calgaushf::CalGausHFDUNE10kt::produce (art::Event &evt)
- need to have a disambiguation algorithm somewhere in here
- Member calgaushf::CalGausHFDUNE35t::produce (art::Event &evt)
- need to have a disambiguation algorithm somewhere in here
- Member calo::CalorimetryAlg::dEdx_AMP (detinfo::DetectorClocksData const &clock_data, detinfo::DetectorPropertiesData const &det_prop, double dQ, double time, double pitch, unsigned int plane, double T0=0) const
- The plane argument should really be for a view instead
- Member calo::CalorimetryAlg::dEdx_AMP (detinfo::DetectorClocksData const &clock_data, detinfo::DetectorPropertiesData const &det_prop, double dQ, double time, double pitch, unsigned int plane, double T0, double EField) const
- The plane argument should really be for a view instead
- Member cheat::RecoCheckAna::CheckRecoEvents (art::Event const &evt, std::string const &label, art::Handle< std::vector< recob::Event >> const &evtcol, std::vector< art::Ptr< recob::Hit >> const &allhits)
- need to divy it up in the case where there is more than 1 true interaction in a spill
- Member cluster::ClusterCheater::produce (art::Event &evt) override
- : The above encoding of the ID probably won't work for DUNE and should be revisited
- Class cluster::ClusterCreator
- Add the example!
- Member cluster::DBcluster::produce (art::Event &evt)
- : need to define start and end positions for this cluster and slopes for dTdW, dQdW
- Member cluster::DBScanAlg::getSimilarity (const std::vector< double > v1, const std::vector< double > v2)
- this code assumes that all planes have the same wire pitch
- Member cluster::DBScanAlg::getSimilarity2 (const std::vector< double > v1, const std::vector< double > v2)
- this code assumes all planes have the same wire pitch
- Member cluster::HoughBaseAlg::FastTransform (detinfo::DetectorClocksData const &clockData, detinfo::DetectorPropertiesData const &detProp, std::vector< art::Ptr< recob::Hit >> const &clusIn, std::vector< art::PtrVector< recob::Hit >> &clusHitsOut, CLHEP::HepRandomEngine &engine, std::vector< double > &slope, std::vector< ChargeInfo_t > &totalQ)
explain where the 0.001 comes from
: the collection plane's characteristic hit width's are,
: on average, about 5 time samples wider than the induction plane's.
: this is hard-coded for now.
should it really be wire_pitch[0] in the if statements below, or the pitch for the plane of the hit?
- Member cluster::HoughBaseAlg::Transform (detinfo::DetectorPropertiesData const &detProp, std::vector< art::Ptr< recob::Hit >> const &hits, double &slope, double &intercept)
could eventually refine this method to throw out hits that are
far from the hough line and refine the fit
- Member cluster::LazyClusterParamsAlg::Width (util::GeometryUtilities const &) override
- provide a description of the algorithm by words
- Member cluster::LineMerger::EndpointCompatibility (float sclstartwire, float sclstarttime, float sclendwire, float sclendtime, float cl2startwire, float cl2starttime, float cl2endwire, float cl2endtime)
- 13.5 ticks/wire. need to make this detector agnostic–spitz
- Member cluster::StandardClusterParamsAlg::Width (util::GeometryUtilities const &gser) override
- provide a description of the algorithm by words
- Member DBG_CTX (x)
- - continutation lines not always recognized
- merging of use-statements with same module name and different only-names
- rename part of use-statement
- links to interface functions
- references to variables
- Member deconvgaushf::DeconvGausHFDUNE10kt::produce (art::Event &evt)
- need to have a disambiguation algorithm somewhere in here
- Member deconvgaushf::DeconvGausHFDUNE35t::produce (art::Event &evt)
- need to have a disambiguation algorithm somewhere in here
- Class details::PointerToData_t< T >
- Document this code and make it into a library
- Member detinfo::DetectorTimings::startTick () const
- The example is not even supported yet!
- Member detinfo::LArPropertiesServiceStandard::LArPropertiesServiceStandard (Parameters const ¶ms, art::ActivityRegistry ®)
- these values should eventually come from a database
- Member detsim::DriftElectronstoPlane::produce (art::Event &evt) override
think about effects of drift between planes.
think about effects of drift between planes
- Member detsim::SimDriftElectrons::produce (art::Event &evt) override
think about effects of drift between planes.
think about effects of drift between planes
check on what happens if we allow the tdc value to be
beyond the end of the expected number of ticks
- Member detsim::WienerFilterAna::analyze (const art::Event &evt)
- Need to change hardcoded values to an automatic determination of noise vs. signal
- Member EDepSim::VertexInfo::SetReaction (const G4String &r)
- We need to define a standard format for the reaction names.
- File electromagnetism.h
- Also belong here: ampere, volt, farad, ohm...
- File energy.h
- Also belong here: joule, watt, eV/c, eV/c^2
- Member evd::GraphCluster::produce (art::Event &evt)
This method appears to produce a recob::Cluster really as it is
a collection of 2D clusters from single planes
- Member evd::RecoBaseDrawer::EndPoint2D (const art::Event &evt, evdb::View2D *view, unsigned int plane)
- - have to verify that we are in the right TPC, but to do that we
- Member evd::RecoBaseDrawer::Event3D (const art::Event &evt, evdb::View3D *view)
- need a better way to grab the vertex module labels,
- Member evd::RecoDrawingOptions::fCaloPSet
- Why are calorimetry related drawing options in RecoDrawingOptions instead of a separate service?
- Member evd::TQPad::BookHistogram ()
- decide if ndivraw and ndivreco are useful
- Member evd::TWireProjPad::Draw (const char *opt=0)
- : Why is kSelectedColor hard coded?
- Member evd::TWQProjectionView::SelectHit (int plane)
- What the heck is this all about???
- Member evgb::CRYHelper::Sample (simb::MCTruth &mctruth, double const &surfaceY, double const &detectorLength, double *w, double rantime=0)
- Check if this time slice passes selection criteria
- Member filter::MuonFilter::filter (art::Event &evt) override
: the delta wire numbers seem a bit magic,
: should also change to using Geometry::ChannelsIntersect method
- Member gar::detinfo::ECALPropertiesServiceStandard::reconfigure (fhicl::ParameterSet const &pset) override
- these values should eventually come from a database
- Member gar::detinfo::GArPropertiesServiceStandard::reconfigure (fhicl::ParameterSet const &pset) override
- these values should eventually come from a database
- Member gar::geo::AuxDetGeometryCore::AuxDet (unsigned int const ad=0) const
what happens if it does not exist?
remove the default parameter?
- Member gar::geo::AuxDetGeometryCore::FindAuxDetAtPosition (double const worldLoc[3]) const
- replace with numeric_limits<>?
- Member gar::geo::AuxDetGeometryCore::NAuxDets () const
use a AutDetID_t instead of unsigned int?
Change return type to size_t
- Member gar::geo::AuxDetGeometryCore::PositionToAuxDet (double const worldLoc[3], unsigned int &ad) const
- what happens if it does not exist?
- Member gar::geo::AuxDetGeometryCore::PositionToAuxDetSensitive (double const worldLoc[3], size_t &ad, size_t &sv) const
- what happens if it does not exist?
- Member gar::geo::GeometryCore::DefaultWiggle () const
- Confirm the definition of wiggle: this one is taken from other doc
- Member gar::geo::GeometryCore::MassBetweenPoints (double *p1, double *p2) const
- Unify the coordinates type
- Member gar::geo::GeometryCore::Material (double x, double y, double z) const
- Unify the coordinates type
- Member gar::geo::GeometryCore::MaterialName (TVector3 const &point)
make this constant
remove return value constantness (or make it a reference)
Unify the coordinates type
- Member gar::geo::GeometryCore::NearestChannel (float const worldLoc[3]) const
remove the integers version
Verify the raw::InvalidChannelID part
- Member gar::geo::GeometryCore::SurfaceY () const
- check that this is actually how it is used
- Member gar::geo::GeometryCore::TotalMass (const char *vol="volWorld") const
- Use GetWorldVolumeName() as default instead
- Member gar::geo::GeometryCore::ValueInRange (double value, double min, double max) const
Use wiggle instead of 10^-6
resort source code for a bit of speed up
- Member gar::geo::GeometryCore::VolumeName (TVector3 const &point) const
Use a reference to TVector3
Use a double[3] instead?
declare it const
what happens if none?
Unify the coordinates type
- Member gar::geo::GeometryCore::WorldBox (float *xlo, float *xhi, float *ylo, float *yhi, float *zlo, float *zhi) const
Replace it with a TPC boundaries style thing?
Unify the coordinates type
- Class genie::BLI2DNonUnifObjectGrid< ZObject, IndexType, XType, YType >
- Think about how to have this class inherit from the other BLI2D classes
- Member genie::BLI2DNonUnifObjectGrid< ZObject, IndexType, XType, YType >::get_bound_indices (const std::vector< Type > *vec, Type val, int &lower_index, int &upper_index) const
- Check that the vector contains at least two entries
- Member genie::HadronTensorType_t
- Document enum values
- Member genie::HadronTensorType_t
- Document enum values
- Member genie::NievesSimoVacasMECPXSec2016::XSec (const Interaction *i, KinePhaseSpace_t k) const
Replace these hard-coded replacements with an equivalent XML configuration
Revisit this assumption, and perhaps implement something more robust
Shouldn't we get this from the nuclear model?
- Member genie::SuSAv2MECPXSec::PairRatio (const Interaction *i) const
- add the different pair configurations for e-scattering
- Member genie::SuSAv2MECPXSec::Qvalue (const Interaction &interaction) const
- Add more hadron tensors so this scaling is not so terrible
- Member genie::SuSAv2MECPXSec::XSec (const Interaction *i, KinePhaseSpace_t k) const
- add the different pair configurations for e-scattering
- Member genie::TabulatedHadronTensorModelI::BuildTensor (const HadronTensorID &ht_id) const
- Switch to using std::unique_ptr here for easy cleanup once C++11 features are allowed in GENIE
- Member genie::TabulatedLabFrameHadronTensor::dSigma_dT_dCosTheta (int probe_pdg, double E_probe, double m_probe, double Tl, double cos_l, double ml, double Q_value) const
Add check if in grid. If not, return 0
Add any needed changes for positron projectiles
- Member genie::TabulatedLabFrameHadronTensor::dSigma_dT_dCosTheta_rosenbluth (int probe_pdg, double E_probe, double m_probe, double Tl, double cos_l, double ml, double Q_value) const
Add check if in grid. If not, return 0
Add any needed changes for positron projectiles
- Member genie::TabulatedLabFrameHadronTensor::TableEntry::add (double x, double y)
- Replace these with lambdas (or something similar) when GENIE updates to C++11. This technique is a bit of a hack.
- Member genie::TabulatedLabFrameHadronTensor::TabulatedLabFrameHadronTensor (const std::string &table_file_name)
Add error checks
Use type name
- Member genie::TabulatedLabFrameHadronTensor::tz (double q0, double q_mag) const
- Think about adding the imaginary part even though it's not needed for the cross section calculation
- Member geo::AuxDetGeometryCore::AuxDet (unsigned int const ad=0) const
what happens if it does not exist?
remove the default parameter?
- Member geo::AuxDetGeometryCore::FindAuxDetAtPosition (double const worldLoc[3], double tolerance=0) const
- replace with numeric_limits<>?
- Member geo::AuxDetGeometryCore::NAuxDets () const
use a AutDetID_t instead of unsigned int?
Change return type to size_t
- Member geo::AuxDetGeometryCore::PositionToAuxDet (double const worldLoc[3], unsigned int &ad, double tolerance=0) const
- what happens if it does not exist?
- Member geo::AuxDetGeometryCore::PositionToAuxDetSensitive (double const worldLoc[3], size_t &ad, size_t &sv, double tolerance=0) const
- what happens if it does not exist?
- Member geo::ChannelMap35Alg::FirstChannelInROP (readout::ROPID const &ropid) const override
- to be completed
- Member geo::ChannelMap35Alg::FirstTPCinTPCset (readout::TPCsetID const &tpcsetid) const override
- to be completed
- Member geo::ChannelMap35Alg::FirstWirePlaneInROP (readout::ROPID const &ropid) const override
- to be completed
- Member geo::ChannelMap35Alg::HasTPCset (readout::TPCsetID const &tpcsetid) const override
- to be completed
- Member geo::ChannelMap35Alg::MaxROPs () const override
- to be completed
- Member geo::ChannelMap35Alg::MaxTPCsets () const override
- to be completed
- Member geo::ChannelMap35Alg::Nchannels (readout::ROPID const &ropid) const override
- to be completed
- Member geo::ChannelMap35Alg::NROPs (readout::TPCsetID const &tpcsetid) const override
- to be completed
- Member geo::ChannelMap35Alg::NTPCsets (readout::CryostatID const &cryoid) const override
- to be completed
- Member geo::ChannelMap35Alg::ROPtoTPCs (readout::ROPID const &ropid) const override
- to be completed
- Member geo::ChannelMap35Alg::ROPtoWirePlanes (readout::ROPID const &ropid) const override
- to be completed
- Member geo::ChannelMap35Alg::TPCsetToTPCs (readout::TPCsetID const &tpcsetid) const override
- to be completed
- Member geo::ChannelMap35Alg::TPCtoTPCset (geo::TPCID const &tpcid) const override
- to be completed
- Member geo::ChannelMap35Alg::WirePlaneToROP (geo::PlaneID const &planeid) const override
- to be completed
- Member geo::ChannelMap35OptAlg::FirstChannelInROP (readout::ROPID const &ropid) const override
- to be completed
- Member geo::ChannelMap35OptAlg::FirstTPCinTPCset (readout::TPCsetID const &tpcsetid) const override
- to be completed
- Member geo::ChannelMap35OptAlg::FirstWirePlaneInROP (readout::ROPID const &ropid) const override
- to be completed
- Member geo::ChannelMap35OptAlg::HasTPCset (readout::TPCsetID const &tpcsetid) const override
- to be completed
- Member geo::ChannelMap35OptAlg::MaxROPs () const override
- to be completed
- Member geo::ChannelMap35OptAlg::MaxTPCsets () const override
- to be completed
- Member geo::ChannelMap35OptAlg::Nchannels (readout::ROPID const &ropid) const override
- to be completed
- Member geo::ChannelMap35OptAlg::NROPs (readout::TPCsetID const &tpcsetid) const override
- to be completed
- Member geo::ChannelMap35OptAlg::NTPCsets (readout::CryostatID const &cryoid) const override
- to be completed
- Member geo::ChannelMap35OptAlg::ROPtoTPCs (readout::ROPID const &ropid) const override
- to be completed
- Member geo::ChannelMap35OptAlg::ROPtoWirePlanes (readout::ROPID const &ropid) const override
- to be completed
- Member geo::ChannelMap35OptAlg::TPCsetToTPCs (readout::TPCsetID const &tpcsetid) const override
- to be completed
- Member geo::ChannelMap35OptAlg::TPCtoTPCset (geo::TPCID const &tpcid) const override
- to be completed
- Member geo::ChannelMap35OptAlg::WirePlaneToROP (geo::PlaneID const &planeid) const override
- to be completed
- Member geo::ChannelMapAPAAlg::FirstChannelInROP (readout::ROPID const &ropid) const override
- to be completed
- Member geo::ChannelMapAPAAlg::FirstTPCinTPCset (readout::TPCsetID const &tpcsetid) const override
- to be completed
- Member geo::ChannelMapAPAAlg::FirstWirePlaneInROP (readout::ROPID const &ropid) const override
- to be completed
- Member geo::ChannelMapAPAAlg::HasTPCset (readout::TPCsetID const &tpcsetid) const override
- to be completed
- Member geo::ChannelMapAPAAlg::MaxROPs () const override
- to be completed
- Member geo::ChannelMapAPAAlg::MaxTPCsets () const override
- to be completed
- Member geo::ChannelMapAPAAlg::Nchannels (readout::ROPID const &ropid) const override
- to be completed
- Member geo::ChannelMapAPAAlg::NROPs (readout::TPCsetID const &tpcsetid) const override
- to be completed
- Member geo::ChannelMapAPAAlg::NTPCsets (readout::CryostatID const &cryoid) const override
- to be completed
- Member geo::ChannelMapAPAAlg::ROPtoTPCs (readout::ROPID const &ropid) const override
- to be completed
- Member geo::ChannelMapAPAAlg::ROPtoWirePlanes (readout::ROPID const &ropid) const override
- to be completed
- Member geo::ChannelMapAPAAlg::TPCsetToTPCs (readout::TPCsetID const &tpcsetid) const override
- to be completed
- Member geo::ChannelMapAPAAlg::TPCtoTPCset (geo::TPCID const &tpcid) const override
- to be completed
- Member geo::ChannelMapAPAAlg::WirePlaneToROP (geo::PlaneID const &planeid) const override
- to be completed
- Member geo::ChannelMapCRMAlg::Nchannels (readout::ROPID const &ropid) const override
- Needs to be implemented
- Member geo::ChannelMapCRUAlg::Nchannels (readout::ROPID const &ropid) const override
- Needs to be implemented
- Member geo::DuneApaChannelMapAlg::FirstTPCinTPCset (readout::TPCsetID const &tpcsetid) const override
- to be completed
- Member geo::DuneApaChannelMapAlg::HasTPCset (readout::TPCsetID const &tpcsetid) const override
- to be completed
- Member geo::DuneApaChannelMapAlg::MaxROPs () const override
- to be completed
- Member geo::DuneApaChannelMapAlg::MaxTPCsets () const override
- to be completed
- Member geo::DuneApaChannelMapAlg::Nchannels (readout::ROPID const &ropid) const override
- to be completed
- Member geo::GeometryCore::AuxDet (unsigned int const ad=0) const
what happens if it does not exist?
remove the default parameter?
- Member geo::GeometryCore::ChannelsIntersect (raw::ChannelID_t c1, raw::ChannelID_t c2, double &y, double &z) const
what happens for channels from different TPCs?
what happens for channels with multiple intersection points?
- Member geo::GeometryCore::Cryostat (geo::CryostatID const &cryoid) const
- Make the cryostat number mandatory (as CryostatID)
- Member geo::GeometryCore::CryostatBoundaries (double *boundaries, geo::CryostatID const &cid) const
- What happen on invalid cryostat?
- Member geo::GeometryCore::DefaultWiggle () const
- Confirm the definition of wiggle: this one is taken from other doc
- Member geo::GeometryCore::DetHalfHeight (geo::TPCID const &tpcid) const
deprecate this function
rename the function
- Member geo::GeometryCore::DetHalfWidth (geo::TPCID const &tpcid) const
deprecate this function
rename the function
- Member geo::GeometryCore::DetLength (geo::TPCID const &tpcid) const
deprecate this function
rename the function
- Member geo::GeometryCore::GetCryostatVolumeName (geo::CryostatID const &cid) const
Use a cryostat ID instead
What if it does not exist?
- Member geo::GeometryCore::GetLArTPCVolumeName (geo::TPCID const &tpcid) const
Use a TPCID instead
What if it does not exist?
- Member geo::GeometryCore::NAuxDets () const
use a AutDetID_t instead of unsigned int?
Change return type to size_t
Change return type to size_t
- Member geo::GeometryCore::Ncryostats () const
- Change return type to size_t
- Member geo::GeometryCore::NearestChannel (const double worldLoc[3], geo::PlaneID const &planeid) const
- remove the integers version
- Member geo::GeometryCore::NearestWire (const double worldLoc[3], geo::PlaneID const &planeid) const
remove the integers version
what happens when no wire is found?
- Member geo::GeometryCore::NearestWireID (const double point[3], geo::PlaneID const &planeid) const
- remove the integers version
- Member geo::GeometryCore::NOpChannels () const
- add explanation of the different IDs
- Member geo::GeometryCore::Nplanes (unsigned int tpc=0, unsigned int cstat=0) const
Make all the arguments mandatory (as TPCID)
Change return type to size_t
what happens if TPC does not exist?
- Member geo::GeometryCore::Nplanes (geo::TPCID const &tpcid) const
- Change return type to size_t
- Member geo::GeometryCore::NTPC (unsigned int cstat=0) const
Make the cryostat number mandatory (as CryostatID)
Change return type to size_t
what happens if it does not exist?
- Member geo::GeometryCore::NTPC (geo::CryostatID const &cryoid) const
- Change return type to size_t
- Member geo::GeometryCore::Nviews () const
- Change return type to size_t
- Member geo::GeometryCore::Nwires (geo::PlaneID const &planeid) const
- Change return type to size_t
- Member geo::GeometryCore::Nwires (unsigned int p, unsigned int tpc=0, unsigned int cstat=0) const
Make all the arguments mandatory (as PlaneID)
Change return type to size_t
what happens if it does not exist?
- Member geo::GeometryCore::OpDetGeoName (unsigned int c=0) const
- Change to use CryostatID
- Member geo::GeometryCore::Plane (unsigned int const p, unsigned int const tpc=0, unsigned int const cstat=0) const
- remove the version with integers
- Member geo::GeometryCore::PlanePitch (geo::TPCID const &tpcid, geo::PlaneID::PlaneID_t p1=0, geo::PlaneID::PlaneID_t p2=1) const
add a version with plane IDs
deprecate this function
add a default version for a given TPCID
add a version with two plane indices for a given TPCID
return the absolute value of the distance (makes the order unimportant)
document what will happen (in the future methods) with planes on different TPCs
- Member geo::GeometryCore::PlaneWireToChannel (WireID const &wireid) const
Verify the raw::InvalidChannelID part
remove the integers version
- Member geo::GeometryCore::PositionToAuxDet (double const worldLoc[3], unsigned int &ad, double tolerance=0) const
- what happens if it does not exist?
- Member geo::GeometryCore::PositionToAuxDet (geo::Point_t const &point, unsigned int &ad, double tolerance=0) const
- what happens if it does not exist?
- Member geo::GeometryCore::PositionToAuxDetSensitive (geo::Point_t const &point, size_t &ad, size_t &sv, double tolerance=0) const
- what happens if it does not exist?
- Member geo::GeometryCore::PositionToAuxDetSensitive (double const worldLoc[3], size_t &ad, size_t &sv, double tolerance=0) const
- what happens if it does not exist?
- Member geo::GeometryCore::SignalType (raw::ChannelID_t const channel) const
- verify that kMysteryType is returned on invalid channel
- Member geo::GeometryCore::SignalType (geo::PlaneID const &pid) const
- verify that kMysteryType is returned on invalid plane
- Member geo::GeometryCore::SurfaceY () const
- check that this is actually how it is used
- Member geo::GeometryCore::TPC (unsigned int const tpc=0, unsigned int const cstat=0) const
- remove the version with integers
- Member geo::GeometryCore::ValueInRange (double value, double min, double max) const
Use wiggle instead of 10^-6
resort source code for a bit of speed up
- Member geo::GeometryCore::VolumeName (geo::Point_t const &point) const
what happens if none?
Unify the coordinates type
- Member geo::GeometryCore::WireCoordinate (double YPos, double ZPos, geo::PlaneID const &planeid) const
- Unify (y, z) coordinate
- Member geo::GeometryCore::WirePitch (geo::PlaneID const &planeid) const
add a version with wire IDs
deprecate this function
document what will happen (in the future methods) with wires on different planes
- Member geo::PlaneGeo::NearestWireID (geo::Point_t const &pos) const
- When the ID is out of range, instead of throwing an exception, return an invalid wire ID with the wire number set to the non-existing wire which would be the nearest to
pos.
- Member geo::PlaneGeo::WidthDepthProjection_t
- the following should be a PositionVector2D
- Class geo::vect::CoordConstIterator< Vector >
- Not fully unit-tested yet!
- Member geo::WireID::planeID () const
- Remove the instances of geo::WireID::planeID() in the code
- Member getGenericProcedureLink (const ClassDef *cd, const char *memberText, CodeOutputInterface &ol)
- implementation
- Member hit::DisambigCheater::InitHitToWids (detinfo::DetectorClocksData const &clockData, const std::vector< art::Ptr< recob::Hit >> &ChHits)
: Why would an IDE xyz position be outside of a TPC?
: Why would an IDE ossociated with a hit return a nearest wire not on the hit's channel? Usually only one off.
- Member hit::DisambigCheater::produce (art::Event &e) override
- : Decide how to handle the hits with multiple-wid activity. For now, randomly choose.
- Member hit::FFTHitFinder::produce (art::Event &evt) override
- just get the integral from the fit for totSig
need to have a disambiguation algorithm somewhere in here
- multiplicity and local_index have to be determined
- Member lar::double_fwd_const_iterator
- write documentation about how to use it
- Class lar::sparse_vector< T >
- backward iteration; reverse iterators; iterator on non-void elements only; iterator on non-void elements only, returning a pair (index;value)
- Member lar::util::TrackProjectedLength (recob::Track const &track, geo::View_t view)
CAREFUL: using view to determine projected length does not work for DUNE. Need to think more about this.
is this right, or should it be dist*cosgamma???
is this right, or should it be dist*cosgamma???
- Member larg4::G4BadIdeaAction::SteppingAction (const G4Step *)
Do not copy the code below. Contact Brian Rebel, Eric Church,
Bill Seligman and Andrzej Szelc for reasons why not to!
- Member larg4::LArVoxelReadout::DriftIonizationElectrons (detinfo::DetectorClocksData const &clockData, G4ThreeVector stepMidPoint, const double simTime, int trackID, unsigned short int cryostat, unsigned short int tpc)
think about effects of drift between planes
think about effects of drift between planes
check on what happens if we allow the tdc value to be
beyond the end of the expected number of ticks
- Class proxy::details::OneTo01Data< Main, Aux, Metadata, Tag >
- Metadata for
proxy::details::OneTo01Data is not supported yet.
- Member proxy::withZeroOrOneMeta (Args &&...args)
- Metadata is not supported yet (the interface is apparently there though).
- Member proxy::withZeroOrOneMetaAs (Args &&...args)
- Metadata is not supported yet.
- Class recob::test::HitCollectionCreatorTest
- Member shwf::ShowerCheater::produce (art::Event &evt) override
- figure out the max transverse width of the shower in the x and y directions
- Member shwf::ShowerFinder::produce (art::Event &evt)
This method appears to produce a recob::Cluster really as it is
a collection of 2D clusters from single planes
NEED TO TAKE OUT THE HOUGH LINES FROM THE PROTOSHOWERS!!!!! !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
really need to determine the values of the arguments of the recob::Shower ctor
check if protoshower from further vertex is also contained in vertex nearer...
- Member shwf::ShowerReco::beginJob ()
- the call to geo->Nplanes() assumes this is a single cryostat and single TPC detector; need to generalize to multiple cryostats and TPCs
- Member shwf::ShowerReco::produce (art::Event &evt)
- really need to determine the values of the arguments of the recob::Shower ctor
- Member testing::ProviderSetupClass< detinfo::DetectorPropertiesStandard >::setup (fhicl::ParameterSet const &pset, detinfo::DetectorPropertiesStandard::providers_type const &providers)
- remove this debug stuff!
- Class Todo
- This still needs to be done.
- Member Todo::todo ()
- more things to do here
- Member trkf::SpacePointAlg::makeSpacePoints (detinfo::DetectorClocksData const &clockData, detinfo::DetectorPropertiesData const &detProp, const art::PtrVector< recob::Hit > &hits, std::vector< recob::SpacePoint > &spts, bool useMC) const
Why are we still checking on whether this is MC or not?
Such checks should not be in reconstruction code.
- Member trkf::SpacePts::produce (art::Event &evt)
- really should fill the direction cosines with unique values
- Member trkf::Track3DKalman::produce (art::Event &evt) override
- : remove this statement, there is no place for checks on isRealData in reconstruction code
- Member trkf::Track3DKalmanSPS::produce (art::Event &evt)
Should never test whether the event is real data in reconstruction algorithms
as that introduces potential data/MC differences that are very hard to track down
Remove this test as soon as possible please
Should never test whether the event is real data in reconstruction algorithms
as that introduces potential data/MC differences that are very hard to track down
Remove this test as soon as possible please
- Member trkf::Track3Dreco::produce (art::Event &evt) override
- : This is very bad practice and should be changed ASAP
- Member trkf::TrackAnaCT::TrackAnaCT (fhicl::ParameterSet const &pset)
- Move this module to DUNE code and remove it from larreco
- Member util::GeometryUtilities::Get2DPointProjection (Double_t *xyz, Int_t plane) const
- : this should use the cryostat and tpc as well in the NearestWire method
- Member util::GeometryUtilities::Get2DPointProjectionCM (double *xyz, int plane) const
- : this should use the cryostat and tpc as well in the NearestWire method
- Member util::GeometryUtilities::Get2DPointProjectionCM (std::vector< double > xyz, int plane) const
- : this should use the cryostat and tpc as well in the NearestWire method
- Member util::GeometryUtilities::PitchInView (UInt_t plane, Double_t phi, Double_t theta) const
switch to using new Geometry::WireAngleToVertical(geo::View_t)
and Geometry::WirePitch(geo::View_t) methods
- Member vertex::PrimaryVertexFinder::produce (art::Event &evt)
- replace with the real query when this module is updated to look for showers too
- Member vertex::VertexFinder2D::produce (art::Event &evt) override
should really get the actual vector of hits corresponding to end point
for now will get all hits from the current cluster
need to actually make tracks and showers to go into 3D vertex
currently just passing empty collections to the ctor
1.8.11