Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Inconsistent graduated filter behaviour in Selective Editing #7232

Open
waynesutton50 opened this issue Oct 27, 2024 · 1 comment
Open

Inconsistent graduated filter behaviour in Selective Editing #7232

waynesutton50 opened this issue Oct 27, 2024 · 1 comment
Labels
tool: selective editing type: bug Something is not doing what it's supposed to be doing
Milestone

Comments

@waynesutton50
Copy link

waynesutton50 commented Oct 27, 2024

Short description

The graduated filters in Selective Editing do not move consistently with the image when panning or zooming

Steps to reproduce

  1. Open an image with the Neutral profile
  2. Create a Global Spot and activate a tool in Standard or Advanced mode (e.g. Color & Light)
  3. Activate the Graduated Filter and set the "Luma gradient strength" to maximum
  4. Pan or Zoom the image. The graduated filter position on the image will vary as the image is zoomed out or panned up or down.

Expected behavior
The graduated filter behaviour should be consistent with the Graduated Filter in the Exposure tab

@Lawrence37 Lawrence37 added type: bug Something is not doing what it's supposed to be doing tool: selective editing labels Oct 27, 2024
@Lawrence37 Lawrence37 added this to the v5.12 milestone Oct 27, 2024
@Desmis
Copy link
Collaborator

Desmis commented Oct 28, 2024

Thank you @waynesutton50 for this report which confirms what you have already reported in the branch and the PR on GHS.

The problem doesn't only concern the “global” or “full image” mode, but as soon as a Spot leaves the Preview's full image.

I tried to find a solution but without success. The 'ghs' PR branch contains support for my attempts.

The algorithm and code are very similar to those of Graduated Filter main.

I've modified iplocallab (and all calling functions: improccoordinator, simpleprocess and especially dcrop), to take into account all the variables used for image size in the pipeline.

This is summarized in the function "calclocalgradientparams" which groups together all possible variables

  • xstart, ystrat, xend, yend, bfw, bfh - size and position Spot
  • oW, oH, transW, transH, transX, transY, fw, fh, skip - size of image and position
  • Previewpops
  • etc.

I also tried some modifications to calcGradientFactor.

For some reason (the pipeline and Selective Editing are complex) the position of the gradient_center is not guaranteed, hence the problems.

If someone more competent than me can look into the problem. Thank you

I will be away from home for 15 days or 3 weeks, but should remain reachable (I hope)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tool: selective editing type: bug Something is not doing what it's supposed to be doing
Projects
None yet
Development

No branches or pull requests

3 participants