Smart UV Project doing very bad UV projection in recent builds #39601

Closed
opened 9 years ago by sanctuary · 20 comments

System Information
Operating system and graphics card
winxp sp3

Blender Version
Broken: blender-2.70-6026295-win32
Worked: blender-2.70-a7242c3-win32

Short description of error
Smart UV Project does not project correctly

Exact steps for others to reproduce the error

  • File -> Load Factory Settings
  • Split screen in two, make one of the windows the UV/Image Editor
  • Select default cube and go to Edit Mode
  • Select All, press U -> Smart UV Project
  • Notice in the UV/Image editor that the faces are now distorted squares, not perfect square as they should be (and were in 2.70 stable or early post-2.70 builds)
**System Information** Operating system and graphics card winxp sp3 **Blender Version** Broken: blender-2.70-6026295-win32 Worked: blender-2.70-a7242c3-win32 **Short description of error** Smart UV Project does not project correctly **Exact steps for others to reproduce the error** - File -> Load Factory Settings - Split screen in two, make one of the windows the UV/Image Editor - Select default cube and go to Edit Mode - Select All, press U -> Smart UV Project - Notice in the UV/Image editor that the faces are now distorted squares, not perfect square as they should be (and were in 2.70 stable or early post-2.70 builds)
Poster

Changed status to: 'Open'

Changed status to: 'Open'
Poster

Added subscriber: @sanctuary

Added subscriber: @sanctuary
Poster

forgot to add screenshots :

bad elongated projected uv in recent blender-2.70-6026295-win32 :
http://i.imgur.com/Cex4ygD.jpg

correct proportioned faces in old blender-2.70-a7242c3-win32 :
http://i.imgur.com/Kkg0Atr.jpg

forgot to add screenshots : bad elongated projected uv in recent blender-2.70-6026295-win32 : http://i.imgur.com/Cex4ygD.jpg correct proportioned faces in old blender-2.70-a7242c3-win32 : http://i.imgur.com/Kkg0Atr.jpg

Added subscriber: @Ace_Dragon

Added subscriber: @Ace_Dragon

It looks like the code was changed to try to fill in the entire image space rather than just a part of it.

This type of thing really should be an option rather than how it works now, I say keeping it as an option because I could see the utility of it for texture painting.

It looks like the code was changed to try to fill in the entire image space rather than just a part of it. This type of thing really should be an option rather than how it works now, I say keeping it as an option because I could see the utility of it for texture painting.
Poster

Yes, if that change is intentional making it as an option in F6 is really needed, as while it would be indeed ok for texture painting, for assigning an existing texture it would only lead into nasty image distorsion on the object surfaces.

Yes, if that change is intentional making it as an option in F6 is really needed, as while it would be indeed ok for texture painting, for assigning an existing texture it would only lead into nasty image distorsion on the object surfaces.
Owner

Added subscriber: @ideasman42

Added subscriber: @ideasman42
Owner

In fact this looks like a float precision issue, if you chance the margin the result flickers back and fourth.

In fact this looks like a float precision issue, if you chance the margin the result flickers back and fourth.
Owner

Added subscriber: @ThomasDinges

Added subscriber: @ThomasDinges
Sergey commented 9 years ago
Owner

Added subscriber: @Sergey

Added subscriber: @Sergey
Sergey commented 9 years ago
Owner

@ideasman42, who maintains smart uv?

@ideasman42, who maintains smart uv?

Added subscriber: @Harvester

Added subscriber: @Harvester

Here on Win7 Pro 64bit, Blender 2.70a 64bit (Hash: f93bc76) there is the same issue and it seems to affect also the "Lightmap Pack" UV projection method.

Here on Win7 Pro 64bit, Blender 2.70a 64bit (Hash: f93bc76) there is the same issue and it seems to affect also the "Lightmap Pack" UV projection method.
Poster

very bad, that nasty bug is now in official 2.70a too , the smart uv project is then not trustable anymore for unwrapping and texturing.

a very bad regression in comparison to official 2.70 in which it didn't happened.

very bad, that nasty bug is now in official 2.70a too , the smart uv project is then not trustable anymore for unwrapping and texturing. a very bad regression in comparison to official 2.70 in which it didn't happened.

I don't get it, it's an important tool for many and I can't see the fix being that hard to do.

If it became widely known that some bugs were treated like this than it will attempt to seal the idea that FOSS = We couldn't care less about what the user needs.

Is it such a formidable task to fix important tools just because you might not maintain it, should we rather not be giving the impression that Blender is not made with the user in mind (and torpedo any chance for FOSS to have a future in 3D)?

