Skip to content

DM-55514: Add method for getting box from sky circle - #69

Merged
timj merged 6 commits into
mainfrom
tickets/DM-55514
Jul 20, 2026
Merged

DM-55514: Add method for getting box from sky circle#69
timj merged 6 commits into
mainfrom
tickets/DM-55514

Conversation

@timj

@timj timj commented Jul 15, 2026

Copy link
Copy Markdown
Member

Checklist

  • ran Jenkins
  • added a release note for user-visible changes to doc/changes

@codecov

codecov Bot commented Jul 15, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 91.66667% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 63.12%. Comparing base (cef7785) to head (74b9b7a).
⚠️ Report is 12 commits behind head on main.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
python/lsst/images/_geom.py 70.58% 5 Missing ⚠️
python/lsst/images/_generalized_image.py 85.71% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #69      +/-   ##
==========================================
+ Coverage   62.98%   63.12%   +0.14%     
==========================================
  Files         132      132              
  Lines       17603    17685      +82     
==========================================
+ Hits        11087    11164      +77     
- Misses       6516     6521       +5     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

@timj timj mentioned this pull request Jul 15, 2026
2 tasks
@timj
timj force-pushed the tickets/DM-55514 branch 3 times, most recently from 895719d to 63ed346 Compare July 17, 2026 19:36
@timj
timj force-pushed the tickets/DM-55514 branch from 63ed346 to 9ee22e3 Compare July 17, 2026 19:54
@timj timj changed the title DM-55514: Migrate stencils code over from dax_images_cutout DM-55514: Add method for getting box from sky circle Jul 17, 2026
Comment thread python/lsst/images/_generalized_image.py
Comment thread python/lsst/images/_generalized_image.py

@natelust natelust left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Comment thread python/lsst/images/_generalized_image.py Outdated
@timj
timj force-pushed the tickets/DM-55514 branch from 19a4930 to 74b9b7a Compare July 20, 2026 17:57
@timj
timj merged commit 004ae44 into main Jul 20, 2026
21 checks passed
@timj
timj deleted the tickets/DM-55514 branch July 20, 2026 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants