-
Notifications
You must be signed in to change notification settings - Fork 41
Add kMatrix to Python bindings #202
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
Conversation
Hi @henryiii @galapaegos and others, |
Sorry, I and my family have been sick this weekend. Will be getting back to work today. |
Added getDecayInfo to python bindings. Style fixes
37328f5
to
19023a9
Compare
I've simplified the history and applied the formatting, will address the actual problem next (computer is updating Xcode so can't build at the moment). |
2eed546
to
928eed7
Compare
I've changed the input to vector, then just check the size to make sure it is the correct size for f_scatter and matrix of poles - you can't default-construct an std::array, and it's not supported by default in pybind11 anyway. |
Keep in mind, kMatix is experimental, and hasn't been throughly validated. It also is off by default on CUDA builds due to the amount of time it takes to compile. |
* Fix bug introduced in #202 (Python could not be run if kMatrix was off) * Make sure goofit is importable * Fix bug with `python -m goofit` randomly failing if no GPU present - Now prints number of GPUs present * Respect GPU change request when GPU = 0 Python + CUDA import now tested in CI!
Dear @henryiii and GooFit experts,
I am trying to add the kMatrix lineshape to the python bindings but it is not working. I was wondering if you could take a look to see if there is something wrong?
Many thanks.