Skip to content

[feat] adds --bad-pixels-path - #315

Merged
antond-weta merged 1 commit into
AcademySoftwareFoundation:mainfrom
antond-weta:bad_pixels
Aug 14, 2026
Merged

[feat] adds --bad-pixels-path#315
antond-weta merged 1 commit into
AcademySoftwareFoundation:mainfrom
antond-weta:bad_pixels

Conversation

@antond-weta

Copy link
Copy Markdown
Contributor

Description

adds --bad-pixels-path parameter which provides a list of bad pixels to libraw.

Tests

yes

Checklist:

  • I have read the contribution guidelines.
  • I have updated the documentation, if applicable. (Check if there is no
    need to update the documentation, for example if this is a bug fix that
    doesn't change the API.)
  • I have ensured that the change is tested somewhere in the testsuite
    (adding new test cases if necessary).
  • My code follows the prevailing code style of this project. If I haven't
    already run clang-format before submitting, I definitely will look at the CI
    test that runs clang-format and fix anything that it highlights as being
    nonconforming.

Signed-off-by: Anton Dukhovnikov <antond@wetafx.co.nz>
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.91%. Comparing base (22b696a) to head (e49be2e).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #315      +/-   ##
==========================================
+ Coverage   91.90%   91.91%   +0.01%     
==========================================
  Files          17       17              
  Lines        3781     3787       +6     
  Branches      559      559              
==========================================
+ Hits         3475     3481       +6     
  Misses        306      306              
Files with missing lines Coverage Δ
include/rawtoaces/image_converter.h 100.00% <ø> (ø)
src/rawtoaces_util/image_converter.cpp 79.70% <100.00%> (+0.08%) ⬆️

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 22b696a...e49be2e. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@soswow soswow left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

fair enough. lgtm

/// A path to the file containing a list of bad bixels in libraw format:
/// a plain text where each line describes a single bad pixel using
/// three numbers separated by whitespace for the column, row and UNIX
/// timestamp.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

what's unix timestamp is for? o_O

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

to ignore the bad pixels which appeared after the image had been taken

@antond-weta
antond-weta merged commit 69abcd6 into AcademySoftwareFoundation:main Aug 14, 2026
28 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.

3 participants