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

pr90x L1T Phase2: Add Stage2 products To RawCollector #17462

Conversation

rekovic
Copy link
Contributor

@rekovic rekovic commented Feb 8, 2017

This PR is an addendum to already merged and then reverted PR #17248.
When re-reverted #17248, complement it with this PR.

Details:
Add CaloLaye2, uGMT, and uGT products to the RawCollector (previously missed).

@cmsbuild
Copy link
Contributor

cmsbuild commented Feb 8, 2017

A new Pull Request was created by @rekovic for CMSSW_9_0_X.

It involves the following packages:

L1Trigger/Configuration

@cmsbuild, @rekovic, @mulhearn, @davidlange6 can you please review it and eventually sign? Thanks.
@Martin-Grunewald, @kreczko this is something you requested to watch as well.
@davidlange6, @smuzaffar you are the release manager for this.

cms-bot commands are listed here #13028

@kpedro88
Copy link
Contributor

@rekovic this has a merge conflict

@davidlange6 will #17248 be un-reverted as is, or does it need to be remade?

@davidlange6
Copy link
Contributor

davidlange6 commented Feb 17, 2017 via email

@kpedro88
Copy link
Contributor

@davidlange6 that's what I thought, thanks

@rekovic
Copy link
Contributor Author

rekovic commented Feb 17, 2017

@davidlange6
Ok. So will remake #17248, and will merge this simple PR to a new PR.

@kpedro88
Copy link
Contributor

@rekovic should this be closed?

@rekovic
Copy link
Contributor Author

rekovic commented Feb 24, 2017

@kpedro88 Yes. This is now part integral part of #17598.

@rekovic rekovic closed this Feb 24, 2017
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.

4 participants