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

Shadow System Enhancements and Renderer Optimizations #51

Merged
merged 14 commits into from
Aug 5, 2024
Merged

Conversation

AlexMollard
Copy link
Collaborator

This pull request includes several improvements to our shadow system, renderer optimizations, and some additional features:

  • Moved shadow map, depth image, and related logic to the renderer
  • Implemented a better camera frustum for shadows, including:
    • Added a bounding box frustum around the camera for shadows
    • Fixed the camera frustum function for shadow maps
    • Optimized camera frustum updates to occur only when necessary
  • Added ability to adjust shadow map size dynamically
  • Refactored light structures
  • Implemented a new shadow system to support multiple shadows and advanced techniques
  • Moved all shadow logic out of the renderer for better organization
  • Added grass material and changed clear color
  • Implemented moving platforms and power-ups in the platformer game
  • Added ability to select a shadow map for debugging purposes
  • Fixed dirlight position issues in the light matrix calculation
  • Re-added ability to update shadow map on the fly
  • Added GLM_ENABLE_EXPERIMENTAL to the slime oddyssey compile definitions
  • Updated README.md image

These changes aim to improve shadow rendering, enhance game features, and optimize our rendering pipeline. Please review and provide feedback.

@AlexMollard AlexMollard self-assigned this Aug 5, 2024
Copy link
Collaborator

@DeclanRogers DeclanRogers left a comment

Choose a reason for hiding this comment

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

Looks good, maybe next time a few more comments

@AlexMollard AlexMollard merged commit 5542513 into main Aug 5, 2024
6 checks passed
@AlexMollard AlexMollard deleted the dev branch August 5, 2024 09:56
@ConnorY97 ConnorY97 added the enhancement New feature or request label Aug 7, 2024
@ConnorY97 ConnorY97 added this to the Pre Release milestone Aug 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants