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

Add templates for wimp example #30

Merged
merged 4 commits into from
Jul 12, 2023
Merged

Add templates for wimp example #30

merged 4 commits into from
Jul 12, 2023

Conversation

hoetzsch
Copy link
Contributor

@hoetzsch hoetzsch commented Jun 30, 2023

This adds two templates in (cS1, log10(cS2)) for the WIMP example:

  • Nuclear recoil signal of a 50GeV WIMP
  • Electronic recoil background with flat energy spectrum

wimp50gev_template er_template

@hoetzsch hoetzsch requested a review from hammannr June 30, 2023 16:05
@zihaoxu98
Copy link
Contributor

Nice @hoetzsch One question, are these templates normalized?

@hoetzsch
Copy link
Contributor Author

Nice @hoetzsch One question, are these templates normalized?

Yes, both are normalized to 1.

@zihaoxu98
Copy link
Contributor

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?

@hammannr
Copy link
Collaborator

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 ?

@hammannr
Copy link
Collaborator

But apart from that it looks great, thanks a lot @hoetzsch !! 😊

@zihaoxu98
Copy link
Contributor

zihaoxu98 commented Jun 30, 2023

@hoetzsch FYI, if you wanna normalize the rate to 1e-45 cm2 xsec, you need to

  1. let norm_type = 'on_sim'
  2. let rate = integral(wimp_spectrum) * exposure

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.

@hammannr
Copy link
Collaborator

I think for our purposes it would be sufficient to just multiply the histograms with some somewhat sensible expectation values.

@zihaoxu98
Copy link
Contributor

I think for our purposes it would be sufficient to just multiply the histograms with some somewhat sensible expectation values.

That's also fine.

@hoetzsch
Copy link
Contributor Author

hoetzsch commented Jul 11, 2023

Updated templates:

  • ER normalised to 200 events, WIMP to 10 events
  • set empty bins to zero (instead of 1.75e-6)
  • increased batch size in sim for production
    er_template
    wimp50gev_template

Copy link
Collaborator

@hammannr hammannr left a 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! 👍🏼

@kdund
Copy link
Collaborator

kdund commented Jul 11, 2023

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 ?

I think at least this is the better option.

@hoetzsch hoetzsch merged commit 27ab321 into master Jul 12, 2023
@hoetzsch hoetzsch deleted the wimp_example branch July 12, 2023 08:35
@dachengx
Copy link
Collaborator

dachengx commented Aug 8, 2023

Hey, @hoetzsch . Sorry for reaching out so late. Where can I find the notebook to reproduce it? Just to make a record.

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

Successfully merging this pull request may close these issues.

5 participants