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

Phase-2 L1T: Fix GT packed format for Correlator e/gamma and tau objects #1168

Conversation

cerminar
Copy link

PR description:

This PR addresses a bug in converting the hardware data format used in the Correlator to the GT one.

The corresponding PR to master is cms-sw#43075

@aloeliger aloeliger added Phase-2 Pertains to phase-2 development Bug fix For minor bug fixing PRs labels Oct 24, 2023
@aloeliger
Copy link

@cerminar We are trying to use phase2-l1t-integration-13_3_0_pre3 as the integration branch. Could you please rebase this work there? (Should just be a clean CMSSW_13_3_0_pre3 release before your PR).

Otherwise the content looks fine. @epalencia I see you've commented on the CMSSW version, I will let you sign there. Any comments for here, or should we just merge it once rebased?

@epalencia
Copy link

Yes, this can be merged once rebased (and master PR is signed already).

@cerminar
Copy link
Author

cerminar commented Nov 6, 2023

@cerminar We are trying to use phase2-l1t-integration-13_3_0_pre3 as the integration branch. Could you please rebase this work there? (Should just be a clean CMSSW_13_3_0_pre3 release before your PR).

Done in: #1177

epalencia added a commit that referenced this pull request Nov 6, 2023
Phase-2 L1T: Fix GT packed format for Correlator e/gamma and tau objects #1168
@epalencia
Copy link

I'm closing this PR as the 13_3 rebased version already exists.

@epalencia epalencia closed this Nov 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug fix For minor bug fixing PRs Phase-2 Pertains to phase-2 development
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants