Skip to content
Snippets Groups Projects

Remove documentation particles from truth record

Merged Mark Sutton requested to merge sutt/athena:fsn into master
1 unresolved thread
1 file
+ 4
4
Compare changes
  • Side-by-side
  • Inline
@@ -4,7+4,7 @@
@@ -4,7+4,7 @@
** @author mark sutton
** @author mark sutton
** @date Sun 2 Nov 2014 11:10:06 CET
** @date Sun 2 Nov 2014 11:10:06 CET
**
**
** Copyright (C) 2002-2019 CERN for the benefit of the ATLAS collaboration
** Copyright (C) 2002-2019 CERN for the benefit of the ATLAS collaboration
    • Hi Mark, sorry to be a pain, but you are supposed to start each file with the copyright in exactly the following format:

      /*
        Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration
      */

      We've not run the scripts for a while, but when we do, this will fail.

      • Author Developer

        Why does the copyright have to be first 3 lines of the file ? and exactly in that format ? Surely just checking for the relevant string and the doing a find and replace for the year should be enough ? I can imaging this would only take about 3 lines for a script. I have a lot of code that uses this format, @author, @date etc for doxygen. In principle even the copyright line itself should really be @copyright. It is not realistic that I go through all the files I have ever written replacing all the headers in this way.

      • Please register or sign in to reply
Please register or sign in to reply
**/
**/
@@ -341,7+341,7 @@
@@ -341,7+341,7 @@
if ( track ) {
if ( track ) {
 
// check it is a final state particle - documentation particles have status() == 3
 
if ( track->status() != 1 ) return false;
 
/// lazy just to avoid a find-replace of measPer to track
/// lazy just to avoid a find-replace of measPer to track
const xAOD::TruthParticle* measPer = track;
const xAOD::TruthParticle* measPer = track;
@@ -399,10 +402,7 @@ bool TrigTrackSelector::selectTrack( const xAOD::TruthParticle* track ) {
@@ -399,10 +402,7 @@ bool TrigTrackSelector::selectTrack( const xAOD::TruthParticle* track ) {
if ( ( track->hasProdVtx() && rp<=inner_radius ) &&
if ( ( track->hasProdVtx() && rp<=inner_radius ) &&
( !track->hasDecayVtx() || rd>outer_radius ) ) final_state = true;
( !track->hasDecayVtx() || rd>outer_radius ) ) final_state = true;
/// leave this in for the moment - we may need to uncomment this afterall
if ( !final_state ) return false;
// if ( track->status() == 3 ) final_state = false; /// check its not a documentation particle
if ( !final_state ) return 0;
double deta = 0;
double deta = 0;
double dphi = 0;
double dphi = 0;
Loading