Difference: HLTPendingIssues (1 vs. 5)

Revision 52007-03-13 - HugoRuiz

Line: 1 to 1
Changed:
<
<
>
>
 

Bugs

  • TrackIPSelection crashes if outputVerticesName is provided  (HR 07/03/2007)
  • TrackIPSelection always executes endExecute   (HR 07/03/2007)
Line: 14 to 14
 
  • Change location of 2D VELO tracks from "Rec/Track/RZVelo" to "Hlt/Track/RZVelo" (HR 07/03/2007)
  • Naming convention to distinguish locations of Pat and Hlt containers? (HR 07/03/2007)
  • Nomenclature about passed/accepted (HR 09/03/2007)
Changed:
<
<
  • What does  long m_nCandidates in HltBaseAlg? If it is really the minimum number of candidates required, put it in the name of the variable (HR 13/03/2007)
>
>
  • What does  long m_nCandidates in HltBaseAlg? If it is really the minimum number of candidates required, put it in the name of the variable (HR 13/03/2007)
  • beginExecute and endExecute are needed? If so, clarify who does what: HltBaseAlgo and HltAlgorithm. (HR 13/03/2007)
 

Robustness

  • size() must produce a WARNING when size = 0, as this is not expected in normal running of HLT   (HR 07/03/2007)

Missing tools

Revision 42007-03-13 - HugoRuiz

Line: 1 to 1
Changed:
<
<
>
>
 

Bugs

  • TrackIPSelection crashes if outputVerticesName is provided  (HR 07/03/2007)
  • TrackIPSelection always executes endExecute   (HR 07/03/2007)
Line: 14 to 14
 
  • Change location of 2D VELO tracks from "Rec/Track/RZVelo" to "Hlt/Track/RZVelo" (HR 07/03/2007)
  • Naming convention to distinguish locations of Pat and Hlt containers? (HR 07/03/2007)
  • Nomenclature about passed/accepted (HR 09/03/2007)
Added:
>
>
  • What does  long m_nCandidates in HltBaseAlg? If it is really the minimum number of candidates required, put it in the name of the variable (HR 13/03/2007)
 

Robustness

  • size() must produce a WARNING when size = 0, as this is not expected in normal running of HLT   (HR 07/03/2007)

Missing tools

Line: 23 to 24
 
  • Vertex ancestry by vertices?
  • Clever functions not to recompute AppendedValues   (HR 07/03/2007)
  • Reactivate candidatesFound   (HR 07/03/2007)
Added:
>
>
    • Counter for nCandidates per event
    • Set filter to passed if nCandidates>minNCandidates
    • Histogram nCandidates per event
    • Count nTotalCandidates (cumulated in all events)
    • What is the use of setDecision on top of setFilterPassed?
 
  • Reactivate nRequiredCandidates   (HR 07/03/2007)

-- HugoRuiz - 07 Mar 2007

Revision 32007-03-09 - HugoRuiz

Line: 1 to 1
Changed:
<
<
>
>
 

Bugs

  • TrackIPSelection crashes if outputVerticesName is provided  (HR 07/03/2007)
  • TrackIPSelection always executes endExecute   (HR 07/03/2007)
Line: 13 to 13
 
  • Give more informative names to vertex "banks"? (HR 07/03/2007)
  • Change location of 2D VELO tracks from "Rec/Track/RZVelo" to "Hlt/Track/RZVelo" (HR 07/03/2007)
  • Naming convention to distinguish locations of Pat and Hlt containers? (HR 07/03/2007)
Added:
>
>
  • Nomenclature about passed/accepted (HR 09/03/2007)
 

Robustness

Changed:
<
<
  • size() produces a WARNING when size = 0, as this is not expected in normal running of HLT   (HR 07/03/2007)
>
>
  • size() must produce a WARNING when size = 0, as this is not expected in normal running of HLT   (HR 07/03/2007)
 

Missing tools

Added:
>
>
  • More information at the end: number of candidates found, number of standard input and output objects read/written (HR 09/03/2007)
  • Add a default PatOutputVertices to the base class (HR/JH 09/03/2007)
 
  • Heritage of AppendedValues to daughter tracks and vertices  (HR 07/03/2007)
  • Vertex ancestry by vertices?
  • Clever functions not to recompute AppendedValues   (HR 07/03/2007)

Revision 22007-03-07 - HugoRuiz

Line: 1 to 1
Changed:
<
<
>
>
 

Bugs

  • TrackIPSelection crashes if outputVerticesName is provided  (HR 07/03/2007)
  • TrackIPSelection always executes endExecute   (HR 07/03/2007)
Changed:
<
<
  • TrackIPSelection returns StatusCode::SUCCESS if beginExecute fails   (HR 07/03/2007)
>
>
  • HltAlgorithm.cpp: typo on a property: vertices is not in capital letters in declareProperty("OutputverticesName",  m_outputVerticesName = "");
 

Clarity

Deleted:
<
<
  • Rename monitor() to something like fillWithAppendedValues()   (HR 07/03/2007)
 
  • Find an appropiate name for AppendedValues... document and name everything accordingly.   (HR 07/03/2007)
Added:
>
>
  • Rename:
    • monitor() to something like fillWithAppendedValues()   (HR 07/03/2007)
    • map() to appendValues() (HR 07/03/2007)
    • size() to monitorContainerSize() (HR 07/03/2007)
  • Give more informative names to vertex "banks"? (HR 07/03/2007)
  • Change location of 2D VELO tracks from "Rec/Track/RZVelo" to "Hlt/Track/RZVelo" (HR 07/03/2007)
  • Naming convention to distinguish locations of Pat and Hlt containers? (HR 07/03/2007)

Robustness

  • size() produces a WARNING when size = 0, as this is not expected in normal running of HLT   (HR 07/03/2007)
 

Missing tools

  • Heritage of AppendedValues to daughter tracks and vertices  (HR 07/03/2007)
  • Vertex ancestry by vertices?

Revision 12007-03-07 - HugoRuiz

Line: 1 to 1
Added:
>
>

Bugs

  • TrackIPSelection crashes if outputVerticesName is provided  (HR 07/03/2007)
  • TrackIPSelection always executes endExecute   (HR 07/03/2007)
  • TrackIPSelection returns StatusCode::SUCCESS if beginExecute fails   (HR 07/03/2007)

Clarity

  • Rename monitor() to something like fillWithAppendedValues()   (HR 07/03/2007)
  • Find an appropiate name for AppendedValues... document and name everything accordingly.   (HR 07/03/2007)

Missing tools

  • Heritage of AppendedValues to daughter tracks and vertices  (HR 07/03/2007)
  • Vertex ancestry by vertices?
  • Clever functions not to recompute AppendedValues   (HR 07/03/2007)
  • Reactivate candidatesFound   (HR 07/03/2007)
  • Reactivate nRequiredCandidates   (HR 07/03/2007)

-- HugoRuiz - 07 Mar 2007

 
This site is powered by the TWiki collaboration platform Powered by PerlCopyright & 2008-2019 by the contributing authors. All material on this collaboration platform is the property of the contributing authors.
Ideas, requests, problems regarding TWiki? Send feedback