search

legacysurvey / legacypipe

Image reduction pipeline for the DESI Legacy Imaging Surveys, using the Tractor framework
BSD 3-Clause "New" or "Revised" License
37 stars 22 forks source link

estimate the variance for GALEX images #691

Closed moustakas closed 3 years ago

moustakas commented 3 years ago

This PR implements a simple model to estimate the variance of the GALEX images.

codecov[bot] commented 3 years ago

Codecov Report

Merging #691 (d87d8e8) into main (5ffe6fb) will decrease coverage by 0.07%. The diff coverage is 9.75%.

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

@@            Coverage Diff             @@
##             main     #691      +/-   ##
==========================================
- Coverage   67.70%   67.62%   -0.08%     
==========================================
  Files          36       36              
  Lines       14505    14497       -8     
  Branches     2560     2562       +2     
==========================================
- Hits         9820     9804      -16     
- Misses       3857     3870      +13     
+ Partials      828      823       -5
Impacted Files Coverage Δ
py/legacypipe/galex.py 5.55% <5.12%> (-0.34%) :arrow_down:
py/legacypipe/unwise.py 84.59% <100.00%> (+0.06%) :arrow_up:
py/legacypipe/cfht.py 23.80% <0.00%> (-0.72%) :arrow_down:
py/legacypipe/utils.py 79.86% <0.00%> (-0.69%) :arrow_down:
py/legacypipe/image.py 67.31% <0.00%> (-0.03%) :arrow_down:
py/legacyzpts/legacy_zeropoints.py 72.95% <0.00%> (+0.04%) :arrow_up:
py/legacyzpts/annotate_ccds.py 84.03% <0.00%> (+0.70%) :arrow_up:
py/legacyzpts/psfzpt_cuts.py 72.41% <0.00%> (+0.74%) :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 5ffe6fb...d2a3683. Read the comment docs.

dstndstn commented 3 years ago

Looks fine -- Please go ahead and merge when you're ready!

coveralls commented 3 years ago

Coverage Status

Coverage decreased (-0.1%) to 73.261% when pulling d2a36836834fc421f51871955d94feef21aa1986 on galex-ivar into 5ffe6fb2458618b68653580badc4a94e1ecb4f04 on main.