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

Fixed analytic normal + Optimizations with LUT + Character controller #1

Merged
merged 34 commits into from
Nov 30, 2023

Conversation

BarthPaleologue
Copy link
Owner

  • Updated terrain generation to fix terrain normal and open the way for asset scattering
  • Added controllable character that can walk on the surface of planets
  • Optimized clouds and rings with precomputed LUTs
  • Improved chunk subdivision system to prevent oscillations

Rather than splitting files by MVC, they are split by functionality.
I.E all code related to planets is in the same folder, all the code for stars are in another and so on
It is now based on a minimal distance between vertices. The max depth will be what is needed to achieve this terrain resolution
+ better debug material
There is an issue at higher resolution though, the cubes just disappear even though the program computes the right number of instances to create
- Fixed mutexes pending forever when a chunk would be disposed before recieving its vertex data
- Distance to chunk now accounts for the average height of the chunk to be more precise
- In order to use the average height, the new terrain generation is used
Adding a flat tolerance instead of a proportional one did the trick.
Maybe the tolerance can be lowered further, it needs to be tested
- Fixed inefficiencies in postprocessmanager
- Fixed toolbar not being enabled
- tree shaking issues where all babylonjs would be imported
- quadtree issue in playground
The character controller can now walk on any planet
screen coordinates of star is computed once on the cpu instead of the fragment shader
in/out will make transition to WebGPU harder, so I brought back varying
It was caused by the fact that the rotation around the parent object also rotated the frame of reference of the moon. Now this is no longer the case.
It will be useful to scale the assets and textures of the project
- Added test texture
- Added back procedural atmosphere LUT for later
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Message that will be displayed on users first pull request

@BarthPaleologue BarthPaleologue added the enhancement New feature or request label Nov 30, 2023
@BarthPaleologue BarthPaleologue merged commit b16f1f8 into main Nov 30, 2023
7 of 8 checks passed
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.

1 participant