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

Frustum culling documentation #1662

Merged
merged 7 commits into from
Jul 29, 2024
Merged

Conversation

heinezen
Copy link
Member

@heinezen heinezen commented Jul 27, 2024

This PR adds several things related to frustum culling:

  • Documentation
    • Frustum Culling
    • Desription of all renderer demos, including screenshots
  • Demo 6 (frustum culling)
    • Code comments
    • Render different angles of sprites
    • Draw the frame around the frustum borders to make it more obvious where the frustum is
  • Fix a bug in Frustum2d where the pixel size was not calculated correctly

@heinezen heinezen added area: renderer Concerns our graphics renderer lang: c++ Done in C++ code documentation Involves the project documentation bugfix Restores intended behavior labels Jul 27, 2024
@heinezen heinezen added this to the 0.6.0 milestone Jul 28, 2024
@heinezen heinezen requested a review from TheJJ July 28, 2024 02:05
TheJJ
TheJJ previously approved these changes Jul 28, 2024
@heinezen heinezen force-pushed the docs/frustum_culling branch from 39265f0 to 5203f52 Compare July 28, 2024 10:41
@heinezen heinezen force-pushed the docs/frustum_culling branch from 5203f52 to ec5d56e Compare July 28, 2024 11:01
@heinezen heinezen requested a review from TheJJ July 28, 2024 11:16
@heinezen heinezen force-pushed the docs/frustum_culling branch 2 times, most recently from 941b43d to ec5d56e Compare July 28, 2024 16:27
@heinezen heinezen force-pushed the docs/frustum_culling branch from ec5d56e to ae9dd8f Compare July 29, 2024 02:02
@TheJJ TheJJ merged commit d66e9f9 into SFTtech:master Jul 29, 2024
13 checks passed
@heinezen heinezen deleted the docs/frustum_culling branch November 25, 2024 23:23
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 bugfix Restores intended behavior documentation Involves the project documentation lang: c++ Done in C++ code
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

2 participants