This is basically a case of, "let it be, it's not my problem", a commercial company doing may eventually get sued out of millions if they knowingly did that. Is the Blender way supposed to be professional or not?

I don't get it, it's an important tool for many and I can't see the fix being that hard to do. If it became widely known that some bugs were treated like this than it will attempt to seal the idea that *FOSS = We couldn't care less about what the user needs*. Is it such a formidable task to fix important tools just because you might not maintain it, should we rather not be giving the impression that Blender is not made with the user in mind (and torpedo any chance for FOSS to have a future in 3D)? This is basically a case of, "let it be, it's not my problem", a commercial company doing may eventually get sued out of millions if they knowingly did that. Is the Blender way supposed to be professional or not?
Owner

Ace Dragon, your comment brings 0 value into this discussion here. This is the bugtracker and not Blenderartists!

The report is open for only 10 days, and as you might have noticed, we have 239 reports in total now. We cannot tackle everything at once.

Ace Dragon, your comment brings 0 value into this discussion here. This is the bugtracker and not Blenderartists! The report is open for only 10 days, and as you might have noticed, we have 239 reports in total now. We cannot tackle everything at once.
Poster

On a Blenderartist thread thanks to some users helping, some odd behaviour of that bug has been noticed, some people actually observed only the correct behaviour in 2.70a

But as soon as they use Load Factory Settings in 2.70a, they will obtain only the buggy behaviour.

If i use the startup and user prefs blend of an user that do not observe the buggy behaviour , i don't see it either (until both of us use Load Factory Settings) , i loaded a 2.69 blend into 2.70a with "load ui" enabled and the bug disappeared too (to re-appear if i did a Load Factory Setting).

It's very strange, but seems to hint there's something in the default factory setting of 2.70a that is different from the default factory setting of previous versions that is contributing to that bug as the users that at first saw the correct behaviour of the smart unwrap have imported their old versions settings into 2.70a (and probably why after load factory settings they observe the bug).

If what i write is not understandable, you can check some blender users helping to test around this bug here :
http://blenderartists.org/forum/showthread.php?318765-Blender-2-7x-development-thread/page38

On a Blenderartist thread thanks to some users helping, some odd behaviour of that bug has been noticed, some people actually observed only the correct behaviour in 2.70a But as soon as they use Load Factory Settings in 2.70a, they will obtain only the buggy behaviour. If i use the startup and user prefs blend of an user that do not observe the buggy behaviour , i don't see it either (until both of us use Load Factory Settings) , i loaded a 2.69 blend into 2.70a with "load ui" enabled and the bug disappeared too (to re-appear if i did a Load Factory Setting). It's very strange, but seems to hint there's something in the default factory setting of 2.70a that is different from the default factory setting of previous versions that is contributing to that bug as the users that at first saw the correct behaviour of the smart unwrap have imported their old versions settings into 2.70a (and probably why after load factory settings they observe the bug). If what i write is not understandable, you can check some blender users helping to test around this bug here : http://blenderartists.org/forum/showthread.php?318765-Blender-2-7x-development-thread/page38
Poster

after testing more, it looks like when indeed you use old blender version the bug disappear, but in appearance only, because it is in fact not buggy at Margin = 0
While it is immediately buggy at Margin = 0 with default factory settings.

But as soon as you increase slowly the margin, even on the situation in which it was appearing not buggy at Margin = 0 we'll still observe the elongated faces appearing after a small margin increase.
Probably confirming what campellbarton said :
https://developer.blender.org/T39601#9

still strange that depending on the default factory or older user settings, the bug is visible or not at Margin = 0

after testing more, it looks like when indeed you use old blender version the bug disappear, but in appearance only, because it is in fact not buggy at Margin = 0 While it is immediately buggy at Margin = 0 with default factory settings. But as soon as you increase slowly the margin, even on the situation in which it was appearing not buggy at Margin = 0 we'll still observe the elongated faces appearing after a small margin increase. Probably confirming what campellbarton said : https://developer.blender.org/T39601#9 still strange that depending on the default factory or older user settings, the bug is visible or not at Margin = 0
Owner

Changed status from 'Open' to: 'Resolved'

Changed status from 'Open' to: 'Resolved'
ideasman42 closed this issue 9 years ago
ideasman42 self-assigned this 9 years ago
Owner

The problem is when packing boxes exactly the same size, decisions about where to place the box were comparing values which were so close to being the same, that it came down to floating point precision giving erratic/unpredictable results.

