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

Fix pipeline integration test workflow #118

Closed HealthyPear closed 3 years ago

HealthyPear commented 3 years ago

I noticed that I forgot to use the estimated energy (both telescope-wise and event-wise )as a classification parameter as it is done currently in a standard analysis with protopipe.

codecov[bot] commented 3 years ago

Codecov Report

Merging #118 (01b3aa2) into master (21b483c) will increase coverage by 1.01%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #118      +/-   ##
==========================================
+ Coverage   46.30%   47.32%   +1.01%     
==========================================
  Files          22       22              
  Lines        1978     1980       +2     
==========================================
+ Hits          916      937      +21     
+ Misses       1062     1043      -19
Impacted Files Coverage Δ
protopipe/scripts/tests/test_pipeline.py 100.00% <100.00%> (ø)
protopipe/pipeline/utils.py 55.75% <0.00%> (+1.21%) :arrow_up:
protopipe/scripts/data_training.py 94.65% <0.00%> (+9.09%) :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 21b483c...01b3aa2. Read the comment docs.