Skip to content

Update BadPixels with new bits - #519

Merged
takluyver merged 1 commit into
masterfrom
feat/badpixel-user-bits
Aug 4, 2026
Merged

Update BadPixels with new bits#519
takluyver merged 1 commit into
masterfrom
feat/badpixel-user-bits

Conversation

@philsmt

@philsmt philsmt commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

Some updates to the BadPixels enum as discussed in CAL dev team:

  • EXPERT_CHOICE for interactive masks set by the operators and experts, e.g. the BadPixelsManual
  • Reserved last two bits for arbitrary use labelled USER1 and USER2

@philsmt
philsmt force-pushed the feat/badpixel-user-bits branch from c6052a6 to 7a1248a Compare August 3, 2026 12:29
@philsmt philsmt added the skip-changelog Tell the CI that a changelog entry isn't necessary label Aug 3, 2026
@codecov

codecov Bot commented Aug 3, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.40%. Comparing base (e84e4c1) to head (7a1248a).

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #519      +/-   ##
==========================================
+ Coverage   75.39%   75.40%   +0.01%     
==========================================
  Files          39       39              
  Lines        7182     7185       +3     
==========================================
+ Hits         5415     5418       +3     
  Misses       1767     1767              

☔ View full report in Codecov by Harness.
📢 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.

@philsmt

philsmt commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator Author

@takluyver will add the changelog entry directly to #518

@takluyver
takluyver merged commit bcfd593 into master Aug 4, 2026
10 of 11 checks passed
@takluyver
takluyver deleted the feat/badpixel-user-bits branch August 4, 2026 08:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog Tell the CI that a changelog entry isn't necessary

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants