spartyjet is hosted by Hepforge, IPPP Durham
close Warning: Can't synchronize with repository "(default)" (/hepforge/svn/spartyjet does not appear to be a Subversion repository.). Look in the Trac log for more information.

Opened 9 years ago

Last modified 9 years ago

#21 assigned task

Check phi conventions

Reported by: cvermilion Owned by: cvermilion
Priority: minor Component: misc
Version: 4.0 Keywords:


PseudoJet::phi() returns [0,2pi], but I think our old convention was [-pi,pi] (see recent versions of EventView?.cc, eg). We should change phi() to phi_std() everywhere that it matters! Potentially a big bug somewhere, maybe not.

Change History (2)

comment:1 Changed 9 years ago by cvermilion

Version: 4.03.6

comment:2 Changed 9 years ago by cvermilion

Component: FastJetCoremisc
Priority: majorminor
Status: newassigned
Version: 3.64.0

Looked through all files containing phi(), I think everything is safe. Not 100% positive for HullMomentTool?, since the mechanics of the cylindrical geometry functions are pretty opaque.

Note: See TracTickets for help on using tickets.