Default UVMap from planes is 90° CW rotated. #37649

New Issue

Heiko Schmidt · 2013-11-28T09:17:23+01:00

Heiko Schmidt commented

2013-11-28 09:17:23 +01:00

System Information
Win 7 64 bit

Blender Version
Broken: Blender 2.69 r ??? (Build Date 28.11.2013 1:11)
Worked: Blender 2.29 release

When importing an image as plane the aspect ratio is wrong looks like the width and high values are muddled up somehow.

**System Information** Win 7 64 bit **Blender Version** Broken: Blender 2.69 r ??? (Build Date 28.11.2013 1:11) Worked: Blender 2.29 release When importing an image as plane the aspect ratio is wrong looks like the width and high values are muddled up somehow.

Heiko Schmidt commented

2013-11-28 09:17:23 +01:00

Changed status to: 'Open'

Heiko Schmidt commented

2013-11-28 09:17:23 +01:00

Added subscriber: @heik

Philipp Oeser commented

2013-11-28 10:49:39 +01:00

Added subscriber: @lichtwerk

Philipp Oeser commented

2013-11-28 10:49:39 +01:00

can confirm this on a simple test, will have a quick look at the code...

Philipp Oeser commented

2013-11-28 13:53:54 +01:00

this is propably caused by cc9f663c54eca82b703979b255ea5e285ad96b45 when the manual quad creation was swapped with bpy.ops.mesh.primitive_plane_add()
Attached a patch that fixes this (it just swappes x & y dimensions).
note though, that I am unsure if we should rather rotate the UVs instead? will have to check behaviour before cc9f663c54eca82b703979b255ea5e285ad96b45...
I was also having difficulties generating a proper diff with git (it seems to not catch changes in submodules here, needs more investigstion on my side...) so this patch is done with meld manually.

this is propably caused by cc9f663c54eca82b703979b255ea5e285ad96b45 when the manual quad creation was swapped with bpy.ops.mesh.primitive_plane_add() Attached a patch that fixes this (it just swappes x & y dimensions). note though, that I am unsure if we should rather rotate the UVs instead? will have to check behaviour before cc9f663c54eca82b703979b255ea5e285ad96b45... I was also having difficulties generating a proper diff with git (it seems to not catch changes in submodules here, needs more investigstion on my side...) so this patch is done with meld manually.

Bastien Montagne was assigned by Brecht Van Lommel

2013-11-28 15:39:59 +01:00

Bastien Montagne commented

2013-11-28 16:28:30 +01:00

No, blender/blender-addons@cc9f663c is old, it worked well after it. Checking right now where it comes from, suspect recent grid primitive optimization changed the order in which the vertices are added to the polygon…

But for sure, switching the dimensions is not what we want, rather rotating the UVs ;)

No, blender/blender-addons@cc9f663c is old, it worked well after it. Checking right now where it comes from, suspect recent grid primitive optimization changed the order in which the vertices are added to the polygon… But for sure, switching the dimensions is not what we want, rather rotating the UVs ;)

Bastien Montagne changed title from ~~Import Images as Planes: aspect ratio wrong~~ to Default UVMap from planes is 90° CW rotated.

2013-11-28 17:07:33 +01:00

Bastien Montagne commented

2013-11-28 17:07:42 +01:00

Yep, can confirm it's another issue from aa3c06b41, not related to the addon (just add a plane, add anew UVMap, assign a texture to it, and you'll get it aligned in wrong direction).

Will see how to fix this.

Yep, can confirm it's another issue from aa3c06b41, not related to the addon (just add a plane, add anew UVMap, assign a texture to it, and you'll get it aligned in wrong direction). Will see how to fix this.

Bastien Montagne removed their assignment 2013-11-28 17:10:10 +01:00

Campbell Barton was assigned by Bastien Montagne

2013-11-28 17:10:10 +01:00

Bastien Montagne commented

2013-11-28 17:10:10 +01:00

Added subscriber: @mont29

Bastien Montagne commented

2013-11-28 17:10:10 +01:00

Actually, would rather avoid messing again things here, so will let Campbell figure out a proper solution (99% sure the issue comes from order of verts, it has changed in this aa3c06b41 commit).

Actually, would rather avoid messing again things here, so will let Campbell figure out a proper solution (99% sure the issue comes from order of verts, it has changed in this aa3c06b41 commit).

Philipp Oeser commented

2013-11-28 17:43:46 +01:00

Sorry Bastien, I spoke too soon... (found out too late that aa3c06b41 was affecting primitive_plane_add() as well...)

Sorry Bastien, I spoke too soon... (found out too late that aa3c06b41 was affecting primitive_plane_add() as well...)

Campbell Barton commented

2013-11-29 00:41:01 +01:00

fixed aa3933b411

fixed aa3933b411

Campbell Barton commented

2013-11-29 00:41:09 +01:00

Changed status from 'Open' to: 'Resolved'

Campbell Barton closed this issue