Grease Pencil: "Set Material Node" in Geometry Nodes doesn't transfer material settings for "Line Art" Masks #100435

New Issue

Christoph Werner · 2022-08-16T11:42:55+02:00

Christoph Werner commented

2022-08-16 11:42:55 +02:00

System Information
Version: 3.2.2, branch: master, commit date: 2022-08-02 18:15, hash: bcfdb14560, type: release
build date: 2022-08-02, 23:41:46
platform: 'Windows-10-10.0.22000-SP0'

Broken: Blender including versions 3.0 - 3.2.2
Worked: Never

Short description of error
When setting / replacing materials via Geometry Nodes that include Grease Pencil Line Art mask settings, the settings are ignored by Grease Pencil or deleted by Geometry Nodes.

Important: If you set or replace the very same material with itself, then all works!

Exact steps for others to reproduce the error
See screenshot and example scene for reproduction.

Example file:
GN_GPencil_LineArt_Error.blend

Screenshot:

**System Information** Version: 3.2.2, branch: master, commit date: 2022-08-02 18:15, hash: bcfdb14560e7, type: release build date: 2022-08-02, 23:41:46 platform: 'Windows-10-10.0.22000-SP0' Broken: Blender including versions 3.0 - 3.2.2 Worked: Never **Short description of error** When setting / replacing materials via Geometry Nodes that include Grease Pencil Line Art mask settings, the settings are ignored by Grease Pencil or deleted by Geometry Nodes. **Important:** If you set or replace the very same material with itself, then all works! **Exact steps for others to reproduce the error** See screenshot and example scene for reproduction. **Example file:** [GN_GPencil_LineArt_Error.blend](https://archive.blender.org/developer/F13393091/GN_GPencil_LineArt_Error.blend) **Screenshot:** ![image.png](https://archive.blender.org/developer/F13393081/image.png)

Christoph Werner commented

2022-08-16 11:42:55 +02:00

Added subscriber: @ChristophWerner

YimingWu commented

2022-08-16 11:46:55 +02:00

Added subscriber: @ChengduLittleA

YimingWu self-assigned this 2022-08-16 11:46:55 +02:00

YimingWu commented

2022-08-16 11:46:55 +02:00

I'll take a look. :)

Christoph Werner commented

2022-08-16 11:55:23 +02:00

In #100435#1404407, @ChengduLittleA wrote:
I'll take a look. :)

Great. Thank you!
Seem to be a basic problem with geometry nodes replacing materials based on object data.

> In #100435#1404407, @ChengduLittleA wrote: > I'll take a look. :) Great. Thank you! Seem to be a basic problem with geometry nodes replacing materials based on object data.

Omar Emara commented

2022-08-16 11:58:38 +02:00

Changed status from 'Needs Triage' to: 'Confirmed'

YimingWu commented

2022-08-16 11:59:23 +02:00

Looks like by adding two material slots in the original object could resolve the issue.

I'm trying if changing BKE_object_material_get to BKE_object_material_get_eval would resolve the issue without requiring extra modification on the user side.

Looks like by adding two material slots in the original object could resolve the issue. I'm trying if changing `BKE_object_material_get` to `BKE_object_material_get_eval` would resolve the issue without requiring extra modification on the user side.

Christoph Werner commented

2022-08-16 12:14:29 +02:00

In #100435#1404416, @ChengduLittleA wrote:
Looks like by adding two material slots in the original object could resolve the issue.

I'm trying if changing BKE_object_material_get to BKE_object_material_get_eval would resolve the issue without requiring extra modification on the user side.

Yes, but this solution works with local objects only. Seem not to work with linked data that has an override.

> In #100435#1404416, @ChengduLittleA wrote: > Looks like by adding two material slots in the original object could resolve the issue. > > I'm trying if changing `BKE_object_material_get` to `BKE_object_material_get_eval` would resolve the issue without requiring extra modification on the user side. Yes, but this solution works with local objects only. Seem not to work with linked data that has an override.

blender-admin commented

2022-08-16 12:29:54 +02:00

This issue was referenced by accf38c1d1

This issue was referenced by accf38c1d1d9c2b34f8e30ee1f1cbbf3cdfb69f6

YimingWu commented

2022-08-16 12:33:21 +02:00

Changed status from 'Confirmed' to: 'Resolved'

YimingWu closed this issue

2022-08-16 12:33:21 +02:00

YimingWu commented

2022-08-16 12:35:18 +02:00

Added subscriber: @PratikPB2123

YimingWu commented

2022-08-16 12:35:18 +02:00

Now it should work fine in 3.3. Should I apply this to 3.2 as well? @PratikPB2123

Pratik Borhade commented

2022-08-16 12:41:27 +02:00

Added subscriber: @ThomasDinges

Pratik Borhade commented

2022-08-16 12:41:27 +02:00

No need :)
3.2.2 was the last corrective release.

In #98661#1401821, @ThomasDinges wrote:
Also closing this one, since there won't be another 3.2.x release.

No need :) 3.2.2 was the last corrective release. - - - > In #98661#1401821, @ThomasDinges wrote: > Also closing this one, since there won't be another 3.2.x release.

YimingWu commented

2022-08-16 15:18:25 +02:00

OK Thanks! :)

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

Grease Pencil: "Set Material Node" in Geometry Nodes doesn't transfer material settings for "Line Art" Masks #100435