Skip to content

CLDReco: add cms parameter and add possible beam spot sizes#63

Merged
andresailer merged 1 commit intomainfrom
beamspots
Feb 7, 2025
Merged

CLDReco: add cms parameter and add possible beam spot sizes#63
andresailer merged 1 commit intomainfrom
beamspots

Conversation

@andresailer
Copy link
Collaborator

Alternative to #62

BEGINRELEASENOTES

  • CLDConfig: add --cms parameter with choices of 91, 160, 240 and 365 (GeV) rounded.
    • Add beam spot sizes for these energies and use them in Vertex Reconstruction

ENDRELEASENOTES

@andresailer andresailer force-pushed the beamspots branch 2 times, most recently from cd6518e to 5e11156 Compare February 7, 2025 10:09
@Zehvogel
Copy link
Collaborator

Zehvogel commented Feb 7, 2025

Can you update to use a permanent link e.g.
https://github.com/HEP-FCC/FCCeePhysicsPerformance/blob/d6ecee2c2c3ed5d76db55a3ae18ced349b2b914a/General/README.md?plain=1#L457-L467

@andresailer andresailer force-pushed the beamspots branch 2 times, most recently from 49b72e7 to 7fedb70 Compare February 7, 2025 10:14
@andresailer
Copy link
Collaborator Author

Copy link

@kjvbrt kjvbrt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

Copy link
Collaborator

@Zehvogel Zehvogel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@andresailer andresailer merged commit f9db347 into main Feb 7, 2025
7 checks passed
@andresailer andresailer deleted the beamspots branch February 7, 2025 11:43
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.

Transverse impact parameter discrepancy between full and fast simulation

3 participants