-
Notifications
You must be signed in to change notification settings - Fork 8
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
Add Quantization effect to METIS detector yamls #170
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! But we can't test whether it works, because the CI fails due to #168. Maybe we should fix that first, because it would not surprise me if the quantization will also break some notebooks.
Try putting quantization as the final effect to avoid numpy issues that otherwise occur when adding floats to ints.
At least one METIS notebook fails with this. I'll get the |
We want to release the METIS Simulations tomorrow (Wednesday), so I'll fix this and merge it. |
casting to uint currently overflows to MAXINT, that is not supposed to happen. Well, negative fluxes shouldn't happen, but if they happen, they shouldn't be cast to extremely high values |
Fixed the overflow in AstarVienna/ScopeSim#414 |
No description provided.