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

feat: optionally depend on ROOT, and use it for boosting in LorentzTransformer #109

Merged
merged 26 commits into from
Feb 22, 2024

Conversation

c-dilks
Copy link
Member

@c-dilks c-dilks commented Feb 19, 2024

Add ROOT as an optional dependence. If a user does not have ROOT, or does not satisfy the minimum version requirement, code which depends on ROOT will be ignored.

Make LorentzTransformer slightly more useful, by adding a boost to the beam electron rest frame.

More useful transformations will require input from certain momenta, such as the virtual photon $q$.

@c-dilks c-dilks linked an issue Feb 19, 2024 that may be closed by this pull request
.github/workflows/ci.yml Outdated Show resolved Hide resolved
.github/workflows/ci.yml Outdated Show resolved Hide resolved
@c-dilks c-dilks marked this pull request as ready for review February 21, 2024 21:57
@c-dilks c-dilks changed the title feat: optionally depend on ROOT feat: optionally depend on ROOT, and use it for boosting in LorentzTransformer Feb 22, 2024
@c-dilks c-dilks merged commit ec55e14 into main Feb 22, 2024
55 checks passed
@c-dilks c-dilks deleted the root branch February 22, 2024 02:18
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.

add optional ROOT dependence
1 participant