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

Close by gun generator #58

Merged
merged 6 commits into from May 2, 2019
Merged

Close by gun generator #58

merged 6 commits into from May 2, 2019

Conversation

apsallid
Copy link
Contributor

@apsallid apsallid commented Apr 6, 2019

A new gun was recently introduced in PRs #26065 and #26277. Here we are adding this really useful tool for clustering and calibration studies to the repository. Although the instructions have been updated, this command [1] will create a photon of 60 GeV in the Scintillator part of the V10 geometry.

[1] python SubmitHGCalPGun.py --datTier GSD --nevts 10000 --evtsperjob 50 --queue tomorrow --partID 22 --nPart 1 --thresholdMin 60 --thresholdMax 60 --etaMin 1.4 --etaMax 3.0 --zMin 429.99 --zMax 430.01 --rMin 179.99 --rMax 180.01 --pointing --gunType E --gunMode closeby --eosArea /eos/path/to/output --tag ${USER}_PDGId22_nPart1_E60_Scint_R180_Z430 [--dry-run]

@rovere @felicepantaleo @clelange

@clelange
Copy link
Contributor

clelange commented Apr 8, 2019

Thanks for the PR @apsallid - I propose we wait until a release has been cut that includes cms-sw/cmssw#26065 and cms-sw/cmssw#26277 (both are merged) until we merge this (and then update the documentation).

@clelange
Copy link
Contributor

Hi @apsallid - CMSSW_10_6_0_pre4 is out, we could move ahead with this PR. Let me know if you have time to update the documentation. If not, I can take care of it next week.

@apsallid
Copy link
Contributor Author

apsallid commented May 2, 2019

Hi @clelange ,

Sorry for this late reply. I think the documentation is updated now. Let me know if anything else is needed.

Change order of particle gun descriptions
@clelange clelange merged commit e0c28d8 into CMS-HGCAL:master May 2, 2019
@clelange
Copy link
Contributor

clelange commented May 2, 2019

Thanks, in addition I've made a few changes in the order that things are explained.

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.

None yet

2 participants