Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@mcara
Copy link
Member

@mcara mcara commented Dec 25, 2022

This PR fixes a unit test failure in tweakreg step and pins tweakwcs=0.8.1.

Checklist for maintainers

@mcara mcara added this to the Build 9.1 milestone Dec 25, 2022
@mcara mcara requested review from hbushouse and nden December 25, 2022 19:13
@mcara mcara self-assigned this Dec 25, 2022
@codecov
Copy link

codecov bot commented Dec 25, 2022

Codecov Report

Base: 80.04% // Head: 79.92% // Decreases project coverage by -0.12% ⚠️

Coverage data is based on head (1b2792f) compared to base (503e7a6).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #7417      +/-   ##
==========================================
- Coverage   80.04%   79.92%   -0.13%     
==========================================
  Files         417      417              
  Lines       38083    38083              
==========================================
- Hits        30485    30437      -48     
- Misses       7598     7646      +48     
Flag Coverage Δ
nightly 79.91% <ø> (-0.13%) ⬇️
unit 52.10% <ø> (ø)
Impacted Files Coverage Δ
jwst/regtest/conftest.py 64.67% <0.00%> (-23.37%) ⬇️
jwst/regtest/regtestdata.py 84.86% <0.00%> (-1.38%) ⬇️
jwst/tweakreg/tweakreg_step.py 61.39% <0.00%> (-0.74%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@mcara
Copy link
Member Author

mcara commented Dec 26, 2022

@mcara mcara merged commit 5894098 into spacetelescope:master Dec 26, 2022
@mcara mcara deleted the pin-tweakwcs-version-0p8p1 branch April 8, 2023 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants