Skip to content

Conversation

@melanieclarke
Copy link
Collaborator

@melanieclarke melanieclarke commented May 14, 2025

Resolves JP-3859

Style and documentation fixes for the flat field step.

Tasks

  • If you have a specific reviewer in mind, tag them.
  • add a build milestone, i.e. Build 12.0 (use the latest build if not sure)
  • Does this PR change user-facing code / API? (if not, label with no-changelog-entry-needed)
    • write news fragment(s) in changes/: echo "changed something" > changes/<PR#>.<changetype>.rst (see changelog readme for instructions)
    • update or add relevant tests
    • update relevant docstrings and / or docs/ page
    • start a regression test and include a link to the running job (click here for instructions)
      • Do truth files need to be updated ("okified")?
        • after the reviewer has approved these changes, run okify_regtests to update the truth files
  • if a JIRA ticket exists, make sure it is resolved properly

@melanieclarke
Copy link
Collaborator Author

melanieclarke commented May 14, 2025

@melanieclarke melanieclarke marked this pull request as ready for review May 15, 2025 15:18
@melanieclarke melanieclarke requested review from a team as code owners May 15, 2025 15:18
Copy link
Collaborator

@emolter emolter left a comment

Choose a reason for hiding this comment

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

Looks great Melanie, thanks! I left a few small comments but TBH despite the +-850 code lines I was unable to find much, including nitpicks

@codecov
Copy link

codecov bot commented May 15, 2025

Codecov Report

❌ Patch coverage is 60.78431% with 80 lines in your changes missing coverage. Please review.
✅ Project coverage is 77.90%. Comparing base (63fe3b5) to head (f9de497).
⚠️ Report is 657 commits behind head on main.

Files with missing lines Patch % Lines
jwst/flatfield/flat_field.py 59.23% 75 Missing ⚠️
jwst/flatfield/flat_field_step.py 73.68% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #9464      +/-   ##
==========================================
+ Coverage   77.80%   77.90%   +0.10%     
==========================================
  Files         363      362       -1     
  Lines       36388    36334      -54     
==========================================
- Hits        28311    28307       -4     
+ Misses       8077     8027      -50     

☔ 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.

@melanieclarke melanieclarke requested a review from tapastro May 15, 2025 18:12
@tapastro tapastro merged commit c95f0b9 into spacetelescope:main May 19, 2025
22 of 23 checks passed
@melanieclarke melanieclarke deleted the jp-3859-flatfield branch July 23, 2025 14:11
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.

3 participants