-
Notifications
You must be signed in to change notification settings - Fork 1
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 templates for wimp example #30
Conversation
Nice @hoetzsch One question, are these templates normalized? |
Yes, both are normalized to 1. |
I think it's more sensible to normalize ER template to, say, 200? And 50GeV WIMP to the rate with 1e-45 cm^2 xsec? |
I think we agreed now that we would prefer to have the nominal expectation value in the templates and only use an external multiplier as a last resort. Right, @kdund ? |
But apart from that it looks great, thanks a lot @hoetzsch !! 😊 |
@hoetzsch FYI, if you wanna normalize the rate to 1e-45 cm2 xsec, you need to
To get the integral of wimp_spectrum, apt.utils.integrate_midpoint is useful. You need to be careful with the time unit of wimp_spectrum, to be the same as the unit of exposure. |
I think for our purposes it would be sufficient to just multiply the histograms with some somewhat sensible expectation values. |
That's also fine. |
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.
Thanks a lot @hoetzsch !! 😊 I gave it a try and everything looked tgood. I’d say you can merge it away! 👍🏼
I think at least this is the better option. |
Hey, @hoetzsch . Sorry for reaching out so late. Where can I find the notebook to reproduce it? Just to make a record. |
This adds two templates in (cS1, log10(cS2)) for the WIMP example: