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

Coordinate system in renderer #1505

Closed
wants to merge 99 commits into from
Closed

Conversation

heinezen
Copy link
Member

@heinezen heinezen commented Apr 12, 2023

Use the coordinate system to freely convert input/world/scene positions.

  • Decouple coordinate types
    • scene coordinate type for renderer
    • convert phys to scene
    • convert input to phys (with camera matrices)
  • Use coordinate system in new engine components
    • scene for renderer positions
    • phys in gamestate
    • input in input system
  • Store game entity positions in component
    • continuous curve for positions
    • send curve updates to renderer
  • Documentation
    • Coordinate system
    • Renderer
    • Input system
  • Tests
    • Curve (sync)
    • Keyframe container (sync)

Misc additions:

  • Support zooming for sprites
  • Move a lot of sprite calculations to the shaders
  • Enable depth testing in terrain and world renderers
  • Flip camera direction and align calculations with how they work in other OpenGL tools like glm
  • Fix index buffer generation for 3D terrain

Depends on #1501

@heinezen heinezen added area: renderer Concerns our graphics renderer lang: c++ Done in C++ code documentation Involves the project documentation labels Apr 12, 2023
@heinezen heinezen force-pushed the coordinate branch 4 times, most recently from b24fcf8 to 11432df Compare April 24, 2023 00:59
@heinezen heinezen marked this pull request as ready for review April 30, 2023 02:30
@heinezen heinezen added the kevin-rebuild-pl0x instruct kevin to rebuild this pull request label Apr 30, 2023
@heinezen
Copy link
Member Author

heinezen commented Apr 30, 2023

Supercedes/Closes #1380

@heinezen heinezen added kevin-rebuild-pl0x instruct kevin to rebuild this pull request and removed kevin-rebuild-pl0x instruct kevin to rebuild this pull request labels May 1, 2023
@heinezen heinezen added kevin-rebuild-pl0x instruct kevin to rebuild this pull request and removed kevin-rebuild-pl0x instruct kevin to rebuild this pull request labels May 14, 2023
@SFTbot SFTbot removed the kevin-rebuild-pl0x instruct kevin to rebuild this pull request label May 14, 2023
@heinezen
Copy link
Member Author

heinezen commented Jul 9, 2023

Resolved by #1515

@heinezen heinezen closed this Jul 9, 2023
@heinezen heinezen deleted the coordinate branch October 15, 2023 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: renderer Concerns our graphics renderer documentation Involves the project documentation lang: c++ Done in C++ code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants