Geometry Node : Added Object attribute nodes #120745

Open

Flo_12344 wants to merge 3 commits from Flo_12344/blender:main into main

When changing the target branch, be careful to rebase the branch in your fork to match. See documentation.

Flo_12344 commented

2024-04-17 17:01:53 +02:00

First-time contributor

A Geometry node that help accessing data from different object without the need of using drivers, (light/camera/mesh at the moment), by copy the data path of their different options. (the original idea come from Cartesian Caramel)

A Geometry node that help accessing data from different object without the need of using drivers, (light/camera/mesh at the moment), by copy the data path of their different options. (the original idea come from Cartesian Caramel) ![image](/attachments/4bcb4dee-c254-442f-b0c5-df56b548044c)

27 KiB

👍 5 👀 2 ❤️ 4 🎉 1 🚀 1

Flo_12344 added 1 commit 2024-04-17 17:02:04 +02:00

d54b53aeea Geometry Node : Added Object attribute nodes

Iliya Katushenock added this to the Nodes & Physics project 2024-04-17 17:08:24 +02:00

Iliya Katushenock added the

label 2024-04-17 17:08:27 +02:00

Iliya Katushenock commented

2024-04-17 17:10:26 +02:00

Member

Thanks for Pull Request. How do you planned to build dependency graph relations for used ID?

Thanks for Pull Request. How do you planned to build dependency graph relations for used ID?

Flo_12344 commented

2024-04-17 17:33:43 +02:00

Author

First-time contributor

Sorry, I feel like I'm going to be a bit annoying but what do you mean ?

Sorry, I feel like I'm going to be a bit annoying but what do you mean ?

Iliya Katushenock commented

2024-04-17 17:37:18 +02:00

Member

Blender build graph of object relations to evaluate all modifiers/properties with animations/drivers/load all data/apply constraints/...
Each object is represented as set of components with separate operations to do.
And if you planned to get access to user to any component of any object what is linked to modifier, how do you planned to build relations?

Blender build graph of object relations to evaluate all modifiers/properties with animations/drivers/load all data/apply constraints/... Each object is represented as set of components with separate operations to do. And if you planned to get access to user to any component of any object what is linked to modifier, how do you planned to build relations?

Flo_12344 commented

2024-04-17 17:43:46 +02:00

Author

First-time contributor

At the moment the node retrieve the object data using the rna access.

At the moment the node retrieve the object data using the rna access.

Flo_12344 added 1 commit 2024-04-17 18:40:06 +02:00

e985830f91 Geometry Node : removed unused include

Iliya Katushenock commented

2024-04-17 18:53:54 +02:00

Member

At the moment the node retrieve the object data using the rna access.

Okay, so, this even not work with animated properties (if this is that you mean).

Current feedback:

There is no way to versioning node groups with this node. In case python API will be changed, there is no way to handle this for geometry nodes. Users never like that they geometry nodes do change result in new version of blender.
Assets. There is no way to handle linking of custom properties in such way.
This increate dependency fishnet.I'd name this like state if we have to increase dependencies for all linked object due to lack of static info about kind of dependency. Even if access if happen to not COW objects, i think, this still require at least access to COW cope of objects in memory racer to real data.
This is overkill for usually geometry nodes. If quote the review from the chat: We should focus on more user friendly ways to expose data to geometry nodes.
There is some issues with raw access to RNA values from geometry nodes, currently we have inconsistency of quaternion normalization.
This can provide access to things that is should not be done as it. For example, with this you can read separate vertex position of the mesh. This is not the thing that should be accepted.

> At the moment the node retrieve the object data using the rna access. Okay, so, this even not work with animated properties (if this is that you mean). Current feedback: 1. There is no way to versioning node groups with this node. In case python API will be changed, there is no way to handle this for geometry nodes. Users never like that they geometry nodes do change result in new version of blender. 2. Assets. There is no way to handle linking of custom properties in such way. 3. This increate dependency fishnet.I'd name this like state if we have to increase dependencies for all linked object due to lack of static info about kind of dependency. Even if access if happen to not COW objects, i think, this still require at least access to COW cope of objects in memory racer to real data. 4. This is overkill for usually geometry nodes. If quote the review from the chat: `We should focus on more user friendly ways to expose data to geometry nodes`. 5. There is some issues with raw access to RNA values from geometry nodes, currently we have inconsistency of quaternion normalization. 6. This can provide access to things that is should not be done as it. For example, with this you can read separate vertex position of the mesh. This is not the thing that should be accepted.

Flo_12344 commented

2024-04-17 21:48:59 +02:00

Author

First-time contributor

It worked fine with animated properties in my tests rotation_euler,rotation_quaternion and location or at least those one works fine and the ones that have deps graph issue using driver works the same way with this method.
I didn't though it was overkill I just though of what I needed when working on some of my projects and tried to access the data for wich I needed drivers to make them easier to maintain by just copying the data path of the properties.
With this method I haven't managed to access the vertex position sadly even if it's something that I have tried at first.
It would be interesting if you have an idea of the things that makes it unusable for versioning so I could try to fix them.
If you want, I can provide a build with it for testing.
Thanks for your feedback.

It worked fine with animated properties in my tests rotation_euler,rotation_quaternion and location or at least those one works fine and the ones that have deps graph issue using driver works the same way with this method. I didn't though it was overkill I just though of what I needed when working on some of my projects and tried to access the data for wich I needed drivers to make them easier to maintain by just copying the data path of the properties. With this method I haven't managed to access the vertex position sadly even if it's something that I have tried at first. It would be interesting if you have an idea of the things that makes it unusable for versioning so I could try to fix them. If you want, I can provide a build with it for testing. Thanks for your feedback.

Flo_12344 added 1 commit 2024-04-20 11:43:37 +02:00

46758ed69b Geometry Node : Added enum to int output

This pull request has changes conflicting with the target branch.

scripts/startup/bl_ui/node_add_menu_geometry.py
source/blender/nodes/geometry/CMakeLists.txt

View command line instructions.

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u main:Flo_12344-main

git checkout Flo_12344-main

Sign in to join this conversation.

No reviewers

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

2 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#120745

No description provided.