Opened 14 years ago
Last modified 14 years ago
#21 assigned task
Check phi conventions
| Reported by: | cvermilion | Owned by: | cvermilion |
|---|---|---|---|
| Priority: | minor | Component: | misc |
| Version: | 4.0 | Keywords: | |
| Cc: |
Description
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 14 years ago by
| Version: | 4.0 → 3.6 |
|---|
comment:2 Changed 14 years ago by
| Component: | FastJetCore → misc |
|---|---|
| Priority: | major → minor |
| Status: | new → assigned |
| Version: | 3.6 → 4.0 |
Note: See
TracTickets for help on using
tickets.
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.