-
Notifications
You must be signed in to change notification settings - Fork 4
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
[WIP] Proposal: new implementation of LorentzVector #13
base: main
Are you sure you want to change the base?
Conversation
doesn't seem to be ready for review as tests don't pass. we can discuss re-writing tests and changing exports whatever later, first let's get existing set of unit tests to pass I think. |
You are right, of course. This implementation also needs JuliaHEP/LorentzVectorBase.jl#24 to be merged. I will try to find some time next week to fix the tests and do some cleanup. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #13 +/- ##
===========================================
- Coverage 84.96% 61.11% -23.86%
===========================================
Files 3 6 +3
Lines 153 108 -45
===========================================
- Hits 130 66 -64
- Misses 23 42 +19 ☔ View full report in Codecov by Sentry. |
Description
This PR proposes a new, flexible implementation of the
LorentzVector
type, designed to work seamlessly with any coordinate system supported byLorentzVectorBase
. This includes both the currently available coordinate systems and any future systems added toLorentzVectorBase
.Key Proposal
The new
LorentzVector
will automatically provide the correct accessor functions for all subtypes ofLorentzVectorBase.AbstractCoordinateSystem
. This approach aims to make theLorentzVectors
more adaptable and interoperable with different coordinate systems.Status
While this implementation is still a work in progress, I would appreciate your feedback on the overall concept and any suggestions for improvement.
TODOs
Base.convert
or just the constructorLorentzVector{CS,T}
).LorentzVectorHEP
to ensure a clean and usable API.add_LorentzVectorBase.jl
and respective CI entry after Streamlining and cleanup LorentzVectorBase.jl#24 is merged andLorentzVectorBase.jl
is registered