Fork me on GitHub

Opened 8 years ago

Last modified 8 years ago

#1066 new Bug

Problems / Feedback: Event Display

Reported by: Soeren Stamm Owned by:
Priority: minor Milestone:
Component: Delphes code Version: Delphes 3
Keywords: Cc:

Description

Hi,

I have a 2 bug reports and some feedback on the Event Display.
(sorry for putting everything into one ticket, I hope it's okay...)

I was following the guide on your webpage:
https://cp3.irmp.ucl.ac.be/projects/delphes/wiki/WorkBook/EventDisplay

Bug 1:
==========================================================
With Delphes 3.4, the Calorimeters in the ATLAS Card changed from "Calorimeter" to "ECal" and "HCal" (I guess the same is true for CMS). Therefore the example script is out-dated.

The default parameters of the script should be (?)

const char *Calorimeters = "ECal,HCal"

instead of "Calorimeter". With this change, the example runs out-of-the box.

Bug 2: The MissingET is not displayed in the EventDisplay.
==========================================================

In "display/DelphesBranchElement.h", in the constructor of DelphesBranchBase, maxPt_ is not initialized, ("maxPt_(maxPt)" is missing)

The constructor should be:

DelphesBranchBase(const char* name="", TClonesArray* branch=NULL, const enum EColor color=kBlack, Float_t maxPt=75.):name_(name),maxPt_(maxPt),branch_(branch),color_(color) {}

Then the missing Et vector is drawn, which brings me to my feedback:

Feedback:
==========================================================
The event display is very good in general. I only have a small issue regarding the missing et.

The TEveArrow class is not projectable. (it does not inherit from TEveProjectable). Therefore, it does not appear in the RPhi view, which is unfortunate.

I would like to have the MissingET displayed as a photon, i.e. as dashed line. I did some testing and I replaced the TEveArrow class with a TEveTrack class.

I attached a screenshot of the modified Event Display and the code that I used. The advantage is, that it is visible in the RPhi view. It might be hard to recognize in the 3D view, but this is also true for photons.

Cheers,
Soeren

Attachments (2)

code_snippet (1.7 KB ) - added by Soeren Stamm 8 years ago.
Code for EventDisplay: MissingET as dashed track
EventDisplay.png (94.0 KB ) - added by Soeren Stamm 8 years ago.

Download all attachments as: .zip

Change History (3)

by Soeren Stamm, 8 years ago

Attachment: code_snippet added

Code for EventDisplay: MissingET as dashed track

by Soeren Stamm, 8 years ago

Attachment: EventDisplay.png added

comment:1 by Pavel Demin, 8 years ago

Hi Soeren,

Many thanks for your feedback!

The bugs are fixed and TEveArrow is replaced with TEveTrack with the following commits:
2d8821c92efaf16a2855168b8beacd13ffa63846/git
7066f9c64a7a3d3e1818637cc48468e590b90958/git
273735da3c93e15993ad2d032a92595bc5ab6259/git

Since the MissingET branch contains only one entry, I've replaced while with if.

Best regards,

Pavel

Note: See TracTickets for help on using tickets.