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

fixed CorrWithOverlapRemovalCondition.cc #36103

Closed

Conversation

namppl
Copy link
Contributor

@namppl namppl commented Nov 12, 2021

PR description:

Recently the overlap-removal condition (for 2 correlated legs + 1 overlap-removal leg) was found to be misconfigured in both L1 firmware and emulator code. The code has been fixed in accordance with the updated L1 firmware.

PR validation:

Unit tests done

@cmsbuild cmsbuild added this to the CMSSW_12_2_X milestone Nov 12, 2021
@namppl namppl changed the base branch from master to CMSSW_12_2_X November 12, 2021 10:58
@cmsbuild cmsbuild changed the base branch from CMSSW_12_2_X to master November 12, 2021 10:58
@cmsbuild
Copy link
Contributor

@namppl, CMSSW_12_2_X branch is closed for direct updates. cms-bot is going to move this PR to master branch.
In future, please use cmssw master branch to submit your changes.

@cmsbuild
Copy link
Contributor

-code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-36103/26599

  • This PR adds an extra 24KB to repository

Code check has found code style and quality issues which could be resolved by applying following patch(s)

@cmsbuild
Copy link
Contributor

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-36103/26600

  • This PR adds an extra 24KB to repository

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @namppl (Kyungwook Nam) for master.

It involves the following packages:

  • L1Trigger/L1TGlobal (l1)

@cmsbuild, @rekovic, @cecilecaillol can you please review it and eventually sign? Thanks.
@Martin-Grunewald, @missirol this is something you requested to watch as well.
@perrotta, @dpiparo, @qliphy you are the release manager for this.

cms-bot commands are listed here

@rekovic
Copy link
Contributor

rekovic commented Nov 18, 2021

this is under discussion.

@rekovic
Copy link
Contributor

rekovic commented Nov 18, 2021

hold

@cmsbuild
Copy link
Contributor

Pull request has been put on hold by @rekovic
They need to issue an unhold command to remove the hold state or L1 can unhold it for all

@qliphy
Copy link
Contributor

qliphy commented Dec 16, 2021

@rekovic Can this PR be closed by #36441 ?

@cecilecaillol
Copy link
Contributor

please test

@cecilecaillol
Copy link
Contributor

unhold

@cmsbuild
Copy link
Contributor

-1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-4498eb/21815/summary.html
COMMIT: f4a9def
CMSSW: CMSSW_12_3_X_2022-01-19-1100/slc7_amd64_gcc10
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmssw/36103/21815/install.sh to create a dev area with all the needed externals and cmssw changes.

This pull request cannot be automatically merged, could you please rebase it?
You can see the log for git cms-merge-topic here: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-4498eb/21815/git-merge-result

@cecilecaillol
Copy link
Contributor

@namppl Can you please close this PR? Changes were merged with #36758

@namppl namppl closed this Jan 26, 2022
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