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

HLT development in May (80X) #14445

Merged
merged 29 commits into from May 27, 2016
Merged

Conversation

Martin-Grunewald
Copy link
Contributor

@Martin-Grunewald Martin-Grunewald commented May 11, 2016

HLT development in May (80X).
Includes HLT PR #14325 as that is not yet merged.
(Based on CMSSW_8_0_7_patch2 and including Egamma migration PR #13845 as well)
Needs 2016 L1T menu v2!
Rebased to CMSSW_8_0_8

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @Martin-Grunewald (Martin Grunewald) for CMSSW_8_0_X.

It involves the following packages:

Configuration/HLT
HLTrigger/Configuration
HLTrigger/Egamma
HLTriggerOffline/Egamma

@perrotta, @cmsbuild, @dmitrijus, @Martin-Grunewald, @deguio, @fwyzard, @vanbesien, @davidlange6 can you please review it and eventually sign? Thanks.
@ghellwig, @geoff-smith, @jalimena 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

@cmsbuild
Copy link
Contributor

Pull request #14445 was updated. @perrotta, @cmsbuild, @dmitrijus, @Martin-Grunewald, @rekovic, @deguio, @fwyzard, @vanbesien, @mulhearn, @davidlange6 can you please check and sign again.

@cmsbuild
Copy link
Contributor

Pull request #14445 was updated. @perrotta, @cmsbuild, @dmitrijus, @Martin-Grunewald, @deguio, @fwyzard, @vanbesien, @davidlange6 can you please check and sign again.

@cmsbuild
Copy link
Contributor

Pull request #14445 was updated. @perrotta, @cmsbuild, @dmitrijus, @Martin-Grunewald, @deguio, @fwyzard, @vanbesien, @davidlange6 can you please check and sign again.

@cmsbuild
Copy link
Contributor

Pull request #14445 was updated. @perrotta, @cmsbuild, @dmitrijus, @Martin-Grunewald, @deguio, @fwyzard, @vanbesien, @davidlange6 can you please check and sign again.

@cmsbuild
Copy link
Contributor

@cmsbuild
Copy link
Contributor

@davidlange6
Copy link
Contributor

any remaining issues for this PR?

@mmusich
Copy link
Contributor

mmusich commented May 27, 2016

+1

@Martin-Grunewald
Copy link
Contributor Author

@davidlange6
From the HLT side yes, there are remaining developments to be included - we have a TSG deadline of Monday. If you want to integrate now, you can do so (and its 81X twin), but we will have further
items on Monday!
Of course, #14336 is even more urgent to be integrated in 80X.

@davidlange6
Copy link
Contributor

eg, the HLT is not ready for the MC prod yet. Ok good to know. We wait on the release then.

On May 27, 2016, at 10:03 AM, Martin Grunewald notifications@github.com wrote:

@davidlange6
From the HLT side yes, there are remaining developments to be included - we have a TSG deadline of Monday. If you want to integrate now, you can do so (and its 81X twin), but we will have further
items on Monday!
Of course, #14336 is even more urgent to be integrated in 80X.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@Martin-Grunewald
Copy link
Contributor Author

@davidlange6 @fwyzard
Hmm, there is a circular dependency then: we expected an (intermediate) release with current IB content plus #14336, before we need to provide the "final" HLT for MC prod - then to be included in another release. So this is no longer to happen? (We need this "intermediate" release to do more ConfDB parsing and then use that for updating the HLT config for MC prod).

@davidlange6
Copy link
Contributor

I think I was never aware of this plan/need, so I can’t comment on any change in plan.

On May 27, 2016, at 10:15 AM, Martin Grunewald notifications@github.com wrote:

@davidlange6 @fwyzard
Hmm, there is a circular dependency then: we expected an (intermediate) release with current IB content plus #14336, before we need to provide the "final" HLT for MC prod - then to be included in another release. So this is no longer to happen? (We need this "intermediate" release to do more ConfDB parsing and then use that for updating the HLT config for MC prod).


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@Martin-Grunewald
Copy link
Contributor Author

@davidlange6
It was drawn up as you said you'd make a new 80X release "this week".

@davidlange6
Copy link
Contributor

good to know the hidden dependency. will try to manage a release over the weekend

@Martin-Grunewald
Copy link
Contributor Author

....but please include #14336 in it!

@Martin-Grunewald
Copy link
Contributor Author

+1
thsi PR as you included its twin in 81X already.
Subsequent HLT will be put into new PRs...

@Martin-Grunewald
Copy link
Contributor Author

@davidlange6
Actually, please do include this PR #14445 in this next 80X release as it contains the HLT/Eg technical migration. Thanks!

@dmitrijus
Copy link
Contributor

+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

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

7 participants