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)
Change History (3)
by , 8 years ago
Attachment: | code_snippet added |
---|
by , 8 years ago
Attachment: | EventDisplay.png added |
---|
comment:1 by , 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
Code for EventDisplay: MissingET as dashed track