Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve how reference catalog is expanded #195

Merged

Conversation

mcara
Copy link
Member

@mcara mcara commented Jan 5, 2024

This PR addresses the issue JP-3468 (excerpt):

When running the TweakRegStep with expand_refcat=True, if aligning an image fails, its sources are still added to the reference catalog. This breaks the astrometric consistency of the reference catalog, and the next image(s) can lock onto these unaligned sources (or just not match and add even more unaligned sources). This can cause a cascade of misaligned images that can report the alignment as successful.

Reported by @Vb2341

@mcara mcara added the bug Something isn't working label Jan 5, 2024
@mcara mcara added this to the Better Awsomeness milestone Jan 5, 2024
@mcara mcara self-assigned this Jan 5, 2024
@mcara mcara requested a review from a team as a code owner January 5, 2024 06:37
@mcara mcara requested review from Vb2341 and mairanteodoro January 5, 2024 06:37
Copy link

codecov bot commented Jan 5, 2024

Codecov Report

Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (26703f8) 95.25% compared to head (48907b2) 93.79%.

Files Patch % Lines
tweakwcs/imalign.py 94.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #195      +/-   ##
==========================================
- Coverage   95.25%   93.79%   -1.47%     
==========================================
  Files          11       11              
  Lines        1939     1982      +43     
==========================================
+ Hits         1847     1859      +12     
- Misses         92      123      +31     

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

@mcara
Copy link
Member Author

mcara commented Jan 5, 2024

JWST regression tests running here: https://plwishmaster.stsci.edu:8081/job/RT/job/JWST/2727/

@mcara
Copy link
Member Author

mcara commented Jan 5, 2024

@Vb2341 Could you, please, confirm that this solves the issue? You can install this code with

pip install git+https://github.com/mcara/tweakwcs.git@dont-add-failed-catalogs-to-expanded-refcat

@mcara mcara merged commit f7ad5a1 into spacetelescope:master Jan 5, 2024
22 of 24 checks passed
@hbushouse
Copy link

Just adding a reference to JP-3468

@@ -4,9 +4,17 @@
Release Notes
=============

.. 0.8.5 (unreleased)
.. 0.8.7 (unreleased)

Choose a reason for hiding this comment

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

Shouldn't this be "0.8.6", as opposed to "0.8.7"?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants