feder-observatory / stellarphot

Stellar aperture photometry
https://stellarphot.readthedocs.io
BSD 3-Clause "New" or "Revised" License
4 stars 12 forks source link

Improve profile tests and use a cutout when possible #354

Closed mwcraig closed 3 months ago

mwcraig commented 3 months ago

This PR does three things:

  1. Flesh out the radial profile test with background a bit more.
  2. Add a test of finding the center of a dim star (fixes #352)
  3. Use a cutout instead of the full data set inside CenterAndProfile for calculating the profile to improve performance (fixes #331)
codecov-commenter commented 3 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 75.33%. Comparing base (36aa59e) to head (823f853). Report is 371 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #354 +/- ## ========================================== + Coverage 75.20% 75.33% +0.12% ========================================== Files 27 27 Lines 3594 3608 +14 ========================================== + Hits 2703 2718 +15 + Misses 891 890 -1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.