Page MenuHome
Create Task
Maniphest T82540

UV Smart Project works wrong
Closed, ResolvedPublic
Actions

Description

System Information
Operating system: Windows-10-10.0.19041-SP0 64 Bits
Graphics card: GeForce RTX 2080 Ti/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 456.71

Blender Version
Broken: version: 2.91.0 Beta, branch: master, commit date: 2020-11-08 13:39, hash: rB06dac0a45320
Worked: (newest version of Blender that worked as expected)

Short description of error
[Please fill out a short description of the error here]

Smart UV unwrap does not react on UV seams

Exact steps for others to reproduce the error
[Please describe the exact steps needed to reproduce the issue]
[Based on the default startup or an attached .blend file (as simple as possible)]

Use Smart UV unwrap with seams on some co-planar edges.

Revisions and Commits

rB Blender
rBe01bf7a92eca Fix T82540: Smart UV project ignores seams
rB05a2382c0840 Fix T82540: Smart UV project ignores seams

Related Objects

Event Timeline

Tomasz Muszynski (muchasty) created this task.Nov 9 2020, 3:31 PM
Philipp Oeser (lichtwerk) changed the task status from Needs Triage to Needs Information from User.Nov 9 2020, 3:55 PM
Philipp Oeser (lichtwerk) added a subscriber: Philipp Oeser (lichtwerk).

Mind sharing an example file where this happens?

Tomasz Muszynski (muchasty) added a comment.EditedNov 9 2020, 5:36 PM

Look at the attached image.
Those "pinkish" lines are seams. And they are completely ignored by smart_project .
Recently Devs have changed something in the smart_project operator cause it gives error when using parameters that are included in current offcial 2.90.1 API and no info in the Changelog that something has been developed recently ( with 2.91 version I have to use Radians instead Degrees ).
You can download my add-on ( sorry for the quality of the code inside it ) and see that I had to use other parameters (watched from the operator call in Bblender ) than are mentioned in official API.

https://github.com/muchasty/UV-Tools/blob/master/LightmapAutoUV.py

Tomasz Muszynski (muchasty) added a comment.Nov 9 2020, 5:38 PM

Here is my project file:
https://www.dropbox.com/s/2m7shajh2gy32ns/USA-Lazienka.rar?dl=0

Philipp Oeser (lichtwerk) closed this task as Archived.Nov 9 2020, 6:17 PM
Philipp Oeser (lichtwerk) claimed this task.

Maybe I am misunderstanding, but Smart UV is not supposed to take Seams into account?
Instead, islands are defined by by Angle Limit?
https://docs.blender.org/manual/en/latest/modeling/meshes/editing/uv.html

So, if this was never working [which I think it wasnt], then this sounds more like a feature request than a bug.
Will have to close (but feel free to comment again if this was working at some point - or if this is a misunderstanding on my side)

Regarding the change between 2.90.1 and 2.91: smart UV was ported to C, see rB850234c1b10a: UV: port smart project from Python to C
Since this is a different subject - having to use radians instead of degrees - than the original report though, I would kindly ask to report this separately [agree, this should be mentioned in the release logs]

Tomasz Muszynski (muchasty) added a comment.Nov 9 2020, 10:40 PM

I am disappointed with that kind of bug filtering !

Please use 2.90.1 and you will see how the smart_project() should work !
Look at the attached image and find the difference ( this is from 2.90.1 with nicely cuts made by seams )
according to the previous image from 2.91Beta !

Philipp Oeser (lichtwerk) reopened this task as Needs Triage.Nov 9 2020, 11:10 PM
Campbell Barton (campbellbarton) changed the task status from Needs Triage to Confirmed.Nov 10 2020, 8:05 AM
Campbell Barton (campbellbarton) closed this task as Resolved by committing rB05a2382c0840: Fix T82540: Smart UV project ignores seams.Nov 10 2020, 9:08 AM
Campbell Barton (campbellbarton) added a commit: rB05a2382c0840: Fix T82540: Smart UV project ignores seams.
Philipp Oeser (lichtwerk) added a subscriber: Campbell Barton (campbellbarton).Nov 10 2020, 9:10 AM

Sorry @Tomasz Muszynski (muchasty) for getting that wrong and thx @Campbell Barton (campbellbarton) for the fix!

Tomasz Muszynski (muchasty) added a comment.Nov 10 2020, 9:36 AM

No problem. Let's go forward and do not look back. :)
I hope that it will be fixed soon cause it is crucial for me for making with my add-on well working lightmaps for UE4.

Thank you.

Philipp Oeser (lichtwerk) added a comment.Nov 10 2020, 9:44 AM
In T82540#1051377, @Tomasz Muszynski (muchasty) wrote:

I hope that it will be fixed soon cause it is crucial for me for making with my add-on well working lightmaps for UE4.

Well, fix is in, should be in tomorrows build from https://builder.blender.org/download/ (and will also be in 2.91 release)

Tomasz Muszynski (muchasty) added a comment.Nov 10 2020, 10:33 AM

THANK YOU !

Tomasz Muszynski (muchasty) added a comment.Nov 19 2020, 2:54 PM

What is going on?
Problem came back in release blender-2.91.0-e580c188533c-windows64

Philipp Oeser (lichtwerk) reopened this task as Confirmed.Nov 19 2020, 3:20 PM
Philipp Oeser (lichtwerk) added a project: Restricted Project.

@Campbell Barton (campbellbarton), looks like rB9296ba867462: Fix T82637: pack UV islands fails with some non-manifold meshes brought this back?

Campbell Barton (campbellbarton) closed this task as Resolved by committing rBe01bf7a92eca: Fix T82540: Smart UV project ignores seams.Nov 19 2020, 3:39 PM
Campbell Barton (campbellbarton) added a commit: rBe01bf7a92eca: Fix T82540: Smart UV project ignores seams.