Fork me on GitHub

Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#363 closed Enhancement (fixed)

CalorimeterTowerDither

Reported by: Keith Pedersen Owned by: Pavel Demin <pavel.demin@…>
Priority: minor Milestone:
Component: Delphes code Version: Delphes 3
Keywords: Cc:

Description

Hello,

I would like to suggest that the dithering of the center of Calorimeter towers be turned into a run-time as a opposed to a compile-time option. I have attached a diff-patch that completes this request.

My assumption is that dithering is there to remove the bias of the extremely repetitive grid, but I may soon start looking at narrow jets, and I don't want the dithering to have a large effect on what gets clustered.

Thank you,
Keith

Attachments (1)

Calorimeter_Dither.patch (1.6 KB ) - added by Keith Pedersen 10 years ago.
A diff patch (i.e. [you@earth]$ patch Calorimeter.cc < Calorimeter_Dither.patch)

Download all attachments as: .zip

Change History (4)

by Keith Pedersen, 10 years ago

Attachment: Calorimeter_Dither.patch added

A diff patch (i.e. [you@earth]$ patch Calorimeter.cc < Calorimeter_Dither.patch)

comment:1 by Pavel Demin <pavel.demin@…>, 10 years ago

Owner: set to Pavel Demin <pavel.demin@…>
Resolution: fixed
Status: newclosed

comment:2 by Pavel Demin, 10 years ago

Dear Keith,

Thank you for the patch.

I've just slightly adjusted it so that the names of the class variable and of the configuration parameter are more consistent: fDitherTowerCenter and DitherTowerCenter.

Regards,

Pavel

comment:3 by Michele Selvaggi, 10 years ago

Hi Keith,

just a comment, we changed the parameter name into : SmearTowerCenter
This is how it will appear in 3.2.0 (which we will release in a few hours).

Michele

Note: See TracTickets for help on using tickets.