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

Support for ctapipe 0.10.5 #124

Closed HealthyPear closed 3 years ago

HealthyPear commented 3 years ago

This PR is a requirement for PR #110, which started in the same way, but I preferred to split API upgrade from the refactoring based on ctapipe.core.Tools.

Now all ctapipe-related API is based on ctapipe 0.10.5.

I also introduced a new conda environment file for released versions.

codecov[bot] commented 3 years ago

Codecov Report

Merging #124 (23dc1be) into master (b58e7ec) will decrease coverage by 1.04%. The diff coverage is 49.15%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #124      +/-   ##
==========================================
- Coverage   49.97%   48.92%   -1.05%     
==========================================
  Files          22       22              
  Lines        1995     2001       +6     
==========================================
- Hits          997      979      -18     
- Misses        998     1022      +24     
Impacted Files Coverage Δ
protopipe/scripts/write_dl2.py 4.90% <3.44%> (-0.74%) :arrow_down:
protopipe/pipeline/utils.py 56.89% <81.81%> (+1.13%) :arrow_up:
protopipe/pipeline/event_preparer.py 69.34% <100.00%> (-7.28%) :arrow_down:
protopipe/scripts/data_training.py 94.59% <100.00%> (-0.06%) :arrow_down:
protopipe/_dev_version.py 60.00% <0.00%> (-20.00%) :arrow_down:
protopipe/version.py 36.36% <0.00%> (-18.19%) :arrow_down:
protopipe/pipeline/temp.py 74.82% <0.00%> (+0.68%) :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 b58e7ec...23dc1be. Read the comment docs.