Project

General

Profile

Idea #12785

review typedef cryo_tpc_view_hitmap

Added by Ruth Pordes over 3 years ago. Updated almost 2 years ago.

Status:
Accepted
Priority:
Low
Assignee:
-
Category:
-
Target version:
Start date:
05/26/2016
Due date:
% Done:

0%

Estimated time:
Experiment:
-
Duration:

Description

The typedef cryo_tpc_view_hitmap should be reviewed in more detail.
Consider turning the three separate integer indices into a dense index. The one case in which
only the first two indices are used to look up a map of view>
vector<Hit> needs careful
consideration.
A general solution supported by the LArSoft geometry service should be considered and adopted
if possible.

http://cd-docdb.fnal.gov/cgi-bin/RetrieveFile?docid=5766&filename=ReportfromPatternMatchingAlgorithmCodeAnalysis.pdf&version=1


Related issues

Related to LArSoft - Feature #12602: Add a compact representation for the plane ID, TPC ID and cryostat IDAccepted05/11/2016

History

#1 Updated by Gianluca Petrillo over 3 years ago

  • Related to Feature #12602: Add a compact representation for the plane ID, TPC ID and cryostat ID added

#2 Updated by Gianluca Petrillo about 3 years ago

  • Status changed from New to Assigned

#3 Updated by Katherine Lato about 3 years ago

  • Status changed from Assigned to New
  • Assignee changed from Ruth Pordes to Katherine Lato

#4 Updated by Katherine Lato about 3 years ago

  • Assignee deleted (Katherine Lato)

#5 Updated by Katherine Lato about 3 years ago

  • Status changed from New to Accepted
  • Target version set to 2016-4-quarter

Will look at this in 4th quarter 2016. (note, not necessary do it in that quarter, but figure out what to do with it.)

#6 Updated by Katherine Lato almost 3 years ago

  • Target version changed from 2016-4-quarter to 2017-1-quarter

#7 Updated by Katherine Lato over 2 years ago

  • Target version changed from 2017-1-quarter to 2017-4-quarter

#8 Updated by Katherine Lato almost 2 years ago

  • Priority changed from Normal to Low
  • Target version changed from 2017-4-quarter to 2019-1-quarter


Also available in: Atom PDF