Skip to content

feat: support coords, find_transform, and apply_transform#48

Open
icweaver wants to merge 4 commits into
mainfrom
coords
Open

feat: support coords, find_transform, and apply_transform#48
icweaver wants to merge 4 commits into
mainfrom
coords

Conversation

@icweaver
Copy link
Copy Markdown
Member

@icweaver icweaver commented Apr 24, 2026

Features

  • Pass list of (x, y) coordinates in place of img_from/img_to to bypass photometry step
  • Split align_frame into a find_transform and apply_transform step

Changes to existing behavior

Only return debug data in find_transform step now instead of in align_frame. Also includes additional photometry info like psf model params.

Documentation preview: https://juliaastro.github.io/Astroalign.jl/previews/PR48/

Closes: #42, #44

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 24, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.32%. Comparing base (e445961) to head (08e9b00).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #48      +/-   ##
==========================================
+ Coverage   99.31%   99.32%   +0.01%     
==========================================
  Files           3        3              
  Lines         145      148       +3     
==========================================
+ Hits          144      147       +3     
  Misses          1        1              
Flag Coverage Δ
unittests 99.32% <100.00%> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@icweaver
Copy link
Copy Markdown
Member Author

Added some features based on our conversations, @RainerHeintzmann. Will eventually trim returned objects down as the initial design phase here settles, but let me know if this addresses the general behavior you requested.

@icweaver icweaver linked an issue Apr 24, 2026 that may be closed by this pull request
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.

return FWHM Split out point map + warp

1 participant