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

GH_CUTS HDU information in DL3 index table #1260

Open
SeiyaNozaki opened this issue Jun 6, 2024 · 1 comment
Open

GH_CUTS HDU information in DL3 index table #1260

SeiyaNozaki opened this issue Jun 6, 2024 · 1 comment
Labels
question Further information is requested

Comments

@SeiyaNozaki
Copy link
Collaborator

From #1227

DL3 index table contains the information of IRF HDU.
https://github.com/cta-observatory/cta-lstchain/blob/main/lstchain/high_level/hdu_table.py#L200-L220

It does not contain the GH_CUTS HDU information since this HDU is not created by pyirf function. We define the similar function inside the lstchain. Do we need to add the GH_CUTS HDU information in this index table?

@chaimain
Copy link
Contributor

chaimain commented Jun 6, 2024

I don't think we use the information from GH_CUTS (or even AL_CUTS) HDU for any high-level analyses. It is mainly used to check the distributions of the cuts applied to get the DL3 file.
Hence it makes less sense to me, to add this information to the HDU index tables (for the sake of completeness).

@morcuended morcuended added the question Further information is requested label Nov 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants