Skip to content

Prepare demosaic for moire detection - #22342

Merged
TurboGit merged 1 commit into
darktable-org:masterfrom
jenshannoschwalm:reserve_moire
Sep 20, 2026
Merged

TurboGit merged 1 commit into
darktable-org:masterfrom
jenshannoschwalm:reserve_moire

Conversation

@jenshannoschwalm

Copy link
Copy Markdown
Collaborator

Summary: Defines and initializes two float parameters used for

  • moire detection

  • providing moire raster masks to be used later in the pipe

  • moire mixer inside the demosaic module working on bayer cfa data.

  • I have read CONTRIBUTING.md and the coding style.

  • I have not merged master into the topic branch.

  • The pull request is one logical change, and every commit compiles on its own.

  • I ran the relevant tests: unit tests, src/tests/integration/ where the pixelpipe is touched, or darktable-cli as a headless smoke test.

  • New user-visible strings use _(), new preferences are registered in data/darktableconfig.xml.in.

No release note required as no working code yet.

Defines and initializes two float parameters used for
 - moire detection
 - providing moire raster masks to be used later in the pipe
 - moire mixer
inside the demosaic module working on bayer cfa data.
@jenshannoschwalm jenshannoschwalm added this to the 5.8 milestone Sep 20, 2026
@jenshannoschwalm jenshannoschwalm added the scope: codebase making darktable source code easier to manage label Sep 20, 2026

@TurboGit TurboGit left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Ok, we'll see how this turns out. We can always revert using a v8 if needed.

@TurboGit
TurboGit merged commit 7414ef4 into darktable-org:master Sep 20, 2026
6 checks passed
@jenshannoschwalm
jenshannoschwalm deleted the reserve_moire branch September 20, 2026 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: codebase making darktable source code easier to manage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants