-
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
Require atleast ctapipe_io_lst 0.18.2 #1003
Conversation
I had to add an option to After the update, this event was correctly labeled as UNKNOWN, but it was the only one previously tagged pedestal, so the test for the calculate noise parameters failed. With default_trigger_type="tib", the one actual pedestal event in that file is found again and the test passes with a slight modification of the tested value. Do we just not look at these old runs? What does osa do with runs like these where TIB would be the better choice? |
Codecov Report
@@ Coverage Diff @@
## master #1003 +/- ##
==========================================
- Coverage 85.75% 85.34% -0.42%
==========================================
Files 78 78
Lines 6619 6645 +26
==========================================
- Hits 5676 5671 -5
- Misses 943 974 +31
Continue to review full report at Codecov.
|
We have been using |
Ok, for me this mainly means that we should include a more recent file in the test data sample that doesn't have all these now solved issues. |
No description provided.