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.

Sign up for GitHub

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

Jump to bottom

Split amplitude coefficients into two-body-decay coefficients #254

Closed
redeboer opened this issue Mar 8, 2022 · 0 comments · Fixed by #256
Closed

Split amplitude coefficients into two-body-decay coefficients #254

redeboer opened this issue Mar 8, 2022 · 0 comments · Fixed by #256
Assignees
@redeboer
Milestone
0.13.3

Comments

@redeboer
Copy link
Member

redeboer commented Mar 8, 2022 •
edited
Loading

#251 makes it possible to configure amplitude coefficient names. It's worth considering to produce a separate coefficient for each two-body-decay. The product of those coefficients then forms the original amplitude coefficient. Currently, coefficients are generated for the amplitude of an entire decay, see Helicity versus canonical.
@redeboer redeboer mentioned this issue Mar 8, 2022
Closed
@redeboer redeboer added 💡 Feature ❔ Question Discuss this matter in the team labels Mar 8, 2022
@redeboer redeboer changed the title Also worth considering: produce a coefficient for each decay, so for instance in production and decay. At the moment, there is only one coefficient for the product of both. This would however require a larger refactoring. Split coefficient names into two-body-decay coefficient names Mar 8, 2022
@redeboer redeboer changed the title Split coefficient names into two-body-decay coefficient names Split amplitude coefficients into two-body-decay coefficients Mar 8, 2022
@redeboer redeboer mentioned this issue Oct 1, 2024
Open
2 tasks
@redeboer redeboer removed the ❔ Question Discuss this matter in the team label Mar 12, 2022
@redeboer redeboer self-assigned this Mar 12, 2022
@redeboer redeboer added this to the 0.13.3 milestone Mar 12, 2022
@redeboer redeboer mentioned this issue Mar 12, 2022
Merged
@redeboer redeboer moved this to Upcoming in Roadmap Mar 12, 2022
@redeboer redeboer added this to Roadmap Mar 12, 2022
@redeboer redeboer moved this from Upcoming to Released in Roadmap Apr 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@redeboer redeboer

Labels
None yet
Projects
Roadmap
Status: Released
Milestone
0.13.3
Development

Successfully merging a pull request may close this issue.

feat: implement switch for helicity couplings redeboer/ComPWA-ampform
1 participant
@redeboer