Regression: Performance regression after support modifiers on curve objects commit #91353

New Issue

Ulysse Martin · 2021-09-12T16:18:00+02:00

Ulysse Martin commented

2021-09-12 16:18:00 +02:00

System Information
Operating system: Windows-10-10.0.22000-SP0 64 Bits
Graphics card: NVIDIA GeForce RTX 3070/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 471.96

Blender Version
Caused by b9febb54a4

Short description of error
Performance regression playing timeline this this file (Modified Mr Elephant):

mr_elephantAnimed.blend

Exact steps for others to reproduce the error
Open the file and play the timeline in eevee rendered mode.

First problematic commit: b9febb54a4

Performances before:
80 fps on my system (idk why the framerate is not limited to 60 btw)

Performances after:
41 fps on my system

**System Information** Operating system: Windows-10-10.0.22000-SP0 64 Bits Graphics card: NVIDIA GeForce RTX 3070/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 471.96 **Blender Version** Caused by b9febb54a4 **Short description of error** Performance regression playing timeline this this file (Modified Mr Elephant): [mr_elephantAnimed.blend](https://archive.blender.org/developer/F10402335/mr_elephantAnimed.blend) **Exact steps for others to reproduce the error** Open the file and play the timeline in eevee rendered mode. First problematic commit: b9febb54a492ac6c93802fb0aa189d4c3fd99b0b Performances before: 80 fps on my system (idk why the framerate is not limited to 60 btw) Performances after: 41 fps on my system

Ulysse Martin commented

2021-09-12 16:18:00 +02:00

Added subscriber: @you.le

Hans Goudey commented

2021-09-13 05:18:38 +02:00

Added subscriber: @HooglyBoogly

Hans Goudey commented

2021-09-13 05:18:38 +02:00

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

Hans Goudey commented

2021-09-13 05:18:38 +02:00

Thanks for the report. There will be some performance regression expected for curve objects in some cases.

I plan to look into further refactoring in the future to avoid an intermediate DispList data type before Mesh when evaluating curve modifiers.
I'd like to profile this file to make sure the regression is coming from expected areas though.

I won't fully triage this yet, since I want to look at it in more depth.

Thanks for the report. There will be some performance regression expected for curve objects in some cases. I plan to look into further refactoring in the future to avoid an intermediate `DispList` data type before `Mesh` when evaluating curve modifiers. I'd like to profile this file to make sure the regression is coming from expected areas though. I won't fully triage this yet, since I want to look at it in more depth.

Philipp Oeser commented

2022-03-29 12:46:10 +02:00

Added subscriber: @lichtwerk

Philipp Oeser commented

2022-03-29 12:46:10 +02:00

This still seems relevant even in 3.2.
@HooglyBoogly : will set to High prio since it is a regression, but feel free to lower prio again.

This still seems relevant even in 3.2. @HooglyBoogly : will set to High prio since it is a regression, but feel free to lower prio again.

Philipp Oeser changed title from ~~Performance regression after recent commit~~ to Regression: Performance regression after support modifiers on curve objects commit

2022-03-29 12:47:32 +02:00

Hans Goudey commented

2022-03-29 20:40:35 +02:00

Changing this to a known issue, since current development is around replacing the curve object type (which will solve this problem).

Jacob Merrill commented

2022-03-30 05:57:41 +02:00

Added subscriber: @JacobMerrill-1

Philipp Oeser removed the

Download

What's New

Blender Studio

Manual

Developers Blog

Documentation

Benchmark

Blender Conference

Development Fund

One-time Donations

Regression: Performance regression after support modifiers on curve objects commit #91353