Hair Curve 'Add' brush initializes strands with 0 radius attr #117101

New Issue

Closed

opened 2024-01-13 22:33:15 +01:00 by Jose Conseco · 8 comments

Jose Conseco commented

2024-01-13 22:33:15 +01:00

System Information
Operating system: Linux-6.6.10-1-MANJARO-x86_64-with-glibc2.38 64 Bits, X11 UI
Graphics card: NVIDIA GeForce RTX 3060/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 545.29.06

Blender Version
Broken: version: 3.6.7, branch: blender-v3.6-release, commit date: 2023-12-11 16:28, hash: cbd81f283d58
Worked: I think never

Short description of error
In new hair curves sculpt mode, if we use 'Add' brush, new hair curves gets initialized with 0 radius. This makes very poor workflow, since we 99% will have to override it with value 1. Old curves are correctly set with radius 1 by default (yes, they do not use new attributes - but still, default radius 0 is unusable).

Exact steps for others to reproduce the error

Open attached blend file
Click few times on sphere, with 'Add' strand brush
You will see no Curve Profile on new strands - their width is 0.

sculpt_add.blend

**System Information** Operating system: Linux-6.6.10-1-MANJARO-x86_64-with-glibc2.38 64 Bits, X11 UI Graphics card: NVIDIA GeForce RTX 3060/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 545.29.06 **Blender Version** Broken: version: 3.6.7, branch: blender-v3.6-release, commit date: 2023-12-11 16:28, hash: `cbd81f283d58` Worked: I think never **Short description of error** In new **hair curves sculpt mode**, if we use 'Add' brush, new hair curves gets initialized with **0 radius**. This makes very poor workflow, since we 99% will have to override it with value 1. Old curves are correctly set with radius 1 by default (yes, they do not use new attributes - but still, default radius 0 is unusable). **Exact steps for others to reproduce the error** 1. Open attached blend file 2. Click few times on sphere, with 'Add' strand brush 3. You will see no Curve Profile on new strands - their width is 0. [sculpt_add.blend](/attachments/643355ca-4404-4854-95d9-35559cff6e35)

sculpt_add.blend

121 KiB

Jose Conseco added the

labels 2024-01-13 22:33:16 +01:00

Iliya Katushenock added the

Nodes & Physics

Sculpt, Paint & Texture

labels 2024-01-13 22:38:37 +01:00

Pratik Borhade commented

2024-01-15 08:21:22 +01:00

Member

Thanks for the report. Can confirm but not sure whether this is intended. You can avoid this with the help of stored name attribute:

Thanks for the report. Can confirm but not sure whether this is intended. You can avoid this with the help of stored name attribute: ![image](/attachments/57ac01cc-56e8-4b8d-ae55-872cf9f1167d)

Pratik Borhade added

Nodes & Physics

Needs Info from Developers

and removed

Nodes & Physics

labels 2024-01-15 08:21:40 +01:00

Philipp Oeser commented

2024-01-15 10:33:50 +01:00

Member

In essence, this is the same problem as in #116577

In essence, this is the same problem as in #116577

Iliya Katushenock commented

2024-01-15 11:36:11 +01:00

Member

In essence, this is the same problem as in #116577

Not really the same. But i know the same bug report was there. And not sure if this was fixed.

> In essence, this is the same problem as in #116577 Not really the same. But i know the same bug report was there. And not sure if this was fixed.

Jose Conseco commented

2024-01-15 14:20:48 +01:00

Author

It is bit similar - in the blend file above, the radius attribute was manually created by me.

thus each new strand receives 'radius' value = 0.
In that case 'radius' attribute values is well defined - my complaint is only that it should default to 1.

But if I did not create 'radius' attr manually, then 'add' brush wont generate it too. In that case:

radius nodes - defaults to radius value 0
'curve to mesh' defaults non existing 'radius' attr to 1.
Not sure how other nodes handle non existing 'radius' attribute.

It is bit similar - in the blend file above, the radius attribute was manually created by me. - thus each new strand receives 'radius' value = 0. In that case 'radius' attribute values is well defined - my complaint is only that it should default to 1. But if I did not create 'radius' attr manually, then 'add' brush wont generate it too. In that case: - radius nodes - defaults to radius value 0 - 'curve to mesh' defaults non existing 'radius' attr to 1. Not sure how other nodes handle non existing 'radius' attribute.

Jose Conseco commented

2024-01-15 14:25:45 +01:00

Author

Thanks for the report. Can confirm but not sure whether this is intended. You can avoid this with the help of stored name attribute:

I cannot see the image. But I assume its about using 'named attribute' > exist input ? It will work only if 'radius' attribute does not exist - in that case yes - I can detect it, and init 'radius' to value 1. But if 'radius' attribute does exist, then new strands will receive value 0- making haircrads invisible ( 'Named Attribute' > Exist - will return true).

> Thanks for the report. Can confirm but not sure whether this is intended. You can avoid this with the help of stored name attribute: ![image](/attachments/57ac01cc-56e8-4b8d-ae55-872cf9f1167d) I cannot see the image. But I assume its about using 'named attribute' > exist input ? It will work only if 'radius' attribute does not exist - in that case yes - I can detect it, and init 'radius' to value 1. But if 'radius' attribute does exist, then new strands will receive value 0- making haircrads invisible ( 'Named Attribute' > Exist - will return true).

Hans Goudey added

Good First Issue

and removed

Needs Info from Developers

labels 2024-02-07 01:14:56 +01:00

Achyutha-S-Bharadwaj commented

2024-02-07 10:48:21 +01:00

@JoseConseco I am really really new to contributing to Open Source projects on GitHub, can I please work on this ?

@JoseConseco I am really really new to contributing to Open Source projects on GitHub, can I please work on this ?

Iliya Katushenock commented

2024-02-07 10:53:28 +01:00

Member

@Achyutha-S-Bharadwaj Welcome, here is manual to start: https://developer.blender.org/docs/handbook/new_developers/

@Achyutha-S-Bharadwaj Welcome, here is manual to start: https://developer.blender.org/docs/handbook/new_developers/

Achyutha-S-Bharadwaj commented

2024-02-08 07:53:39 +01:00

@mod_moder Thank You so much

@mod_moder Thank You so much

XDzZyq referenced this issue

2024-02-15 17:26:36 +01:00

Sculpt: Initialize hairs with custom radius #118339

Hans Goudey referenced this issue from a commit

2024-02-27 18:22:17 +01:00

Sculpt: Add interpolate radius option for curve radius

Hans Goudey closed this issue

2024-02-27 18:22:19 +01:00

Blender Bot added

and removed

labels 2024-02-27 18:22:21 +01:00

Jose Conseco referenced this issue

2024-02-27 22:03:43 +01:00

Sculpt: Initialize hairs with custom radius #118339

Sign in to join this conversation.

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

5 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#117101

No description provided.