Change the default Subsurf Viewport Level from 1 to 2 #114108

Raimund Klink · 2023-10-24T15:42:08+02:00

Raimund Klink commented

2023-10-24 15:42:08 +02:00

This is a PR to change the default value of the Subdivision Surface Modifier.

We frequently get users who wonder why their scene is unrenderable because everything was fine in the Viewport.
On certain meshes, the RAM difference between level 1 and 2 is HUGE.

Currently, it is not "What you see is what you get". Instead, it is "What you see is crashing for some unknown reason if you try to render."
(Blender doesn't tell you what object (modifier) needed too much RAM, causing an "Out of RAM error")

An alternative to #114163

This is a PR to change the default value of the Subdivision Surface Modifier. We frequently get users who wonder why their scene is unrenderable because everything was fine in the Viewport. On certain meshes, the RAM difference between level 1 and 2 is HUGE. Currently, it is not "What you see is what you get". Instead, it is "What you see is crashing for some unknown reason if you try to render." (Blender doesn't tell you what object (modifier) needed too much RAM, causing an "Out of RAM error") An alternative to https://projects.blender.org/blender/blender/pulls/114163

❤️ 1

Raimund Klink added 1 commit 2023-10-24 15:42:19 +02:00

605626238a Update source/blender/makesdna/DNA_modifier_defaults.h

Iliya Katushenock added this to the Modeling project 2023-10-24 15:47:03 +02:00

Raimund Klink requested review from Pablo Vazquez 2023-10-24 15:50:35 +02:00

Raimund Klink requested review from Campbell Barton 2023-10-24 15:50:35 +02:00

Raimund Klink requested review from Howard Trickey 2023-10-24 15:50:36 +02:00

Raimund Klink commented

2023-10-24 15:52:29 +02:00

Tagging Pablo for UI and Campbell and Howard as modeling module.

TheRedWaxPolice commented

2023-10-25 01:32:42 +02:00

First-time contributor

Coming from other software, I'd say 2 for render is fine...
In fact, most software use 2 for viewport as well...
So, if anything, I'd change the viewport default to 2 to match the render, like most software...

Coming from other software, I'd say 2 for render is fine... In fact, most software use 2 for viewport as well... So, if anything, I'd change the viewport default to 2 to match the render, like most software...

Raimund Klink commented

2023-10-25 08:38:19 +02:00

I'm okay with that, too.
I just want to prevent a rude awakening.

I'm okay with that, too. I just want to prevent a rude awakening.

Raimund Klink added 1 commit 2023-10-25 08:42:39 +02:00

dfc0a0825c Update source/blender/makesdna/DNA_modifier_defaults.h

Raimund Klink changed title from ~~Change the default Subsurf Render Level from 2 to 1~~ to Change the default Subsurf Viewport Level from 1 to 2

2023-10-25 08:43:11 +02:00

Daniel Gryningstjerna commented

2023-10-25 20:26:08 +02:00

First-time contributor

Please don't change the default viewport to 2. Blender can just barely keep up with 1 as the default on more complex models right now, and increasing it with 4 times more tris is going to cause a lot more crashes and slowdowns when adding modifers. I know many other industry-standard software defaults to 2, but they also have subdivision that is way faster then Blenders. I think it's better to change the render to 1 by default, like it was originally.

👍 1

Raimund Klink referenced this pull request

2023-10-26 06:29:56 +02:00

Change the default Subsurf Render Level from 2 to 1 #114163

~~Raimund Klink referenced this pull request 2023-10-31 15:05:19 +01:00~~

Change the default Subsurf Render Level from 2 to 1 #114163

Raimund Klink referenced this pull request

2023-10-31 15:19:00 +01:00

Change the default Subsurf Render Level from 2 to 1 #114163

Campbell Barton refused to review 2023-10-31 23:39:18 +01:00

Campbell Barton commented

2023-10-31 23:44:50 +01:00

Note, while we don't have strict rules for these things, changing defaults is often not especially helpful:

See: https://wiki.blender.org/wiki/Developer_Intro/Advice#Pick_a_Project

Mainly because changing defaults is more of a design decision than something developers aren't able to do.
While it's always possible we need to re-evaluate design decisions, forcing others to discuss opinionated design decisions using the PR's distracts from other developments, from the response to !114163 it seems like it may be best leaving this as-is.

Note, while we don't have strict rules for these things, changing defaults is often not especially helpful: See: https://wiki.blender.org/wiki/Developer_Intro/Advice#Pick_a_Project Mainly because changing defaults is more of a design decision than something developers aren't able to do. While it's always possible we need to re-evaluate design decisions, forcing others to discuss opinionated design decisions using the PR's distracts from other developments, from the response to !114163 it seems like it may be best leaving this as-is.

Raimund Klink commented

2023-11-03 13:44:18 +01:00

Too bad, I would have liked to have seen the levels equated.
But I understand if this is not the right place.

Too bad, I would have liked to have seen the levels equated. But I understand if this is not the right place.

Raimund Klink closed this pull request

2023-11-03 13:44:20 +01:00

Pull request closed

Please reopen this pull request to perform a merge.

Sign in to join this conversation.

No reviewers

No Label

Download

What's New

Blender Studio

Manual

Developers Blog

Documentation

Benchmark

Blender Conference

Development Fund

One-time Donations

Change the default Subsurf Viewport Level from 1 to 2 #114108

Pull request closed