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

Refine interface of Correlator e/g objects #124

Draft
wants to merge 4 commits into
base: L1PF_12_5_X
Choose a base branch
from

Conversation

cerminar
Copy link

@cerminar cerminar commented Oct 5, 2023

PR description:

This PR addresses several issues with the interface of TkElectron and TkEm objects:

  • add ID score to both the hardware and CMSSW electron objects;
  • add interface to retrieve object in GT hw format;
  • remove Ref. to Egamma standaole objects since they are not the actual ancestors due to architectural choice in L1T hardware. The ref is replaced by a edm::Ptr to the calorimeter primitive (different in barrel and endcap).

NOTE: backward compatibility is maintained via IO rule in the reflex dictionary.

PR validation:

Succesfully tested reading files produced with previous versions of the object data formats.

@cerminar cerminar force-pushed the eg_refine_interface_v0 branch from b3d5232 to d27a578 Compare November 8, 2023 11:15
@cerminar cerminar force-pushed the eg_refine_interface_v0 branch from 5f4b728 to 634f1b4 Compare November 16, 2023 16:47
@cerminar cerminar force-pushed the eg_refine_interface_v0 branch from 634f1b4 to 60dbc4c Compare November 17, 2023 09:27
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.

1 participant