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

Feat/props #60

Merged
merged 31 commits into from
Jan 7, 2025
Merged

Feat/props #60

merged 31 commits into from
Jan 7, 2025

Conversation

piellardj
Copy link
Member

This PR adds new class PropsViewer in the engine that will be used to display ground clutter (grass, rocks for instance).

It does so by using instancing which allows rendering of 10000s of props with great performance.
Also, adds support to make the grass react to the player.

image

@piellardj piellardj requested a review from Sceat January 5, 2025 22:06
@piellardj piellardj self-assigned this Jan 5, 2025
@Sceat
Copy link
Member

Sceat commented Jan 7, 2025

/merge

Copy link

github-actions bot commented Jan 7, 2025

Unable to merge, is this PR ready and approved ?

The stock threejs frustum culling works but does not take distance into account (since the frustum distance is way greater than the props view distance).
So, complete this frustum culling by manually hiding batches that are too far to be seen.
@Sceat
Copy link
Member

Sceat commented Jan 7, 2025

/merge

Copy link

github-actions bot commented Jan 7, 2025

Success! Fast forwarded master to feat/props! git checkout master && git merge feat/props --ff-only

@github-actions github-actions bot merged commit f69a728 into master Jan 7, 2025
2 checks passed
@Sceat Sceat deleted the feat/props branch January 7, 2025 07:12
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