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

Merge l1t-tsg-v6-cand and l1t-tsg-v7-cand into CMSSW #292

Open
mulhearn opened this issue May 3, 2016 · 4 comments
Open

Merge l1t-tsg-v6-cand and l1t-tsg-v7-cand into CMSSW #292

mulhearn opened this issue May 3, 2016 · 4 comments

Comments

@mulhearn
Copy link
Member

mulhearn commented May 3, 2016

Now that the L1T integration branch has been rebased to CMSSW_8_0_7, we can prepare several PRs to update official CMSSW to l1t-tsg-v6-cand. Once l1t-tsg-v6 is declared (likely equal to l1t-tsg-v6-cand) we will make small incremental updates.

The relevant PRs for l1t-tsg-v6-cand are:

The relevant PRs for l1t-tsg-v7-cand are:

The PR for cleaning up unpacker error messages is here:

An additional PR for updates to handle OMTF muons at edge of OMTF acceptance is here:

@mulhearn
Copy link
Member Author

mulhearn commented May 4, 2016

PRs for 80x and 81x are ready. 80x will likely need a simple rebase when underlying PRs are merged.

@mulhearn
Copy link
Member Author

mulhearn commented May 4, 2016

Both PRs pass Jenkin's tests, but I'll run the problematic IB RelVals locally as well.

@mulhearn mulhearn changed the title Merge l1t-tsg-v6-cand into CMSSW Merge l1t-tsg-v6-cand and l1t-tsg-v7-cand into CMSSW May 19, 2016
@mulhearn
Copy link
Member Author

Looks like IB with l1t-tsg-v7-cand PR went well for us. No problems from us as far as I can tell:

https://cmssdt.cern.ch/SDT/html/showIB.html

@mulhearn
Copy link
Member Author

The 80x PRs that bring us up to date with l1t-tsg-v7-cand are now in 80x too. We'll see how tonights IB goes. Last thing is to prepare the 80x version of the small incremental fixes for unpacker error quieting and the OMTF fix for muons at edge OMTF acceptance (already merged in 81x).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant