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

Distribution of trigger cells to stage 2 FPGAs #492

Conversation

snwebb
Copy link

@snwebb snwebb commented Jul 21, 2021

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

@snwebb
Copy link
Author

snwebb commented Aug 3, 2021

Thanks @jbsauvan, I've made the suggested changes.

@jbsauvan
Copy link

Thanks @snwebb
Running test/testHGCalL1T_newProcessors_V11_cfg.py (with these additional lines

from L1Trigger.L1THGCal.customTriggerGeometry import custom_geometry_decentralized_V11
process = custom_geometry_decentralized_V11(process, links='signaldriven', implementation=2)

in order to use the latest geometry) it seems to produce an empty cluster collection.
I tracked this down to the geometry getStage2FpgasFromStage1Fpga which seems to return empty sets.
Do you see the same?

@snwebb
Copy link
Author

snwebb commented Aug 18, 2021

Yes sorry there's a bug there in the geometry shall I fix it here, or would you like a new request?

@jbsauvan
Copy link

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

@snwebb
Copy link
Author

snwebb commented Aug 18, 2021

thanks, done.

@jbsauvan
Copy link

Thanks @snwebb, there are now output clusters.
But I see some holes in the cluster phi distribution, which I wouldn't expect, as if there was no overlap between Stage 2 regions:
image

Do you have an idea where it could come from?

@snwebb
Copy link
Author

snwebb commented Aug 20, 2021

thanks @jbsauvan - there were a couple of bugs, which are hopefully now fixed.

@jbsauvan
Copy link

Thanks @snwebb
The distributions look fine now. There are regions with an excess of clusters, but this is expected as there is no overlap removal in Stage 2 regions at the moment.
image

@jbsauvan jbsauvan merged commit 815df44 into PFCal-dev:hgc-tpg-devel-CMSSW_12_0_0_pre3 Aug 31, 2021
@jbsauvan jbsauvan moved this from Review to Ready for Integration in HGC L1 Simulation and Algorithms Sep 28, 2021
@jbsauvan jbsauvan moved this from Ready for Integration to Integration In Progress in HGC L1 Simulation and Algorithms Dec 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
HGC L1 Simulation and Algorithms
Integration In Progress
Development

Successfully merging this pull request may close these issues.

None yet

2 participants