Line Art feature update: Option to keep contour when using face mark filtering
ClosedPublic
Actions

Authored by YimingWu (NicksBest) on Jan 17 2022, 1:24 AM.

Details

Reviewers

Antonio Vazquez (antoniov)
Aleš Jelovčan (frogstomp)

Group Reviewers

Grease Pencil

Maniphest Tasks

T87739: Line Art further improvement list

Commits

rBdde997086ce2: LineArt: Option to keep contour when using face mark filtering.

Summary

When enabled, it will keep contour around the object instead of hide them by rule of face mark, so the object can always have full contour while filtering out some of the feature lines inside certain regions.

From left to right: mesh setup, face mark filtering without "keep contour", with "keep contour"

Diff Detail

Repository: rB Blender

Event Timeline

YimingWu (NicksBest) requested review of this revision.Jan 17 2022, 1:24 AM

YimingWu (NicksBest) created this revision.

YimingWu (NicksBest) added a task: T87739: Line Art further improvement list.

YimingWu (NicksBest) edited the summary of this revision. (Show Details)Jan 17 2022, 1:36 AM

There are several white space changes, probably caused by clang format? hummm...

YimingWu (NicksBest) added a reviewer: Grease Pencil.Jan 17 2022, 1:39 AM

Just a few comments about format. The code LGTM, but we need some testing.

@Aleš Jelovčan (frogstomp) could you test the patch?

source/blender/gpencil_modifiers/intern/lineart/lineart_cpu.c
1900–1901	Not sure if clang is moving `*/` to next line.
2457	`* files. */`
source/blender/makesdna/DNA_gpencil_modifier_defaults.h
318–319	extra blank line here?