Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Normals aren't correct for SC datastreams with 16 byte structures. #5

Closed
Markemp opened this issue Oct 25, 2017 · 5 comments
Closed
Assignees
Milestone

Comments

@Markemp
Copy link
Owner

Markemp commented Oct 25, 2017

The normal part of the VertsUV struct seem to have been changed to Color, and the Normal is a calculated value from the Tangents datastream. Need to calculate the correct normals from the Tangents datastream by taking the cross product of the tangent and bitangent.

@pixelpath
Copy link

To fix normals not displaying correctly in 3DS Max: Select the model and add an Edit Normal modifier to the stack. Now press CTRL + A to highlight all normals and click the Reset button in the Edit Normal modifier panel. Now the models explicit normals should be working.

@Markemp
Copy link
Owner Author

Markemp commented Sep 12, 2020

Thank you for the workaround!

@pixelpath
Copy link

pixelpath commented Sep 14, 2020

No problem! I think by default, 3DS Max has them rotated in a different default direction and doing the reset forces it to rotate the normals relative to surface / tangent space but I am still trying to figure out the different normal modes. FWIW the 3DS Max SDK docs list some info on them that may shed more light on the situation: http://docs.autodesk.com/3DSMAX/16/ENU/3ds-Max-SDK-Programmer-Guide/files/GUID-88FB1FC7-F557-4B78-9659-0DE6045AD45B.htm and http://docs.autodesk.com/3DSMAX/16/ENU/3ds-Max-SDK-Programmer-Guide/files/GUID-9B83270D-1D20-4613-AB35-5EE9832C162E.htm

@Baconator650
Copy link

The newer "weighted normals" produces a more accurate result.

@Markemp Markemp added this to the v1.3.1 milestone Aug 3, 2021
@Markemp
Copy link
Owner Author

Markemp commented Sep 27, 2021

This has (finally) been resolved. I think. Closing the issue, but will reopen if necessary.

@Markemp Markemp closed this as completed Sep 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants