Use saturation as multiplier for saturation filter
ClosedPublic
Actions

Authored by Ramil Roosileht (Limarest) on Apr 29 2022, 7:43 PM.

Details

Reviewers

Julien Kaspar (JulienKaspar)
Joseph Eagar (joeedh)

Maniphest Tasks

T97697: Sculpt Mode: Correct color filter saturation

Commits

rB2fc6563a597a: D14808: Use saturation as multiplier for saturation filter

Summary

A change proposed in T97697, using existing saturation as a multiplier for the filter operation

Diff Detail

Repository: rB Blender

Event Timeline

Ramil Roosileht (Limarest) requested review of this revision.Apr 29 2022, 7:43 PM

Ramil Roosileht (Limarest) created this revision.

Ramil Roosileht (Limarest) added a task: T97697: Sculpt Mode: Correct color filter saturation.

While this patch seems to fix the most obvious artefacts, I'd like to hold off for a bit and let us investigate a better solution.
If we can't get one in time for the 3.2 release then we should commit this though as a short term solution for this release.

This revision was not accepted when it landed; it landed in state Needs Review.Apr 30 2022, 5:17 AM

Closed by commit rB2fc6563a597a: D14808: Use saturation as multiplier for saturation filter (authored by Ramil Roosileht (Limarest), committed by Joseph Eagar (joeedh)). · Explain Why

This revision was automatically updated to reflect the committed changes.

Joseph Eagar (joeedh) added a commit: rB2fc6563a597a: D14808: Use saturation as multiplier for saturation filter.

Ramil Roosileht (Limarest) mentioned this in D14818: Experiment - use lightness for saturation masking.May 1 2022, 10:27 PM