-
Notifications
You must be signed in to change notification settings - Fork 3
Add new struct FkTable
#68
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
Comments
I agree this will simplify the interaction at boundaries (both eko and fit). The moment you provide the implementation we can migrate the python interface, but there is not so much to actually migrate: being a special case of Personal interest: being that there is no inheritance, how are you going to implement this structure? My natural answer would have been to promote the API to a trait (with some default implementations), and have two trait implementers, but doing this a posteriori would require a massive code rework (and after all it's an approximation to subclassing, I'm too used to it...). |
Like this 😄 : struct FkTable {
table: Grid,
} |
That's a fair solution. Simple and effective 👌 |
I added a first implementation in commit 497a446. |
Implemented in the pyo3 branch: #51. |
Add a new struct
FkTable
, which is a special case ofGrid
, butSubgrid
sx1
andx2
grids are the same for eachSubgrid
initial_state_1
andinitial_state_2
are implicitly set to2212
(proton) if not present inGrid
, but forFkTable
they must be explicitly setlumi_id_types
which is eitherpdg_mc_ids
if the luminosity function uses MC PDG IDs (for the flavour basis), orevo_basis_ids
for the evolution basis, which has the following possible ids:21
(gluon),22
(photon),100
(V),103
(V3),108
(V8),115
(V15),124
(V24),135
(V35),200
(sigma),203
(T3),208
(T8),215
(T15),224
(T24) and235
(T35)FkTable
The text was updated successfully, but these errors were encountered: