cms-nanoAOD / cmssw

CMS NanoAOD software integration repository
http://cms-sw.github.io/
Apache License 2.0
3 stars 10 forks source link

Store x1 and x2, plus add the mass for high mass gen particles #45

Closed arizzi closed 7 years ago

arizzi commented 7 years ago

as @azzurip suggested we should store Energy rather than Mass for GenPart so that incoming partons x1, x2 can be computed

gpetruc commented 7 years ago

i'd rather store x1 and x2 separately.

for most of the other particles, the mass is both more practical and better compressed.

Giovanni

Il 11 Ott 2017 09:17, "arizzi" notifications@github.com ha scritto:

as @azzurip suggested we should store Energy rather than Mass for GenPart so that incoming partons x1, x2 can be computed

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/cms-nanoAOD/cmssw/issues/45, or mute the thread https://github.com/notifications/unsubscribe-auth/AEbbR3QfFhB6JNVJvEu6ZDUsFruacuYsks5srGuWgaJpZM4P1BEg .

arizzi commented 7 years ago

another option is to store pz or energy only for particles with eta > 1000 "?abs(eta)>1000?E():0"

arizzi commented 7 years ago

@azzurrip

azzurrip commented 7 years ago

Ok the pz of partons at very large eta seems fine to me

arizzi commented 7 years ago

@gpetruc do you have a definition of x1 and x2 that is good for all generators? Othwise I'd go for eta >1000 and store pz of all of them

arizzi commented 7 years ago

I mean: are we sure that x1 and x2 can be taken from first two genparts?

gpetruc commented 7 years ago

the x1 and x2 are in the generator block

Il 11 Ott 2017 12:34, "arizzi" notifications@github.com ha scritto:

I mean: are we sure that x1 and x2 can be taken from first two genparts?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/cms-nanoAOD/cmssw/issues/45#issuecomment-335768256, or mute the thread https://github.com/notifications/unsubscribe-auth/AEbbR8gshSWiz-Edh2YXHQhNwf1C_RT9ks5srJnKgaJpZM4P1BEg .

azzurrip commented 7 years ago

Would also be nice to have the incoming parton ids of the hard process.

On 11 Oct 2017, at 13:00, Giovanni Petrucciani notifications@github.com<mailto:notifications@github.com> wrote:

the x1 and x2 are in the generator block

Il 11 Ott 2017 12:34, "arizzi" notifications@github.com<mailto:notifications@github.com> ha scritto:

I mean: are we sure that x1 and x2 can be taken from first two genparts?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/cms-nanoAOD/cmssw/issues/45#issuecomment-335768256, or mute the thread https://github.com/notifications/unsubscribe-auth/AEbbR8gshSWiz-Edh2YXHQhNwf1C_RT9ks5srJnKgaJpZM4P1BEg .

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/cms-nanoAOD/cmssw/issues/45#issuecomment-335773940, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGlek-3FKnG0mwp-pzpNDQikUjD0XXd5ks5srJ_MgaJpZM4P1BEg.

gpetruc commented 7 years ago

https://github.com/cms-sw/cmssw/blob/master/SimDataFormats/GeneratorProducts/interface/GenEventInfoProduct.h ---> https://github.com/cms-sw/cmssw/blob/master/SimDataFormats/GeneratorProducts/interface/PdfInfo.h

On Wed, Oct 11, 2017 at 1:57 PM, azzurrip notifications@github.com wrote:

Would also be nice to have the incoming parton ids of the hard process.

On 11 Oct 2017, at 13:00, Giovanni Petrucciani <notifications@github.com< mailto:notifications@github.com>> wrote:

the x1 and x2 are in the generator block

Il 11 Ott 2017 12:34, "arizzi" <notifications@github.com<mailto: notifications@github.com>> ha scritto:

I mean: are we sure that x1 and x2 can be taken from first two genparts?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/cms-nanoAOD/cmssw/issues/45#issuecomment-335768256,

or mute the thread https://github.com/notifications/unsubscribe-auth/AEbbR8gshSWiz- Edh2YXHQhNwf1C_RT9ks5srJnKgaJpZM4P1BEg .

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/cms- nanoAOD/cmssw/issues/45#issuecomment-335773940, or mute the thread< https://github.com/notifications/unsubscribe-auth/AGlek-3FKnG0mwp- pzpNDQikUjD0XXd5ks5srJ_MgaJpZM4P1BEg>.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/cms-nanoAOD/cmssw/issues/45#issuecomment-335786024, or mute the thread https://github.com/notifications/unsubscribe-auth/AEbbR43njBlvOZKPBwDCJYctdQVzYnB3ks5srK0mgaJpZM4P1BEg .

arizzi commented 7 years ago
arizzi commented 7 years ago

what is the difference between "x" and "xPDF" and which one do we want to store?

arizzi commented 7 years ago

@perrozzi @bendavid can some of you clarify the difference between x and xPDF ?

arizzi commented 7 years ago

54