-
Notifications
You must be signed in to change notification settings - Fork 12
Charm DIS injection #74
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
Changes from 4 commits
8b5eb26
f1fb7cf
3ce2b8f
a6d66aa
4414cbb
e1da327
7e21f18
9f5b7d9
a9c3ac1
6a54bd0
8ea969d
abf6883
4dfb9ce
5a40bff
526c057
10d86d5
4a70aec
ddf7d2c
d94184c
206c13d
252d7be
435eaab
86a9e56
b234445
2f57b2c
e3a7398
51fd813
0abd9f2
1c30c18
8e58b32
562679d
3341768
1110a2d
60925fd
d77526f
1d43080
2cbd4d0
99c0d82
5241785
e97f340
8560b14
e00a15e
8a2f6b0
163862b
be32bd1
409dea8
0a3dffd
0f0c8c4
eb5efcd
9d32939
24f298a
cd7366b
dd67478
3842959
588709c
52fa76d
d7764ae
b301354
08d410e
b95d84e
9d1c99a
799d53e
46b0542
bc99e8c
3908809
34d5dd9
3731160
2c31202
f1751c6
4b7baf4
10ca063
497d2ab
58c36a1
9b8517a
4b2ce81
5686bff
6027ca6
6f754ae
b8e761a
5c0cc37
dea7873
87f7703
db997b6
d0b2dde
530a75c
c9335c1
a97d23d
62f498b
21baf78
a335b96
7458466
7273556
718f63f
bafdcab
516d890
4705b58
cb8c074
685e498
79715cb
67c9d48
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -56,6 +56,12 @@ X(TauPlus, -15) | |
| X(TauMinus, 15) | ||
| X(NuTau, 16) | ||
| X(NuTauBar, -16) | ||
| X(Charm, 4) | ||
| X(CharmBar, -4) | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. can we not use the c and cBar particles defined above? also probably good to group K0/K0Bar with the other mesons
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Done — removed |
||
| X(K0, 311) | ||
| X(K0Bar, -311) | ||
|
|
||
|
|
||
|
|
||
| /* Nuclei */ | ||
| X(HNucleus, 1000010010) | ||
|
|
||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Was there a specific issue that the I'm wondering if there is another underlying issue that this is covering up, or if we really need this fix.
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. see thread below
nickkamp1 marked this conversation as resolved.
|
|
MiaochenJin marked this conversation as resolved.
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Instead of adding special login in this class checks for a hadronization interaction to change the class behavior, you should implement a new secondary vertex distribution that has the appropriate behavior for hadronization.
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @MiaochenJin are we still using Secondary{Bounded,Physical}VertexDistribution for charms? I think it's probably ok as long as we add a check that any particle with hadronization interactions has no scattering or decay interactions present, since hadronization would override those
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. You are right. we do not currently have any hadronization logic any more. A new commit will remove all dead code, and we shouldn't need to change the Secondary*Distribution definitions any more. |
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same comment as for the bounded vertex distribution:
|
Uh oh!
There was an error while loading. Please reload this page.