Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bsunanda:Run2-alca44 Recover from 0 event selection in IsoTrack Trigger #14516

Merged
merged 1 commit into from May 31, 2016

Conversation

bsunanda
Copy link
Contributor

Backporting PR 14514 and needed to go for HLT

@bsunanda
Copy link
Contributor Author

@cmsbuild Please test

@cmsbuild cmsbuild added this to the Next CMSSW_8_0_X milestone May 16, 2016
@cmsbuild
Copy link
Contributor

cmsbuild commented May 16, 2016

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @bsunanda for CMSSW_8_0_X.

It involves the following packages:

Calibration/HcalIsolatedTrackReco
HLTrigger/special

@perrotta, @cmsbuild, @ghellwig, @cerminar, @Martin-Grunewald, @franzoni, @fwyzard, @mmusich, @davidlange6 can you please review it and eventually sign? Thanks.
@Martin-Grunewald, @ghellwig, @tocheng this is something you requested to watch as well.
@slava77, @Degano, @smuzaffar you are the release manager for this.

cms-bot commands are list here #13028

@fwyzard
Copy link
Contributor

fwyzard commented May 16, 2016 via email

@cmsbuild
Copy link
Contributor

@bsunanda
Copy link
Contributor Author

Hi Andrea

I am doing this with Sourav Dey who ran it on the recent data. I shall ask him to prepare a few slides with the changes made and corresponding rates + timing information. Regards

Sunanda


From: Andrea Bocci [notifications@github.com]
Sent: 16 May 2016 19:48
To: cms-sw/cmssw
Cc: Sunanda Banerjee; Mention
Subject: Re: [cms-sw/cmssw] bsunanda:Run2-alca44 Recover from 0 event selection in IsoTrack Trigger (#14516)

Dear Sunanda,

before approving this change to the HLT, I would like to see a short
description of the change, and the usual HLT rate and running studies.

Looking at some of the changes in confused. Did this ever run with the
Stage 2 L1 trigger ?

Ciao,
.Andrea


You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHubhttps://github.com//pull/14516#issuecomment-219494215

@cmsbuild
Copy link
Contributor

@bsunanda
Copy link
Contributor Author

Hi Andrea,
Here I have attached the same slides with three different formats.

Regards,
Sourav


From: Sunanda Banerjee
Sent: 16 May 2016 23:21
To: cms-sw/cmssw; cms-sw/cmssw
Cc: Mention; Sourav Dey
Subject: RE: [cms-sw/cmssw] bsunanda:Run2-alca44 Recover from 0 event selection in IsoTrack Trigger (#14516)

Hi Andrea

I am doing this with Sourav Dey who ran it on the recent data. I shall ask him to prepare a few slides with the changes made and corresponding rates + timing information. Regards

Sunanda


From: Andrea Bocci [notifications@github.com]
Sent: 16 May 2016 19:48
To: cms-sw/cmssw
Cc: Sunanda Banerjee; Mention
Subject: Re: [cms-sw/cmssw] bsunanda:Run2-alca44 Recover from 0 event selection in IsoTrack Trigger (#14516)

Dear Sunanda,

before approving this change to the HLT, I would like to see a short
description of the change, and the usual HLT rate and running studies.

Looking at some of the changes in confused. Did this ever run with the
Stage 2 L1 trigger ?

Ciao,
.Andrea


You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHubhttps://github.com//pull/14516#issuecomment-219494215

@fwyzard
Copy link
Contributor

fwyzard commented May 17, 2016

Hi Sourav,
PDF is usually enough, but thanks.

The details in the slides are very scarce - could you elaborate a bit more on what you are trying to achieve, what are the proposed changes, what is the impact on the HLT rates and timing, and its expected evolution with pileup and/or luminosity ?

@Martin-Grunewald
Copy link
Contributor

+1
Remember to file a JIRA ticket for any changes to the HLT python configuration of your paths!

@mmusich
Copy link
Contributor

mmusich commented May 23, 2016

+1

@cmsbuild
Copy link
Contributor

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

@fwyzard
Copy link
Contributor

fwyzard commented May 24, 2016

-1

Please provide enough details to understand the impact on the HLT.

@fwyzard
Copy link
Contributor

fwyzard commented May 30, 2016

+1

@cmsbuild
Copy link
Contributor

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

@davidlange6
Copy link
Contributor

+1

@cmsbuild cmsbuild merged commit 82f9fdd into cms-sw:CMSSW_8_0_X May 31, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants