Page MenuHome
Differential D2968

Proposed fix for bug T53572
ClosedPublic
Actions

Authored by Martin Felke (scorpion81) on Dec 17 2017, 3:40 PM.

Details

Reviewers
Sybren A. Stüvel (sybren)
Kévin Dietrich (kevindietrich)
Commits
rBSb1a036861ddd: Fix T53572: Alembic imports UV maps incorrectly
rB0f841e24b0b3: Fix T53572: Alembic imports UV maps incorrectly
rBb1a036861ddd: Fix T53572: Alembic imports UV maps incorrectly
Summary

This is a possible fix for Bug T53572: Alembic - Broken UVs after export and test re-import.
Since in Alembic the loop order seems to be reversed when exporting and importing, and this was the only place where it was not, I was thinking to match this to the convention of reversing the loop order as well.

Diff Detail

Repository
rB Blender
Branch
arcpatch-D2968
Build Status
Buildable 1043
Build 1043: arc lint + arc unit

Event Timeline

Martin Felke (scorpion81) created this revision.Dec 17 2017, 3:40 PM
Martin Felke (scorpion81) retitled this revision from Proposed fix for bug {T53572} to Proposed fix for bug T53572.
Martin Felke (scorpion81) edited the summary of this revision. (Show Details)Dec 17 2017, 3:52 PM
Martin Felke (scorpion81) added reviewers: Sybren A. Stüvel (sybren), Kévin Dietrich (kevindietrich).Dec 17 2017, 5:19 PM
Sybren A. Stüvel (sybren) updated this revision to Diff 9720.Dec 20 2017, 10:41 AM
  • calculate the reverse loop offset only once per poly
Harbormaster completed remote builds in B1043: Diff 9720.Dec 20 2017, 10:41 AM
Sybren A. Stüvel (sybren) added a comment.Dec 20 2017, 10:42 AM

Thanks for tracking down this issue and solving it, it indeed fixes T53572!

Sybren A. Stüvel (sybren) accepted this revision.Dec 20 2017, 10:43 AM
This revision is now accepted and ready to land.Dec 20 2017, 10:43 AM
Closed by commit rBb1a036861ddd: Fix T53572: Alembic imports UV maps incorrectly (authored by Martin Felke (scorpion81), committed by Sybren A. Stüvel (sybren)). · Explain WhyDec 20 2017, 10:55 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
source/
blender/
alembic/
intern/
6 lines

Diff 9720

View Options

source/blender/alembic/intern/abc_customdata.cc

Loading...