tasoc / photometry

The TASOC Photometry Pipeline
https://tasoc.dk
GNU General Public License v3.0
18 stars 4 forks source link

Replace Travis CI with GitHub Actions #25

Closed rhandberg closed 3 years ago

rhandberg commented 3 years ago

Replace Travis CI continuous integration with GitHub Actions.

codecov[bot] commented 3 years ago

Codecov Report

Merging #25 (b3ec3e6) into devel (d7931f5) will decrease coverage by 0.84%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##            devel      #25      +/-   ##
==========================================
- Coverage   84.10%   83.26%   -0.85%     
==========================================
  Files          32       33       +1     
  Lines        4171     4194      +23     
==========================================
- Hits         3508     3492      -16     
- Misses        663      702      +39     
Impacted Files Coverage Δ
run_download_cache.py 91.30% <100.00%> (ø)
run_tessphot_mpi.py 23.65% <0.00%> (-69.90%) :arrow_down:
photometry/tessphot.py 34.28% <0.00%> (-8.58%) :arrow_down:
photometry/taskmanager.py 92.85% <0.00%> (-1.34%) :arrow_down:
photometry/BasePhotometry.py 94.30% <0.00%> (-0.13%) :arrow_down:
photometry/AperturePhotometry/k2p2v2.py 73.25% <0.00%> (+0.27%) :arrow_up:
photometry/spice.py 93.38% <0.00%> (+0.73%) :arrow_up:
run_tessphot.py 89.61% <0.00%> (+2.59%) :arrow_up:
photometry/psf_photometry.py 22.36% <0.00%> (+5.26%) :arrow_up:
... and 3 more

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 d7931f5...b3ec3e6. Read the comment docs.

rhandberg commented 3 years ago

It is going to be next to impossible to test the deploy-stage, which automatically creates new releases on GitHub, before actually putting this into production. We will have to debug that on-the-fly,

rhandberg commented 3 years ago

Only issue seems to be that sometimes (at random) coverage fails to combine the different reports from the two separate calls.

rhandberg commented 3 years ago

Will merge this now, since it is performing as well (or better!) than Travis CI and any further issues can be fixed along the way.