-
Notifications
You must be signed in to change notification settings - Fork 2
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
Distribution of trigger cells to stage 2 FPGAs #492
Distribution of trigger cells to stage 2 FPGAs #492
Conversation
L1Trigger/L1THGCal/plugins/backend/HGCalBackendLayer2Processor3DClustering_SA.cc
Outdated
Show resolved
Hide resolved
L1Trigger/L1THGCal/plugins/backend/HGCalBackendLayer2Processor3DClustering_SA.cc
Outdated
Show resolved
Hide resolved
L1Trigger/L1THGCal/interface/backend/HGCalStage2ClusterDistribution.h
Outdated
Show resolved
Hide resolved
L1Trigger/L1THGCal/interface/backend/HGCalStage2ClusterDistribution.h
Outdated
Show resolved
Hide resolved
L1Trigger/L1THGCal/src/backend/HGCalStage2ClusterDistribution.cc
Outdated
Show resolved
Hide resolved
L1Trigger/L1THGCal/src/backend/HGCalStage2ClusterDistribution.cc
Outdated
Show resolved
Hide resolved
Thanks @jbsauvan, I've made the suggested changes. |
Thanks @snwebb
in order to use the latest geometry) it seems to produce an empty cluster collection. |
Yes sorry there's a bug there in the geometry shall I fix it here, or would you like a new request? |
Probably better to fix it here |
thanks, done. |
Thanks @snwebb, there are now output clusters. Do you have an idea where it could come from? |
thanks @jbsauvan - there were a couple of bugs, which are hopefully now fixed. |
Thanks @snwebb |
PR description:
Implemented distribution of trigger cells to stage 2 FPGAs in a similar way to how it is done for stage 1.
PR validation:
Compiles and runs without errors