Skip to content

Commit

Permalink
fix: Fix units in RootParticleReader (acts-project#2477)
Browse files Browse the repository at this point in the history
Effectively only time was broken but I made the other units also explicit.
  • Loading branch information
andiwand authored and AJPfleger committed Sep 29, 2023
1 parent fc7a9b1 commit 1b5793a
Showing 1 changed file with 8 additions and 4 deletions.
12 changes: 8 additions & 4 deletions Examples/Io/Root/src/RootParticleReader.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -147,12 +147,16 @@ ActsExamples::ProcessCode ActsExamples::RootParticleReader::read(

p.setProcess(static_cast<ActsFatras::ProcessType>((*m_process)[i]));
p.setPdg(static_cast<Acts::PdgParticle>((*m_particleType)[i]));
p.setCharge((*m_q)[i]);
p.setMass((*m_m)[i]);
p.setCharge((*m_q)[i] * Acts::UnitConstants::e);
p.setMass((*m_m)[i] * Acts::UnitConstants::GeV);
p.setParticleId((*m_particleId)[i]);
p.setPosition4((*m_vx)[i], (*m_vy)[i], (*m_vz)[i], (*m_vt)[i]);
p.setPosition4((*m_vx)[i] * Acts::UnitConstants::mm,
(*m_vy)[i] * Acts::UnitConstants::mm,
(*m_vz)[i] * Acts::UnitConstants::mm,
(*m_vt)[i] * Acts::UnitConstants::ns);
// NOTE: depends on the normalization done in setDirection
p.setDirection((*m_px)[i], (*m_py)[i], (*m_pz)[i]);
p.setAbsoluteMomentum((*m_p)[i]);
p.setAbsoluteMomentum((*m_p)[i] * Acts::UnitConstants::GeV);

particleContainer.insert(particleContainer.end(), p);
priVtxCollection.push_back((*m_vertexPrimary)[i]);
Expand Down

0 comments on commit 1b5793a

Please sign in to comment.