[Bf-blender-cvs] [b51554bb937] temp-lineart-contained: LineArt: Remove geometry space chaining and clean up vertex group selection code.
YimingWu
noreply at git.blender.org
Fri Mar 19 04:50:33 CET 2021
Commit: b51554bb9378983031d15089439cc0f9f1a81eb1
Author: YimingWu
Date: Fri Mar 19 09:01:43 2021 +0800
Branches: temp-lineart-contained
https://developer.blender.org/rBb51554bb9378983031d15089439cc0f9f1a81eb1
LineArt: Remove geometry space chaining and clean up vertex group selection code.
===================================================================
M source/blender/gpencil_modifiers/intern/MOD_gpencillineart.c
M source/blender/gpencil_modifiers/intern/lineart/MOD_lineart.h
M source/blender/gpencil_modifiers/intern/lineart/lineart_chain.c
M source/blender/gpencil_modifiers/intern/lineart/lineart_cpu.c
M source/blender/makesdna/DNA_gpencil_modifier_defaults.h
M source/blender/makesdna/DNA_gpencil_modifier_types.h
M source/blender/makesrna/intern/rna_gpencil_modifier.c
===================================================================
diff --git a/source/blender/gpencil_modifiers/intern/MOD_gpencillineart.c b/source/blender/gpencil_modifiers/intern/MOD_gpencillineart.c
index b66e970b107..9a7ed53cedb 100644
--- a/source/blender/gpencil_modifiers/intern/MOD_gpencillineart.c
+++ b/source/blender/gpencil_modifiers/intern/MOD_gpencillineart.c
@@ -388,8 +388,7 @@ static void chaining_panel_draw(const bContext *UNUSED(C), Panel *panel)
uiItemR(col, ptr, "fuzzy_everything", 0, NULL, ICON_NONE);
col = uiLayoutColumn(layout, true);
- uiItemR(col, ptr, "chaining_geometry_threshold", 0, NULL, ICON_NONE);
- uiItemR(col, ptr, "chaining_image_threshold", 0, NULL, ICON_NONE);
+ uiItemR(col, ptr, "chaining_threshold", 0, NULL, ICON_NONE);
uiItemR(layout, ptr, "resample_length", UI_ITEM_R_SLIDER, NULL, ICON_NONE);
@@ -422,7 +421,7 @@ static void vgroup_panel_draw(const bContext *UNUSED(C), Panel *panel)
col, ptr, "vertex_group", &ob_ptr, "vertex_groups", IFACE_("Target"), ICON_NONE);
}
- uiItemR(col, ptr, "soft_selection", 0, NULL, ICON_NONE);
+ uiItemR(col, ptr, "binary_weights", 0, NULL, ICON_NONE);
}
static void baking_panel_draw(const bContext *UNUSED(C), Panel *panel)
diff --git a/source/blender/gpencil_modifiers/intern/lineart/MOD_lineart.h b/source/blender/gpencil_modifiers/intern/lineart/MOD_lineart.h
index c01b1a4c86b..dfa8d4c1b6f 100644
--- a/source/blender/gpencil_modifiers/intern/lineart/MOD_lineart.h
+++ b/source/blender/gpencil_modifiers/intern/lineart/MOD_lineart.h
@@ -288,8 +288,7 @@ typedef struct LineartRenderBuffer {
double near_clip, far_clip;
float shift_x, shift_y;
float crease_threshold;
- float chaining_image_threshold;
- float chaining_geometry_threshold;
+ float chaining_threshold;
float angle_splitting_threshold;
/* FIXME: (Yiming) Temporary solution for speeding up calculation by not including lines that
@@ -512,7 +511,7 @@ void MOD_lineart_destroy_render_data(struct LineartGpencilModifierData *lmd);
void MOD_lineart_chain_feature_lines(LineartRenderBuffer *rb);
void MOD_lineart_chain_split_for_fixed_occlusion(LineartRenderBuffer *rb);
-void MOD_lineart_chain_connect(LineartRenderBuffer *rb, const bool do_geometry_space);
+void MOD_lineart_chain_connect(LineartRenderBuffer *rb);
void MOD_lineart_chain_discard_short(LineartRenderBuffer *rb, const float threshold);
void MOD_lineart_chain_split_angle(LineartRenderBuffer *rb, float angle_threshold_rad);
@@ -552,7 +551,8 @@ void MOD_lineart_gpencil_generate(LineartRenderBuffer *rb,
float resample_length,
const char *source_vgname,
const char *vgname,
- int modifier_flags);
+ int modifier_flags,
+ float weight_threshold);
float MOD_lineart_chain_compute_length(LineartLineChain *rlc);
diff --git a/source/blender/gpencil_modifiers/intern/lineart/lineart_chain.c b/source/blender/gpencil_modifiers/intern/lineart/lineart_chain.c
index fae5bcd7c2e..99a316de33a 100644
--- a/source/blender/gpencil_modifiers/intern/lineart/lineart_chain.c
+++ b/source/blender/gpencil_modifiers/intern/lineart/lineart_chain.c
@@ -694,8 +694,7 @@ static LineartChainRegisterEntry *lineart_chain_get_closest_cre(LineartRenderBuf
int occlusion,
unsigned char transparency_mask,
float dist,
- int do_geometry_space,
- float *result_new_len,
+ float *new_dist,
LineartBoundingArea *caller_ba)
{
@@ -739,13 +738,12 @@ static LineartChainRegisterEntry *lineart_chain_get_closest_cre(LineartRenderBuf
}
}
- float new_len = do_geometry_space ? len_v3v3(cre->rlci->gpos, rlci->gpos) :
- len_v2v2(cre->rlci->pos, rlci->pos);
- if (new_len < dist) {
+ float newd = len_v2v2(cre->rlci->pos, rlci->pos);
+ if (newd < dist) {
closest_cre = cre;
- dist = new_len;
- if (result_new_len) {
- (*result_new_len) = new_len;
+ dist = newd;
+ if (new_dist) {
+ (*new_dist) = newd;
}
}
}
@@ -758,30 +756,22 @@ static LineartChainRegisterEntry *lineart_chain_get_closest_cre(LineartRenderBuf
if (dist_to < dist && dist_to > 0) { \
LISTBASE_FOREACH (LinkData *, ld, list) { \
LineartBoundingArea *sba = (LineartBoundingArea *)ld->data; \
- adjacent_closest = lineart_chain_get_closest_cre(rb, \
- sba, \
- rlc, \
- rlci, \
- occlusion, \
- transparency_mask, \
- dist, \
- do_geometry_space, \
- &adjacent_new_len, \
- ba); \
+ adjacent_closest = lineart_chain_get_closest_cre( \
+ rb, sba, rlc, rlci, occlusion, transparency_mask, dist, &adjacent_new_len, ba); \
if (adjacent_new_len < dist) { \
dist = adjacent_new_len; \
closest_cre = adjacent_closest; \
} \
} \
}
- if (!do_geometry_space && !caller_ba) {
+ if (!caller_ba) {
LRT_TEST_ADJACENT_AREAS(rlci->pos[0] - ba->l, &ba->lp);
LRT_TEST_ADJACENT_AREAS(ba->r - rlci->pos[0], &ba->rp);
LRT_TEST_ADJACENT_AREAS(ba->u - rlci->pos[1], &ba->up);
LRT_TEST_ADJACENT_AREAS(rlci->pos[1] - ba->b, &ba->bp);
}
- if (result_new_len) {
- (*result_new_len) = dist;
+ if (new_dist) {
+ (*new_dist) = dist;
}
return closest_cre;
}
@@ -789,20 +779,19 @@ static LineartChainRegisterEntry *lineart_chain_get_closest_cre(LineartRenderBuf
/* This function only connects two different chains. It will not do any clean up or smart chaining.
* So no: removing overlapping chains, removal of short isolated segments, and no loop reduction is
* implemented yet. */
-void MOD_lineart_chain_connect(LineartRenderBuffer *rb, const bool do_geometry_space)
+void MOD_lineart_chain_connect(LineartRenderBuffer *rb)
{
LineartLineChain *rlc;
LineartLineChainItem *rlci_l, *rlci_r;
LineartBoundingArea *ba_l, *ba_r;
LineartChainRegisterEntry *closest_cre_l, *closest_cre_r, *closest_cre;
- float dist = do_geometry_space ? rb->chaining_geometry_threshold : rb->chaining_image_threshold;
+ float dist = rb->chaining_threshold;
float dist_l, dist_r;
int occlusion, reverse_main;
unsigned char transparency_mask;
ListBase swap = {0};
- if ((!do_geometry_space && rb->chaining_image_threshold < 0.0001) ||
- (do_geometry_space && rb->chaining_geometry_threshold < 0.0001)) {
+ if ((rb->chaining_threshold < 0.00001)) {
return;
}
@@ -825,26 +814,10 @@ void MOD_lineart_chain_connect(LineartRenderBuffer *rb, const bool do_geometry_s
rlci_r = rlc->chain.last;
while ((ba_l = lineart_bounding_area_get_end_point(rb, rlci_l)) &&
(ba_r = lineart_bounding_area_get_end_point(rb, rlci_r))) {
- closest_cre_l = lineart_chain_get_closest_cre(rb,
- ba_l,
- rlc,
- rlci_l,
- occlusion,
- transparency_mask,
- dist,
- do_geometry_space,
- &dist_l,
- NULL);
- closest_cre_r = lineart_chain_get_closest_cre(rb,
- ba_r,
- rlc,
- rlci_r,
- occlusion,
- transparency_mask,
- dist,
- do_geometry_space,
- &dist_r,
- NULL);
+ closest_cre_l = lineart_chain_get_closest_cre(
+ rb, ba_l, rlc, rlci_l, occlusion, transparency_mask, dist, &dist_l, NULL);
+ closest_cre_r = lineart_chain_get_closest_cre(
+ rb, ba_r, rlc, rlci_r, occlusion, transparency_mask, dist, &dist_r, NULL);
if (closest_cre_l && closest_cre_r) {
if (dist_l < dist_r) {
closest_cre = closest_cre_l;
diff --git a/source/blender/gpencil_modifiers/intern/lineart/lineart_cpu.c b/source/blender/gpencil_modifiers/intern/lineart/lineart_cpu.c
index 77de09a5526..c3cd2e4ba03 100644
--- a/source/blender/gpencil_modifiers/intern/lineart/lineart_cpu.c
+++ b/source/blender/gpencil_modifiers/intern/lineart/lineart_cpu.c
@@ -2644,8 +2644,7 @@ static LineartRenderBuffer *lineart_create_render_buffer(Scene *scene,
rb->crease_threshold = cos(M_PI - lmd->crease_threshold);
rb->angle_splitting_threshold = lmd->angle_splitting_threshold;
- rb->chaining_image_threshold = lmd->chaining_image_threshold;
- rb->chaining_geometry_threshold = lmd->chaining_geometry_threshold;
+ rb->chaining_threshold = lmd->chaining_threshold;
rb->fuzzy_intersections = (lmd->calculation_flags & LRT_INTERSECTION_AS_CONTOUR) != 0;
rb->fuzzy_everything = (lmd->calculation_flags & LRT_EVERYTHING_AS_CONTOUR) != 0;
@@ -3676,32 +3675,19 @@ int MOD_lineart_compu
@@ Diff output truncated at 10240 characters. @@
More information about the Bf-blender-cvs
mailing list