That it even gave the results it did before, was mostly accidental, it would flicker when changing margin size.

Committed some changes to box packing, so its more deterministic,

6c57ee53bf
72e9052713
d5647a37b4
e6c1a23341
69de54f99a
ae71729c8b

The problem is when packing boxes *exactly* the same size, decisions about where to place the box were comparing values which were so close to being the same, that it came down to floating point precision giving erratic/unpredictable results. That it even gave the results it did before, was mostly accidental, it would flicker when changing margin size. Committed some changes to box packing, so its more deterministic, 6c57ee53bf 72e9052713 d5647a37b4 e6c1a23341 69de54f99a ae71729c8b
Sign in to join this conversation.
No Label
Interest/Alembic
Interest/Animation & Rigging
Interest/Asset Browser
Interest/Asset Browser Project Overview
Interest/Audio
Interest/Automated Testing
Interest/Blender Asset Bundle
Interest/Collada
Interest/Compositing
Interest/Core
Interest/Cycles
Interest/Dependency Graph
Interest/Development Management
Interest/Eevee & Viewport
Interest/Freestyle
Interest/Geometry Nodes
Interest/Grease Pencil
Interest/ID Management
Interest/Images & Movies
Interest/Import/Export
Interest/Line Art
Interest/Masking
Interest/Modeling
Interest/Modifiers
Interest/Motion Tracking
Interest/Nodes & Physics
Interest/Overrides
Interest/Performance
Interest/Performance
Interest/Physics
Interest/Pipeline, Assets & I/O
Interest/Platforms, Builds, Tests & Devices
Interest/Python API
Interest/Render & Cycles
Interest/Render Pipeline
Interest/Sculpt, Paint & Texture
Interest/Text Editor
Interest/Translations
Interest/Triaging
Interest/Undo
Interest/USD
Interest/User Interface
Interest/UV Editing
Interest/VFX & Video
Interest/Video Sequencer
Interest/Virtual Reality
legacy module/Animation & Rigging
legacy module/Core
legacy module/Development Management
legacy module/Eevee & Viewport
legacy module/Grease Pencil
legacy module/Modeling
legacy module/Nodes & Physics
legacy module/Pipeline, Assets & IO
legacy module/Platforms, Builds, Tests & Devices
legacy module/Python API
legacy module/Rendering & Cycles
legacy module/Sculpt, Paint & Texture
legacy module/Triaging
legacy module/User Interface
legacy module/VFX & Video
legacy project/1.0.0-beta.2
legacy project/Asset Browser (Archived)
legacy project/BF Blender: 2.8
legacy project/BF Blender: After Release
legacy project/BF Blender: Next
legacy project/BF Blender: Regressions
legacy project/BF Blender: Unconfirmed
legacy project/Blender 2.70
legacy project/Code Quest
legacy project/Datablocks and Libraries
legacy project/Eevee
legacy project/Game Animation
legacy project/Game Audio
legacy project/Game Data Conversion
legacy project/Game Engine
legacy project/Game Logic
legacy project/Game Physics
legacy project/Game Python
legacy project/Game Rendering
legacy project/Game UI
legacy project/GPU / Viewport
legacy project/GSoC
legacy project/Infrastructure: Websites
legacy project/LibOverrides - Usability and UX
legacy project/Milestone 1: Basic, Local Asset Browser
legacy project/Nodes
legacy project/OpenGL Error
legacy project/Papercut
legacy project/Pose Library Basics
legacy project/Retrospective
legacy project/Tracker Curfew
legacy project/Wintab High Frequency
Meta/Good First Issue
Meta/Papercut
migration/requires-manual-verification
Module › Animation & Rigging
Module › Core
Module › Development Management
Module › Eevee & Viewport
Module › Grease Pencil
Module › Modeling
Module › Nodes & Physics
Module › Pipeline, Assets & IO
Module › Platforms, Builds, Tests & Devices
Module › Python API
Module › Render & Cycles
Module › Sculpt, Paint & Texture
Module › Triaging
Module › User Interface
Module › VFX & Video
Platform/FreeBSD
Platform/Linux
Platform/macOS
Platform/Windows
Priority › High
Priority › Low
Priority › Normal
Priority › Unbreak Now!
Status › Archived
Status › Confirmed
Status › Duplicate
Status › Needs Information from Developers
Status › Needs Information from User
Status › Needs Triage
Status › Resolved
Type › Bug
Type › Design
Type › Known Issue
Type › Patch
Type › Report
Type › To Do
No Milestone
No project
No Assignees
6 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: blender/blender#39601
Loading…
There is no content yet.