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

Add code for WIRO camera #718

Closed dstndstn closed 1 year ago

dstndstn commented 1 year ago

This adds code for WIRO, as well as refactoring to make it easier to add more cameras.

codecov[bot] commented 1 year ago

Codecov Report

Base: 64.81% // Head: 63.69% // Decreases project coverage by -1.12% :warning:

Coverage data is based on head (caa1989) compared to base (452b9cd). Patch coverage: 43.01% of modified lines in pull request are covered.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #718 +/- ## ========================================== - Coverage 64.81% 63.69% -1.13% ========================================== Files 39 42 +3 Lines 15997 16437 +440 Branches 2814 2873 +59 ========================================== + Hits 10369 10469 +100 - Misses 4739 5050 +311 - Partials 889 918 +29 ``` | [Impacted Files](https://codecov.io/gh/legacysurvey/legacypipe/pull/718?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=legacysurvey) | Coverage Δ | | |---|---|---| | [py/legacypipe/bits.py](https://codecov.io/gh/legacysurvey/legacypipe/pull/718/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=legacysurvey#diff-cHkvbGVnYWN5cGlwZS9iaXRzLnB5) | `100.00% <ø> (ø)` | | | [py/legacypipe/detection.py](https://codecov.io/gh/legacysurvey/legacypipe/pull/718/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=legacysurvey#diff-cHkvbGVnYWN5cGlwZS9kZXRlY3Rpb24ucHk=) | `77.77% <ø> (-0.37%)` | :arrow_down: | | [py/legacypipe/utils.py](https://codecov.io/gh/legacysurvey/legacypipe/pull/718/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=legacysurvey#diff-cHkvbGVnYWN5cGlwZS91dGlscy5weQ==) | `83.52% <ø> (+5.31%)` | :arrow_up: | | [py/legacyzpts/psfzpt\_cuts.py](https://codecov.io/gh/legacysurvey/legacypipe/pull/718/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=legacysurvey#diff-cHkvbGVnYWN5enB0cy9wc2Z6cHRfY3V0cy5weQ==) | `58.71% <0.00%> (-3.73%)` | :arrow_down: | | [py/legacyzpts/update\_ccd\_cuts.py](https://codecov.io/gh/legacysurvey/legacypipe/pull/718/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=legacysurvey#diff-cHkvbGVnYWN5enB0cy91cGRhdGVfY2NkX2N1dHMucHk=) | `19.68% <0.00%> (ø)` | | | [py/legacypipe/outliers.py](https://codecov.io/gh/legacysurvey/legacypipe/pull/718/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=legacysurvey#diff-cHkvbGVnYWN5cGlwZS9vdXRsaWVycy5weQ==) | `57.21% <8.33%> (-10.34%)` | :arrow_down: | | [py/legacypipe/wiro.py](https://codecov.io/gh/legacysurvey/legacypipe/pull/718/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=legacysurvey#diff-cHkvbGVnYWN5cGlwZS93aXJvLnB5) | `19.71% <19.71%> (ø)` | | | [py/legacypipe/forced\_photom.py](https://codecov.io/gh/legacysurvey/legacypipe/pull/718/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=legacysurvey#diff-cHkvbGVnYWN5cGlwZS9mb3JjZWRfcGhvdG9tLnB5) | `53.04% <28.57%> (-0.66%)` | :arrow_down: | | [py/legacypipe/runs.py](https://codecov.io/gh/legacysurvey/legacypipe/pull/718/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=legacysurvey#diff-cHkvbGVnYWN5cGlwZS9ydW5zLnB5) | `52.63% <30.76%> (-6.46%)` | :arrow_down: | | [py/legacypipe/suprime.py](https://codecov.io/gh/legacysurvey/legacypipe/pull/718/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=legacysurvey#diff-cHkvbGVnYWN5cGlwZS9zdXByaW1lLnB5) | `41.93% <41.93%> (ø)` | | | ... and [18 more](https://codecov.io/gh/legacysurvey/legacypipe/pull/718/diff?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=legacysurvey) | | Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=legacysurvey). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=legacysurvey)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.