Skip to content

Commit

Permalink
Clean up
Browse files Browse the repository at this point in the history
  • Loading branch information
mandrenguyen committed Jul 24, 2013
1 parent d747509 commit 5c02aae
Showing 1 changed file with 1 addition and 20 deletions.
21 changes: 1 addition & 20 deletions Validation/RecoHI/plugins/HitPixelLayersTPSelector.h
Original file line number Diff line number Diff line change
Expand Up @@ -96,26 +96,7 @@ class HitPixelLayersTPSelector
std::vector<bool> pixelHitPattern( const TrackingParticleRef& simTrack, const TrackerTopology *tTopo )
{
std::vector<bool> hitpattern(5,false); // PXB 0,1,2 PXF 0,1
/*
#warning "This file has been modified just to get it to compile without any regard as to whether it still functions as intended"
#ifdef REMOVED_JUST_TO_GET_IT_TO_COMPILE__THIS_CODE_NEEDS_TO_BE_CHECKED
for(std::vector<PSimHit>::const_iterator simHit = simTrack->pSimHit_begin();simHit!= simTrack->pSimHit_end();simHit++){
DetId id = DetId(simHit->detUnitId());
uint32_t detid = id.det();
uint32_t subdet = id.subdetId();
if (detid == DetId::Tracker) {
if (subdet == PixelSubdetector::PixelBarrel)
hitpattern[tTopo->pxbLayer(id)-1]=true;
else if (subdet == PixelSubdetector::PixelEndcap)
hitpattern[tTopo->pxfDisk(id)+2]=true;
}
}// end simhit loop
#endif
*/
// Indeed this no longer functions as intended, but this code should no longer be accessed, since tripletSeedOnly_ has been changed to False. We will clean up when we figure out how to do this properly - Matt Nguyen, 24/7/2013
// This currently will always return false, since we can no loger use the sim hits to check for triplets. This would need to be fixed if we want to enable this feature, but it's not being used at the moment, since tripletSeedOnly is always set to False - Matt Nguyen, 24/7/2013

return hitpattern;
}
Expand Down

0 comments on commit 5c02aae

Please sign in to comment.