Skip to content

Stacker - #30

Closed
RainerHeintzmann wants to merge 4 commits into
JuliaAstro:mainfrom
RainerHeintzmann:stacker
Closed

Stacker#30
RainerHeintzmann wants to merge 4 commits into
JuliaAstro:mainfrom
RainerHeintzmann:stacker

Conversation

@RainerHeintzmann

Copy link
Copy Markdown
Contributor

This adds some code to stack data based on the alignment routine.

A fully working version can be seen at this commit which was an original PR that was succesively splitted:
RainerHeintzmann@67c80f1

@codecov

codecov Bot commented Apr 5, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 96 lines in your changes missing coverage. Please review.
✅ Project coverage is 41.46%. Comparing base (43e9fd4) to head (60fcd48).
⚠️ Report is 58 commits behind head on main.

Files with missing lines Patch % Lines
src/stacker.jl 0.00% 61 Missing ⚠️
src/utils.jl 0.00% 35 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##              main      #30       +/-   ##
============================================
- Coverage   100.00%   41.46%   -58.54%     
============================================
  Files            5        6        +1     
  Lines           68      164       +96     
============================================
  Hits            68       68               
- Misses           0       96       +96     
Flag Coverage Δ
unittests 41.46% <0.00%> (-58.54%) ⬇️

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

@RainerHeintzmann

Copy link
Copy Markdown
Contributor Author

Thanks for the link to the discusssion. If I find some time, I will also try to see, if I can adapt the Stacker and Drizzle stuff to your new Ransac implementation.

@icweaver

icweaver commented Apr 23, 2026

Copy link
Copy Markdown
Member

Just doing some bookkeeping. Closing this PR and #33 now that AstroStacker.jl exists.

Happy to explore #29 for the com_psf bit. Thanks for your contributions!

@icweaver icweaver closed this Apr 23, 2026
@icweaver icweaver mentioned this pull request Apr 23, 2026
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.

2 participants