Habib Gahbiche zazizizou
  • Joined on 2015-02-21
Habib Gahbiche commented on pull request blender/blender#114764 2023-11-17 12:29:30 +01:00
Fullframe compositor: unify scaling behavior with realtime compositor

@OmarEmaraDev @Sergey maybe you have opinions here too...

Habib Gahbiche pushed to com-split-node at zazizizou/blender 2023-11-17 12:26:16 +01:00
87e0925f4c Fix build errors
62baa54b70 Merge remote-tracking branch 'origin/main' into com-split-node
eeb8d74a88 Coding style
f6569b7726 Cleanup: Make format
10fae637aa Refactor: Extract function to initialize a BezTriple
Compare 92 commits »
Habib Gahbiche commented on pull request blender/blender#114245 2023-11-17 12:26:01 +01:00
Compositor: Make split viewer a regular split node

👍

Habib Gahbiche commented on pull request blender/blender#114245 2023-11-17 11:57:39 +01:00
Compositor: Make split viewer a regular split node

ELEM is used in other places for single element comparison. I'm leaving out node_gizmo.cc, to better conform with the coding style rule "When making changes, conform to the style and…

Habib Gahbiche commented on pull request blender/blender#114755 2023-11-16 14:34:54 +01:00
Fullframe compositor: unify behavior of size inference with realtime-compositor

Ideally we shouldn't need that flag at all, since all operations can be constant, and the flag is_constant_operation exists for operations that are definately constant.

I will refactor in…

Habib Gahbiche commented on issue blender/blender#114967 2023-11-16 14:30:37 +01:00
Blender 4.0.0 compositor crashes when opening a file that has been migrated from 3.6.5

I can reproduce the freeze, Blender shows following error: error

Did you save the 3.6 with that addon enabled? I think Blender should not…

Habib Gahbiche commented on pull request blender/blender#114755 2023-11-16 14:17:53 +01:00
Fullframe compositor: unify behavior of size inference with realtime-compositor

does it make sense to set can_be_constant to true by default? That could work too, I was hoping this way shows the behavior more explicitly in operations. I don't have a strong opinion…

Habib Gahbiche pushed to com-split-node at zazizizou/blender 2023-11-16 14:01:32 +01:00
05ba55801d Address review comments: remove node storage and remove node from name
Habib Gahbiche commented on pull request blender/blender#114245 2023-11-16 13:57:36 +01:00
Compositor: Make split viewer a regular split node

Ok, I don't mind either way

Habib Gahbiche commented on pull request blender/blender#114245 2023-11-16 13:04:33 +01:00
Compositor: Make split viewer a regular split node

Change type, manually free storage and add output socket.... isn't deleting and constructing a new node cleaner at this point? :)

Habib Gahbiche commented on pull request blender/blender#114245 2023-11-16 12:37:22 +01:00
Compositor: Make split viewer a regular split node

CONVERTER and FILTER don't sound right either... Switch node also uses NODE_CLASS_LAYOUT. Do you suggest I change that as well (in a separate patch) ?

Habib Gahbiche commented on pull request blender/blender#114245 2023-11-16 12:33:51 +01:00
Compositor: Make split viewer a regular split node

I'm not sure how to achieve that. node doesn't have an output socket so linking to viewer node doesn't work. So I took version_geometry_nodes_replace_transfer_attribute_node() from `versioning_…

Habib Gahbiche commented on pull request blender/blender#114755 2023-11-14 23:06:03 +01:00
Fullframe compositor: unify behavior of size inference with realtime-compositor

That's the setup I tried to demonstrate in my previous post. For some reason images are not showing properly...

I don't think we can cover that case using the way constant folding is implemented…

Habib Gahbiche commented on pull request blender/blender#114755 2023-11-13 22:28:14 +01:00
Fullframe compositor: unify behavior of size inference with realtime-compositor

Not sure if this answers your question, but levels is not included in this patch, so when connected to an image it will behave as before, but when it's input is not connected to an input it will…

Habib Gahbiche commented on pull request blender/blender#114764 2023-11-13 21:09:29 +01:00
Fullframe compositor: unify scaling behavior with realtime compositor

@Jeroen-Bakker do you know the historic reasons behind limiting the image size for scale operation? I was not able to get Blender to crash when scaling images with very large factors, but the…

Habib Gahbiche pushed to com-split-node at zazizizou/blender 2023-11-13 21:01:40 +01:00
5147126fa6 Merge remote-tracking branch 'origin/main' into com-split-node
c061eaef9f Revert to original behavior: make first image decide the canvas size
8f362a175d Cleanup: Combine crazyspace C header with C++ header
5371f5516c Cleanup: Add const to MutableSpan methods
8e82c82074 Revert "Fix #114582: Replace unknown node types with an 'undefined' node"
Compare 69 commits »
Habib Gahbiche commented on pull request blender/blender#114245 2023-11-13 20:36:38 +01:00
Compositor: Make split viewer a regular split node

The GPU compositor uses the domain of the first linked input, the Full Frame Compositor was also designed like this from the start,.

This doesn't make sense to me for though, at least for…

Habib Gahbiche commented on pull request blender/blender#114755 2023-11-13 20:15:21 +01:00
Fullframe compositor: unify behavior of size inference with realtime-compositor

Constant folding of the node graph happens every time COM_execute() gets called. It replaces operations such as TranslateOperation with ConstantOperation at runtime, right after building the…