-
Notifications
You must be signed in to change notification settings - Fork 77
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
Fix fake errors in DL2 to DL3 & Index creation step #1227
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1227 +/- ##
==========================================
+ Coverage 73.05% 73.11% +0.06%
==========================================
Files 134 134
Lines 14039 14029 -10
==========================================
+ Hits 10256 10258 +2
+ Misses 3783 3771 -12 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe also update the DL2_to_DL3 notebook to remove the fake errors for better documentation
@SeiyaNozaki, this got stuck... can you please address the comments? |
…into fix_fake_irf_error
Regarding DL2_to_DL3 notebook, can you re-run the notebook with this branch? @morcuended
|
Sure, I will try it out. Regarding the problem with the irfs, I think it is because I produced in the same directory an IRF file with different cuts or something like this. I removed the file I think is problematic. You can try again. I will do so as well. |
I will update the notebook as well, in my to-do list |
Okay thank you @morcuended ! @chaimain @maxnoe Could you please review the code again (apart from DL2_to_DL3 notebook which will be updated by Daniel later)? Thanks! |
Fix #1225 with some code refactoring.
I updated codes adding case handling for IRF interpolation and index creation (point-like IRF: effective area and energy dispersion. full-enclosure: + PSF and background model). I added a warning message for background IRF.
Actually I also found that there was a small typo/bug in PSF interpolation and pytest of the interpolation for full-enclosure IRF, so I also fixed it.