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.

Sign up for GitHub

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

Jump to bottom

Lifetime broadening makes calculation run 180 times slower #52

Open
ajm143 opened this issue Mar 10, 2022 · 1 comment
Open

Lifetime broadening makes calculation run 180 times slower #52

ajm143 opened this issue Mar 10, 2022 · 1 comment
Assignees
@ajm143

Comments

@ajm143
Copy link
Contributor

ajm143 commented Mar 10, 2022

Describe the bug
Whilst working on the MZ correction we found that lifetime broadening make the NaGe core-loss calculation take minutes, rather than seconds. I need to investigate if there's something that can speed it up. Is there something sub-optimal in the loops?

To Reproduce
Add LAI broadening to a CORE calc.

Expected behavior
it should be faster!
@ajm143 ajm143 self-assigned this Mar 10, 2022
@ajm143
Copy link
Contributor Author

ajm143 commented Mar 10, 2022

Bin Spacing. Time w/o LAI. Time w. LAI
0.1. 1.1s 2.7s
0.05 2.2s 9.2s
0.025 4.3s 35.0s

Makes sense as we're doing nbin*nbin operations for the convolution

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ajm143 ajm143

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ajm143