Regression: Compositor Split node with only colors does not work (in 3DView, compositor backdrop is fine) #115780

New Issue

Closed

opened 2023-12-05 04:41:14 +01:00 by Sun Kim · 9 comments

Sun Kim commented

2023-12-05 04:41:14 +01:00

Contributor

System Information
Operating system: Windows-10-10.0.19045-SP0 64 Bits
Graphics card: NVIDIA GeForce RTX 2060 SUPER/PCIe/SSE2 NVIDIA Corporation 4.6.0 NVIDIA 546.17

Blender Version
Broken: version: 4.1.0 Alpha, branch: main, commit date: 2023-12-04 01:28, hash: 92e8b7e85027
Worked: version: 4.1.0 Alpha, branch: main, commit date: 2023-12-02 19:08, hash: 8933284518a2

Likely caused by 153f14be2b

Short description of error
Compositor Split node with only colors no longer work.

**System Information** Operating system: Windows-10-10.0.19045-SP0 64 Bits Graphics card: NVIDIA GeForce RTX 2060 SUPER/PCIe/SSE2 NVIDIA Corporation 4.6.0 NVIDIA 546.17 **Blender Version** Broken: version: 4.1.0 Alpha, branch: main, commit date: 2023-12-04 01:28, hash: `92e8b7e85027` Worked: version: 4.1.0 Alpha, branch: main, commit date: 2023-12-02 19:08, hash: `8933284518a2` Likely caused by 153f14be2b **Short description of error** Compositor Split node with only colors no longer work. <video src="/attachments/fd69145e-aa94-48f9-90ad-2042c59eea04" title="compositor_split_colors2.mp4" controls></video>

compositor_split_colors2.mp4

909 KiB

Sun Kim added the

labels 2023-12-05 04:41:15 +01:00

Iliya Katushenock changed title from ~~Compositor Split node with only colors does not work~~ to Regression: Compositor Split node with only colors does not work

2023-12-05 09:23:18 +01:00

Iliya Katushenock added the

Nodes & Physics

labels 2023-12-05 09:23:24 +01:00

Philipp Oeser commented

2023-12-05 10:20:37 +01:00

Member

It works as the Compositor Backdrop, but not in 3D View, so will confirm

It works as the Compositor Backdrop, but not in 3D View, so will confirm

Philipp Oeser changed title from ~~Regression: Compositor Split node with only colors does not work~~ to Regression: Compositor Split node with only colors does not work (in 3DView, compositor ackdrop is fine)

2023-12-05 10:21:10 +01:00

Philipp Oeser added

and removed

labels 2023-12-05 10:21:34 +01:00

Evan Wilson changed title from ~~Regression: Compositor Split node with only colors does not work (in 3DView, compositor ackdrop is fine)~~ to Regression: Compositor Split node with only colors does not work (in 3DView, compositor backdrop is fine)

2023-12-06 02:43:11 +01:00

Pratik Borhade commented

2023-12-06 04:30:53 +01:00

Member

Likely caused by 153f14be2b

> Likely caused by 153f14be2b @zazizizou ^

Habib Gahbiche self-assigned this 2023-12-06 09:24:25 +01:00

Habib Gahbiche commented

2023-12-06 09:28:57 +01:00

Member

Thanks for the report. This was intentional. The Fullframe compositor also behaves the same way. But this seems to be perceived as a bug / non desired behavior, so I will change it to match the Tiled compositor.

Thanks for the report. This was intentional. The Fullframe compositor also behaves the same way. But this seems to be perceived as a bug / non desired behavior, so I will change it to match the Tiled compositor.

Omar Emara commented

2023-12-06 11:31:49 +01:00

Member

@zazizizou I don't think we should change it for now. The node is no longer a viewer and should not depend on global information like the render size.

@zazizizou I don't think we should change it for now. The node is no longer a viewer and should not depend on global information like the render size.

Habib Gahbiche commented

2023-12-06 14:25:50 +01:00

Member

@OmarEmaraDev but the expectation seems to be that unconnected input behaves the same way as connected input with an image of same color.

For example, would you expect those two setups to behave the same?
Linked:

Unlinked

@OmarEmaraDev but the expectation seems to be that unconnected input behaves the same way as connected input with an image of same color. For example, would you expect those two setups to behave the same? Linked: <img src="/attachments/14187385-237f-4e77-a850-54ece9039cbd" width="500"/> Unlinked <img src="/attachments/861eefe0-4bc0-41d4-81ca-62478cc9c478" width="500"/>

unlinked.jpg

110 KiB

194 KiB

Omar Emara commented

2023-12-06 14:30:07 +01:00

Member

I wouldn't expect them to behave the same, since the unlinked one is doing size inference, which is the opposite of what we are trying to do in #115733.

The only issue here is that versioning will not work for unliked sockets, which can try to fix, but it doesn't seem worth it.

I wouldn't expect them to behave the same, since the unlinked one is doing size inference, which is the opposite of what we are trying to do in #115733. The only issue here is that versioning will not work for unliked sockets, which can try to fix, but it doesn't seem worth it.

Habib Gahbiche commented

2023-12-08 16:55:01 +01:00

Member

@persun Do you have a practical use case in mind where the behavior you describe is useful? Otherwise I'd suggest to close this report...

@persun Do you have a practical use case in mind where the behavior you describe is useful? Otherwise I'd suggest to close this report...

Sun Kim commented

2023-12-09 08:53:48 +01:00

Author

Contributor

@zazizizou I just felt that it works differently with the expectation. I think most use cases people can think with this can be done with some math or the texture node (or upcoming texture nodes support).

But I think this at least warrants a note in the manual.

Thanks for taking a look!

@zazizizou I just felt that it works differently with the expectation. I think most use cases people can think with this can be done with some math or the texture node (or upcoming texture nodes support). But I think this at least warrants a note in the manual. Thanks for taking a look!

Sun Kim closed this issue

2023-12-09 08:54:30 +01:00

Blender Bot added

and removed

labels 2023-12-09 08:54:54 +01:00

Habib Gahbiche commented

2023-12-09 11:02:09 +01:00

Member

@persun ok, thanks for testing!

@persun ok, thanks for testing!

Sign in to join this conversation.

No Label

Animation & Rigging

Asset Browser Project Overview

Automated Testing

Blender Asset Bundle

Dependency Graph

Development Management

EEVEE & Viewport

Images & Movies

Motion Tracking

Nodes & Physics

Pipeline, Assets & IO

Platforms, Builds & Tests

Render & Cycles

Render Pipeline

Sculpt, Paint & Texture

Video Sequencer

Virtual Reality

Blender 2.8 Project

Milestone 1: Basic, Local Asset Browser

Good First Issue

Animation & Rigging

Development Management

EEVEE & Viewport

Nodes & Physics

Pipeline, Assets & IO

Platforms, Builds & Tests

Render & Cycles

Sculpt, Paint & Texture

Needs Info from Developers

Needs Information from User

No Milestone

No project

No Assignees

5 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: blender/blender#115780

No description provided.