Skip to content

Commit

Permalink
quick fix for eta and phi
Browse files Browse the repository at this point in the history
  • Loading branch information
Duchstf committed Jun 7, 2023
1 parent 71c40f3 commit ca2da92
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions L1Trigger/Phase2L1ParticleFlow/plugins/L1NNTauProducer.cc
Original file line number Diff line number Diff line change
Expand Up @@ -266,8 +266,8 @@ void L1NNTauProducer::makeTau_HW(const l1t::PFCandidate& seed,
//Firmware Tau
l1ct::Tau l1ctTau;
l1ctTau.hwPt = l1ct::pt_t(pt); //l1gt is <16,11> and currently <16,14>
l1ctTau.hwEta = l1ct::Scales::makeGlbEta(float(eta));

This comment has been minimized.

Copy link
@perrotta

perrotta Jun 7, 2023

Contributor

Then the variables eta and phi computed right above are not needed any more, and you should remove them: correct?

l1ctTau.hwPhi = l1ct::Scales::makeGlbPhi(float(phi));
l1ctTau.hwEta = l1ct::Scales::makeGlbEta(seed.eta()); // seed.eta() and seed.phi() are in physical coordinates
l1ctTau.hwPhi = l1ct::Scales::makeGlbPhi(seed.phi());

l1ctTau.hwSeedPt = seed.pt();
l1ctTau.hwSeedZ0 = seed.hwZ0();
Expand Down

0 comments on commit ca2da92

Please sign in to comment.