Autosmooth/sculpt/viewport bug #46726

New Issue

Paul · 2015-11-08T17:31:40+01:00

Paul commented

2015-11-08 17:31:40 +01:00

System Information
windows 8.1 surface pro intel HD 4000 ;
windows 7 nvidia geforece Gtx 560

Blender Version
Broken: 2.76b
Worked: (optional)

Short description of error
viewport does not update shading of model while sculpting on meshes with matcap set throught shadeless material with texture mapped to normal and Autosmoth checkbox checked in Normals section of object Data.

Exact steps for others to reproduce the error
Create suzanne, add material in Blender Internal and check shadeless in shading section, go to texture tab. Add texture and choose any of this round ball images like here
https://www.google.pl/search?q=matcap+material&source=lnms&tbm=isch&sa=X&ved=0CAcQ_AUoAWoVChMIlrG2vZqByQIVCf5yCh1U2QDa&biw=2000&bih=969&dpr=0.8

go to mapping section and choose normal from coordinates section. Then go to object Data tab and check autosmooth checkbox.

Then add multiresolution subdivide few times, go to sculpt mode and start sculpting. Shading of model will update only if you exit sculpt mode and move back to sculpt mode.
blender_bug.blend

**System Information** windows 8.1 surface pro intel HD 4000 ; windows 7 nvidia geforece Gtx 560 **Blender Version** Broken: 2.76b Worked: (optional) **Short description of error** viewport does not update shading of model while sculpting on meshes with matcap set throught shadeless material with texture mapped to normal and Autosmoth checkbox checked in Normals section of object Data. **Exact steps for others to reproduce the error** Create suzanne, add material in Blender Internal and check shadeless in shading section, go to texture tab. Add texture and choose any of this round ball images like here https://www.google.pl/search?q=matcap+material&source=lnms&tbm=isch&sa=X&ved=0CAcQ_AUoAWoVChMIlrG2vZqByQIVCf5yCh1U2QDa&biw=2000&bih=969&dpr=0.8 go to mapping section and choose normal from coordinates section. Then go to object Data tab and check autosmooth checkbox. Then add multiresolution subdivide few times, go to sculpt mode and start sculpting. Shading of model will update only if you exit sculpt mode and move back to sculpt mode. [blender_bug.blend](https://archive.blender.org/developer/F253787/blender_bug.blend)

Paul commented

2015-11-08 17:31:40 +01:00

Changed status to: 'Open'

Paul commented

2015-11-08 17:31:40 +01:00

Added subscriber: @riceart

Antonis Ryakiotakis commented

2015-11-09 14:02:32 +01:00

Added subscriber: @Psy-Fi

Antonis Ryakiotakis commented

2015-11-09 14:02:32 +01:00

Autosmooth was never supported in sculpt mode as far as I know.

Antonis Ryakiotakis commented

2015-11-09 22:04:40 +01:00

Changed status from 'Open' to: 'Archived'

Antonis Ryakiotakis closed this issue

2015-11-09 22:04:40 +01:00

Antonis Ryakiotakis self-assigned this 2015-11-09 22:04:40 +01:00

Antonis Ryakiotakis commented

2015-11-09 22:04:40 +01:00

I'll be closing this as not supported. Regenerating autosmooth normals in sculpt mode would make sculpting quite slow anyway. It's just not possible to work like that.

Paul commented

2015-11-09 22:19:41 +01:00

Im not asking here to add autosmooth to sculpting. I think that turning autosmooth option off everytime someone enters sculpt mode might be solution for this. Leaving this unresolved will be confusing for some people.

Regards.

Im not asking here to add autosmooth to sculpting. I think that turning autosmooth option off everytime someone enters sculpt mode might be solution for this. Leaving this unresolved will be confusing for some people. Regards.

Antonis Ryakiotakis commented

2015-11-09 22:28:59 +01:00

Changed status from 'Archived' to: 'Open'

Antonis Ryakiotakis reopened this issue

2015-11-09 22:28:59 +01:00

Antonis Ryakiotakis commented

2015-11-09 22:28:59 +01:00

You have a point let me check if it;s possible

Antonis Ryakiotakis commented

2015-11-10 08:53:35 +01:00

As said above, just adding

	Mesh *me = (Mesh *)ob->data;

	if (me->flag & ME_AUTOSMOOTH)
		DAG_id_tag_update(&me->id, 0);

in sculpt_flush_update makes this work. You can even see custom normals now, but speed-wise it's unusable. The problem with mesh autosmooth is that it's a mesh variable and at the time we upload normal uploads GPU we only have access to the derivedmesh. It's probably possible to somehow hack this in though.

As said above, just adding ``` Mesh *me = (Mesh *)ob->data; if (me->flag & ME_AUTOSMOOTH) DAG_id_tag_update(&me->id, 0); ``` in sculpt_flush_update makes this work. You can even see custom normals now, but speed-wise it's unusable. The problem with mesh autosmooth is that it's a mesh variable and at the time we upload normal uploads GPU we only have access to the derivedmesh. It's probably possible to somehow hack this in though.

Paul commented

2015-11-10 22:44:29 +01:00

Sorry to bother, but trick with autosmooth checkbox works only if i use multiresolution modifier. I just discovered that if i apply midifier i get same visual bug no matter what position autosmooth check box is left. You can check this by downloading file that i atached in first post and just apply multiress modifier then try to sculpt.

Regards,

Sorry to bother, but trick with autosmooth checkbox works only if i use multiresolution modifier. I just discovered that if i apply midifier i get same visual bug no matter what position autosmooth check box is left. You can check this by downloading file that i atached in first post and just apply multiress modifier then try to sculpt. Regards,

blender-admin commented

2015-11-13 21:58:30 +01:00

This issue was referenced by 4d33c37c9e

This issue was referenced by 4d33c37c9eb4fc46482f7b2a7227b54c5cfda07a

Antonis Ryakiotakis commented

2015-11-13 21:59:32 +01:00

Changed status from 'Open' to: 'Resolved'

Antonis Ryakiotakis closed this issue

2015-11-13 21:59:32 +01:00

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

Autosmooth/sculpt/viewport bug #46726