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

Need to decide how to implement rechit-cluster association with sharing enabled #63

Open
clelange opened this issue Sep 10, 2018 · 1 comment
Labels
ntuple format Changes/issues related to the ntuple format question

Comments

@clelange
Copy link
Contributor

The ntupliser curently assumes that sharing is not enabled and therefore a 1-to-1 association between rechits and layer clusters exists: https://github.com/CMS-HGCAL/reco-ntuples/blob/master/HGCalAnalysis/plugins/HGCalAnalysis.cc#L1831-L1851

With sharing enabled, a rechit can be associated to more than one cluster. If we want to preserve this information in the ntuple, we would need to change rechit_cluster2d_ (the variable that stores the index of the associated layer cluster, https://github.com/CMS-HGCAL/reco-ntuples/blob/master/HGCalAnalysis/plugins/HGCalAnalysis.cc#L1917) to a vector of integers.

Any comments or alternative suggestions?

@artlbv
Copy link
Member

artlbv commented Sep 10, 2018

I guess, in that case one would also need to add a vector like rechit_cluster2d_fraction_ for the corresponding fractions?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ntuple format Changes/issues related to the ntuple format question
Projects
None yet
Development

No branches or pull requests

2 participants