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

update LTV1 by difference in CRPIX1 between direct and spectral image #23

Merged
merged 5 commits into from
Nov 22, 2023

Conversation

n-bachmann
Copy link
Contributor

solves problem of misaligned trace

@sebastian-zieba
Copy link
Owner

Hi @n-bachmann ,
Thanks a lot! I will try this on a few datasets to make sure that it works regardless of subarray size and grism.

@sebastian-zieba sebastian-zieba self-assigned this Nov 21, 2023
@sebastian-zieba
Copy link
Owner

Relevant for issue #21 and issue #24

@sebastian-zieba
Copy link
Owner

Looks like this PR fixes the bug for:

  • TOI-1201 (GO17192) Array256 G141
  • TOI-561 (GO17192) Array256 G141

I will test it for some other datasets (with a range of subarrays and grims) to be sure that this PR won't create new problems. I'll try:

  • KELT11 (GO15255) Array512 G141 (worked pre-PR)
  • HD189 (GO16180) Array512 G141 (worked pre-PR)
  • HD209 (GO 12181) Array256 G141 (experienced the same bug like the datasets above)
  • GJ1214 (GO13021) Array256 G141 (worked pre-PR)
  • KELT11 (GO15926) Array512 G102 (worked pre-PR)

These tests shouldn't take too long.

@sebastian-zieba
Copy link
Owner

Looks good. I tested the datasets:

KELT11 (GO15255) Array512 G141
KELT11_15255

HD189 (GO16180) Array512 G141
HD189_16180

HD209 (GO 12181) Array256 G141
HD209_12181

GJ1214 (GO13021) Array256 G141
GJ1214_13021

KELT11 (GO15926) Array512 G102
KELT11_15926

Copy link
Owner

@sebastian-zieba sebastian-zieba left a comment

Choose a reason for hiding this comment

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

Looks all good! Thanks!

Copy link

codecov bot commented Nov 22, 2023

Codecov Report

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

Comparison is base (2a6fe89) 0.00% compared to head (ac3e189) 0.00%.

Files Patch % Lines
src/pacman/s10_direct_images.py 0.00% 4 Missing ⚠️
Additional details and impacted files
@@          Coverage Diff           @@
##           master     #23   +/-   ##
======================================
  Coverage    0.00%   0.00%           
======================================
  Files          57      57           
  Lines        4838    4840    +2     
======================================
- Misses       4838    4840    +2     

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

@sebastian-zieba sebastian-zieba merged commit f7a9908 into sebastian-zieba:master Nov 22, 2023
5 of 7 checks passed
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.

Issue with TOI-561 data: Stage 20 Issue with TOI-1201 data optimal extraction
2 participants