Fork me on GitHub

Opened 10 years ago

Closed 10 years ago

Last modified 5 years ago

#346 closed Bug (fixed)

A correction in a comment

Reported by: Sho IWAMOTO Owned by:
Priority: minor Milestone:
Component: Delphes code Version: Delphes 3
Keywords: Cc:

Description

Sorry for a stupid ticket, but for an improvement.

Best,
Sho

  • modules/ParticlePropagator.cc

    old new  
    139139
    140140    if(TMath::Abs(q) < 1.0E-9 || TMath::Abs(fBz) < 1.0E-9)
    141141    {
    142       // solve pt2*t^2 + 2*(px*x + py*y)*t + (fRadius2 - x*x - y*y) = 0
     142      // solve pt2*t^2 + 2*(px*x + py*y)*t - (fRadius2 - x*x - y*y) = 0
    143143      tmp = px*y - py*x;
    144144      discr2 = pt2*fRadius2 - tmp*tmp;

Change History (3)

comment:1 by Pavel Demin, 10 years ago

Indeed.

I've already noticed it while answering another ticket (#327) but forgot to fix it.

The fixed version is now in the git repository and it will be available in the next release.

Thanks for finding this problem and reminding me about it.

comment:2 by Sho IWAMOTO, 10 years ago

Oh, delphes now on Github! Great!!!
I didn't know. Thank you very much for the transition!

(but why don't you have no tags on the git tree?)

Last edited 10 years ago by Sho IWAMOTO (previous) (diff)

comment:3 by Sho IWAMOTO, 10 years ago

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.