Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 1, 2025

Added markdown documentation for all engine-specific math implementations (Source, Unity, OpenGL, IW, Unreal) matching the existing Frostbite documentation structure.

Changes

Created docs/engines/{engine_name}/ directories with 4 files each (20 files total):

  • constants.md — Coordinate system conventions (Y-up vs Z-up, handedness), matrix storage (row/column-major), angle type definitions
  • formulas.md — Direction vector computation, rotation/view matrix builders, perspective projection helpers
  • camera_trait.mdCameraTrait integration with omath::projection::Camera (look-at, view, projection)
  • pred_engine_trait.mdPredEngineTrait for projectile prediction (ballistic calculations, aim angles, target tracking)

Key differences documented

Each engine's coordinate system and conventions are clearly specified:

Engine Up Axis Handedness Matrix Layout Pitch Range
Source Z Right Row-major ±89°
Unity Y Left Row-major ±90°
OpenGL Y Right Column-major ±90°
IW Z Right Row-major ±89°
Unreal Z Left Row-major ±90°

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: orange-cpp <59374393+orange-cpp@users.noreply.github.com>
Copilot AI changed the title [WIP] Add additional documentation for engines Add documentation for game engine math modules Nov 1, 2025
Copilot AI requested a review from orange-cpp November 1, 2025 09:38
Copilot finished work on behalf of orange-cpp November 1, 2025 09:38
@orange-cpp orange-cpp marked this pull request as ready for review November 1, 2025 09:40
@orange-cpp orange-cpp merged commit b9ae356 into feature/docs Nov 1, 2025
@orange-cpp orange-cpp deleted the copilot/sub-pr-96 branch November 1, 2025 09:40
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