Skin modifier crash with null skin radii (C++ assert) #39097

New Issue

gandalf3 · 2014-03-10T20:58:29+01:00

gandalf3 commented

2014-03-10 20:58:29 +01:00

System Information
Archlinux
Nvidia GTX 460

Blender Version
Broken: 2.7 RC1

Short description of error
Setting the skin radius to a smaller value with numinput crashes blender

Exact steps for others to reproduce the error
Based on lightnining.blend

Open attached file and press Ctrl+A .5
Blender crashes

**System Information** Archlinux Nvidia GTX 460 **Blender Version** Broken: 2.7 RC1 **Short description of error** Setting the skin radius to a smaller value with numinput crashes blender **Exact steps for others to reproduce the error** Based on [lightnining.blend](https://archive.blender.org/developer/F80570/lightnining.blend) 1. Open attached file and press Ctrl+A .5 2. Blender crashes

gandalf3 commented

2014-03-10 20:58:29 +01:00

Changed status to: 'Open'

gandalf3 commented

2014-03-10 20:58:29 +01:00

Added subscriber: @gandalf3

blender-admin commented

2014-03-10 20:58:29 +01:00

#44566 was marked as duplicate of this issue

Bastien Montagne commented

2014-03-10 23:27:50 +01:00

Added subscriber: @mont29

Bastien Montagne self-assigned this 2014-03-10 23:28:01 +01:00

Bastien Montagne commented

2014-03-10 23:45:30 +01:00

Actually, issue is caused by skin radius being set to zero (the only dot in numinput will evaluate to 0…).

Bastien Montagne removed their assignment 2014-03-11 00:11:16 +01:00

Bastien Montagne commented

2014-03-11 00:11:16 +01:00

Grmlll… looks like this tool should not crash on Zero radius…

Not my area at all, have no clue how this stuff works… :/

Grmlll… looks like this tool should not crash on Zero radius… Not my area at all, have no clue how this stuff works… :/

Bastien Montagne changed title from ~~Crash when using numinput to adjust skin modifier~~ to Skin modifier crash with null skin radii (C++ assert)

2014-03-11 00:12:35 +01:00

Nicholas Bishop was assigned by Lukas Tönne

2014-03-25 17:33:03 +01:00

Lukas Tönne commented

2014-03-25 17:33:03 +01:00

Added subscribers: @NicholasBishop, @LukasTonne

Lukas Tönne commented

2014-03-25 17:33:03 +01:00

@NicholasBishop: Maybe you could have a look at this issue? I'm getting nan values in the skin modifier data. Culprit seems to be this function (maybe dangling pointers exist in Frame.merge before):
https://developer.blender.org/diffusion/B/browse/master/source/blender/modifiers/intern/MOD_skin.c;a7ed1db51fa32a80b943816d354b45b413d98fa3$1471

There was another issue in relation to skin modifier in this report: #39341
The actual regression was fixed, but if you have a look at this reduced test file Vase_Céramique4.blend1, you can see that enter/exit editmode constantly changes the vertex/face count. I'm pretty sure these two things are both related to skin modifier applied on a edge-only mesh.

@NicholasBishop: Maybe you could have a look at this issue? I'm getting `nan` values in the skin modifier data. Culprit seems to be this function (maybe dangling pointers exist in Frame.merge before): https://developer.blender.org/diffusion/B/browse/master/source/blender/modifiers/intern/MOD_skin.c;a7ed1db51fa32a80b943816d354b45b413d98fa3$1471 There was another issue in relation to skin modifier in this report: #39341 The actual regression was fixed, but if you have a look at this reduced test file [Vase_Céramique4.blend1](https://archive.blender.org/developer/F82548/Vase_Céramique4.blend1), you can see that enter/exit editmode constantly changes the vertex/face count. I'm pretty sure these two things are both related to skin modifier applied on a edge-only mesh.

Sergey Sharybin commented

2014-05-29 10:19:02 +02:00

Added subscribers: @ideasman42, @Sergey

Sergey Sharybin commented

2014-05-29 10:19:02 +02:00

@ideasman42, remember you've added some clamping to skin modifier, guess we need some more here?

Sergey Sharybin commented

2014-12-26 10:07:55 +01:00

@NicholasBishop, any hope you'll have a look in observable future?

michael knubben commented

2015-04-30 20:55:01 +02:00

Added subscriber: @michaelknubben

michael knubben commented

2015-04-30 20:55:01 +02:00

Apologies, but I've created a new bug report about this, as I didn't find this in the search earlier.
It's #44566, and I've also attached a .blend there.
I can't edit it anymore, or I'd make it a subtask!
And as I didn't mention it in my report (can't edit), I'm on Windows 7, 64bit, with an Nvidia GTX 670 (driver: 347.52)

Apologies, but I've created a new bug report about this, as I didn't find this in the search earlier. It's #44566, and I've also attached a .blend there. I can't edit it anymore, or I'd make it a subtask! And as I didn't mention it in my report (can't edit), I'm on Windows 7, 64bit, with an Nvidia GTX 670 (driver: 347.52)

Campbell Barton commented

2015-05-18 06:03:53 +02:00

Changed status from 'Open' to: 'Resolved'

Campbell Barton closed this issue

2015-05-18 06:03:53 +02:00

Campbell Barton commented

2015-05-18 06:03:53 +02:00

7969b238db fixes this. closing.

7969b238db fixes this. closing.

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

Skin modifier crash with null skin radii (C++ assert) #39097