Compositing node Vector -> Normalize inputs and outputs scalars not vectors #61875

New Issue

Finn Bear · 2019-02-23T07:14:48+01:00

Finn Bear commented

2019-02-23 07:14:48 +01:00

System Information
Operating system: Ubuntu 18:04

Blender Version
Broken: 2.8 from February 4, 2019

Short description of error
The Vector -> Normalize node, visible in the compositor, takes/returns scalars (numbers denoted by a grey dot) instead of vectors (two values denoted by a blue dot). It is impossible to normalize scalars (unless you just set them to equal 1), so this shouldn't even be an option. I'm guessing this is a cosmetic issue.

Exact steps for others to reproduce the error
Open the compositor, enable "use nodes," add a Vector -> Normalize node

**System Information** Operating system: Ubuntu 18:04 **Blender Version** Broken: 2.8 from February 4, 2019 **Short description of error** The Vector -> Normalize node, visible in the compositor, takes/returns scalars (numbers denoted by a grey dot) instead of vectors (two values denoted by a blue dot). It is impossible to normalize scalars (unless you just set them to equal 1), so this shouldn't even be an option. I'm guessing this is a cosmetic issue. **Exact steps for others to reproduce the error** Open the compositor, enable "use nodes," add a Vector -> Normalize node

Finn Bear commented

2019-02-23 07:14:48 +01:00

Added subscriber: @finnb

Jacques Lucke commented

2019-02-25 15:05:15 +01:00

Added subscriber: @JacquesLucke

Jacques Lucke self-assigned this 2019-02-25 15:05:15 +01:00

Jacques Lucke commented

2019-02-25 15:05:15 +01:00

The actual problem here seems to be the documentation, not the implementation.

In the code it says: Normalize single channel, useful for z buffer (https://developer.blender.org/diffusion/B/browse/master/source/blender/nodes/composite/nodes/node_composite_normalize.c$28).

While the manual explains something very different (https://docs.blender.org/manual/en/latest/compositing/types/vector/normalize.html).

The actual problem here seems to be the documentation, not the implementation. In the code it says: `Normalize single channel, useful for z buffer` (https://developer.blender.org/diffusion/B/browse/master/source/blender/nodes/composite/nodes/node_composite_normalize.c$28). While the manual explains something very different (https://docs.blender.org/manual/en/latest/compositing/types/vector/normalize.html).

blender-admin commented

2019-02-25 15:11:19 +01:00

This issue was referenced by 4574

Jacques Lucke commented

2019-02-25 15:22:45 +01:00

Changed status from 'Open' to: 'Resolved'

Jacques Lucke closed this issue

2019-02-25 15:22:45 +01:00

Finn Bear commented

2019-02-26 03:56:37 +01:00

@JacquesLucke Now I'm wondering why this node is in the "Vector" category, and that actually applies to most of the nodes there. These nodes, except vector curves, really stretch the meaning of "vector."

Jacques Lucke commented

2019-02-26 13:38:14 +01:00

Added subscriber: @WilliamReynish

Jacques Lucke commented

2019-02-26 13:38:14 +01:00

@finnb, you are right. That does not really make sense.

@WilliamReynish, where should this be?

@finnb, you are right. That does not really make sense. @WilliamReynish, where should this be?

William Reynish commented

2019-02-26 13:43:49 +01:00

As @finnb points out, that could be argued for almost all the nodes in the Vector category.

We could put them inside the Converter category? Or we could add a new category called 'Values' or something like that?

Although if we use the logic of the main input or output socket, the Vector category would be almost empty.

As @finnb points out, that could be argued for almost all the nodes in the Vector category. We could put them inside the Converter category? Or we could add a new category called 'Values' or something like that? Although if we use the logic of the main input or output socket, the Vector category would be almost empty.

Jacques Lucke commented

2019-02-26 13:50:01 +01:00

Hmmm, I'm not sure... Maybe this should be discussed in a design task.

Finn Bear commented

2019-02-28 07:38:01 +01:00

The vector category wouldn't be almost empty if a true "normalize" node was added. The current "normalize" node could be renamed "normalize channel."

Finn Bear commented

2019-02-28 07:39:23 +01:00

Also, where can I find "design tasks?"

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

Compositing node Vector -> Normalize inputs and outputs scalars not vectors #61875