Page MenuHome

Recent change flips profile curve extruded along spline upside down
Closed, ArchivedPublic

Description

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: rBfb87d236edb7
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 rB6ac378e685f3: 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.

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

Event Timeline

Philipp Oeser (lichtwerk) changed the task status from Needs Triage to Confirmed.Aug 4 2021, 3:45 PM
Philipp Oeser (lichtwerk) claimed this task.

Can confirm, will check

Philipp Oeser (lichtwerk) changed the task status from Confirmed to Needs Information from Developers.Aug 4 2021, 4:48 PM
Philipp Oeser (lichtwerk) removed Philipp Oeser (lichtwerk) as the assignee of this task.

Behavior changed in rB6ac378e685f3: Fix: avoid creating improper rotation matrix

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 @Hans Goudey (HooglyBoogly) decide.

@Philipp Oeser (lichtwerk) Here's the example with Attribute Curve Map node

@Hans Goudey (HooglyBoogly) Can you have a look at this again. To me it feels like this can be closed now, not sure if there is really a good way to "solve" this.