Skip to content
Snippets Groups Projects
Commit aac94489 authored by Christopher Rob Jones's avatar Christopher Rob Jones Committed by Christopher Rob Jones
Browse files

RichTrackSegment: revert (for now) change to phi computed by angleToDirection

parent 9c045cb5
No related branches found
No related tags found
2 merge requests!4666RichTrackSegment: General Cleanup,!4647Synchronize master branch with 2024-patches
Pipeline #8100483 passed
......@@ -360,9 +360,11 @@ namespace LHCb {
theta = my_atan2<THETA_PRECISION>( perp, r.z() );
phi = my_atan2<PHI_PRECISION>( r.y(), r.x() );
// correct phi to range 0 - 2PI
constexpr TYPE pi = TYPE( M_PI );
// constexpr TYPE pi = TYPE( M_PI );
constexpr TYPE twopi = TYPE( 2.0 * M_PI );
phi += pi;
/// @todo : Fix me. Minor bug fix but will changes a lot of refs so defer to later on
if ( phi < 0 ) { phi += twopi; }
// phi += pi;
} else if constexpr ( LHCb::SIMD::is_SIMD_v<TYPE> ) {
// SIMD version
// create vector in track reference frame
......@@ -372,7 +374,9 @@ namespace LHCb {
theta = my_atan2<THETA_PRECISION>( perp, r.z() );
phi = my_atan2<PHI_PRECISION>( r.y(), r.x() );
// correct phi to range 0 - 2PI
phi += TYPE( M_PI );
/// @todo : Fix me. Minor bug fix but will changes a lot of refs so defer to later on
phi( phi < TYPE::Zero() ) += TYPE( 2.0 * M_PI );
// phi += TYPE( M_PI );
} else {
// If get here unknown types so force compilation failure
TYPE::WillFail();
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment