Skip to content

Commit 6727644

Browse files
sawenzelclaude
andcommitted
Carry the rotation angles of Detector::Matrix in double
This widens the six angle arguments of o2::base::Detector::Matrix from Float_t to Double_t. - The helper only forwards to TVirtualMC::Matrix, which already takes Double_t, so the narrowing served no purpose. - Every detector that places a rotated volume goes through it, so the rotation of every placement was quantised to float. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> (cherry picked from commit 86e074bddc867b1d5278fdc5b6dc42062837cfc3)
1 parent fced507 commit 6727644

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

‎Detectors/Base/include/DetectorsBase/Detector.h‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -87,8 +87,8 @@ class Detector : public FairDetector
8787
/// \param phi1 azimuthal angle for axis I
8888
/// \param phi2 azimuthal angle for axis II
8989
/// \param phi3 azimuthal angle for axis III
90-
void Matrix(Int_t& nmat, Float_t theta1, Float_t phi1, Float_t theta2, Float_t phi2, Float_t theta3,
91-
Float_t phi3) const;
90+
void Matrix(Int_t& nmat, Double_t theta1, Double_t phi1, Double_t theta2, Double_t phi2, Double_t theta3,
91+
Double_t phi3) const;
9292

9393
static void setDensityFactor(Float_t density)
9494
{

‎Detectors/Base/src/Detector.cxx‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -101,8 +101,8 @@ void Detector::SpecialProcess(Int_t numed, EProc parID, int val)
101101
mgr.SpecialProcess(GetName(), numed, parID, val);
102102
}
103103

104-
void Detector::Matrix(Int_t& nmat, Float_t theta1, Float_t phi1, Float_t theta2, Float_t phi2, Float_t theta3,
105-
Float_t phi3) const
104+
void Detector::Matrix(Int_t& nmat, Double_t theta1, Double_t phi1, Double_t theta2, Double_t phi2, Double_t theta3,
105+
Double_t phi3) const
106106
{
107107
TVirtualMC::GetMC()->Matrix(nmat, theta1, phi1, theta2, phi2, theta3, phi3);
108108
}

0 commit comments

Comments
 (0)