User feedback requested on masking and blending parameters

A glorious patch submitted in June to make the masking and blending fully scene-referred-compliant (and unbounded) is currently being reviewed for merging in darktable 3.4.

We would like to take this opportunity to declutter and remove the output masking parameter, which nobody seems to use. Here:

Screenshot_20201030_234058

Does anyone has a case to make against this ?

The input masking takes the image before the current module is applied, creates an opacity (alpha) mask parametrically from the input, and blends the output of the module on top of its input.

The output masking does the same, but the opacity masks is built parametrically from the output of the current module, which depends on the other settings in the current module, such that if you change the parameters in the current module, it might invalidate your mask and you might need to redo it.

I find this design leading to circular edits and bad worflow. But I have seen before users developing creative solutions I never thought of, so… here is your chance to say something.

7 Likes

@Bruce_Williams once explained the output parameters in one of his videos in such a professional way that I wonder, he has a use case. Hence I ping him here.

Me I vote for ā€˜eliminate’

I had to look at my Darktable mentor’s video on parametric masks to find something relevant. He starts explaining the top sliders at 19:30 into the video. Granted, this is pretty out of date.

https://www.youtube.com/watch?v=x1RRaXbnixQ

@AxelG I’ll have to look at Bruce Williams’ video to see what he has to add to this.

1 Like

I personally not going to miss it. I read about how it works somewhere (I suppose the manual, I don’t know) and I discarded it :man_shrugging:t2:

Honestly i never had a use case for the output slider.
Just rewatched Bruce’s episode about it on YouTube. But i believe all the mask ā€œrefinementsā€ achieved in this presentation would have been possible on the input stage as well.
Anyway: i feel like there’s a certain use case for it.

I only ever use it for sharpening modules (contrast eq, and sharpen). It can help reduce halos, and effects detail in a different way.

The below is screenshot of a 200% zoom. Have only applied lens correction, exposure and filmic, same for each.

  1. Sharpen module, blending with ā€˜input’ only

  1. Sharpen module, blending with ā€˜output’ only

Note:
Using ā€˜input’ only we get more highlights in the bush, around the door buzzers, and less darkness on the vertical line at edge of doorframe. These may or may not be desirable pending the image, but I think the differences are distinct enough to warrant keeping both for sharpening modules at least. Don’t think I’d miss them for other modules.

4 Likes

I don’t use the output often, but when I do I use it as a ā€œblend ifā€ function to modify how the adjustment is applied.

So, my workflow is use the input filter to select what I want to modify, then make my adjustment, then use the output filter to adjust how it’s applied, if I need to.

1 Like

It doesn’t hurt anyone to leave it in imho.

Honestly, the only time I have ever used the output sliders was when I was making that video!
I, for one, would not miss them if they were removed.
It certainly seemed to me that any mask you could build with the output sliders could also be achieved with the input sliders if done correctly. Just my 2 cents.

3 Likes

I’ve never used it. Does anyone?

(If there’s doubt about this, would a survey help guide the decision?)

If it’s redundant, it should go. ā€˜Declutter the GUI’ is a good design principle!

1 Like

I also never used the output sliders.

What did catch me sometimes was having used one of the channels, and then forgetting about it later. Using a different channel gave unexpected effects. Figuring out why has cost me a few hairs…

Would it be possible to add a visual marker when a channel has one of the sliders off it’s default value?

I’ve tried output masks a few times, but wouldn’t miss them.

1 Like

Yes it hurts. The masking options are very crowded now, and having a buffer reading at input and at output is twice the I/O penalty, so it degrades performance a bit.

1 Like

Argh, this is a very hacky way of working around the flaws of the sharpening module, which, as usual, should be addressed in the module itself. Masks should not be used to circumvent pixels operations flaws, flaws should be fixed where they are.

4 Likes

I think i used it very seldom. But won’t such change possibly break old edits?
Maybe a button in settings could solve the clutter issue.

@anon41087856: I never used it so I don’t care if it is removed. Anyway, @danny idea is good: maybe just hide it and let a button (or preferences setting) to show it if some wants it. Many users will don’t see this thread.

To be clear, the PR mentionned above adds a new option blending option on top of the old ones. Output masking will be kept for old blendings, the point is to remove it if possible for the new option.

Adding a new button to hide/show it defeats the purpose. Better keep it than making it optional.

Never felt the need to use it. So removing it is fine by me.

One more for removing.

I vote for removing too