[Bf-committers] Color Unpremultiply

[Troy Sobotka]

Matt Ebb wrote:
> Eh? I presume you're talking about the render option that does this at
> the end of the pipeline before saving imagery out of the
> renderer/compositor, and not the option per image that does this when
> loading the image, right?

My bad. It was the only place I could find that made as much sense
from a global perspective.

It _really_ belongs in the output panel. I suppose I should patch this. Erf.

> For situations where you're just rendering through BI straight to disk
> with no compositor, and the system knows it's Premul, you may have a
> point. But afaics that's not how this option works - it's a bit
> misleading having this option in the shading panel, which is full of
> BI related properties, since this is something that happens at the end
> of blender's entire imaging pipeline. If you want to make things
> clearer, perhaps the option should be on the compositor output node
> itself, and when you're not using the compositor, rendering straight
> out of BI, it could just do the right thing automatically since it has
> all the information to make that decision itself. Much of a muchness
> really, though...

Gespertino wrote:
> However, I still wonder if that checkbox doesn't belong to the compositing
node instead of the shading panel.

Yes. Yes. Yes. Completely agree. It's my fault!

I did _try_ to get the blasted thing on the output node, but the UI
goes fuzzy on some of those special nodes, of which the output is one.

If anyone wants to step up and demo how to put interface elements on
it, I'd love it.

Matt Ebb wrote:
> Well, the image viewer kind of 'implies' that output images should be
> premultiplied (otherwise they'll look bad), but this is probably also
> the fault of there being no option to choose how to interpret imagery
> that you're viewing, to correct accordingly.

I really would love to see the patch[1] for toggling associated versus
unassociated view committed to trunk on this note. I think it is poor
form to provide a default view that doesn't accurately depict the
internal alpha state, but alas...

With respect,
TJS

[1] http://projects.blender.org/tracker/?func=detail&atid=127&aid=28782&group_id=9