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

Implementing full-enclosure IRFs #41

Closed
3 tasks done
TarekHC opened this issue Aug 25, 2022 · 3 comments
Closed
3 tasks done

Implementing full-enclosure IRFs #41

TarekHC opened this issue Aug 25, 2022 · 3 comments
Assignees

Comments

@TarekHC
Copy link
Collaborator

TarekHC commented Aug 25, 2022

After discussion between Orel, Victor, Juan and Tarek, we came up with the following recipe:

  • The current training (regression) should be good enough for the offset-dependent IRFs

  • The key will at the testing stage: we need to define the event type thresholds taking into account the offset. At the testing stage we probably should:

    • Bin test gamma-cone sample (not used in the training) in offset steps (requiring enough statistics to define event types across the energy, as we do now. These bins may be smaller than those offset bins we use for computing IRFs. This is an attempt to not produce FoV effects on different types.

    • Once event types are defined properly taking into account the offset, it will just be a matter of defining broad off-axis bins (as done in ED) to compute the IRFs

    • Modifications to pyirf are stil needed, as we want to compute full-enclosure IRFs (removing the angular cut from the optimization). It should be a simple change, specially if Max is operative.

@TarekHC
Copy link
Collaborator Author

TarekHC commented Aug 25, 2022

We checked current pyirf code, and it seems effective area and energy migration is already being computed for the following cuts:

masks = {
    "": gammas["selected"],
    "_NO_CUTS": slice(None),
    "_ONLY_GH": gammas["selected_gh"],
    "_ONLY_THETA": gammas["selected_theta"],
}

To test if these full-enclosure effective areas are consistent with the ones computed within ED, we could:

  1. Compare ED point-like vs full-enclosure effective area.
  2. Compute standard effective area with OnSource gammas with pyirf.
  3. Compute _ONLY_GH effective area with a 1-deg gamma-cone file with pyirf.
  4. Compare these two, and see if the difference is roughly consistent with ED.

@TarekHC
Copy link
Collaborator Author

TarekHC commented Sep 28, 2022

Some of the modifications needed at the pyirf stage (either within pyirf, or scripts we need to develop on our side) are being discussed here and here.

@JBernete JBernete self-assigned this Feb 15, 2023
@JBernete
Copy link
Collaborator

I think we can close this issue. Event-type-wise full-enclosure IRFs have already been tested with Gammapy.

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

No branches or pull requests

2 participants