New Curve interpolation in Color Management works as the old method unless it gets reset. #47366

New Issue

Marco G · 2016-02-08T22:23:51+01:00

Marco G commented

2016-02-08 22:23:51 +01:00

System Information
Win7 64bit Nvidia 750Ti 4Gb
system-info_276-hash-578f70f.txt

Blender Version
Broken: 2.76 578f70f

Short description of error
New Curve interpolation in Color Management needs to be reset to get it working with the new method.

Exact steps for others to reproduce the error
It's not file specific, it can be reproduced even on factory settings here, that's why i have no blends to attach.

Steps:

Load factory settings
Fresh new default file
Go to Color Management tab and tweak the curve by adding a point like the image attached
(Look how the curve behaviour is the old one)
Now Reset curve from the Tools
Tweak again the curve by adding a point
Now it is working as expected

**System Information** Win7 64bit Nvidia 750Ti 4Gb [system-info_276-hash-578f70f.txt](https://archive.blender.org/developer/F280991/system-info_276-hash-578f70f.txt) **Blender Version** Broken: 2.76 578f70f **Short description of error** New Curve interpolation in Color Management needs to be reset to get it working with the new method. **Exact steps for others to reproduce the error** It's not file specific, it can be reproduced even on factory settings here, that's why i have no blends to attach. Steps: - Load factory settings - Fresh new default file - Go to Color Management tab and tweak the curve by adding a point like the image attached - (Look how the curve behaviour is the old one) - Now Reset curve from the Tools - Tweak again the curve by adding a point - Now it is working as expected ![ColorManagementCurve.jpg](https://archive.blender.org/developer/F280994/ColorManagementCurve.jpg)

Marco G commented

2016-02-08 22:23:51 +01:00

Changed status to: 'Open'

Marco G commented

2016-02-08 22:23:51 +01:00

Added subscriber: @marcog

Adam Friesen commented

2016-02-08 23:33:00 +01:00

Added subscriber: @Ace_Dragon

Adam Friesen commented

2016-02-08 23:33:00 +01:00

That's because the points on each end of a new curve are now of the vector type by default (as opposed to the auto type they were before).

If you have an existing curve, change the point type for the ones at each end to 'vector'. You can do that by clicking the little blue wrench icon.

That's because the points on each end of a new curve are now of the *vector* type by default (as opposed to the *auto* type they were before). If you have an existing curve, change the point type for the ones at each end to 'vector'. You can do that by clicking the little blue wrench icon.

Sergey Sharybin commented

2016-02-09 10:12:31 +01:00

Added subscriber: @Sergey

Sergey Sharybin commented

2016-02-09 10:12:31 +01:00

@marcog: I'm not sure what are the old and new methods? Can you explain that? And does this happen with facture Blender settings?

@Ace_Dragon, this i can not confirm. All the points are AUTO handle after the curve mapping is being enabled first time. So it's some operation which you do makes them VECTOR. Please give us exact steps you're doing to get point handle type from a factory startup.

@marcog: I'm not sure what are the old and new methods? Can you explain that? And does this happen with facture Blender settings? @Ace_Dragon, this i can not confirm. All the points are AUTO handle after the curve mapping is being enabled first time. So it's some operation which you do makes them VECTOR. Please give us exact steps you're doing to get point handle type from a factory startup.

Marco G commented

2016-02-09 14:19:53 +01:00

@Sergey , yes, as stated, this happens with factory settings too.
I can't find the commit but i remember (probably you?) that the behaviour of the curve mapping has been modified (now it's better btw, IMHO), there is an example here in this thread, now a single point give a sort of automatic S shape ---> link BA

The bug i report is because to get this "new" way about how curve interpolates, it does not work out of the box, the user MUST reset the curve manually (even on a fresh new file on factory settings), otherwhise it works as the "old" way.

Please look the far left and far right shape of the curve in the image I attached. The far left is how it worked before, the far right is how works now (improved). Try following the steps i wrote.
The fact the user needs to reset manually the curve is IMHO, a bug, should work out of the box instead.

@Ace_Dragon , the fact you need to reset or set to AUTO manually is the bug in my opinion.

@Sergey , yes, as stated, this happens with factory settings too. I can't find the commit but i remember (probably you?) that the behaviour of the curve mapping has been modified (now it's better btw, IMHO), there is an example here in this thread, now a single point give a sort of automatic S shape ---> [link BA ](http://www.blenderartists.org/forum/showthread.php?282540-Cycles-tonemapping&p=3005300&viewfull=1#post3005300) The bug i report is because to get this "new" way about how curve interpolates, it does not work out of the box, the user MUST reset the curve manually (even on a fresh new file on factory settings), otherwhise it works as the "old" way. Please look the **far left** and **far right** shape of the curve in the image I attached. The far left is how it worked before, the far right is how works now (improved). Try following the steps i wrote. The fact the user needs to reset manually the curve is IMHO, a bug, should work out of the box instead. @Ace_Dragon , the fact you need to reset or set to AUTO manually is *the bug* in my opinion.

Marco G commented

2016-02-09 14:42:23 +01:00

@Sergey , i think D1658 is the patch i was looking for to explain the "old" vs the "new" method, so the far left in the image I attached is before the patch, far right is after the patch has been committed. But(!), to get this new result, you need to reset the curve, it's not working automatically as soon as the user adds a point to the curve.

Hope it's clearer.

@Sergey , i think [D1658](https://archive.blender.org/developer/D1658) is the patch i was looking for to explain the "old" vs the "new" method, so the far left in the image I attached is *before* the patch, far right is *after* the patch has been committed. But(!), to get this new result, you need to reset the curve, it's not working automatically as soon as the user adds a point to the curve. Hope it's clearer.

blender-admin commented

2016-02-09 14:51:32 +01:00

This issue was referenced by 24e6411be6

This issue was referenced by 24e6411be6d0795d3107cc2d5dca8ae838e54663

Sergey Sharybin commented

2016-02-09 14:53:28 +01:00

Changed status from 'Open' to: 'Resolved'

Sergey Sharybin closed this issue

2016-02-09 14:53:28 +01:00

Sergey Sharybin commented

2016-02-09 14:55:53 +01:00

Just to clarify, there was never changes in interpolation or so. There was only a change for the shutter curve in 4f6b01ce1 which lead to all LINEAR curve preset to switch to a VECTOR handle. Official startup file has AUTO handles by default for the color mapping.

Now made it so Reset behaves same as in all previous releases for color management curve (and rest of single-slope curves actually). This conflicts with original report, but that's really how it used to work all the time and how it is expected to work.

Just to clarify, there was never changes in interpolation or so. There was only a change for the shutter curve in 4f6b01ce1 which lead to all LINEAR curve preset to switch to a VECTOR handle. Official startup file has AUTO handles by default for the color mapping. Now made it so Reset behaves same as in all previous releases for color management curve (and rest of single-slope curves actually). This conflicts with original report, but that's really how it used to work all the time and how it is expected to work.

Marco G commented

2016-02-09 15:11:45 +01:00

Ah, ok thanks for clarifying.

Sign in to join this conversation.

No Label

Download

What's New

Blender Studio

Manual

Developers Blog

Documentation

Benchmark

Blender Conference

Development Fund

One-time Donations

New Curve interpolation in Color Management works as the old method unless it gets reset. #47366