search

cta-observatory / protopipe

Prototype data analysis pipeline for the Cherenkov Telescope Array Observatory
https://protopipe.readthedocs.io/en/latest/
Other
5 stars 13 forks source link

Implement DL2 integration tests #126

Closed HealthyPear closed 3 years ago

HealthyPear commented 3 years ago

Requirements for merging

Description

Through this PR we implement a set of 3 integration tests for DL2 (1 per particle type).

Probably 1 could have been sufficient, but they are relatively fast, using about <~10 shower each.

codecov[bot] commented 3 years ago

Codecov Report

Merging #126 (9f1427f) into master (61722f9) will increase coverage by 11.37%. The diff coverage is 100.00%.

:exclamation: Current head 9f1427f differs from pull request most recent head 294b850. Consider uploading reports for the commit 294b850 to get more accurate results Impacted file tree graph

@@             Coverage Diff             @@
##           master     #126       +/-   ##
===========================================
+ Coverage   48.93%   60.30%   +11.37%     
===========================================
  Files          23       23               
  Lines        2058     2091       +33     
===========================================
+ Hits         1007     1261      +254     
+ Misses       1051      830      -221
Impacted Files Coverage Δ
protopipe/scripts/tests/test_pipeline.py 100.00% <100.00%> (ø)
protopipe/scripts/write_dl2.py 82.45% <0.00%> (+77.54%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 61722f9...294b850. Read the comment docs.