Spektrafilm darktable module implementation discussion

[mikae1]

@piratenpanda

Spektrafilm should do the same as values above one don’t make sense and only confuse people that’s why I hard limited it. Also different films break at different values above one so I’d rather not explain to the users why their values break the edits.

I get it. I don’t think I ever used a value above 1.10 or 1.15 in Spektrafilm. It breaks fast above that, but it might be valuable to have it there with a really limited range above 1 and if it’s technically a more correct way of increasing saturation for film (compared to just increasing in color balance rgb).

@piratenpanda

Here’s an updated resize module:
resize-module.patch (49.6 KB)

Nice! Have not attempted to benchmark. Do you know if it mathematically should be a penalty of settings px vs factor scaling?

Sync bug

The two sliders seem to have a tendency of going out of sync. Have not figured out a way to provoke it, but it l can look like this:

UI proposals

• Move the interpolation row to under the “9568 × 6376 → 8420 × 5611 px” row (in my screenshot).
• Align the row that ends with 5611 px (in my screenshot) with the row that ends with “8420 px” (in my screenshot).