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
Calculate sensitivity from event lists #29
Comments
The condition can be satisfied prior to compute the IRFs (I think it is the case now). |
Can you explain what you mean by "calculating sensitivity from an event list"? Do you mean without binning it or generating an IRF? The only requirement is that the method used is the "CTA Official" one, which so far uses aperture photometry (meaning a theta^2 cut), rather than a likelihood, but that can be done from binned quantities, so no event list needed. you only need a few inputs to make a sensitivity curve after you've optimized cuts, etc:
With that info, you can produce effective areas, and sensitivities. That means the inputs are: binned spectra, E_migration + (not used here, but still necessary) PSF |
But you optimize the cuts for the best sensitivity, so you have to calculate it during cuts optimization.
For what is the energy migration matrix needed in the sensitivity? It does not enter anywhere afaik. |
Yes, but isn't that what is already done? Currently the code starts with event lists, optimizes and applies cuts, makes histograms, and then computes sensitivity. That should be broken into steps, and I think the sensitivity part only needs to work on histograms (the first steps of course need the event list). There is also the CTools method of computing sensitivity which does it by simulating event lists, but I guess that's not what you mean here? (http://cta.irap.omp.eu/ctools/users/reference_manual/cssens.html) |
No, the actual sensitivity was calculated from the IRFS using a gammapy function. Acutally, both was done, during the cut optimization, the sensitivity was calculated from the event lists / histograms. But the final sensitivity was calculated using this |
Fixed in #36 |
I think it is required to calculate sensitivities from event lists, not from the IRFs, otherwise conditions on the number of events are not applicable.
This also directly rules out usage of the gammapy
SensitivityEstimator
.The text was updated successfully, but these errors were encountered: