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

Consider combining or renaming dynamics.phasespace #293

Open
redeboer opened this issue May 23, 2022 · 0 comments
Open

Consider combining or renaming dynamics.phasespace #293

redeboer opened this issue May 23, 2022 · 0 comments
Labels
⚠️ Interface Breaking changes to the API ❔ Question Discuss this matter in the team

Comments

@redeboer
Copy link
Member

Since #292, there are two modules named phasespace: dynamics.phasespace (phase space factors) and kinematics.phasespace (phase space boundaries). There is a physical relation between the two, but this is not clear from the documentation.

Ideally, the modules are combined in a way that makes sense physically (see PDG Kinematics review in particular). Alternatively, simply rename to phsp_factor or something along those lines.

@redeboer redeboer added ❔ Question Discuss this matter in the team ⚠️ Interface Breaking changes to the API labels May 23, 2022
@redeboer redeboer self-assigned this May 23, 2022
@redeboer redeboer removed their assignment Jan 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚠️ Interface Breaking changes to the API ❔ Question Discuss this matter in the team
Projects
None yet
Development

No branches or pull requests

1 participant