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

mark sources that need initial flux estimates #642

Closed dstndstn closed 4 years ago

dstndstn commented 4 years ago

saturated and SGA-preburn sources have poor initial flux estimates. mark them with a "need_initial_flux" attribute. fit just the fluxes of those sources at the beginning. (and drop initial fitting of all fluxes for small blobs)

coveralls commented 4 years ago

Coverage Status

Coverage decreased (-0.007%) to 76.232% when pulling d9113952169e04d435babcbf5e04716dbba8ff42 on needflux into 1ad571a5aced33c009807e841229aa0f83bddf41 on master.

codecov[bot] commented 4 years ago

Codecov Report

Merging #642 into master will decrease coverage by 0.02%. The diff coverage is 62.50%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #642      +/-   ##
==========================================
- Coverage   70.64%   70.61%   -0.03%     
==========================================
  Files          27       27              
  Lines       11573    11582       +9     
  Branches     2053     2059       +6     
==========================================
+ Hits         8176     8179       +3     
- Misses       2746     2749       +3     
- Partials      651      654       +3
Impacted Files Coverage Δ
py/legacypipe/runbrick.py 79.09% <50.00%> (-0.06%) :arrow_down:
py/legacypipe/oneblob.py 81.16% <63.63%> (-0.21%) :arrow_down:
py/legacypipe/reference.py 81.05% <100.00%> (+0.04%) :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 1ad571a...d911395. Read the comment docs.