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

Randomize seed missing for GPU height map etc. #9

Open
andypoly opened this issue Aug 31, 2020 · 0 comments
Open

Randomize seed missing for GPU height map etc. #9

andypoly opened this issue Aug 31, 2020 · 0 comments

Comments

@andypoly
Copy link

andypoly commented Aug 31, 2020

Great code, only:

You seem to be missing the line "seed = (randomizeSeed) ? Random.Range( -10000, 10000 ) : seed;" from the GenerateHeightMapGPU function which is in the CPU version, needed to add it to allow randomizing.

The thread count for height generation can be too large for older GFX cards so you have to cut it into chunks - you could test for 64k max maybe rather doing 2048*2048 for example

Also I wonder if the height map gen can be made the same for both GPU & CPU to truly test them side by side as CPU is using basic perlin noise, GPU simplex.

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

No branches or pull requests

1 participant