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

Akalinow update omtf pt overflow label #552

Conversation

akalinow
Copy link

@akalinow akalinow commented Jun 1, 2017

This PR should be merged together with an update to cms-l1t-offline/L1Trigger-L1TMuon
cms-l1t-offline/L1Trigger-L1TMuon#5

@thomreis
Copy link

thomreis commented Jun 1, 2017

@akalinow please rebase this PR on 9_2_0 and make the PR to the l1t-integration-CMSSW_9_2_0 branch.

@akalinow akalinow changed the base branch from l1t-integration-v94.8 to l1t-integration-CMSSW_9_2_0 June 1, 2017 09:48
@akalinow
Copy link
Author

akalinow commented Jun 1, 2017

@thomreis done. It was made on top of 9_2_0. The large number of commits was because I made a PR into l1t-integration-v94.8. Now it shows only my two commits.

@thomreis
Copy link

Hi @akalinow this needs to be rebased to 9_3_0_pre1, the new default release.

@rekovic
Copy link

rekovic commented Jul 27, 2018

@akalinow
Do we still need this PR merged ?

@akalinow
Copy link
Author

Yes. In fact I forgot about it, as it was already a year ago.

@rekovic
Copy link

rekovic commented Jul 27, 2018

@akalinow
In fact, no need to merge this.
Change was done by commit 000b66e which is in the integration branch.

@rekovic rekovic closed this Jul 27, 2018
@akalinow akalinow deleted the akalinow-update-OMTF-pt-overflow-label branch November 6, 2023 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants