Fix T62730 Overlay: Selected edit hair points highlight is incorrect
This was due to the fact the drawing code was expecting the editpoints to be equaly spaced. Reuse the code in particle.c to output the select mask in red color channel of the particle (which is unused in new code).
This commit is contained in:
parent
ca572e9970
commit
b2034c6ba2
Notes:
blender-bot
2024-04-29 13:07:32 +02:00
Referenced by issue #62730, Subdivide a hair segment causes the high light to appear at the wrong place
|
@ -3246,22 +3246,24 @@ static void psys_cache_edit_paths_iter(void *__restrict iter_data_v,
|
|||
}
|
||||
}
|
||||
else {
|
||||
/* HACK(fclem): Instead of setting the color we pass the select state in the red channel.
|
||||
* This is then picked up in DRW and the gpu shader will do the color interpolation. */
|
||||
if ((ekey + (pind.ekey[0] - point->keys))->flag & PEK_SELECT) {
|
||||
if ((ekey + (pind.ekey[1] - point->keys))->flag & PEK_SELECT) {
|
||||
copy_v3_v3(ca->col, iter_data->sel_col);
|
||||
ca->col[0] = 1.0f;
|
||||
}
|
||||
else {
|
||||
keytime = (t - (*pind.ekey[0]->time)) / ((*pind.ekey[1]->time) - (*pind.ekey[0]->time));
|
||||
interp_v3_v3v3(ca->col, iter_data->sel_col, iter_data->nosel_col, keytime);
|
||||
ca->col[0] = 1.0f - keytime;
|
||||
}
|
||||
}
|
||||
else {
|
||||
if ((ekey + (pind.ekey[1] - point->keys))->flag & PEK_SELECT) {
|
||||
keytime = (t - (*pind.ekey[0]->time)) / ((*pind.ekey[1]->time) - (*pind.ekey[0]->time));
|
||||
interp_v3_v3v3(ca->col, iter_data->nosel_col, iter_data->sel_col, keytime);
|
||||
ca->col[0] = keytime;
|
||||
}
|
||||
else {
|
||||
copy_v3_v3(ca->col, iter_data->nosel_col);
|
||||
ca->col[0] = 0.0f;
|
||||
}
|
||||
}
|
||||
}
|
||||
|
|
|
@ -637,23 +637,6 @@ static void particle_batch_cache_fill_segments_proc_pos(ParticleCacheKey **path_
|
|||
}
|
||||
}
|
||||
|
||||
static float particle_key_select_ratio(const PTCacheEdit *edit, int strand, float t)
|
||||
{
|
||||
const PTCacheEditPoint *point = &edit->points[strand];
|
||||
float edit_key_seg_t = 1.0f / (point->totkey - 1);
|
||||
if (t == 1.0) {
|
||||
return (point->keys[point->totkey - 1].flag & PEK_SELECT) ? 1.0f : 0.0;
|
||||
}
|
||||
else {
|
||||
float interp = t / edit_key_seg_t;
|
||||
int index = (int)interp;
|
||||
interp -= floorf(interp); /* Time between 2 edit key */
|
||||
float s1 = (point->keys[index].flag & PEK_SELECT) ? 1.0f : 0.0;
|
||||
float s2 = (point->keys[index + 1].flag & PEK_SELECT) ? 1.0f : 0.0;
|
||||
return s1 + interp * (s2 - s1);
|
||||
}
|
||||
}
|
||||
|
||||
static float particle_key_weight(const ParticleData *particle, int strand, float t)
|
||||
{
|
||||
const ParticleData *part = particle + strand;
|
||||
|
@ -673,8 +656,8 @@ static float particle_key_weight(const ParticleData *particle, int strand, float
|
|||
}
|
||||
|
||||
static int particle_batch_cache_fill_segments_edit(
|
||||
const PTCacheEdit *edit, /* NULL for weight data */
|
||||
const ParticleData *particle, /* NULL for select data */
|
||||
const PTCacheEdit *UNUSED(edit), /* NULL for weight data */
|
||||
const ParticleData *particle, /* NULL for select data */
|
||||
ParticleCacheKey **path_cache,
|
||||
const int start_index,
|
||||
const int num_path_keys,
|
||||
|
@ -697,8 +680,8 @@ static int particle_batch_cache_fill_segments_edit(
|
|||
seg_data->color = (weight < 1.0f) ? weight : 1.0f;
|
||||
}
|
||||
else {
|
||||
float selected = particle_key_select_ratio(edit, i, strand_t);
|
||||
seg_data->color = selected;
|
||||
/* Computed in psys_cache_edit_paths_iter(). */
|
||||
seg_data->color = path[j].col[0];
|
||||
}
|
||||
GPU_indexbuf_add_generic_vert(elb, curr_point);
|
||||
curr_point++;
|
||||
|
|
Loading…
Reference in New Issue