-
Notifications
You must be signed in to change notification settings - Fork 19
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
Clarify FCCeeCLD detector model #16
Comments
@EmanuelPerez can you confirm? From what I remember:
So this might be better off with the name FCCee_01_v05? |
On 10 Feb 2022, at 14:06, Valentin Volkl ***@***.******@***.***>> wrote:
@EmanuelPerez<https://github.com/EmanuelPerez> can you confirm? From what I remember:
The starting point was FCCee_o1_v04. The other files are obsolete or were used
for tests I think, except o2_v01, which has a smaller beam-pipe (backgrounds are still
to be studied in detail with this model, to check that such a small beam-pipe is acceptable
or not).
Yes. o1_v04 is the configuration with the R=15 mm beam-pipe, while o2 corresponds to the smaller beam-pipe.
I do not remember a o2_v02. But obviously, it is indeed the wrong vertex file that has been put in this directory.
The beam-pipe file may also be wrong, that needs to be checked.
E.
—
Reply to this email directly, view it on GitHub<#16 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ABI7YBGTTAM52AQ4K57U6WDU2OZ5PANCNFSM5OAUDZOQ>.
Triage notifications on the go with GitHub Mobile for iOS<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
From the name of the model FCCee_o2_v02 I would assume that this is based on FCCee_o2_v01, which is the CLD detector model with the smaller 10 mm beampipe, and the corresponding smaller vertex detector
cf. https://github.com/iLCSoft/lcgeo/blob/4a5b6bb293ce5526ed23d14a3b5157a31326caf8/FCCee/compact/FCCee_o2_v01/FCCee_o2_v01.xml#L284
But looking at the vertex detector used in FCC_o2_v02 I see:
FCCDetectors/Detector/DetFCCeeCLD/compact/FCCee_o2_v02/Vertex.xml
Lines 45 to 47 in ce0513d
Instead of
https://github.com/iLCSoft/lcgeo/blob/4a5b6bb293ce5526ed23d14a3b5157a31326caf8/FCCee/compact/FCCee_o2_v01/Vertex_o4_v05_smallBP.xml#L41-L43
The text was updated successfully, but these errors were encountered: