UV Smart Project works wrong #82540

New Issue

Tomasz Muszynski · 2020-11-09T15:31:54+01:00

Tomasz Muszynski commented

2020-11-09 15:31:54 +01:00

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: 06dac0a453
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.

**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: `06dac0a453` 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. ![SmartUV-ProblemAGAIN.JPG](https://archive.blender.org/developer/F9245605/SmartUV-ProblemAGAIN.JPG)

Tomasz Muszynski commented

2020-11-09 15:31:54 +01:00

Added subscriber: @TomaszMuszynski

Philipp Oeser commented

2020-11-09 15:55:28 +01:00

Added subscriber: @lichtwerk

Philipp Oeser commented

2020-11-09 15:55:28 +01:00

Changed status from 'Needs Triage' to: 'Needs User Info'

Philipp Oeser commented

2020-11-09 15:55:28 +01:00

Mind sharing an example file where this happens?

Tomasz Muszynski commented

2020-11-09 17:36:00 +01:00

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

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 commented

2020-11-09 17:38:14 +01:00

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

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

Philipp Oeser commented

2020-11-09 18:17:54 +01:00

Changed status from 'Needs User Info' to: 'Archived'

Philipp Oeser closed this issue

2020-11-09 18:17:54 +01:00

Philipp Oeser self-assigned this 2020-11-09 18:17:54 +01:00

Philipp Oeser commented

2020-11-09 18:17:54 +01:00

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 850234c1b1
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]

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 850234c1b1 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 commented

2020-11-09 22:40:58 +01:00

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 !

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 ! ![Przechwytywanie.JPG](https://archive.blender.org/developer/F9248042/Przechwytywanie.JPG)

Philipp Oeser commented

2020-11-09 23:10:41 +01:00

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

Philipp Oeser reopened this issue

2020-11-09 23:10:41 +01:00

Campbell Barton commented

2020-11-10 08:05:25 +01:00

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

blender-admin commented

2020-11-10 09:05:07 +01:00

This issue was referenced by 05a2382c08

This issue was referenced by 05a2382c08402b250d6b5c4105b449f4474f6cf2

Campbell Barton commented

2020-11-10 09:08:37 +01:00

Changed status from 'Confirmed' to: 'Resolved'

Campbell Barton closed this issue

2020-11-10 09:08:37 +01:00

Philipp Oeser commented

2020-11-10 09:10:25 +01:00

Added subscriber: @ideasman42

Philipp Oeser commented

2020-11-10 09:10:25 +01:00

Sorry @TomaszMuszynski for getting that wrong and thx @ideasman42 for the fix!

Tomasz Muszynski commented

2020-11-10 09:36:36 +01:00

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.

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 commented

2020-11-10 09:44:52 +01:00

In #82540#1051377, @TomaszMuszynski 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)

> In #82540#1051377, @TomaszMuszynski 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 commented

2020-11-10 10:33:22 +01:00

THANK YOU !