-
Notifications
You must be signed in to change notification settings - Fork 18
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
Add OpenSimplex Module #15
Comments
On first glance, OpenSimplex looks pretty interesting. I'll look into it, but I don't know that I can afford the time to prepare a decent implementation. I'd be more than happy to accept contributions, though ;-) I actually wasn't aware that Simplex noise has an associated US patent. For me personally, being based in Germany, that doesn't present much of an issue, but I can see the problem for someone wanting to use the code commercially or distribute it in the US. So perhaps the module should be behind a #ifdef of something. |
Ach du kommst auch aus Deutschland? Moin ;-) Well, yes, there are some unclear patent issues. That's why OpenSimplex exist. As far as I know the patent only effects textures. If I use it for e.g. heightmaps it should be ok, but I'm not sure. If you add OpenSimplex you can hit two birds with one stone. Get rid of the patent issue (or at least give the user the option) and add the seed to a simplex noise. |
I played with FastNoise and OpenSimplex. If I didn't make a mistake OpenSimplex is significantly faster than FastNoise SimplexFractal. https://github.com/MarkuBu/noiseperf I would contribute myself, but I'm a C# noob and your project is a little bit to complex for me EDIT: only 2D is faster. 3D is slow |
It appears that OpenSimplex has been ported to C# and is perhaps available from here: |
I would really appreciate it, if you where adding Opensimplex noise as module because Perlin noise has some ugly artifacts and Simplex has some patent issues.
The text was updated successfully, but these errors were encountered: