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

Merge MAGIC data subrun-wise is not needed? #126

Open
YoshikiOhtani opened this issue Nov 22, 2022 · 3 comments
Open

Merge MAGIC data subrun-wise is not needed? #126

YoshikiOhtani opened this issue Nov 22, 2022 · 3 comments

Comments

@YoshikiOhtani
Copy link
Collaborator

In the last weekly meeting Julian gave me a comment that M1 and M2 files may contain different sets of events even if the subrun ID is identical. Then I realized that it could affect the stereo reconstruction of MAGIC-only data, since I usually merge MAGIC DL1 files subrun-wise beforehand, but it may lose some events wrongly recognized as mono events. The best option would be to merge the files run-wise and use them for the stereo reconstruction (which takes more time per job though), and in that case the option --subrun-wise in merge_hdf_files.py is not needed anymore.

@YoshikiOhtani
Copy link
Collaborator Author

In order to avoid a confusion in the analysis, I would suggest to remove this option but I'm not sure who actually use it for the analysis. Please let me know what you think.

@aleberti
Copy link
Collaborator

Hi Yoshiki, yes, better to remove it at this point. MARS indeed in superstar looks at the subrun before and after the current subrun to look for matching events. If merging run-wise gets all the subruns putting them in a single df and then does the matching, then it is the same behavior as MARS and should be the only one allowed.

@YoshikiOhtani
Copy link
Collaborator Author

YoshikiOhtani commented Nov 22, 2022

OK thanks Alessio, then I will open a pull request to remove the option.

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