Skip to content

Fix model transformation, orientation and tangent generation#113

Open
RaYRoD-TV wants to merge 6 commits intomonkeyman192:pathfinderfrom
RaYRoD-TV:pathfinder
Open

Fix model transformation, orientation and tangent generation#113
RaYRoD-TV wants to merge 6 commits intomonkeyman192:pathfinderfrom
RaYRoD-TV:pathfinder

Conversation

@RaYRoD-TV
Copy link

@RaYRoD-TV RaYRoD-TV commented Apr 1, 2025

  • Coordinate System Fix: Properly transformed between Blender's Z-up and No Man's Sky's Y-up coordinate systems

- Coordinate System Fix: Properly transformed between Blender's Z-up and No Man's Sky's Y-up coordinate systems
- Vertex Processing: Enhanced handling of vertices, normals, and tangents for correct orientation

- Smooth Shading: Improved normal averaging for shared vertices

- Vector Dimension Fix: Corrected the tangent calculation to use 3D vectors during processing and only convert to 4D at the end

- Tangent Vector Dimension Fix: Fixed dimension mismatch by:
Using 3D vectors during calculation instead of 4D
Only converting to 4D vectors (w=1.0) at the end

- Transformation Matrix: Improved transformation matrix calculations for consistent model positioning and rotation

- Model Orientation: Fixed upside-down and incorrectly rotated models in-game
addon_script.py Outdated

return tangents

# FIXED: Improved transform application function for better orientation
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this function has no fixes to it?

return AnimationFiles

#Main Mesh parser - Fixed to eliminate faceting
# FIXED: Fixed mesh parser for proper orientation and smoothing
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also no fixes here...

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

Successfully merging this pull request may close these issues.

2 participants