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 updates on top of the menus merged right after CMSSW_745 - 75X #9775

Conversation

cmsbuild
Copy link
Contributor

Title says all...
Most of the updates refer to the preparation of the 50ns menu for the start of data taking with it
Automatically ported from CMSSW_7_5_X #9684 (original by @cms-tsg-storm).

@cmsbuild
Copy link
Contributor Author

A new Pull Request was created by @cmsbuild for CMSSW_7_6_X.

HLT updates on top of the menus merged right after CMSSW_745 - 75X

It involves the following packages:

HLTrigger/Configuration

@Martin-Grunewald, @perrotta, @cmsbuild, @fwyzard can you please review it and eventually sign? Thanks.
@Martin-Grunewald, @geoff-smith, @jalimena this is something you requested to watch as well.
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.
@Degano you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

@cmsbuild
Copy link
Contributor Author

Pull request #9775 was updated. @Martin-Grunewald, @perrotta, @cmsbuild, @fwyzard can you please check and sign again.

@Martin-Grunewald
Copy link
Contributor

@perrotta
So the L1 problem was a non-issue?

@perrotta
Copy link
Contributor

@Martin-Grunewald : this is 75X!

@Martin-Grunewald
Copy link
Contributor

There is supposed to be no difference!

@perrotta
Copy link
Contributor

The difference is that in 75X we (you, by the way...) already set the postLS1 customization of the 50ns L1 menu to v3: d8fb461
Which is something we would like to avoid in 74X, as we discussed yesterday.

@Martin-Grunewald
Copy link
Contributor

please test

@cmsbuild
Copy link
Contributor Author

The tests are being triggered in jenkins.

@cmsbuild
Copy link
Contributor Author

Pull request #9775 was updated. @Martin-Grunewald, @perrotta, @cmsbuild, @fwyzard can you please check and sign again.

@Martin-Grunewald Martin-Grunewald force-pushed the hltUpdatesOnTopOf745plusEpsilon_75X branch from 3bd2fd9 to c08e3c8 Compare June 25, 2015 08:49
@Martin-Grunewald
Copy link
Contributor

also

@cmsbuild
Copy link
Contributor Author

Pull request #9775 was updated. @Martin-Grunewald, @perrotta, @cmsbuild, @fwyzard can you please check and sign again.

1 similar comment
@cmsbuild
Copy link
Contributor Author

Pull request #9775 was updated. @Martin-Grunewald, @perrotta, @cmsbuild, @fwyzard can you please check and sign again.

@cmsbuild
Copy link
Contributor Author

@Martin-Grunewald Martin-Grunewald force-pushed the hltUpdatesOnTopOf745plusEpsilon_75X branch from dd7bd79 to ba6bca2 Compare June 25, 2015 12:00
@cmsbuild
Copy link
Contributor Author

Pull request #9775 was updated. @Martin-Grunewald, @perrotta, @cmsbuild, @fwyzard can you please check and sign again.

@cmsbuild
Copy link
Contributor Author

Pull request #9775 was updated. @perrotta, @cmsbuild, @civanch, @mdhildreth, @Martin-Grunewald, @fwyzard, @mulhearn can you please check and sign again.

@cmsbuild
Copy link
Contributor Author

Pull request #9775 was updated. @perrotta, @cmsbuild, @civanch, @mdhildreth, @Martin-Grunewald, @fwyzard, @mulhearn can you please check and sign again.

@Martin-Grunewald
Copy link
Contributor

@nclopezo @smuzaffar
Could you please close this PR as being obsolete and superseeded by #10209 ?
Thanks!

@cmsbuild
Copy link
Contributor Author

Pull request #9775 was updated. @perrotta, @cmsbuild, @civanch, @diguida, @mdhildreth, @Martin-Grunewald, @cerminar, @fwyzard, @mmusich, @mulhearn can you please check and sign again.

@cmsbuild
Copy link
Contributor Author

Pull request #9775 was updated. @perrotta, @cmsbuild, @civanch, @diguida, @mdhildreth, @Martin-Grunewald, @cerminar, @fwyzard, @mmusich, @mulhearn can you please check and sign again.

@cmsbuild
Copy link
Contributor Author

Pull request #9775 was updated. @perrotta, @cmsbuild, @civanch, @diguida, @mdhildreth, @Martin-Grunewald, @cerminar, @fwyzard, @mmusich, @mulhearn can you please check and sign again.

@smuzaffar
Copy link
Contributor

closing as request in #10243

@smuzaffar smuzaffar closed this Jul 16, 2015
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