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

Standardize CKM matrix #68

Open
scarlehoff opened this issue Mar 25, 2024 · 3 comments
Open

Standardize CKM matrix #68

scarlehoff opened this issue Mar 25, 2024 · 3 comments

Comments

@scarlehoff
Copy link
Member

The CKM matrix (should be) a list of numbers, however it is declared as a string

https://github.com/NNPDF/theories_slim/blob/04a3716991015717bb442038adbe25dec57d2650/data/theory_cards/700.yaml#L3

Personally I'd put a list everywhere. Is there a reason why a string would be preferred?

@felixhekhorn
Copy link
Contributor

it was just historically a string since we were bound to sqlite ...

in yadism we wrote a conversion to support that https://github.com/NNPDF/yadism/blob/9db321d2f3496d8ba85ee9b7587b1b7231d8ecc3/src/yadism/coefficient_functions/coupling_constants.py#L460

@felixhekhorn
Copy link
Contributor

do you want to do a list + .reshape(3,3) or directly a matrix?

@scarlehoff
Copy link
Member Author

I see, if we have the conversion in separate programs we might as well convert it to a list everywhere.
Maybe directly a 3x3 matrix makes more sense? but that part I don't really mind

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

No branches or pull requests

2 participants