add castor lowPTjet trigger and reweight threshold via gain v2

[cwohrman]

Dear all,
this update is to implement the newest castor technical triggers how the will be used during VdM run in 2015.
The 4 trigger bits are
60 gap trigger
61 high pt jet trigger
62 low pt jet trigger (new)
63 muon trigger

Because VdM run will be the only data taking with castor and upcoming MC simulation are using CMSSW_7_4_X.

Especiall the new trigger bit 62 has changed significantly compared to the old version. Also the MC samples are already in proccess of being requested.

So it would be very nice to get it into CMSSW_7_4_X.
The same pull request will be done also for CMSSW 7_5_X.

Any help is welcome.

Thanks,
Hauke

[cmsbuild]

A new Pull Request was created by @cwohrman (Hauke) for CMSSW_7_5_X.

add castor lowPTjet trigger and reweight threshold via gain v2

It involves the following packages:

SimCalorimetry/CastorTechTrigProducer

@cmsbuild, @civanch, @nclopezo, @mdhildreth can you please review it and eventually sign? Thanks.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.
If you are a L2 or a release manager you can ask for tests by saying 'please test' in the first line of a comment.
@nclopezo you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[civanch]

@cwohrman , I may be do not understand the code of CastorTTRecord but for me there is a problem:
new class member reweighted_gain is computed inside the loop over digi collection in the method getEnergy_fC, its value come the last digi, so may have whatever value. After that it is applied to all octants in the method getTriggerDecisionPerOctant method. Is this desired?

[cwohrman]

@civanch, the gain is for all Digis the same. And there is no plan in Sim to change this seperatly for the digis. But the gains over all can change very much from 1 to 5 to 20 depending on the collision for pp, pPb, PbPb. Also a gain weighted summution of the digis I wanted to avoid so that also the thresholds in the config script are in fC related to gain 1.

I should also point out that the gain in castor sim is not really used in a commen way. In principle the sim changes also the 'electron response' so you get for example lower fC with higher gain value. And this electron response is coded as one factor for whole castor and thats why you have the same gain for all digis.

[civanch]

please test

[cmsbuild]

The tests are being triggered in jenkins.

[civanch]

@cwohrman , OK, this become clear. How this reweighted_gain appears in the digi collection?

[cwohrman]

@civanch , the gain is in the and comming from database. And there it looks in the table for the corresponding digi to get the value. But it is the same for every digi.

The point was to avoid adc saturation by higher gain the number of photon electrons produced by cherenkov photons will be reduced. In the end you get then the correct energy again. But the formular which calculates the photoelectrons from cherekov ones is for all digis the same.

[cmsbuild]

+1
Tested at: ab18135
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-8420/3633/summary.html

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_5_X IBs unless changes (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @nclopezo, @ktf, @smuzaffar

[cwohrman]

Is this ORP meeting one which I should join or is it more like mandatory?

[davidlange6]

@civanch , @cwohrman - are there some validation plots of this change in indico? I'm not sure how this can be checked via the usual relvals.

[civanch]

@davidlange6 , there is a short report from last Friday: https://indico.cern.ch/event/368377/session/16/contribution/67/material/slides/2.pdf
Obviously, validation was private.

[davidlange6]

I knew about that… but I included the word “plots” for a specific reason..

On Mar 24, 2015, at 10:47 AM, Vladimir Ivantchenko notifications@github.com wrote:

@davidlange6 , there is a short report from last Friday: https://indico.cern.ch/event/368377/session/16/contribution/67/material/slides/2.pdf
Obviously, validation was private.

—
Reply to this email directly or view it on GitHub.

[cwohrman]

Here are DQM plots which shows events with the new simulated castor low pT jet trigger (bit62) combined with central PF jets in HLT path.

[cwohrman]

@civanch, @davidlange6, I have also a plot to show the trigger efficiency of the new TTbit62 compared to the hottest sector energy in castor on reco level. The TTbit62 is actually just cutting on the sector energy in castor so this show that the trigger sim does what it should do.

Trig eff also for GenJet pT in castor

I'm not sure if this is what requested.

Cheers,
Hauke

[davidlange6]

+1