Skip to content

Commit c11b077

Browse files
author
Ibrahim Safa
committed
Merge branch 'release/v10_03_01'
sbncode release for larsoft v10_03_01
2 parents 40af989 + 3c7cbcf commit c11b077

3 files changed

Lines changed: 23 additions & 20 deletions

File tree

CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616
cmake_minimum_required(VERSION 3.20 FATAL_ERROR)
1717

1818
find_package(cetmodules 3.20.00 REQUIRED)
19-
project(sbncode VERSION 10.01.04 LANGUAGES CXX)
19+
project(sbncode VERSION 10.03.01 LANGUAGES CXX)
2020

2121
message(STATUS "\n\n ========================== ${PROJECT_NAME} ==========================")
2222

sbncode/Cluster3D/SnippetHit3DBuilderSBN_tool.cc

Lines changed: 20 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@
2525
// LArSoft includes
2626
#include "larcore/Geometry/Geometry.h"
2727
#include "larcore/Geometry/WireReadout.h"
28+
#include "larcorealg/Geometry/WireReadoutGeom.h"
2829
#include "lardata/ArtDataHelper/HitCreator.h"
2930
#include "lardata/DetectorInfoServices/DetectorClocksService.h"
3031
#include "lardata/DetectorInfoServices/DetectorPropertiesService.h"
@@ -369,6 +370,7 @@ void SnippetHit3DBuilderSBN::configure(fhicl::ParameterSet const &pset)
369370
m_outputHistograms = pset.get<bool >("OutputHistograms", false);
370371
m_makeAssociations = pset.get<bool >("MakeAssociations", false);
371372

373+
m_geometry = art::ServiceHandle<geo::Geometry const>{}.get();
372374
m_wireReadout = &art::ServiceHandle<geo::WireReadout const>{}->Get();
373375

374376
// Returns the wire pitch per plane assuming they will be the same for all TPCs
@@ -1822,27 +1824,28 @@ void SnippetHit3DBuilderSBN::CollectArtHits(const art::Event& evt) const
18221824
std::map<geo::PlaneID,double> planeIDToPositionMap;
18231825

18241826
// Initialize the plane to hit vector map
1825-
for(auto const& tpcID : m_geometry->Iterate<geo::TPCID>())
1827+
for(auto const& tpcGeo : m_geometry->Iterate<geo::TPCGeo>())
18261828
{
1827-
m_planeToSnippetHitMap[geo::PlaneID(tpcID,0)] = SnippetHitMap();
1828-
m_planeToSnippetHitMap[geo::PlaneID(tpcID,1)] = SnippetHitMap();
1829-
m_planeToSnippetHitMap[geo::PlaneID(tpcID,2)] = SnippetHitMap();
1829+
m_planeToSnippetHitMap[geo::PlaneID(tpcGeo.ID(),0)] = SnippetHitMap();
1830+
m_planeToSnippetHitMap[geo::PlaneID(tpcGeo.ID(),1)] = SnippetHitMap();
1831+
m_planeToSnippetHitMap[geo::PlaneID(tpcGeo.ID(),2)] = SnippetHitMap();
18301832

1831-
// Should we provide output?
1832-
if (!m_weHaveAllBeenHereBefore)
1833-
{
1834-
std::ostringstream outputString;
1833+
// Should we provide output?
1834+
if (!m_weHaveAllBeenHereBefore)
1835+
{
1836+
std::ostringstream outputString;
18351837

1836-
outputString << "***> plane 0 offset: " << m_PlaneToT0OffsetMap.find(geo::PlaneID(tpcID,0))->second
1837-
<< ", plane 1: " << m_PlaneToT0OffsetMap.find(geo::PlaneID(tpcID,1))->second
1838-
<< ", plane 2: " << m_PlaneToT0OffsetMap.find(geo::PlaneID(tpcID,2))->second << "\n";
1839-
outputString << " Det prop plane 0: " << det_prop.GetXTicksOffset(geo::PlaneID(tpcID,0)) << ", plane 1: " << det_prop.GetXTicksOffset(geo::PlaneID(tpcID,1)) << ", plane 2: " << det_prop.GetXTicksOffset(geo::PlaneID(tpcID,2)) << ", Trig: " << trigger_offset(clock_data) << "\n";
1840-
debugMessage += outputString.str() + "\n";
1841-
}
1838+
outputString << "***> plane 0 offset: " << m_PlaneToT0OffsetMap.find(geo::PlaneID(tpcGeo.ID(),0))->second
1839+
<< ", plane 1: " << m_PlaneToT0OffsetMap.find(geo::PlaneID(tpcGeo.ID(),1))->second
1840+
<< ", plane 2: " << m_PlaneToT0OffsetMap.find(geo::PlaneID(tpcGeo.ID(),2))->second << "\n";
1841+
outputString << " Det prop plane 0: " << det_prop.GetXTicksOffset(geo::PlaneID(tpcGeo.ID(),0)) << ", plane 1: " << det_prop.GetXTicksOffset(geo::PlaneID(tpcGeo.ID(),1)) << ", plane 2: " << det_prop.GetXTicksOffset(geo::PlaneID(tpcGeo.ID(),2)) << ", Trig: " << trigger_offset(clock_data) << "\n";
1842+
1843+
debugMessage += outputString.str() + "\n";
1844+
}
18421845

1843-
geo::PlaneID const planeID2{tpcID, 2};
1844-
double xPosition(det_prop.ConvertTicksToX(0., planeID2));
1845-
planeIDToPositionMap[planeID2] = xPosition;
1846+
geo::PlaneID const planeID2{tpcGeo.ID(), 2};
1847+
double xPosition(det_prop.ConvertTicksToX(0., planeID2));
1848+
planeIDToPositionMap[planeID2] = xPosition;
18461849
}
18471850

18481851
if (!m_weHaveAllBeenHereBefore)

ups/product_deps

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -254,11 +254,11 @@ libdir fq_dir lib
254254
product version qual flags <table_format=2>
255255
genie_xsec v3_04_00 -
256256
larcv2 v2_2_6 -
257-
larsoft v10_01_04 -
257+
larsoft v10_03_01 -
258258
sbnanaobj v09_23_02_01 -
259259
sbndaq_artdaq_core v1_10_04 -
260260
sbndata v01_07 -
261-
sbnobj v10_00_01 -
261+
sbnobj v10_00_02 -
262262
systematicstools v01_04_04 -
263263
nusystematics v1_05_04 -
264264
cetmodules v3_24_01 - only_for_build

0 commit comments

Comments
 (0)