Vertex normals, VTXNORMS chunk #69
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Since smoothing groups are not exported and the normal data is only exported with the glTF option most users never have the custom normals imported as they use ActorX.
Another issue is that psa animations don't really work with glTF armatures as-is.
I have a fork of UEViewer (floxay/UEViewer@cc0542d) that saves the normals into a VTXNORMS chunk and I'm opening this PR mainly because this 'unofficial' support for normals should not break importing old psk/pskx (probably should force it to pskx only) files but could be useful when encountering this 'new' chunk.
Edit: I only added it to the 280 script as I'm not sure if the 279 one would work the same way or not.added to 270 too
Edit 2: Vertex normals and ActorX [gildor.org]
If anyone reading this is familiar enough with MAXScript to implement this correctly into the 3ds Max importer (link above) that'd be appreciated, I tried but failed :D