GPencil: Fill tool on the surface will not be fill in correct place. #99459

New Issue

shoji kawano · 2022-07-05T14:03:45+02:00

shoji kawano commented

2022-07-05 14:03:45 +02:00

System Information
Operating system: Windows 10
Graphics card: GTX1080

Blender Version
Broken: 3.2.0, 3.3.0-alpha+master.6e879c39987c
Worked: 2.93.109

Short description of error
The use of the fill tool on the surface of the grease pencil will not be fill in correct place.

Exact steps for others to reproduce the error

Start Blender.
Create a grease pencil object and enter draw mode.
Set "Stroke Placement" to "Surface".
Draw a stroke on the default cube with the Draw tool.
Create a new material, uncheck Stroke and check Fill.
Select the material you created and fill the stroke with the Fill tool.
Filled the wrong position.

**System Information** Operating system: Windows 10 Graphics card: GTX1080 **Blender Version** Broken: 3.2.0, 3.3.0-alpha+master.6e879c39987c Worked: 2.93.109 **Short description of error** The use of the fill tool on the surface of the grease pencil will not be fill in correct place. ![image.png](https://archive.blender.org/developer/F13260730/image.png) ![image.png](https://archive.blender.org/developer/F13260735/image.png) **Exact steps for others to reproduce the error** 1. Start Blender. 2. Create a grease pencil object and enter draw mode. 3. Set "Stroke Placement" to "Surface". 4. Draw a stroke on the default cube with the Draw tool. 5. Create a new material, uncheck Stroke and check Fill. 6. Select the material you created and fill the stroke with the Fill tool. 7. Filled the wrong position.

shoji kawano commented

2022-07-05 14:03:45 +02:00

Added subscriber: @umiyuki

Omar Emara commented

2022-07-05 14:43:22 +02:00

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

Antonio Vazquez commented

2022-07-21 16:05:51 +02:00

Added subscriber: @antoniov

Antonio Vazquez commented

2022-07-21 16:05:51 +02:00

Looking at the code, not sure we can solve this easily. The fill tool is working in an "internal" 2D flat representation of the fill area and cannot access to the surface. Also, the fill tool is not designed to work in areas with volume (not flat), so the result can be unexpected.

I would consider this a know issue because the right (a maybe unique) way of solve this is the right way is to have boolean operations. There were some plans about this boolean library for 2D, but now is paused.

Looking at the code, not sure we can solve this easily. The fill tool is working in an "internal" 2D flat representation of the fill area and cannot access to the surface. Also, the fill tool is not designed to work in areas with volume (not flat), so the result can be unexpected. I would consider this a know issue because the right (a maybe unique) way of solve this is the right way is to have boolean operations. There were some plans about this boolean library for 2D, but now is paused.

Antonio Vazquez changed title from ~~The use of the fill tool on the surface of the grease pencil will not be fill in correct place.~~ to GPencil: Fill tool on the surface will not be fill in correct place.

2022-07-21 16:06:19 +02:00

shoji kawano commented

2022-07-25 10:26:16 +02:00

Changed status from 'Confirmed' to: 'Needs Developer To Reproduce'

shoji kawano commented

2022-07-25 10:26:17 +02:00

I don't think this is a known issue. It works fine with Blender 2.93 and 3.0. I'm having problems with 3.1 and later. In other words, it's a bug that was recently caused by some mistake.

Antonio Vazquez commented

2022-07-25 11:31:47 +02:00

Added subscriber: @OmarEmaraDev

Antonio Vazquez commented

2022-07-25 11:31:47 +02:00

@OmarEmaraDev I guess this could be introduced in this commit: https://developer.blender.org/rBaa0ac0035a0d3601672a0c732e3f8f932a36fc04

I have tried to compile several times but I cannot compile that version...could you try to find if this commit breaks the fill. In theory, previous commit must work.

Here a simple test file.
T99459_test_file.blend

@OmarEmaraDev I guess this could be introduced in this commit: https://developer.blender.org/rBaa0ac0035a0d3601672a0c732e3f8f932a36fc04 I have tried to compile several times but I cannot compile that version...could you try to find if this commit breaks the fill. In theory, previous commit must work. Here a simple test file. [T99459_test_file.blend](https://archive.blender.org/developer/F13314666/T99459_test_file.blend)

Antonio Vazquez commented

2022-07-25 11:32:54 +02:00

Added subscriber: @mano-wii

Antonio Vazquez commented

2022-07-25 11:32:54 +02:00

@mano-wii could you try to test this bug? I always get a cmake error with the suspicious commit.

Antonio Vazquez commented

2022-07-25 11:33:06 +02:00

Changed status from 'Needs Developer To Reproduce' to: 'Confirmed'

Antonio Vazquez commented

2022-07-25 11:52:40 +02:00

@mano-wii I have done a partial revert to test and without this commit the fill is working as expected, so something in the depths array is not working as expected in Fill.

blender-admin commented

2022-07-25 18:41:35 +02:00

This issue was referenced by 703dff333c

This issue was referenced by 703dff333ce6cbc25c990042858a0c40f9a212cb

Germano Cavalcante commented

2022-07-25 19:14:18 +02:00

Changed status from 'Confirmed' to: 'Resolved'

Germano Cavalcante closed this issue

2022-07-25 19:14:18 +02:00

Germano Cavalcante self-assigned this 2022-07-25 19:14:18 +02:00

shoji kawano commented

2022-07-26 07:14:55 +02:00

Thanks for your efforts to investigate and fix the bug!

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

GPencil: Fill tool on the surface will not be fill in correct place. #99459