[Brahms-dev-l] bug fixed in banapp MicroTreeReader

From: <i.c.arsene_at_fys.uio.no>
Date: Fri, 20 Oct 2006 17:38:02 +0200 (CEST)
     Hi,

   I spotted a serious bug in banapp, MicroTreeReader::InsideCent()
function. This is what i found in the latest banapp version:
Bool_t MicroTreeReader::InsideCent()
{
  if (fGlobal->GetCentrality() < fMinVtx || fGlobal->GetCentrality() >
fMaxVtx)
    return kFALSE;

  return kTRUE;
}
   As you can see, the centrality cut is done using the VERTEX limits. The
correct way is
Bool_t MicroTreeReader::InsideCent()
{
  if (fGlobal->GetCentrality() < fMinCent || fGlobal->GetCentrality() >
fMaxCent)
    return kFALSE;

  return kTRUE;
}
   I corrected the bug and submitted the change to CVS. I don't know if
this function is used to select events in method2 of banapp, but for
sure it is used in method1. So whoever is using method1 programs
(tree2datamap.cxx) should be aware of this.

              Cheers,
                  Ionut

_______________________________________________
Brahms-dev-l mailing list
Brahms-dev-l_at_lists.bnl.gov
http://lists.bnl.gov/mailman/listinfo/brahms-dev-l
Received on Fri Oct 20 2006 - 11:39:22 EDT

This archive was generated by hypermail 2.2.0 : Fri Oct 20 2006 - 11:39:46 EDT