"Aurelien said : basecurve is bad"

You don’t have to tell me how open source works. I’ve contributed to open source projects, I’ve even been a technical lead/founder for one. (Now retired, Android platform development is hell.)

Perhaps, next time, you should not make assumptions, such as assuming that no developers were willing to put time and effort into fixing the issue. Not one, but TWO developers were willing to put in the time and effort.

The effort was shot down because “the module is too early in the pipeline”. Never mind that it took a single screenshot to show that a single mouse drag fixed that deficiency. Such unfounded attacks continued that “it was too early in the pipeline” despite screenshots showing it capable of operating (and operating well) as the final step before the last colorspace conversion.

Here we are, around 3-4 years later, and Aurelien has finally figured out how to drag a module with the mouse, even after being shown a screenshot of it being done.

The patch was a WIP, and I was willing to put in the effort to determine a way to fix that flaw, even if it meant doing what @jandren is basically being told to do by yourself in New Sigmoid Scene to Display mapping - #423 by paperdigits and put two completely different codepaths into single module. But it was made clear that such effort would be wasted as “it was too early in the pipeline” no matter how many screenshots were provided indicating that flaw was fixable in a matter of seconds by the user, and in fact the changeset was being tested with the module moved to a “better” place.

Such attacks continued even after another developer split out the functionality into a completely separate module (and fixed a number of corner cases I had not yet fixed), and it was made clear that no matter what, any continued effort was pointless because the modules were “too early in the pipeline”.

Also, in general Aurilien inconsistently both hates local tone mapping because halos - New Sigmoid Scene to Display mapping - #458 by anon41087856 - and three posts later says to use it anyway - New Sigmoid Scene to Display mapping - #461 by anon41087856 - exposure fusion is a form of local tonemapping so it was bad (but we should use local tonemapping anyway?)

1 Like