Recent change flips profile curve extruded along spline upside down #90432

New Issue

Miro Horváth · 2021-08-04T15:19:14+02:00

Miro Horváth commented

2021-08-04 15:19:14 +02:00

System Information
Operating system: Windows-10-10.0.19041-SP0 64 Bits
Graphics card: GeForce GTX 970/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 461.09

Blender Version
Broken: version: 3.0.0 Alpha, branch: master, commit date: 2021-08-03 20:35, hash: fb87d236ed
Worked: patch D11522 from July 21st https://builder.blender.org/download/patch/archive/blender-3.0.0-alpha+master-D11522.871f6c4bd0c6-windows.amd64-release.zip
Behavior changed in 6ac378e685: Fix: avoid creating improper rotation matrix

Exact steps for others to reproduce the error

Attached blend file uses correct orientation of profile curve in TOP view(previously working), but shows incorrect result.
bug.blend

Also, the same(incorrect) result appears when curve is done via Attribute Curve Map node

**System Information** Operating system: Windows-10-10.0.19041-SP0 64 Bits Graphics card: GeForce GTX 970/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 461.09 **Blender Version** Broken: version: 3.0.0 Alpha, branch: master, commit date: 2021-08-03 20:35, hash: `fb87d236ed` Worked: patch [D11522](https://archive.blender.org/developer/D11522) from July 21st https://builder.blender.org/download/patch/archive/blender-3.0.0-alpha+master-[D11522](https://archive.blender.org/developer/D11522).871f6c4bd0c6-windows.amd64-release.zip Behavior changed in 6ac378e685: Fix: avoid creating improper rotation matrix **Exact steps for others to reproduce the error** ![bug.jpg](https://archive.blender.org/developer/F10265278/bug.jpg) Attached blend file uses correct orientation of profile curve in TOP view(previously working), but shows incorrect result. [bug.blend](https://archive.blender.org/developer/F10265285/bug.blend) Also, the same(incorrect) result appears when curve is done via Attribute Curve Map node ![image.png](https://archive.blender.org/developer/F10265293/image.png)

Miro Horváth commented

2021-08-04 15:19:14 +02:00

Added subscriber: @MiroHorvath

blender-admin commented

2021-08-04 15:19:14 +02:00

#92220 was marked as duplicate of this issue

Philipp Oeser commented

2021-08-04 15:45:20 +02:00

Added subscriber: @lichtwerk

Philipp Oeser commented

2021-08-04 15:45:20 +02:00

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

Philipp Oeser self-assigned this 2021-08-04 15:45:20 +02:00

Philipp Oeser commented

2021-08-04 15:45:20 +02:00

Can confirm, will check

Philipp Oeser removed their assignment 2021-08-04 16:48:58 +02:00

Philipp Oeser commented

2021-08-04 16:48:58 +02:00

Added subscriber: @HooglyBoogly

Philipp Oeser commented

2021-08-04 16:48:58 +02:00

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

Philipp Oeser commented

2021-08-04 16:48:58 +02:00

Behavior changed in 6ac378e685

Not sure how the Attribute Curve Map node is used for this (maybe you can share that file as well)?
For the profile, I would also expect it the other way around (also since doing the same with the profile as the curve bevel object with "old-school" curves would also do it the other way around).
Soo, not sure at which point we'll have to make adjustments (if at all):

spline.evaluated_normals ?
the old curve code?
using inverted normals?
none of the above?

Will let @HooglyBoogly decide.

Behavior changed in 6ac378e685 Not sure how the Attribute Curve Map node is used for this (maybe you can share that file as well)? For the profile, I would also expect it the other way around (also since doing the same with the profile as the curve bevel object with "old-school" curves would also do it the other way around). Soo, not sure at which point we'll have to make adjustments (if at all): - spline.evaluated_normals ? - the old curve code? - using inverted normals? - none of the above? Will let @HooglyBoogly decide.

Miro Horváth commented

2021-08-04 17:18:35 +02:00

@lichtwerk Here's the example with Attribute Curve Map node
bug1.blend

@lichtwerk Here's the example with Attribute Curve Map node [bug1.blend](https://archive.blender.org/developer/F10265535/bug1.blend)

Hans Goudey commented

2021-10-15 16:01:00 +02:00

Added subscriber: @MikhailRachinskiy