 |
 |
Hi Norman, I can fix change this in the offline clustering code. I would like to remind you that sampling fractions are actually on the order of about 0.89 until corrected. I strongly feel that we need to preserve both of these energies because I would like to compare what we really find with how I calculated the energy corrections in simulation. Using the setEnergy() method later in reconstruction will overwrite the original energy. At the moment, I've removed energy corrections from the clustering since it's necessary that getEnergy() returns the raw (uncorrected) energy for use in the event display in the counting house. All corrections will be done later in reconstruction with cluster track matching, but I don't think it's good to overwrite the raw/uncorrected energy yet. On Mon, Dec 15, 2014 at 4:13 PM, Graf, Norman A. <[log in to unmask]> wrote: > > Hello Holly et al., > > Here’s my proposal: > > We change the call to getRawEnergy() in > HPSEcalClusterIC.recalculateForParticleID() > > to getEnergy(). Our sampling fractions are 1.0 so the returned energy is > the same in either > > case. This method is not called at present, so it does not change any > behavior. > > We then eliminate the references to raw energy in > org.lcsim.event.baseBaseCluster. > > I can do this now, or we can discuss this further in detail. > > Norman > > > > *From:* Holly Vance [mailto:[log in to unmask]] > *Sent:* Monday, December 15, 2014 1:06 PM > *To:* Graf, Norman A. > *Cc:* Uemura, Sho; Kyle McCarty; hps-software > > *Subject:* Re: EVIO to LCIO: Raw Hit Energy > > > > Hi all, > > > > The raw energy is calculated separately and then set because it includes > considerations for hits shared between clusters (energy is distributed > then). Also, the raw energy is used separately to calculate the corrected > energy. > > > > It is a bit of a question of semantics, but there must be a distinction > for the offline clustering since this is where the corrections happen. > > > > Let me know how we wish to proceed. > > > > On Mon, Dec 15, 2014 at 3:10 PM, Graf, Norman A. <[log in to unmask]> > wrote: > > Hello All, > This is complicated by the fact that HPSEcalClusterIC actually calls > getRawEnergy() before > calculating the cluster position and energy corrections. I am checking to > understand > whether this is just a semantic issue. > Norman > > > -----Original Message----- > From: [log in to unmask] [mailto: > [log in to unmask]] On Behalf Of Sho Uemura > Sent: Monday, December 15, 2014 10:54 AM > To: Kyle McCarty > Cc: hps-software > Subject: Re: EVIO to LCIO: Raw Hit Energy > > This is a known incompatibility between the LCSim builtin classes - a > CalorimeterHit read from LCIO (SIOCalorimeterHit) has a corrected energy > but no raw energy, and the LCSim base implementation of Cluster > (BaseCluster) uses the hit raw energy. We've always tried to avoid this by > using corrected energy when possible. > > The HPS subclasses of BaseCluster do not override that behavior; it's not > "our fault" that the cluster code crashes. But either BaseCluster gets > changed so it doesn't use raw energy, or we override the methods of > BaseCluster that do. > > On Mon, 15 Dec 2014, Kyle McCarty wrote: > > > Hello hps-software, > > > > I am trying to write a version of the GTP clustering algorithm to run > > over EVIO readout data and form clusters from the hits. This is useful > > both to get cluster data that closely matches what the hardware is > > actually seeing and also for trigger verification and diagnostic > > drivers, since these will need to run the trigger over clusters that > match the hardware. > > > > I am running into a problem with this because the readout hits do not > > declare a raw energy and attempting to access it causes a > RuntimeException. > > Normally, we use corrected energy, which is available, but the addHit > > method in HPSEcalCluster apparently calls getRawEnergy at some point > > and thusly crashes the simulation. > > > > Is there a way to fix this so that the hits in the EVIO readout will > > have this value when converted to LCIO? Alternatively, should we > > change the HPSEcalCluster to use the corrected energy? > > > > Thanks, > > > > Kyle > > > > ###################################################################### > > ## > > Use REPLY-ALL to reply to list > > > > To unsubscribe from the HPS-SOFTWARE list, click the following link: > > https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=HPS-SOFTWARE&A=1 > > > > ######################################################################## > Use REPLY-ALL to reply to list > > To unsubscribe from the HPS-SOFTWARE list, click the following link: > https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=HPS-SOFTWARE&A=1 > > ######################################################################## > Use REPLY-ALL to reply to list > > To unsubscribe from the HPS-SOFTWARE list, click the following link: > https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=HPS-SOFTWARE&A=1 > > > -- > BEGIN-ANTISPAM-VOTING-LINKS > ------------------------------------------------------ > > NOTE: This message was trained as non-spam. If this is wrong, > please correct the training as soon as possible. > > Teach CanIt if this mail (ID 01NrUbF6w) is spam: > > Spam: about:blank > > Not spam: about:blank > > Forget vote: about:blank > ------------------------------------------------------ > END-ANTISPAM-VOTING-LINKS > > ------------------------------ > > Spam > <https://www.spamtrap.odu.edu/canit/b.php?i=01NrVeyNR&m=21768ab16559&t=20141215&c=s> > Not spam > <https://www.spamtrap.odu.edu/canit/b.php?i=01NrVeyNR&m=21768ab16559&t=20141215&c=n> > Forget previous vote > <https://www.spamtrap.odu.edu/canit/b.php?i=01NrVeyNR&m=21768ab16559&t=20141215&c=f> > ######################################################################## Use REPLY-ALL to reply to list To unsubscribe from the HPS-SOFTWARE list, click the following link: https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=HPS-SOFTWARE&A=1