Skip to content

Commit

Permalink
Apply time offset only for reco-wire. This is not a hack but may not …
Browse files Browse the repository at this point in the history
…be an ideal solution. Basically this timeoffset was introduced to provide a user capability to deal with different reco-wire time offset used in larsoft production (which is not stored in data). But now timeoffset can be varied, in the data produciton, separately for wire and MC. We now use supera timeoffset parameter to deal with ONLY wire, and not MC. If we need a capability to SEPARATELY vary timeoffset for MC, we'll implment that change when needed. --kazu
  • Loading branch information
drinkingkazu committed Aug 16, 2016
1 parent 47ae9cc commit 2fd261e
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion app/Supera/larfmwk_shared/SuperaCore.inl
Original file line number Diff line number Diff line change
Expand Up @@ -613,9 +613,10 @@ namespace larcv {

// Initialize column vector
for (auto& v : column) v = (float)(::larcv::kROIUnknown);
//for (auto& v : column) v = (float)(-1);

for (auto const tick_ides : sch.TDCIDEMap()) {
int tick = (TPCTDC2Tick((double)(tick_ides.first)) + time_offset);
int tick = TPCTDC2Tick((double)(tick_ides.first));
if (tick < meta.min_y()) continue;
if (tick >= meta.max_y()) continue;
// Where is this tick in column vector?
Expand Down

0 comments on commit 2fd261e

Please sign in to comment.